为啥 fullstack-angular 生成器使用 Lo-Dash 的合并而不是 document.set？

Posted 2023-03-11

【中文标题】为啥 fullstack-angular 生成器使用 Lo-Dash 的合并而不是 document.set？

【英文标题】：Why does the fullstack-angular generator use Lo-Dash's merge instead of document.set?为什么 fullstack-angular 生成器使用 Lo-Dash 的合并而不是 document.set？

【发布时间】：2015-02-18 15:56:59

【问题描述】：

这是更新的默认代码：

exports.update = function(req, res) 
  if(req.body._id)  delete req.body._id; 
  Thing.findById(req.params.id, function (err, thing) 
    if (err)  return handleError(res, err); 
    if(!thing)  return res.send(404); 
    var updated = _.merge(thing, req.body);
    updated.save(function (err) 
      if (err)  return handleError(res, err); 
      return res.json(200, thing);
    );
  );
;

注意上面写着：var updated = _.merge(thing, req.body); 的行。这仅适用于作为基元的 Schema 属性。例如。它不适用于作为数组的架构属性。

问题在于thing 是一个mongoose 文档，而req.body 是一个javascript 对象。

对于真正的问题，我有两个假设：

_.merge() 要求第一个参数是一个对象，但它是 接收文件。 _.merge() 仅在两个对象具有相同的键和类型时才合并。因为一个是文档，另一个是对象，所以它们是 不同。

考虑这个例子。 name 已正确更新，但 arr 未正确更新。但是，当我使用 thing.set(req.body) 而不是 var updated = _.merge(thing, req.body) 时，两者都会正确更新。

thing.html

<div class="container">
  <form ng-submit="update()">
    <div class="form-group">
      <label>Name</label>
      <input class="form-control" ng-model="thing.name">
    </div>
    <div class="form-group">
      <label>Arr</label>
      <input
        type="text"
        class="form-control"
        ng-repeat="el in thing.arr"
        ng-model="thing.arr[$index]">
    </div>
    <button 
      class="btn btn-default"
      type="submit">
      Update
    </button>
  </form>

  thing | json
</div>

thing.controller.js（前端）

.controller('MainCtrl', function ($scope, $http) 
  $http.get('/api/things')
    .success(function(things) 
      $scope.thing = things[0];
    );
  $scope.update = function() 
    $http.put('/api/things/' + $scope.thing._id, $scope.thing)
      .success(function(newThing) 
        console.log('updated thing: ', newThing);
      )
      .error(function() 
        console.log('unable to update thing');
      );
  ;
);

thing.model.js

'use strict';

var mongoose = require('mongoose'),
    Schema = mongoose.Schema;

var ThingSchema = new Schema(
  name: String,
  arr: []
);

module.exports = mongoose.model('Thing', ThingSchema);

thing.controller.js（后端）

'use strict';

var _ = require('lodash');
var Thing = require('./thing.model');

exports.update = function(req, res) 
  if(req.body._id)  delete req.body._id; 
  Thing.findById(req.params.id, function (err, thing) 
    if (err)  return handleError(res, err); 
    if(!thing)  return res.send(404); 
    var updated = _.merge(thing, req.body);
    updated.save(function (err) 
      if (err)  return handleError(res, err); 
      return res.json(200, thing);
    );
  );
;

function handleError(res, err) 
  return res.send(500, err);

两个问题：

使用 _.merge() 有什么问题？ 为什么angular-fullstack generator 的创建者选择使用_.merge()？我相信他们知道它的行为，所以他们一定是有原因的。

【问题讨论】：

是的 - ***.com/questions/27559061/…。我已将其删除，因为我可以使用 set()，但我只是取消删除它以供您参考。

@JohnnyHK 绝对！你说得对，我一开始就应该这样做。我用新示例编辑了问题。

在这种情况下，您需要手动将 arr 字段标记为已修改，因为您尚未定义其元素包含的内容，因此 Mongoose 无法判断它何时被修改。即updated.markModified('arr');

@JohnnyHK 好的，谢谢。你认为这比使用set 有优势吗？也许它更快，因为它只合并更新的东西而不是所有东西？

【参考方案1】：

在 _.merge 返回值上调用“markModified”（如上一个答案所建议的那样）并没有为我解决这个问题。 Adam Zerner 报告的问题是 _merge 导致“更新”对象包含数组属性的无效数据。我在使用 angular-fullstack 生成器时也遇到了这个问题，例如，如果我从数组中删除元素，则此更改不会存储在 Mongoose/mongo 中。在 mongoose 文档上调用 set 函数（例如 thing.set(...，正如 Zerner 所建议的那样）为我解决了这个问题。我还阅读了这个讨论线程：https://github.com/DaftMonk/generator-angular-fullstack/issues/637；它建议用 _extend 替换 _merge（另见 @ 987654322@) 这也适用于我。

【参考方案2】：

您需要用户“markModified” http://mongoosejs.com/docs/api.html#document_Document-markModified

var updated = _.merge(store, req.body);
updated.markModified('foo_name');
updated.save(function (err) 
  if (err) 
    return handleError(res, err);
  
  return res.json(200, store);
);

【讨论】：

查看 user3608315 的回答

【参考方案3】：

我从 Lodash 3.10.1 升级到 4.6.1（当时的最新版本）并且 _.merge 现在可以按照您的需要运行。我遇到了一段时间的麻烦，并且使用最新版本的 generator-angular-fullstack (3.3.0) 将 _.merge 更改为 _.assign 并没有像以前的版本那样对我有用。 Lodash v4.6.1 似乎可以完成这项工作，虽然我看到一堆将 Lodash v3.10.1 列为依赖项的 npm 模块，但我还没有遇到任何问题。