тестовое задание
в общем имеются следующие условия:
http://jsbin.com/ladobefoji/edit?js,output сделал вот так : https://jsfiddle.net/t73w9obn/1/ по поводу сохранения данных на сервер не пойму что они от меня тут хотят... |
DynkanMaclaud,
Uncaught TypeError: Cannot read property 'push' of undefined (index):138 В примере запись можно сохранять, изменять и удалять. Цитата:
Объект сериализовать и отдавать на сервер через протокол http, аяксом. |
Rasy,
изменил ссылку на рабочий пример... |
DynkanMaclaud,
у тебя класс Notes сочетает в себе и модель, и вьюху. По сути, наследуется от модели. Возможно, правильнее было бы оставить модель отдельно, и, например, передавать её в конструктор вьюхи, или ещё как-то. В самой модели предусмотреть асинхронный интерфейс (для взаимодействия с сервером). В идеале - модель должна быть EventEmitter, тогда вьюха просто сможет на неё подписываться и обновляться. При таком раскладе всякие прочие компоненты смогут тоже работать с моделью, а не с вьюхой. В общем, как-то так. |
Alexandroppolus,
т.е предлагаешь паттерн observeble (обозреватель) впилить ?? |
DynkanMaclaud,
ну это если совсем правильно делать :) повторюсь - это чтобы другие компоненты страницы могли с твоим компонентом работать, если вдруг такой кейс понадобится. по рендеру - наверно, для обработки событий правильнее будет использовать делегирование, а конкретно вот такой подход: https://learn.javascript.ru/behavior - т.е. вешать один обработчик конкретного типа события на родительский элемент, смотреть target, ну и т.д. Чтобы при каждом обновлении обработчики не ставить на отдельные элементы. |
DynkanMaclaud,
Работа с localStorage и JSON.parse без try...catch. Поправьте, пока никто не поранился. |
Имхо, если предусмотрено общение с сервером, то локальное хранилище лишнее:)
|
destus,
можно пример того как вызов JSON.parse именно в этом коде, может выдать ошибку ... ? другими словами что пользователь может написать что код не отработает... ?) |
|
| Часовой пояс GMT +3, время: 01:18. |