куча замечаний
Closed this issue · 0 comments
sergeythrees commented
- именование
- id должен начинаться только с букв
- заменить listeners на handlers
- sass
- оформить как сss
- сделать вложенность элементов через скобки{} (.display.content)
(calc.Event)
- оформление
- использовать двойные "кавычки" а не 'одинарные'
- переносить аннотации у функций
- до и после операторов в коде ставить пробелы
- приложение, код
- элементы в дом дереве генерировать через js
- использовать namespace чтобы не конфликтовать с зарезервированными переменными
- "магические числа" в константы
- ! в аннотациях (строгая типизация)
- sender в классе Event лишний
- убрать абсолютные пути
- тесты
- использовать sinon
- тесты для функционала калькулятора
- не работает
- после вычисления, при нажатии смены знака или запятой, меняется второй операнд, а не первый