MVC架构下数据查询等方法的存放位置咨询

嘿，我太懂你这种把数据库逻辑混在模型get/set里导致混乱的感受了——我刚入行的时候也踩过一模一样的坑！让我给你理清楚怎么调整更合理：

首先，别再把数据库操作塞进模型的get/set里了！你的/models文件夹应该只放「纯数据载体」——也就是只负责封装数据、做简单的字段校验或者字段格式化（比如把日期字符串转成Date对象），它的核心职责是「代表数据是什么」，而不是「怎么获取/保存数据」。把查询、插入这类数据库逻辑混进去，会让模型变得臃肿不堪，后续维护起来简直噩梦。

那正确的姿势是什么？给你两个常见的分层方案：

方案1：新增/repositories（仓库层） #

专门创建一个/repositories文件夹，这里是所有数据库操作的「专属地盘」。每个模型对应一个仓库类，比如你有User模型，就写一个UserRepository，里面集中存放所有和用户相关的数据库操作：

• 查询单个用户、查询所有用户
• 创建用户、更新用户信息
• 删除用户

举个简单的代码例子：

/models/User.js（纯数据载体） #

class User {
  constructor(id, name, email) {
    this._id = id;
    this._name = name;
    this._email = email;
  }

  // 只保留和字段相关的get/set
  get id() { return this._id; }
  get name() { return this._name; }
  set name(newName) {
    if (!newName || newName.length < 2) {
      throw new Error("用户名长度不能小于2位");
    }
    this._name = newName;
  }
  get email() { return this._email; }
}

/repositories/UserRepository.js（数据库操作中心） #

// 假设你用的是某个数据库连接工具，比如mysql2
const db = require('../db/connection');

class UserRepository {
  async getUserById(userId) {
    const [rows] = await db.query('SELECT id, name, email FROM users WHERE id = ?', [userId]);
    if (rows.length === 0) return null;
    const userData = rows[0];
    return new User(userData.id, userData.name, userData.email);
  }

  async getAllActiveUsers() {
    const [rows] = await db.query('SELECT id, name, email FROM users WHERE is_active = 1');
    return rows.map(row => new User(row.id, row.name, row.email));
  }

  async createUser(userData) {
    const [result] = await db.query(
      'INSERT INTO users (name, email) VALUES (?, ?)',
      [userData.name, userData.email]
    );
    return new User(result.insertId, userData.name, userData.email);
  }
}

/controllers/UserController.js（处理请求，调用仓库） #

const UserRepository = require('../repositories/UserRepository');

class UserController {
  async getUserProfile(req, res) {
    try {
      const userId = req.params.id;
      const userRepo = new UserRepository();
      const user = await userRepo.getUserById(userId);
      
      if (!user) {
        return res.status(404).json({ message: "用户不存在" });
      }
      res.json({ user });
    } catch (err) {
      res.status(500).json({ message: "获取用户信息失败", error: err.message });
    }
  }
}

方案2：如果业务逻辑复杂，再加/services（服务层） #

如果你的项目有很多复杂的业务规则（比如「创建用户时要同时生成默认角色、发送欢迎邮件」），那可以在控制器和仓库之间加一个/services层，把业务逻辑放在这里：

• 控制器：只负责接收请求、返回响应，调用服务层
• 服务层：处理业务逻辑，调用仓库层获取/保存数据
• 仓库层：只和数据库打交道

这样分层后，每个模块的职责都清清楚楚，再也不会出现「模型里混着SQL查询」的混乱情况了。

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