面向对象设计疑问：SingleAnswer的getAnswers()返回单元素List是否合理？

首先，你的这个设计有其合理性，但也存在一些可以优化的语义细节，咱们一步步拆解来看：

一、当前设计的可取之处 #

你通过让SingleAnswer和MultiAnswer统一实现getAnswers()返回List<String>，最大的好处是上层代码无需区分题型——比如你的populateFormElements方法，只需要统一调用getAnswers()拿到值，再传给element.setValue()即可，完全不用写if (answer instanceof SingleAnswer)这类分支判断。这很好地遵循了里氏替换原则，让代码的扩展性更强（以后加新的答案类型，只要实现Answer抽象类，上层逻辑基本不用改）。

对于开放题、单选题来说，返回单元素List是完全可行的——很多表单组件的setValue方法本身就可以接受集合类型（比如单选框可能只取集合的第一个元素，但统一接口的话，不用单独为单选框写特殊逻辑）。

二、当前设计的小瑕疵 #

不过确实存在语义上的小问题：

1. 方法名与实际行为的冲突：SingleAnswer的getAnswers()是复数形式，但实际只会返回一个元素，容易让其他协作的开发者产生误解——看到返回List就下意识认为可能有多个值，需要额外看注释或实现细节才清楚是单元素。
2. set方法的冗余设计：SingleAnswer的setAnswers(Set<String> answers)需要判断集合大小为1，其实完全可以直接接受String类型的参数，既符合单一答案的语义，又避免了集合操作的额外开销和判断逻辑。

另外注意你的代码里有个小笔误：SingleAnswer的getAnswer()方法签名应该是getAnswers()（和父类保持一致），否则会因为方法名不匹配导致编译错误哦。

三、优化方案建议 #

方案1：保留统一接口，优化语义与实现 #

这是最推荐的方案，既能保留统一接口的优势，又解决语义问题：

• 给SingleAnswer新增setAnswer(String answer)方法，替代原来的setAnswers(Set<String>)，让单一答案的设置更直观：

  public class SingleAnswer extends Answer {
      private String answer;
      public SingleAnswer(Question question) { super(question); }
      
      @Override
      public List<String> getAnswers() { 
          return Collections.singletonList(answer); 
      }
      
      public void setAnswer(String answer) {
          this.answer = answer;
      }
  }
  

• 给SingleAnswer的getAnswers()方法加上清晰的JavaDoc注释，明确说明返回单元素列表：

  /**
   * 返回单元素列表，对应开放题/单选题的唯一答案
   * @return 包含单个答案的不可变列表
   */
  @Override
  public List<String> getAnswers() { 
      return Collections.singletonList(answer); 
  }
  

方案2：拆分接口（权衡复杂度后可选） #

如果非常在意语义一致性，可以考虑拆分出两个子接口：

public interface SingleValuedAnswer extends Answer {
    String getSingleAnswer();
}

public interface MultiValuedAnswer extends Answer {
    List<String> getMultiAnswers();
}

然后让SingleAnswer实现SingleValuedAnswer，MultiAnswer实现MultiValuedAnswer。但这样的话，你的populateFormElements方法就需要做类型判断，会增加代码复杂度：

if(answer.isPresent()){
    if (answer.get() instanceof SingleValuedAnswer) {
        element.setValue(Collections.singletonList(((SingleValuedAnswer) answer.get()).getSingleAnswer()));
    } else if (answer.get() instanceof MultiValuedAnswer) {
        element.setValue(((MultiValuedAnswer) answer.get()).getMultiAnswers());
    }
}

这种方案适合你需要严格区分单/多答案语义，且表单元素需要不同处理逻辑的场景，但会损失一部分代码简洁性。

总结 #

回到你的核心疑问：当前设计并非过度担忧，但可以通过优化语义让代码更易读。如果你的表单元素统一接受List类型的值，那么保留统一的getAnswers()接口是更优的选择，只需要调整SingleAnswer的set方法和注释即可。

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