2
私のコードは機能しますが、脂肪モデルとスキニーコントローラを持つことが最善であると理解しています。nodejsコードの構造はどうですか？脂肪モデル/スキニーコントローラ
しかし、私は3つの異なるモデルを使用しており、残念なことに私のコントローラを脂肪にしました。このコードを（脂肪モデル/スキニーコントローラのコンセプトで）整理する最良の方法は何ですか？私はコード構造について読んできましたが、私はベストプラクティスを少しは確信しています。
コントローラー：
var Product = require('../models/product');
var Collection = require('../models/collection');
var Vote = require('../models/vote');
exports.topSearch = function(req, res) {
console.log(req.body, "search product")
Product.search({
query_string: {
query: req.body.search
}
},req.body.searchObject,
function(err, results) {
if (err) console.log('ERR', err);
if (results) {
var data = results.hits.hits;
Vote.find({
user: req.user._id
}, function(err, votes) {
if (!err) {
for (var i = 0; i < votes.length; i++) {
for (var j = 0; j < data.length; j++) {
if (data[j]['_id'] == votes[i]['product']) {
data[j]['voteId'] = votes[i]['_id'];
data[j]['userVote'] = votes[i]['vote'];
}
}
}
}
Collection.find({
user: req.user._id
}, function(err, collections) {
if (!err) {
for (var i = 0; i < collections.length; i++) {
for (var j = 0; j < data.length; j++) {
if (data[j]['_id'] == collections[i]['product']) {
console.log('match')
data[j]['collected'] = true;
data[j]['collectId'] = collections[i]['_id'];
data[j]['favorite'] = collections[i]['favorite'];
} else if (data[j]['_id'] !== collections[i]['product'] && data[j]['collected'] !== true) {
data[j]['collected'] = false;
}
}
}
res.send(data);
}
});
});
} else {
res.send({
errmsg: 'results not defined'
})
}
});
};
私はその後、私のルートでこれを呼び出す：
app.post('/products-search', users.ensureAuthenticated, products.topSearch);