Показать сообщение отдельно
  #24 (permalink)  
Старый 17.11.2017, 12:48
Аватар для Vlasenko Fedor
Профессор
Отправить личное сообщение для Vlasenko Fedor Посмотреть профиль Найти все сообщения от Vlasenko Fedor
 
Регистрация: 13.03.2013
Сообщений: 1,572

Плохо
Разные стили function Storage, class Notes
this.data = load() ? load() : [];

зачем дважды выполнять load(), если он вернет результат
var storageEnable = (function () {
        var state = false;
        try {
            window.localStorage.setItem('test', 'a');
            state = window.localStorage.getItem('test') === 'a';
            window.localStorage.removeItem('test');
        } catch (err) {
            console.log('Local storage disable');
        }
        return state;
    }()); // утка если крякает

this.remove
splice
this.set очень интересно. здесь либо this возвращать, для использования в дальнейшем цепочки вызовов или присваиваемое значение для плюшек в синтаксисе
если несколько раз используете document.createElement
то можно для создания написать функцию
<script>
function buildElement(tagName, props) {
  var element = document.createElement(tagName);
  for (var propName in props) element[propName] = props[propName];
  return element;
}

document.addEventListener("DOMContentLoaded", function() {
  var btn = buildElement('input', {
    type: 'button',
    value: 'Delete',
    style: 'background-color: red'
  });
  document.body.appendChild(btn);
});
</script>

и наверное лучше было раз создать, а затем по необходимости клонировать эти элементы
Ответить с цитированием