IP
к в этот модуль правильно передать объект управления записями в субд?
Size: a a a
2016 May 20
IP
для примера, в монге это довольно удобно реализовано через коллекции, вот например как правильно передать в модуль коллекцию?
IP
у меня в голове только два варианта, замкнуть каждый метод на объявленную ранее колелекцию, user/save = (user) => collection.save(save)
IP
или передавать коллекцию явно user/save = (collection, user) => collection.save(user)
NK
я так делаю
NK
может и неправильно зато в любой точке видно все что на входе и все что на выходе, а-ля полу-функциональный подход :)
NK
/**
* Create resource if not exists
*/
function findResourceOrCreate(resourceCollection, resourceId, payload = {}) {
const _id = resourceId
const created = new Date()
const doc = Object.assign({}, payload, { _id, created, count: {} })
return resourceCollection.findAndModify({ _id }, [], {
$setOnInsert: doc
}, {
new: true,
upsert: true
})
}
* Create resource if not exists
*/
function findResourceOrCreate(resourceCollection, resourceId, payload = {}) {
const _id = resourceId
const created = new Date()
const doc = Object.assign({}, payload, { _id, created, count: {} })
return resourceCollection.findAndModify({ _id }, [], {
$setOnInsert: doc
}, {
new: true,
upsert: true
})
}
IP
я тоже так сделал сначала
IP
ой, но скачала скажи, как ты передаешь resourceCollection в модуль?
IP
по идее, при таком подходе, у тебя в модуль должна быть одна точка входа
NK
модуль это набор независимых функций, ни одна из них не использует вернхюю область видимости
NK
module.exports = {
...
findResourceOrCreate,
...
}
...
findResourceOrCreate,
...
}
IP
ага, т.е. ты бы сделал module.exports.save = (collection, user) => collection.save(user)
NK
а в нужном месте:
const model = require('../models/resource')
...
const resourceCollection = mongodb.collection('resources')
...
return model.findResourceOrCreate(resourceCollection, resourceId, input)
const model = require('../models/resource')
...
const resourceCollection = mongodb.collection('resources')
...
return model.findResourceOrCreate(resourceCollection, resourceId, input)
NK
мне так удобнее тестировать
NK
потому что обычно коллекция доступна не сразу (когда модуль реквайрится) а после определенного времени когда подключение к базе произойдет
NK
а тут я в любой момент времени при вызове функции передаю актуальную коллекцию
NK
в тестах мокаю если надо
NK
ну и заменять легко... по крайней мере в текущий момент я такой подход юзаю, может через пару месяцев окажется говнокодом и буду по-другому, хз :)
IP
а у тебя бывает такое, что вызов model.findResourceOrCreate происходит в разных местах?