在Java中创建一个新的自定义对象vs将同一对象的多个值设置为Null

Question

所以，我遇到了这样的情况

我有一个Restful控制器方法，它返回一个列表对象match

@RequestMapping(value = "/fetchFootballMatchesToday", method = RequestMethod.GET)
public @ResponseBody List<Match> fetchMatchesScheduledToday() throws ParseException {
    return matchesToday;
   }

List<Match> matchesToday = matchInvService.fetchMatchByDate(today);

for (Match stats : matchesToday) {
 if (stats != null) {
/* user should not know these details, so they are being set to null */
    stats.setMatchSuccessIndex(null);
    stats.setTeamRanking(null);
    stats.setStadiumInfrastructureLevel(null); 
    ..... & so on
  }
}

List<Match> matchesToday = matchInvService.fetchMatchByDate(today);
List<Match> responseList = new ArrayList<Match>();  

for (Match stats : matchesToday) {
 if (stats != null) {
   Match tmpMatch = new Match();
   match.setProperty1(stats.getProperty1());
  }
 responseList.add(tmpMatch); 
}

 return responseList;

需要你们的建议伙计们。这是在编写更多代码和节省内存之间进行权衡吗？解决这一问题的最佳方法是什么？

Answer 1

对象匹配有15个属性&我只需要UI上的3个属性。我不希望用户看到对象的其他属性。

将所有字段设置为null确实很麻烦，而且容易出错。
所以我将避免使用第一种方法。

第二种方法似乎朝着更好的方向发展。
但是您不应该害怕将对象映射到其他对象，因为客户端需要查看它们的特定视图。
为“合理的”列表大小进行简单的映射操作是便宜的。并且我假设您不会在UI中显示数百万行，所以大小应该是合理的。
否则您应该通过考虑分页概念来重新思考整个UI设计。

List<Match> matchesToday = matchInvService.fetchMatchByDate(today);
List<MatchDTO> responseList = new ArrayList<Match>();  

for (Match stats : matchesToday) {
 if (stats != null) {
    MatchDTO match = new MatchDTO(stats);
    responseList.add(match); 
  } 
}

return responseList;

public MatchDTO(Match match){
  this.foo = match.getFoo();
  this.bar = match.getBar();
  this.fooBar = match.getFooBar();
}

或在Java 8中：

List<MatchDTO> responseList =   
    matchInvService.fetchMatchByDate(today)
                   .stream()
                   .filter(Objects::nonNull)
                   .map(MatchDTO::new)
                   .collect(Collectors.toList);