Express与Sequelize发数据库请求时报Cannot read property 'create' of undefined错误如何修复

你遇到的Cannot read property 'create' of undefined报错，本质是调用方法时对应的对象本身不存在（值为undefined）。结合你贴的代码，一共存在6处核心问题，按优先级排序如下：

1. 控制器文件内容完全错位、分层失效 #

你贴的productController.js内容和productRouter.js完全重复，文件里只写了路由定义，根本没有导出all、one、create、update、delete这些控制器方法，路由调用时拿到的控制器对象上根本没有对应方法值，一执行就会报错。同时这个错误的控制器文件里还自己require自己，形成循环引用，整个分层逻辑完全失效。

错误代码示例：

// productController.js 里错误重复写了路由逻辑，没有导出任何业务处理方法
const Router = require("express");
const router = new Router();
const productController = require("../controllers/productControllers");

正确的productController.js应当和路由完全拆分：路由负责定义接口路径、匹配参数，控制器负责处理业务逻辑、调用数据库操作，最后把结果返回给路由。

2. Sequelize 模型引用错误，直接触发create报错 #

你的models/index.js会自动扫描models目录下的模型文件，把每个模型挂到db对象上，属性名是模型配置里的modelName——也就是你在product.js里设置的modelName: 'products'。
原来业务代码里的引用方式：

// 错误写法：db上根本没有productsModel这个属性，值为undefined
const newProduct = await db.productsModel.create(product);

正确的引用应该是直接从db取注册好的模型：

// 正确写法：取db上注册的products模型实例
const newProduct = await db.products.create(product);

3. 模型字段类型配置错误，导致模型初始化失败 #

你的product.js模型里把价格字段类型写成了DataTypes.NUMBER，Sequelize不存在这个内置类型，会导致模型初始化异常，最终db上的products模型注册失败，也会出现undefined问题。
修正为匹配业务的类型：

products.init({
  product_name: DataTypes.STRING,
  // 价格用DECIMAL存小数，比INTEGER更适合商品场景
  price: DataTypes.DECIMAL(10, 2),
  product_description: DataTypes.STRING
}, {
  sequelize,
  modelName: 'products',
});

4. 通用数据库操作方法调用错误 #

你写的业务逻辑层（productModels.js）里，直接调用了db.findAll()、db.update()、db.destroy()——这些是Sequelize模型类上的方法，不是聚合db对象上的方法。
所有数据库操作必须绑定到具体的模型上：

• 查全量商品：db.products.findAll({ raw: true })
• 更新商品：db.products.update(newData, { where: id })
• 删除商品：db.products.destroy({ where: id })
  另外代码里的Discount模型没有从db实例里引入，是全局未定义变量，执行到查询折扣的逻辑时会直接抛错，需要改为db.discounts.findAll(...)的正确引用方式。

5. 路由错误判断逻辑完全失效，触发未处理Promise警告 #

原来的路由代码里写的错误判断：

if(resultOfGetAllProducts === Error) { ... }

Error是JS内置的错误构造函数，业务方法的返回值永远不会等于这个构造函数本身，这个判断永远无法触发。同时路由里的async异步逻辑没有任何异常捕获，抛出的错误没有对应处理者，就会触发你看到的UnhandledPromiseRejectionWarning警告。
另外res.sendStatus()方法会直接设置状态码并结束响应，后续链式调用.json()、.send()完全不会执行——比如res.sendStatus(200).json(result)，实际只会返回空的200响应，接口返回的业务数据完全丢失。
修正后路由的写法示例：

router.get("/products", async (req, res, next) => {
  try {
    const products = await productController.all();
    return res.status(200).json(products);
  } catch (err) {
    // 所有异常都在这里捕获，不会触发未处理的Promise警告
    return res.status(500).json({ message: err.message });
  }
});

6. 业务层的回调逻辑和async/await风格冲突 #

你在业务层写了传统回调的写法return cb(Error, null)，但是路由调用时根本没有传入callback函数，而且用await等待方法返回值，两种异步写法完全不兼容。
业务层应该直接async/await返回结果或者抛出异常，不要混用回调：

// 正确的业务层create方法
exports.create = async function createProduct(product) {
  const newProduct = await db.products.create(product);
  return newProduct;
};

内容的提问来源于stack exchange，提问作者Alex Vislohuzov