Оцените качество кода?

Question

[Антон Мяснов]

Доброго времени суток.
Писал скрипт для калькулятора расчета стоимости. Выбираем материал, цвета, толщину и площадь и получаем сумму.
Прошу помощи в оценке кода JS:
1) Стала ли новая версия лучше?
2) Какие есть ошибки и что можно сделать лучше?
Вставил бы код, но он слишком длинный.
Старая версия: GitHub
Новая версия: GitHub
GitHub Pages Старая версия
GitHub Pages Новая версия
Спасибо.

Comments

[PetrPo]

Вставил бы код, но он слишком длинный.

Не хотел открывать, но все-таки:
- старая версия весит - 8.06 KB
- новая версия весит - 5.63 KB

Резюме:
поздравляю, стало лучше, на 8.06 KB - 5.63 KB = подсчитай на своем калькуляторе))

Answer 1

[Антон Шаманов]

по моему очевидно что в данном случае получать/передавать свойства по одному не особо удобно + явное дублирование кода - проще сделать универсальную функцию что-то вроде data.each(function (property, values) {...})