неизменные JS: сделать функцию более .. функциональные

Question

У меня есть функция редуктора, который работает на Immutable.js записях Calendar:

const Calendar = new Record({ 
    'events': new List(), // list of Events 
}); 

, который представляет собой список Event с:

const Event = new Record({ 
    'start': null, 
    'end': null, 
    'title': null, 
}); 

I хотите добавить новые события из объектов Javascript в этот список и вернуть новую запись Calendar, но только в том случае, если в объекте нет объектов:

(state = new Calendar)=> { 

    const receivedEvent = new Event({ 
     start: <a date>, 
     end: <another date>, 
     title: <a string>, 
    }); 

    let newState; 

    if (state.get('events').contains(receivedEvent)){ 
     newState = state; 
    } else { 
     newState = state 
      .updateIn(['events'], (events)=> events.push(receivedEvent)); 
    } 

    return newState; 
} 

Это лучшее, что я могу сделать прямо сейчас в 4 часа ночи, но это похоже на ужасный подход.

Есть ли способ лучше использовать функциональные парадигмы, такие как предоставленные Immutable для выполнения этой же задачи?

Answer 1

Если вы имеете дело с коллекцией, где членство важно, то вы, вероятно, хотите использовать Set не список.

Набор представляет собой набор уникальных ценностей.

const Calendar = new Record({ 
    'events': new Set(), // set of Events 
}); 

Затем вы можете добавить мероприятие в набор, не беспокоясь о том, чтобы в итоге получить повторяющиеся значения. Реализация структуры данных не позволяет вам проверить, уже ли она содержит значение.

(state = new Calendar) => 
    state.updateIn(['events'], calendar => calendar.add(new Event({ 
    start: <a date>, 
    end: <another date>, 
    title: <a string>, 
    })); 

Как вы представляете календарь, порядок, в котором будут добавлены события, вероятно, не важно, как вы можете использовать дату себя событие для сортировки в виде списка, в противном случае, вы могли бы использовать OrderedSet, чтобы сохранить упорядоченность.

Answer 2

Проблема, с которой вы сталкиваетесь, заключается в том, что List.contains будет сравнивать каждый Event с receivedEvent с использованием ===.

Несмотря на то, что receivedEvent может предоставлять те же атрибуты, что и в вашем штате, они не являются тем же самым объектом.

Пример:

const x = { a: 1, b: 2 }; 
const y = { a: 1, b: 2 }; 
const z = x; 

x === y; // -> false 
x === x; // -> true 
x === z; // -> true 

Итак, вам нужно будет сравнить каждый атрибут receivedEvent в каждом случае вы сравниваете его. К счастью, есть handy Immutable.is method, которые вы могли бы использовать так:

if (state.get('events').some(event => Immutable.is(receivedEvent, event))) { 
    // already exists in events 
} else { 
    // doesn't already exist in events 
}