1

Хочу сохранить модель после изминений:

Statistic.findOne({ 'user': user, 'type': type })
    .then(function(commonStatistic) {
    commonStatistic.data.words += words.length;
    commonStatistic.data.comments += comments.length;
    commonStatistic.data.visits++;
    commonStatistic.updated = Date.now();
    commonStatistic.save(function (err) {
        if (err) {
            // TODO: Add handler
        }
    });
});

Все поля, кроме поля (объекта) data сохраняются нормально, а дата не хочет ни в какую, что это за бред такой, все поля сохраняет а это нет, это баг или я не правильно что-то делаю?

UPD: Короче как оказалось просто так вложенные объекты не сохраняются, для того что бы они сохранялись при save() нужно, перед сохранением модели, их пометить: commonStatistic.markModified('data');. Мне данное решение помогло, но возможно есть решения лучше?

2
  • я при таких случаях в схеме модели указываю вложенную схему где указываю какие поля там будут
    – num8er
    21 июл 2017 в 23:42
  • 1
    Да, но у меня так не получится, потому что объект data у меня может быть рандомной структуры, так было изначально задумано. Но за ответ спасибо, то есть получается он не сохраняет наверно именно из-за того, что у меня объект data простой объект и монгус просто не знает его схему, скорей всего из-за этого он и не сохранял.
    – SkyDancer
    21 июл 2017 в 23:50

1 ответ 1

1

я бы коротко сделал бы так:

const 
  query = {user, type},
  update = {
    '$inc': {
      'data.words': words.length, 
      'data.comments': comments.length, 
      'data.visits': 1
    },
    '$set': {
       updated: Date.now()
    }
  };
Statistic.update(query, update, callback); 

данный пример не создаст инстанс модели в памяти (экономия))) и изменит 1 документ

если важно иметь под рукой результирующий документ то:

Statistic.findOneAndUpdate(query, update, callback);
5
  • Ну на самом деле вариант commonStatistic.markModified('data'); Как то покороче выглядит... Но на счет того что при save() создается копия в памяти я и не знал, если это действительно так и это сильно влияет на производительность тогда да, лучше написать на пару строчек кода больше. Спасибо за пример.
    – SkyDancer
    22 июл 2017 в 0:02
  • марк модифаед и всякие доп заморочки вызывают доп поведения в монгузе что опять таки лишние телодвижения. а то что в примере оно тупо на низком уровне минуя монгуз вызывает монго-дб запрос. как бы монгуз берет имя коллекции с модели и тупо проксирует поля вызывая updateOne по протоколу mongodb
    – num8er
    22 июл 2017 в 0:03
  • p.s. save не создает инстанс (: инстанс модели уже создан при вызове findOne. еще один p.s. если не нужен функционал монгуза при изьятии данных то можно экономить так: Statistics.findOne(query).lean().then() но учтите что lean не дополняет результирующий обьект мусором ODM-ки, так что .save не заработает. итог: пользуемся нужными фичами там где надо
    – num8er
    22 июл 2017 в 0:06
  • да и еще при большой конкуренции запросов лучше всетаки делать операции инк/декремента на стороне базы, иначе 2 паралельных изменений на уровне ODM-ки могут друг-друга затирать, а в моем примере об этом заботится база данных и Вы не потеряете запрос.
    – num8er
    22 июл 2017 в 0:12
  • Понял, спасибо за развернутый ответ!
    – SkyDancer
    22 июл 2017 в 0:13

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.