通常，set()方法可以在Java中返回一个值吗？


Conventionally, can a set() method return a value in Java?

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
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()方法，因为它返回一个值？

相关讨论

传统上严格 - 不，设定者通常会返回空白。

话虽如此，如果你愿意，你可以自由地返回boolean - 约定经常被破坏(即使在内部java api中)，方法签名(包括它的返回类型)应该激发感兴趣的开发人员导航到代码中看到究竟为什么返回boolean。

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

1
2
3
4
5
6
7
 public void setTires(Tires tires){
     if(!hasAxels()) {
         throw new IllegalStateException("no axels!");
     }
     mTires = tires;
     mHasTires = true;
 }

顺便提一下，mHasTires是还原剂，您可以随时检查是否tires == null。

最后，您可以避免使用java中的m或匈牙利符号(按照惯例)，如下所示：

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

setSomething方法不应返回任何内容

trySetSomething必须返回一个布尔值，表示该集合是否成功。

为什么？当您使用Java，C ++，任何语言编写代码时，您希望代码的任何读者(可能是人类)可以访问方法所做的大部分知识，只需读取他的名字。

为了完成这个断言，我们可以研究set可能失败的情况。处理故障有两种可能性，具体取决于方法的范围：

  • 如果setSomething的范围是受保护的，私有的或包的，则意味着您可以作为API开发者控制它的调用方式。可以使用断言或RuntimeException来管理可能的失败(因为在方法签名中声明throws子句不是必需的)。

  • 如果setSomething的范围是公共的，则意味着您无法控制它作为API开发者的控制方式。您必须警告API的用户setSomething不是错误安全的。您必须使用必须在throw子句中声明的异常来管理可能的失败。

    • 您"可以"返回一个值，但按照惯例，setter不返回值。由于setter通常以一种甚至不引用返回值的方式使用，因此类的任何调用者都很容易忽略您返回的值。什么都没有阻止被叫做：

    myCar.setTires(aTire);

    ...并忽略你甚至返回一个布尔值，因为这是调用和使用setter的正常方式。如果调用者可以忽略它并且代码看起来正确，则这会使您返回值的目的失败。

    我的第一个倾向是抛出java.lang.IllegalArgumentException而不是返回false。