search

bem / bem-mvc

Yet another MVC for i-bem
30 stars 27 forks source link

Objects equality #115

Open verkholantsev opened 10 years ago

verkholantsev commented 10 years ago

Реализовал сравнение объектов без учета вложенности. Возможно, стоит притянуть какую-либо реализацию deepEqual.

dosyara commented 10 years ago

Проверка объектов без вложенности скорее добавит неоднозначности, чем поможет. Думаю лучше реализовать deepEqual

verkholantsev commented 10 years ago

Может быть стоит притянуть какую-нибудь библиотеку, чтобы не переизобретать deepEqual? В lodash реализация _.isEqual занимает полтора десятка строк: https://github.com/lodash/lodash/blob/master/dist/lodash.js#L1743

dosyara commented 10 years ago

Только на своем уровне :) В bem-mvc тянуть еще библиотеку не правильно.

verkholantsev commented 10 years ago

Может быть тогда выпилить всю эту измененную логику и научить поля модели принимать какое-нибудь свойство навроде comparator, наличие которого поле будет проверять при isEqual() и использовать его вместо базовой реализации?

dosyara commented 10 years ago

Да, хорошая идея.