如何在Controller中正确创建并返回DTO？

你当前的写法不存在功能或规范层面的错误，用静态内部类聚合同业务域DTO本身也是很多小项目常用的实践，核心问题只是代码组织和写法上的可读性优化空间，不需要推翻现有实现。

解决类型辨识度低的问题 #

• 不要把所有响应DTO都塞进命名模糊的外层类里，专门定义语义明确的DTO容器类即可：比如用户看板模块下所有响应DTO，统一放在UserDashboardDtos这个final类中，类的构造器设为私有，仅作为静态内部DTO的命名空间使用。
• 配合静态导入消除冗长的类前缀：在Controller类顶部直接静态导入需要用到的内部DTO，之后方法签名、实例化代码里都不需要再写外层类名。

  // 顶部导入
  import static com.yourpackage.dashboard.UserDashboardDtos.UserInfoListRes;
  
  // 方法签名直接简化成
  public ResponseEntity<UserInfoListRes> findUserInfoList()
  

  这种写法下，DTO类型的辨识度和单独创建类文件没有任何区别，同时保留了单文件聚合DTO的低维护成本优势。

解决实例化代码可读性差的问题 #

• 高版本Java（16+）直接用record定义内部DTO：record自带全参构造、访问器方法、基础通用方法，不需要手写任何模板代码，语义比普通类更清晰，明确表示这是个仅承载数据的传输对象。

  // DTO容器类定义示例
  public final class UserDashboardDtos {
      private UserDashboardDtos() {} // 禁止实例化容器类
      public record UserInfoListRes(List<UserInfo> userList) {}
  }
  

• 低版本Java可以给DTO加静态工厂方法，代替直接调用new构造：
  在UserInfoListRes类内部增加静态工厂方法：

  public static UserInfoListRes of(List<UserInfo> userList) {
      return new UserInfoListRes(userList);
  }
  

  实例化时直接写UserInfoListRes.of(userDashBoardService.findUserInfoList())，语义比直接new更明确，一眼就能识别出是在构造响应DTO。
• 项目引入Lombok的话，直接给内部DTO类加@AllArgsConstructor+@Value（不可变DTO推荐）或者@Data注解，不需要手写构造方法，配合静态导入写法同样简洁。

注意避坑 #

• 不要为了省代码直接返回Service层查出的数据库实体、视图对象，会直接暴露内部数据结构，后续字段调整很容易引发接口兼容性问题，违背DTO做层间隔离的核心作用。
• 不要把跨业务域的无关DTO全部塞进同一个容器类里，否则后期查找、修改字段的成本会远高于省下来的建文件成本。

内容的提问来源于stack exchange，提问作者바보린