映射器中的API调用被认为是不好的做法吗？

Question

使用DTO作为API模型是很常见的。通常，您需要在之后将这些DTO映射到其他模型。我将用下面的例子让它变得非常简单：

class RequestDto {
  private String companyId;
  // more fields ..
  // getter, setter etc..
}

class SomeModel {
  private Company company;
  // more fields ..
  // getter, setter etc..
}

class RequestMapper {
  public SomeModel mapRequestToSomeModel(RequestDto request){
    Company company = fetchCompanyFromApi(request.getCompanyId()); // makes a request to another service
    
    SomeModel someModel = new SomeModel();
    someModel.setCompany(company);
    // map more fields..

    return someModel;
  }
}

Answer 1

在我看来有点臭。我（个人）对映射器的期望是非常非常便宜，特别是如果其他映射器很琐碎的话。不涉及数据库或API调用。我更希望创建某种转换服务，它执行相同的步骤，但调用不同。

对于名为get…的函数，经常会出现类似的问题，在这些函数中，我永远不会期望有昂贵的操作。