Обновить
0
Yurk@yurk

Пользователь

Отправить сообщение
если не использовать сломанный минификатор

Неизвестно кто какой минификатор будет использовать для моего кода, поэтому мне не сложно обезопасить себя от недоразумений, даже если они и случались с нулевой вероятностью два раза.
Статьей хотел пояснить, что такая позиция может привести к затрудненной читаемости кода того же npm. Потом уже понял, что это и был ваш style-guide. Nevermind)
Пожалуй, вряд ли мы сможем обсудить «вероятность синтаксических ошибок при минификации скрипта». Ошибочно считать, что кому-то есть дело до чужого кода.
В статье по ссылке хорошо описаны несколько «злых шуткок», но, в тоже время, она оправдывает ";" в начале строк. На мой взгляд, это конечная стадия пути «расставляем везде -> расставляем нужные -> расставляем на всякий случай в начале». Читать такой код невозможно, надеюсь, и на ваш взгляд тоже.

Чтобы расставить точки с запятой над i. Я ставлю знаки везде в конце строк — это исключает вероятность синтаксических ошибок при минификации скрипта + считаю, что такой код удобнее читать + это удобная привычка для С-like языков, коих много.
Это все Phyton виноват.

Хозяин — барин, но порекомендую habrahabr.ru/post/136860/

Тесты поправил, JSON.stringify вернул (увидел, что обрабатываете "\n" в строке. В этом случае — да, использование JSON.stringify оправдано, но не забывайте о старых ИЕ — им ничего неизвестно о JSON)
Ваш код не содержал ни одной ;, поэтому позволил себе напомнить) Все же, их использование является таким себе правилом хорошего тона.
Как злая шутка:
var b = 1, c = 1, a = b + c
(1+2).toString()

Тесты, каюсь, не запускал, правил онлайн. Падают скорее всего потому, что я изменил названия моих любимых переменных foo и bar. Попробую залить пачсет позже.

Кстати, вчера на заметил, но предпочтительнее использовать obj.hasOwnProperty("' + bar + '") для проверки наличия проперти.
Вам pull request.

Немного перемудрили с «JSON.stringify» — это лишнее; кеширование регекспа даст +5% к производительности; и не предавайте забвению точку с запятой, что может сыграть злую шутку в джаваскрипте;.

Информация

В рейтинге
Не участвует
Откуда
Украина
Дата рождения
Зарегистрирован
Активность