CastError:对于模型“customer”的路径“\u id”处的值“customers”，转换为ObjectId失败

Question

我希望有人能帮我找出我在这里做错了什么。我一直在寻找，我可以找到很多类似的问题，但没有一个我足够聪明来解决我的问题。我得到以下错误：

CastError:对于模型“customer”的路径“\u id”处的值“customers”，转换为ObjectId失败

它以前是有效的，我设法打破了它，我解开了所有我认为我改变了的东西，我仍然得到了错误。

这是我的模式：

const Schema = mongoose.Schema;

const CustomerSchema = new Schema({

  custName: {
    type: String,
    required: true
  },
  custStreet: {
    type: String
  },
  custCity: {
    type: String
  },
  custState: {
    type: String
  },
  custZip: {
    type: String
  }
});

module.exports = Customer = mongoose.model('customer', CustomerSchema); 

我的路线：

const router = express.Router();

const customerController = require('../../controllers/customer')

const Customer = require('../../models/Customer');

router.route('/')
  .get(customerController.index)
  .post(customerController.newCustomer);

router.route('/:customerID')
  .get(customerController.getCustomer)

module.exports = router;

和我的控制器：

module.exports = {

  index: async (req, res, next) => {
    try {
    const customers = await Customer.find({})
    res.status(200).json(customers);
  } catch(err) {
      next(err);
    }
  },

  newCustomer: async (req, res, next) => {
    try {
    const newCustomer = await Customer(req.body);
    const customer = await newCustomer.save();
    res.status(201).json(customer);
      } catch(err) {
          next(err);
      }
  }, 

  getCustomer: async(req, res, next) => {
    try {
    const { customerID } = req.params;
    const customer= await Customer.findById(customerID);
    res.status(200).json(customer);  
    } catch(err) {
        next(err);
    }
  }
};

此外，以下是我得到的全部信息：

CastError:对于模型中的模型“customer”，路径“\u id”处的值“customers”，转换为ObjectId失败。查询模型上的exec（C:\Users\natha\Desktop\Coding\HAMMER\node\u modules\mongoose\lib\query.js:4371:21）。查询查询然后（C:\Users\natha\Desktop\Coding\HAMMER\node\u modules\mongoose\lib\query.js:4463:15）在processtick和rejections（internal/process/task\u queues.js:93:5）

还有一些让我困惑的事情，我的数据库中有另一个集合的路由文件和控制器。它基本上是相同的代码，除了没有路由到“/：Customers”之外，它路由到“/：Tickets”，并且工作正常。我看不出这个代码是如何工作的，客户代码不是。

路线：

const router = express.Router();

const ticketController = require('../../controllers/ticket');

router.route('/')
  .get(ticketController.index)
  .post(ticketController.newTicket);

router.route('/:ticketID')
  .get(ticketController.getTicket)
  .put(ticketController.replaceTicket)
  .patch(ticketController.updateTicket);

module.exports = router;

控制器：

module.exports = {

 index: async(req, res, next) => {
   try {
     const tickets = await Ticket.find({})
     res.status(200).json(tickets);
   } catch(err) {
     next(err);
   }
 },

 newTicket: async (req, res, next) => {
   try {
     const newTicket = await Ticket(req.body);
     const ticket = await newTicket.save();
     res.status(201).json(ticket);
   } catch(err) {
      next(err);
   }
 },

 getTicket: async(req, res, next) => {
  try {
  const { ticketID } = req.params;
  const ticket = await Ticket.findById(ticketID);
  res.status(200).json(ticket);
  } catch(err) {
      next(err);
  }
 }, 

 replaceTicket: async(req, res, next) =>{
   try {
     const { ticketID } = req.params;
     const newTicket = req.body;
     const result = await Ticket.findByIdAndUpdate(ticketID, newTicket);
     res.status(200).json({ Success: true });
   } catch(err) {
      next(err);
   }
 },

 updateTicket: async(req, res, next) =>{
   try {
     const { ticketID } = req.params;
     const newTicket = req.body;
     const result = await Ticket.findByIdAndUpdate(ticketID, newTicket);
     res.status(200).json({ Success: true });
   } catch(err) {
      next(err);
   }
 } 
};

我将感谢任何帮助！

谢谢

-N8

Answer 1

我对OP的解决方案投了更高的票。为了节省大量时间，我只想解释为什么这是一个问题，以及其他人可以在OP服务器中像我一样关注什么。js，“/”排在其他路由之前，这些路由违反规则首先列出特定路由。也就是说，如果在其他路由问题解决后移动了“/”。更重要的是，这个错误来自哪里？当您调用任何其他路由时，您的应用程序将首先调用“/”并使用客户的路由，同时将路由路径（即“/customers”、““/materials”、““/tickets”）作为id参数传递，然后mongo将尝试在客户集合中将客户、物料和票据作为id强制转换。

app.use('/customers', customers);
app.use('/materials', materials)
app.use('/tickets', tickets)
app.use('/', customers);

这将解决OP的问题。

就我而言，这就是问题所在：

router.get('/', getAllUser);
router.get('/:id', getUserById)
router.get('/facebook', facebookPassport)
router.get('/google', customersPassport);

因此，我在这里为模型用户的路径“\u id”处的值“facebook”/“google”得到这个错误投射错误，因为最后两个路由中的字符串“facebook”和“google”是特定的，但永远不会到达，因为“/：id”路由将接受“/”之后的任何值作为id。当我这样重新排列时：

router.get('/', getAllUser);
router.get('/facebook', facebookPassport)
router.get('/google', customersPassport);
router.get('/:id', getUserById)

带走：在应用级别和路由文件中将特定路由移动到动态路由（带参数的路由）的顶部。

Answer 2

您在getCustomer处理程序中出错。

您不应该使用字符串id来查找客户，而是必须首先使用mongoose。类型。ObjectId（字符串ID）。

getCustomer: async(req, res, next) => {
    try {
    const { customerID } = req.params;
    const customer= await Customer.findById(mongoose.Types.ObjectId(customerID));
    res.status(200).json(customer);  
    } catch(err) {
        next(err);
    }
  }

Answer 3

我发现了问题。我server.js档案里有这个：

app.use('/', customers);
app.use('/customers', customers);
app.use('/materials', materials)
app.use('/tickets', tickets)

一旦我摆脱了应用程序。使用（“/”，客户） 一切都很好。