JavaScript

[viv81ster]

индусы?

[lomalkin]

Нет, скорее всего просто студенты на практике. =) А вообще российская крупная компания.
Вооще странно, что человек знает сокращенную запись оператора условия, но не воспользовался простой инверсией. Ну что ж, у всех бывает :)

[viv81ster]

А если не секрет, какая компания :) ?

[lomalkin]

Я там недавно, и пока боюсь, что секрет. :) Постарайтесь меня понять.

[lomalkin]

нет =)

[dab512]

а как еще в одну строчку инвертировать булеву переменную?

[lomalkin]

a=!b; // =)

[lomalkin]

//прошу прощения:
a=!a;

[w0lf]

Я ничуть не оправдываю вышеприведенный код, но что будет если до инверсии xthtp ! obj_.checked будет undefined?

[lomalkin]

Будет эксцепшн. :) Ну а в данной конкретной реализации там на 2 строки выше была проверка на существование obj_, а если объект существует - то это свойство в этой конкретной реализации обязательно задано, это же JS :)

[krig]

У чекбокса при снятом флажке свойство checked выставляется в undefined, а совсем не в false, как все ожидают. Сам с этим однажды столкнулся, и дебажил потом алертами.

[VasilioRuzanni]

Зачем алертами? Есть же FireBug :)

[maxic]

Для тех кто не пользуются FF, DragonFly тоже есть ;)
Но для быстроты можно и алертами :)

[lomalkin]

Когда нет возможности использовать альтернативный браузер, то приходится алертами. :) Ощутил на собственном опыте, с `той` системой, где работаю я - из браузеров не работает ничего, кроме осла. Ибо особенность единой авторизации.

[VasilioRuzanni]

Интересно, а что за "та" система?)

[lomalkin]

Корпоративный проект, раскрывать подробности не имею возможности, моя роль там не так велика. А причина пригодности только одного браузера, если простым языком - единая авторизация в базе данных приложения и системе (информация доступна через веб интерфейс после успешного входа в систему, что-то типа NTLM там)

[VasilioRuzanni]

Ну про NTLM-то я в курсе, и проверку Kerberos, встроенные в операционную систему. Мы сами это используем, создавая решения для MS Office SharePoint Server (а там вообще только с IE можно нормально работать).
Я просто думал, что для других браузеров тоже можно настроить.

[lomalkin]

Конечно, скорее всего можно. При таких то возможностях в браузеров для плагинов. Но есть еще такое понятие как "корпоративный стандарт".

[Gr2k]

Совершенно спокойно работаю с MS SP service 3.0 через фаирфокс. About:config - network.automatic-ntlm-auth.trusted-uris вписываете урл до главного узла и все.

[bat]

Тоже доводилось дебажить JS по IE, пользовался Microsoft Script Editor (MSE), кажется входит в состав мс офиса.

[middle]

obj.checked = !obj["checked"]

[BarAusWald]

obj_.checked = !obj_.checked
?

[dab512]

блин точно - че то протупил )

[Goganchic]

кажись нашелся автор кода :))

[Borro]

В один символ "!" ;)

[fxposter]

a = !!!a; :)

[subecho]

это не так уж далеко от истины - оператор !! приводит любой объект к булеву типу. Так что вышло очень даже в стиле языка со статической типизацией.

[fxposter]

А вы думаете я "от балды" это написал? :)

[subecho]

да и я смайл забыл поставить ;)

[DmitryBaranovskiy]

Удивительно, но оператор ! тоже приводит всё к булеву типу. :)

[vectoroc]

как эмоционально а = !!!!!!!!!а;

[efix]

Никогда не понимал, почему минусуют безобидные вопросы.
Все-таки нельзя же сказать, что этот комментарий именно плохой?
А так человек в следующий раз побоится сказать что он что-то не знает.
Ведь это так ценно, когда люди открыто говорят, что они что-то не знают, особенно в нашей отрасли!

[lomalkin]

Абсолютно согласен! Вам плюс в карму (а за комментарии кончились плюсы уже =) ), а dab512 плюс - за "моральный ущерб" =)
У всех бывают "тупняки". И раньше все чего-то не знали! И никогда не стесняйтесь задавать вопросы! Это нормально чего-то не знать! А только за желание что-то узнать уже можно уважать!
А минусуют в паблике - добро пожаловать в почту, даже лично к себе приглашаю, еще никогда не отказывал в совете! И надеюсь мало кто откажет при личном обращении. Совет это не затратно и не трудно для помогающего. Взаимопомощь это то, что помогает выжить в современном мире, и по крайней мере странно этого не понимать. Удачи ;)

[spiritedflow]

Такой код не редкость. Пишется он обычно в 4-5 утра под подсказку дружной компании: мышки, кофе и скрепки из ворда

[maxic]

Бывает...
Как говорят футболисты премьер-лиги, когда заху...т мяч на трибуну: "У мастеров бывает, у чайников - норма"

[soltpain]

я такое вот встречал в сторонних коммерческих скриптах.

window.location.href='javascript:history.back()'

[80InchNail]

Ну зачем сразу критиковать? Не я писал ;), но код вполне мог быть сгенереным. В таком случае даже респект программеру, который выкрутился и в поле HREF в дизайнере указал javascript:history.back()... (Например)

[neuotq]

НА самом деле часто бывает. И я на подобные штуки попадался. Это когда простейшим задачкам находишь почемуто сложные решения. Вообщем спать нужно больше.

[Stepanow]

Когда голова не соображает хорошо если код в итоге рабочий получится. А вылизать можно в следующем билде. Кстати интересно как по скорости различаются варианты с инверсией и без неё?

[lomalkin]

Думаю при инверсии быстрее. Ибо проверка условия - это сравнение значений.

[yurr]

вполне жизнеспособный код и, что важно, удобочитаемый

[AlborTholus]

Ну на счет удобочитаемости Вы загнули...

obj_.checked=(obj_.checked==true)?false:true;
obj_.checked=(obj_.checked==false)?false:true;
obj_.checked=(obj_.checked==true)?true:false;

В какой строчке была инверсия? А, может, их две?

[lomalkin]

инверсия только в первой =)

[Goganchic]

это ж тест на внимательность!

[dohlik]

Не так уж все и страшно... Код не лишний, просто неоптимальный.

[krig]

На самом деле ничего смешного в коде нет.

Сам лично написал примерно то же самое около месяца назаж, помучившись минут 10 из-за того что поведение было совсем не таким, как планировалось. В итоге оставил для себя же (маловероятно что в тот код кто-то другой полезет править) небольшой камент:

// need this because when unchecked attr gives undefined value instead of false
var checked = $(this).attr('checked') == true ? true : false;

[w0lf]

Я писал о чем то подобном в http://habrahabr.ru/blog/code_wtf/47054.html#comment1010291
Именно из за того что JScript строго себя ведет при операциях с undefined значениями и приходится иногда так извращаться, усложняя логику программы. С другой стороны подобный контроль неявных инициализаций оберегает от возможных ошибок.

[Cellard]

А разве такой способ не обходит проблемы с undefined

var checked = $(this).attr('checked') ? true : false;

[telega2]

Можно использовать двойную инверсию !!value

[krig]

Спасибо, не додумался.

[fxposter]

var checked = $(this).attr('checked') == true;

достаточно такого... оператор == возвращает bool.

[DamnedTiny]

вот так работает оператор not :o)

[habrausername]

ага, слово "йа" можно писать одной буквой? :)

[haikuos]

зато после выполнения кода переменная гарантированно проинициализированна

[naishin]

Да йопт, это вы еще не видели заголовочные файлы от гцц.
Там перлы просто. Перлы.
#define true true

[akral]

Это очень разумная строчка, кстати. Защищает от умников, обиженных на компанию со строками
#define true false
#define false true

[lomalkin]

Ребята, я конечно все понимаю, но скажите мне, зачем срать в карму? Рейтинг статьи растет вверх, а карма падает. А ведь хотел просто поделиться с вами интересной по моему мнению информацией, и при этом вроде бы не задел ни чьих интересов. Разочарован. Реально после такой реакции 10 раз подумаешь перед тем как куда-то что-то писать. :(

[VasilioRuzanni]

Ага, странный эффект. Постоянно всплывает.
Up.

[IntenT]

вобщем-то думать надо перед написанием статьи, а не после..
подумал-бы - глядишь, и промолчал. и карма в целости, и ничего в топ не всплывает.

[lomalkin]

Я с вами согласен, но "парадокс" (впрочем довольно частый) именно в том, что рейтинг статьи и карма "растут" в разные стороны. Если бы они изменялись сонаправлено - у меня бы не возникало таких вопросов - все было бы сразу понятно, или инфа - говно (все в минусе) или всем понравилось (все в плюсе). И не было бы подобных вот постов с моей стороны.

[dparfyonov]

Довольно часто встречаю у новичков подобную конструкцию. Это проходит с возрастом... должно пройти)

[alexbig]

ice = (ice) ? !cei : ice!

[Pilot34]

иногда написать лучше длиннее, но надежнее. В этом случае у нас вконце точно получится либо true либо false, никаких null и undefined. Насчет !a я не уверен, undefined не получится?

[DmitryBaranovskiy]

Не получится. a = !a — это самое правильное решение в данном случае.

[w0lf]

Обсуждалось уже выше. В случае если "a" было undefined (а для свойства checked у объекта checkbox это может быть легко) то !a вызовет эксепшн.

[DmitryBaranovskiy]

Да, что вы говорите! А вы попробуйте :) Если а в принципе определена как переменная никакого exception не будет. Учите матчасть. ;)

[Pilot34]

ну вобщем видите - вызвало сомнения у нас новичков, а писать лучше так, что сомнений нет. Не исключаю бы что и сам так написал в 5 утра. Осуждать лучше кривые алгоритмы, а такие штуки - плохой стиль, говнокод, но на работу-то и на скорость даже не особо влияет.

ps еще бывают генераторы кода, я себе стандартную половину программы на php обычно генерирую. Получается совсем не идеально, но быстро.

[w0lf]

Сходу в тестовом примере (простая статичная форма, IE 6, FF 2.0.0.16, Opera 9.24) добиться undefined действительно не получилось, но по опыту помню что в некоторых случаях это вполне возможно. В любом случае как говорил мой препод по ТРПО (теории разработки программного обеспечения) - "сохраняйся и предохраняйся". Лично я стараюсь никогда не делать действий над объектами - элементами формы, предварительно не проверив их.

[DmitryBaranovskiy]

В данном случае оригинальный код это не a = !a, а a.checked = !a.checked; Данный код может вызвать exception только если а == undefined. Но тогда и обсуждать нечего. Проблема будет ещё в левой части. Хочется bulletproof решения?
a && (a.checked = !a.checked);
Удачи.

[Alexus]

бывает, что не можешь вспомнить какое-нибудь очень простое слово по-английски. Вот и с очевидными решениями так. Видимо кодил начинающий и в тот день был явно не в духе.