#node.js #mongodb #mongoose
Вопрос:
В моей схеме я добавил 3 необязательных параметра ratings
, totalRating
которые служат для автоматического расчета averageRating
. При обновлении существующей записи все идет хорошо, но опалить параметры создания записи не удается Product.create error: Error: Product validation failed: averageRating: Cast to Number failed for value "NaN" (type number) at path "averageRating"
. Я попытался добавить ratings
и totalRating
в тело запроса и установить значение по умолчанию 0, но ничего не помогло. Все еще получаю ошибку . Вы можете понять, почему??
Схема:
const mongoose = require('mongoose');
const productSchema = new mongoose.Schema({
createdOnDate: { type: Number, required: true },
name: { type: String, required: true },
brand: { type: String, required: true },
price: { type: Number, required: true },
description: { type: String, required: true },
category: { type: String, required: true },
city: { type: String, required: true },
region: { type: String, required: true },
country: { type: String, required: true },
vendor: { type: String, required: true },
barcode: { type: String, required: true },
imageUrl: { type: String, required: true },
fullImages: { type: Array, required: true },
thumbNails: { type: Array, required: true },
// productImage: Uint8Array,
minimumStock: { type: Number, required: true },
availableQuantity: { type: Number, required: true },
soldQuantity: { type: Number, required: true },
totalRating: { type: Number, required: false, default: 0 },
ratings: { type: Number, required: false, default: 0 },
averageRating: {
type: Number, required: false, default: function () {
return (this.totalRating / this.ratings)
}
},
},
{ timestamps: true });
module.exports = mongoose.model('Product', productSchema, 'Products');
отправил json:
Mongoos createProduct req.body: {
name: 'someName',
quantity: 5,
price: 5.5,
city: 'Bologna',
region: 'Emilia-Romagna',
country: 'Italy',
category: 'bikes',
vendor: 'zazza zenigata',
createdOnDate: 132468754,
availableQuantity: 5,
soldQuantity: 0,
minimumStock: 10,
imageUrl: 'someUrl',
barcode: 'someCode',
description: 'someDescription',
brand: 'someBrand',
totalRating: 0,
ratings: 0
}
ошибка:
Product.create error: Error: Product validation failed: averageRating: Cast to Number failed for value "NaN" (type number) at path "averageRating"
at ValidationError.inspect (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/error/validation.js:47:26)
at formatValue (node:internal/util/inspect:757:19)
at inspect (node:internal/util/inspect:336:10)
at formatWithOptionsInternal (node:internal/util/inspect:1999:40)
at formatWithOptions (node:internal/util/inspect:1881:10)
at console.value (node:internal/console/constructor:327:14)
at console.log (node:internal/console/constructor:363:61)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/api/src/controllers/product.controller.js:79:17
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/helpers/promiseOrCallback.js:16:11
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:4892:21
at _done (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3118:16)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3133:18
at callbackWrapper (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3087:20)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:4868:18
at processTicksAndRejections (node:internal/process/task_queues:76:11) {
errors: {
averageRating: CastError: Cast to Number failed for value "NaN" (type number) at path "averageRating"
at SchemaNumber.cast (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/schema/number.js:373:11)
at SchemaNumber.SchemaType.applySetters (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/schematype.js:1106:12)
at SchemaNumber.SchemaType.getDefault (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/schematype.js:1052:25)
at $__applyDefaults (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/document.js:437:24)
at model.Document (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/document.js:165:5)
at model.Model (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:105:12)
at new model (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:4706:15)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3094:22
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3130:7
at Array.forEach (<anonymous>)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3129:15
at promiseOrCallback (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/helpers/promiseOrCallback.js:9:12)
at Mongoose._promiseOrCallback (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/index.js:1149:10)
at Function.create (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3064:23)
at exports.createProduct (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/api/src/controllers/product.controller.js:42:11)
at handleReturn (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/express-promise-router/lib/express-promise-router.js:24:23) {
stringValue: '"NaN"',
messageFormat: undefined,
kind: 'Number',
value: NaN,
path: 'averageRating',
reason: AssertionError [ERR_ASSERTION]: The expression evaluated to a falsy value:
assert.ok(!isNaN(val))
at castNumber (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/cast/number.js:28:10)
at SchemaNumber.cast (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/schema/number.js:371:12)
at SchemaNumber.SchemaType.applySetters (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/schematype.js:1106:12)
at SchemaNumber.SchemaType.getDefault (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/schematype.js:1052:25)
at $__applyDefaults (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/document.js:437:24)
at model.Document (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/document.js:165:5)
at model.Model (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:105:12)
at new model (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:4706:15)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3094:22
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3130:7
at Array.forEach (<anonymous>)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3129:15
at promiseOrCallback (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/helpers/promiseOrCallback.js:9:12)
at Mongoose._promiseOrCallback (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/index.js:1149:10)
at Function.create (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3064:23)
at exports.createProduct (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/api/src/controllers/product.controller.js:42:11) {
generatedMessage: true,
code: 'ERR_ASSERTION',
actual: false,
expected: true,
operator: '=='
},
valueType: 'number'
}
},
_message: 'Product validation failed'
}
Product.create: undefined
/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/helpers/promiseOrCallback.js:19
throw error;
^
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
at new NodeError (node:internal/errors:329:5)
at ServerResponse.setHeader (node:_http_outgoing:579:11)
at ServerResponse.header (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/express/lib/response.js:771:10)
at ServerResponse.send (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/express/lib/response.js:170:12)
at ServerResponse.json (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/express/lib/response.js:267:15)
at ServerResponse.send (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/express/lib/response.js:158:21)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/api/src/controllers/product.controller.js:85:23
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/helpers/promiseOrCallback.js:16:11
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:4892:21
at _done (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3118:16)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3133:18
at callbackWrapper (/Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:3087:20)
at /Volumes/ProjectsSSD/FixitServer/fixit_server_node/node_modules/mongoose/lib/model.js:4868:18
at processTicksAndRejections (node:internal/process/task_queues:76:11) {
code: 'ERR_HTTP_HEADERS_SENT'
}
Комментарии:
1. Похоже, код в рабочем состоянии! Я думаю, что «проблема» заключается в том, что ваши значения по умолчанию разрешаются значением NaN, т. Е. 0/0 = NaN. И мангуст ожидает, что тип будет числом, а не значением NaN.
2. 0/0, почему бы не сохранить значение по умолчанию 0,
3. а ну понятно. Так что, возможно, исправлением будет значение по умолчанию 1
Ответ №1:
И оценки, и усреднение должны быть виртуальными полями: в противном случае разделение /0 приведет к сбою вашей схемы в тот момент, когда у вас появится запись в вашей коллекции product.ratings = 0
. Поэтому я бы сделал, во-первых, определил рейтинги с помощью функции получения для виртуального поля:
productSchema.virtual("numberRatings").get(function() {
return this.ratings.length;
});
Предполагая, что this.ratings
это массив, в котором хранятся все оценки для данного конкретного продукта. Таким образом, при каждом доступе product.numerRatings
будет отображаться обновленное количество записей в этом массиве.
Имея это в виду, вам просто нужно добавить немного условной логики к вашему усреднению:
productSchema.virtual("averageRating").get(function() {
return this.ratings.length > 0 ? this.totalRating / this.ratings.length : 0;
});
До тех пор, пока в массиве нет оценок, product.averageRating
будет выдаваться 0, без попыток какого-либо проблемного деления. В тот момент, когда массив начнет содержать оценки, будет вычислено среднее значение.
Комментарии:
1. Я вижу,
ratings
это количество полученных оценок,totalRating
это значение, которое нужно разделитьratings
для расчетаaverageRating
. Итак, если я сделаю 3 параметра виртуальными, я должен быть в порядке, сохраняя их какNumber
правильные? Если изменить их наratings
массив, мне придется изменитьupdate
метод.2. Я не принял ваш ответ только потому, что он предлагал подход, основанный на массиве, вместо использования того, что у меня есть в качестве схемы, но я поддержал его, поскольку он указал мне правильное направление. Большое спасибо
3. Рад помочь. Основной проблемой было разделение de /0. Остальное действительно зависит от ваших потребностей
4. Действительно, в этом и заключалась проблема. Еще раз большое спасибо, все же я хочу выяснить, почему я использую
averageRating
в качестве реального параметра и.pre
не обновляю его.
Ответ №2:
Как указал @amda, используя virtual
я мог бы сделать averageRating
так, чтобы это вычислялось при каждом обновлении записи. Его ответ предложил подход, основанный на массиве, но я хотел использовать схему, которая у меня уже есть, а не для рефакторинга метода обновления, который у меня уже был.
Я попытался использовать .pre
, чтобы установить averageRating
свойство как:
const productSchema = new mongoose.Schema({
createdOnDate: { type: Number, required: true },
name: { type: String, required: true },
brand: { type: String, required: true },
price: { type: Number, required: true },
description: { type: String, required: true },
category: { type: String, required: true },
city: { type: String, required: true },
region: { type: String, required: true },
country: { type: String, required: true },
vendor: { type: String, required: true },
barcode: { type: String, required: true },
imageUrl: { type: String, required: true },
fullImages: { type: Array, required: true },
thumbNails: { type: Array, required: true },
// productImage: Uint8Array,
minimumStock: { type: Number, required: true },
availableQuantity: { type: Number, required: true },
soldQuantity: { type: Number, required: true, default: 0 },
totalRating: { type: Number, required: false, default: 0 },
ratings: { type: Number, required: false, default: 0 },
// using .pre => doesn't update averageRating
averageRating: { type: Number, required: false, default: 0 },
},
{
timestamps: true
});
productSchema.pre('save', function (next) {
// this.averageRating = this.totalRating > 0 ? this.totalRating / this.ratings : 0;
this.averageRating = this.totalRating / this.ratings
next();
});
но averageRating
никогда не обновляет..
Использование virtual
вместо этого работает, мне просто нужно было добавить возвращаемый объект ToObject/toJSON, чтобы включить виртуалы, и теперь он работает так, как я ожидал:
const productSchema = new mongoose.Schema({
createdOnDate: { type: Number, required: true },
name: { type: String, required: true },
brand: { type: String, required: true },
price: { type: Number, required: true },
description: { type: String, required: true },
category: { type: String, required: true },
city: { type: String, required: true },
region: { type: String, required: true },
country: { type: String, required: true },
vendor: { type: String, required: true },
barcode: { type: String, required: true },
imageUrl: { type: String, required: true },
fullImages: { type: Array, required: true },
thumbNails: { type: Array, required: true },
// productImage: Uint8Array,
minimumStock: { type: Number, required: true },
availableQuantity: { type: Number, required: true },
soldQuantity: { type: Number, required: true, default: 0 },
totalRating: { type: Number, required: false, default: 0 },
ratings: { type: Number, required: false, default: 0 },
// using .pre doesn't update
// averageRating: { type: Number, required: false, default: 0 },
},
{
timestamps: true,
toObject: { virtuals: true },
toJSON: { virtuals: true }
});
productSchema.virtual("averageRating").get(function () {
return this.totalRating > 0 ? this.totalRating / this.ratings : 0;
});