|
Цитата:
|
|
Судя по коду вы не используете ide (по крайней мере при reformatCode не осталось бы выравненных присвоений). Это не плохо но с ide удонее. Мне нравится этот гайд по стилю кода.
|
Нет, ide не использую, использую sublime text. Посмотрел гайд, очень ко многому придется привыкать, но, думаю, постепенно буду соблюдать=)
|
Цитата:
|
|
Стало лучше. Мне не ясно зачем тут localStorage? потому как если данные тянутся с сервера то их должно быть не много. И они должны тянуться постронично. На сколько я понимаю это просто для себя. Ок.
|
Задумка в том, чтоб получить данные сервера, а ковырять уже на клиенте, localStorage для того, что б дальше работать с данными.
|
Цитата:
|
|
clearStorage - удалить ключи можно в 1 проход.
|
так?
for (let key in sessionStorage){
if (/^Data__\d+$/.test(key)){
sessionStorage.removeItem(key);
}
}
|
Цитата:
|
|
getData - зачем колбек если у вас есть промис? Возвращайте промис, красиво же!
|
А вот этого, к сожалению, не понял
|
Цитата:
|
|
Вы много используете красивых итераторов, но по данным бежите обчсным фором. Почему? В коде должно быть единообразие по максимому. Допустимы сильные отличия в пользу производительности там где она нужна.
|
Потому, что тут циклы работают с цифрами.
по возможности действия метода должны нести описательный характер, например в setPage вы проверяете валидность индекса, но намного лучше это вынести в отдельный метод это в 10 раз повысит читабельность метода
Я на практике убедился что много однострочных методов - это хорошо, и условия тоже хорошо бывыносить. Это позволяет не вникать в скобки и прочее, а сразу понять суть проверки по названию метода.
Возьму на заметку, спасибо.