java - 按照惯例，set() 方法可以在 Java 中返回一个值吗？

Question

我正在学习如何用 Java 编写整洁有序的代码。 set() 方法是否可以返回一个值，或者是否有更有效/可读性更高的方法？

public class Car {

     private boolean mHasAxles;
     private boolean mHasTires;
     private Tires mTires;

     public setAxels(boolean hasAxles) {
          mHasAxels = hasAxles;
     }

     public boolean hasAxles() {
          return mHasAxles;
     }

     public boolean setTires(Tires tires) {
          if(hasAxles()){
               mTires = tires;
               mHasTires = true;
               return true;      // Returns true if it was able to set tires
          }
      return false;  // Returns false because car did not have axels
                     // Therefore, tires could not be set
     }
}

在这个例子中，我的问题专门针对 setTires() 方法。类应该在设置轮胎时检查汽车是否有车轴，还是应该将逻辑留给使用 Car 的类？由于 setTires() 方法返回一个值，是否应该调用其他方法？

Answer 1

严格按照惯例 - 不，setter 通常返回 void。

话虽如此，如果您愿意，您可以自由地返回 boolean - 约定经常被打破(即使在内部 java api 中)并且方法签名，包括它的返回类型，应该激发感兴趣的人开发人员导航到代码以查看返回 boolean 的确切原因。

为了更清楚，您可能希望使用不同的方法名称，例如setTiresIfAble(Tires tires)，或者您也可以返回 void 并抛出异常，如下所示:

 public void setTires(Tires tires){
     if(!hasAxels()) {
         throw new IllegalStateException("no axels!");
     }
     mTires = tires;
     mHasTires = true;
 }

顺便说一句，mHasTires 是多余的，您可以随时检查是否 tires == null。

最后，您可以在 java 中避免使用 m 或匈牙利语表示法(按照惯例)，如下所示:

 public setAxels(boolean hasAxels){
      this.hasAxels = hasAxels;
 }