我正在构建一个 Spring MVC Web 应用程序,它使用 JPA/Hibernate 来存储对象。我有一个关于域对象以及如何正确使用它们的问题。有一些具有依赖关系的域对象。一个示例可以是分配了区域实体的公司实体。
我有一个从控制器调用的简单帮助器类,该类的任务是从 URL 地址读取 XML 文件,解析其内容并根据该内容返回新的 Company 对象类。
class MyCustomHelper {
MyCustomService myCustomService;
//I have to pass myCustomService to be able to fetch existing Region entities from db
public MyCustomHelper(MyCustomService myCustomService){
this.myCustomService = myCustomService;
}
public Company createCompanyObjectFromUrl(){
//I get these values via xml parser in the app
String name = "Company name";
String address = "Address of a company";
Integer regionId = 19;
//Create new instance of entity and fill it with data
Company company = new Company();
company.setName(name);
company.setAddress(address);
//Assign already existing region to new Company
Region region = this.myCustomService.findOneRegion(regionId);
company.setRegion(region);
return company;
}
}
这种方法有效,但我不知道它的设计是对还是错。如果我的 Company 对象只是没有任何依赖关系的普通对象,那么创建新 Company 并为其设置 String 和 Integer 值会很容易。但事实并非如此。
创建新公司时,我还必须创建与现有区域的连接。我通过在我的助手的构造函数中传递一个服务对象来完成它,它从数据库中获取一个现有的区域。
一旦一个新公司被传回控制器,就会为其设置一些其他属性,然后将其保存到数据库中。
我觉得这不是一个很好的方法(将 Service 实例传递给辅助类)。也许在帮助程序中创建某种 DTO 对象,将其返回给控制器,然后将其映射到域对象会更干净。
还是没关系?