Лишняя запятая в JS

Question

[Kant]

Возник спор по поводу ошибки, возникающей в IE, если встречается лишняя запятая в коде.
Ситуация простая:
items: [
{xtype: 'textfield', cls: 'fake-field', value: 'Manager Login'},
new Ext.form.TextField(Ext.applyIf({fieldLabel: 'Username', name: 'login', allowBlank:false}, fieldConfig)),
this.passwordField,
this.passwordConfirmField,
new Ext.form.Hidden({name: 'companyId'}),
]

Если последнюю запятую не убрать — IE не отрабатывает скрипт. Все остальные браузеры отработают на ура.
Вопрос: Кто прав? Кодер и его запятая (вместе со всеми браузерами) или же IE (который не хочет обрабатывать явный промах программиста).

Answer 1

[charon]

на мой непрофессиональный взгляд неправ программист. Запятая явно тут лишняя, в какой грамматике ни думай.

Comments

[gro]

Запятая здесь явно не лишняя и неправ IE.

[alex_blank]

Ваш взгляд очень непрофессиональный. Закрывающую запятую в перечислениях программисты ставят издавна (ещё со времен C), это немного сокращает время на написание кода, когда нужно поменять местами или добавлять элементы, не думая о возможных опечатках.

Answer 2

[Hackett]

Всё объясняет.

Comments

[Марк]

Это объект, а в вопросе массив.

Answer 3

[Chvanikoff]

Запятым в конце перечислений быть!
А IE… Ну он всегда был «особенным».

Answer 4

[Kant]

Вообщем разобрались: в 5 издании ECMA-262 запятые допускаются, но браузеры поддерживают только 3 издание, в котором запятые не разрешены.

Comments

[lugansk]

Нет.
Перевод приведённой мною выше цитаты:

11.1.4: Издание 5 проясняет тот факт, что завершающая запятая в конце ArrayInitialiser не увеличивает длину массива. Это не семантическое изменение по сравнению с изданием 3, однако некоторые реализации ранее могли неверно интерпретировать это.

Т.е., если сказать то же проще: «ввиду того, что разработчики ИЕ имеют альтернативное восприятие окружающей действительности, в 5-й редакции специально для них добавлено особое разъяснение, чтоб уж точно не ошиблись».

Answer 5

[Антон Корзунов]

Не прав прогер, и у нас на работе почему-то много таких вот не правых.
И не только в JS
наверное потому что люди не Сишники и даже не Паскалисты — а любые «нормальные» языки любое нарушение синтаксиса жестко убивают.
И приучают не писать лишние зпт очень даже быстро

Comments

[alex_blank]

Лол. Это в стандарте C как раз явно прописано:

enum-specifier:
enum identifier { enumerator-list }
enum identifier { enumerator-list, }

[Антон Корзунов]

гм, я и про енумы могу длинные гневные посты написать :)
Оставте ка хвостит в определении структуры или обычного байтового массива.

Я вообще пока (в этом году) не увидел «хвостанутые» запятые у коллег никогда бы не додумался что так можно. Уж больно не понятно зачем( чтобы быстрее дописавать новые элементы в хвост? это не ответ )

[alex_blank]

Да, чтобы банально дописывать новые элементы в хвост. Вам почему-то сложно в это поверить, а ведь дописать запятую в конце элемента перечисления — это раз плюнуть, это экономия времени.

Вы забываете также о макросах, когда макросами генерируются энумы, то это необходимая фича.

С точки зрения логики — логичнее когда каждый элемент в перечислении выглядит одинаково. С какого хуя последний должен отличаться от предыдущих на одну запятую? ;)

Answer 6

[Павло Пономаренко]

неправ ИЕ, потому что, согласно стандартам, запятая в конце стоять может. это сделано для удобства программирования и кодогенерации

Answer 7

[MT]

С массивами-то (Array) в IE (даже 6) вроде такой проблемы нет, а что касается объектов, используемых в качестве ассоциативных массивов, начиная с IE8 это уже исправлено:

var obj = {
	'lorem' : 'first',
	'ipsum' : 'second',
};

Итого, прав программист, но из-за IE6-7 пока приходится делать неудобно — без запятой после последнего элемента.

Answer 8

[Марк]

По стандарту там запятую нельзя. Прав IE (это, наверное, единственный такой случай).

Comments

[lugansk]

11.1.4: Edition 5 clarifies the fact that a trailing comma at the end of an ArrayInitialiser does not add to the
length of the array. This is not a semantic change from Edition 3 but some implementations may have
previously misinterpreted this.

(c ) ECMA-262, Edition 5

Не прав MSIE, т.к. он создаёт лишний пустой элемент массива, хотя по стандарту не должен.

Answer 9

[Александр Колобов]

Меня вот другое интересует, почему ie падает, натыкаясь на эту запятую? У меня он в подобных случаях работал вполне нормально.

Comments

[Kant]

Под «падает» я подразумеваю «не обрабатывает скрипт», а не полностью ложиться браузер.

[Stalker_RED]

потому, что он особенный.
вам еще иешных багов найти?

[my_own_parasite]

он не падает. он просто добавляет undefined в конце массива. если вы пройдётесь по этому массиву в цикле и будете у каждого вызывать какие-нибудь методы, то на последней итерации он упадёт, потому что у undefined нет никаких методов