Let's walk through fixing your issues step by step—there are a couple of key problems in your Sequelize logic and a potential route setup issue causing that Cannot GET error.

1. Correct the Sequelize Query Logic #

Your original SQL joins LectureReservation with Lecture and filters on Lecture.mentor_id, but your current Sequelize code has misplaced filters and incorrect association handling:

• Wrong filter placement: You're trying to filter mentor_id on LectureReservation, but this field lives in the Lecture table. Move it inside the include object's where clause.
• Invalid association condition: The Lecture_id: Lecture.id line in your include is unnecessary (and won't work) if you've set up proper model associations. If you haven't defined associations, you'll need to explicitly set the join on clause.
• Using findOne instead of findAll: Your SQL will return all reservations for the mentor, but findOne only grabs the first match. Use findAll to get all results.

Here's the revised controller code:

module.exports = {
  get: (req, res) => {
    // First, validate the mentor ID exists in the request
    if (!req.params.id) {
      return res.status(400).json({ message: 'Mentor ID is required' });
    }

    LectureReservation.findAll({
      include: [
        {
          model: Lecture,
          where: { mentor_id: req.params.id }, // Filter on Lecture's mentor_id here
          // Uncomment below if you haven't set up model associations yet
          // on: {
          //   lecture_id: Sequelize.col('Lecture.id')
          // },
          attributes: [] // Don't fetch extra Lecture fields we don't need
        }
      ],
      attributes: ['reservation_datetime'], // Only fetch the field we care about
    })
    .then((results) => {
      if (results.length) {
        res.status(200).json({ 
          message: 'OK!',
          reservations: results // Return the actual reservation data to the client
        });
      } else {
        res.status(404).json({ message: 'No pre-ordered lectures found for this mentor' });
      }
    })
    .catch((err) => {
      console.error('Error fetching reservations:', err); // Log error for debugging
      res.status(500).json({ message: 'Failed to fetch reservations' });
    });
  },
};

2. Fix the Route Configuration #

The Cannot GET /mentor/mypage/pre-ordered_Lectures/1 error means your Express app doesn't have a route handler mapped to that path. Make sure you've registered the route correctly in your main server file (e.g., server.js or app.js):

const express = require('express');
const app = express();
// Import your controller (update the path to match your file structure)
const reservationController = require('./controllers/lectureReservationController');

// Register the GET route for mentor pre-ordered lectures
app.get('/mentor/mypage/pre-ordered_Lectures/:id', reservationController.get);

// Start your server
const PORT = process.env.PORT || 3000;
app.listen(PORT, () => {
  console.log(`Server running on port ${PORT}`);
});

3. Verify Model Associations #

For the include clause to work seamlessly, ensure you've set up the correct associations between your LectureReservation and Lecture models:

In models/LectureReservation.js: #

module.exports = (sequelize, DataTypes) => {
  const LectureReservation = sequelize.define('LectureReservation', {
    reservation_datetime: DataTypes.DATE,
    lecture_id: DataTypes.INTEGER
  });

  LectureReservation.associate = (models) => {
    // Define that a reservation belongs to a single lecture
    LectureReservation.belongsTo(models.Lecture, {
      foreignKey: 'lecture_id',
      as: 'Lecture' // Match this alias if you use it in the include
    });
  };

  return LectureReservation;
};

In models/Lecture.js: #

module.exports = (sequelize, DataTypes) => {
  const Lecture = sequelize.define('Lecture', {
    mentor_id: DataTypes.INTEGER
    // Add other Lecture fields here
  });

  Lecture.associate = (models) => {
    // Define that a lecture has many reservations
    Lecture.hasMany(models.LectureReservation, {
      foreignKey: 'lecture_id'
    });
  };

  return Lecture;
};

Quick Debugging Tips #

• Add console.log('Received mentor ID:', req.params.id) at the start of your controller to confirm the ID is being passed correctly.
• Check your server logs for detailed error messages—this will help you spot issues like missing model imports or incorrect association names.
• Use 404 (Not Found) instead of 409 (Conflict) when no reservations exist, as 409 is meant for resource conflicts (like duplicate entries).

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