Приведение к номеру не удалось для значения «NaN» (номер типа) в пути в вычисляемом поле Мангуста

#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;
});