设置方法，无参数的多个或带值的单个？

Question

我正在和一个朋友一起开发基于Java的游戏，我注意到他在可维护性方面采取了一种与我有关的方法。

对于一个表示可玩Character的类，他不仅仅是创建一种设置对象属性的方法，还创建了将属性设置为特定值的单独方法。

这两个选项中的哪一个是将来最好的选择？

• 选项1

   public void runFast() { this.character.speed = 5.0f } public void walk() { this.character.speed = 2.0f } public void stop() { this.character.speed = 0.0f; } 

• 选项2

   public void setSpeed(float speedTemp) { this.character.speed = speedTemp; } 

Answer 1

为什么不使用枚举来设置速度-那么您仍然可以

void setSpeed(Speed speed) {
  this.character.speed = speed.getAmount();
}

有：

enum Speed {
  FAST(5.0f), WALK(2.0f), STOP(0.0f);

  private final float amount;
  private Speed(flaot a) { this.amount = a; }
  public float getAmount() {
    return amount;
  }
}

这样，您可以快速更新值，但仍具有预定义的数量。 它灵活且易于维护。 您可能要保存枚举而不是浮点数。

Answer 2

恕我直言，最好的选择是声明常量/枚举，并使用选项2。

---

示例（常量）：

public static final float STOP = 0.0f;
public static final float WALK = 2.0f;
public static final float FAST = 5.0f;

setSpeed(STOP|WALK|FAST);

---

示例（枚举）：

public enum Speed
{
    FAST(5.5f),
    STOP(0),
    WALK(2.5f);

    float value;

    Speed(float pValue)
    {
        this.value = pValue;
    }

    public float getValue()
    {
        return this.value;
    }
}

setSpeed(Speed.FAST);

Answer 3

我的解决方案是改用Enums，

如果将来与您的速度maxHeartRate有更多关系，它会更干净，具有更多上下文并易于扩展。

public class Character {

    private Speed speed;

    public Speed getSpeed() {
        return speed;
    }

    public void setSpeed(Speed speed) {
        this.speed = speed;
    }
};



public enum Speed {

    STOP(0),

    RUN(5.5),

    WALK(2.5);

    double value;

    Speed(double value) {
        this.value = value;
    }

    public double getValue() {
        return value;
    }
};

Answer 4

这取决于。 例如

• 速度是否限制在几个预定义的值？ 在这种情况下，使用enum将是一个很好的解决方案。

• 步行/跑步/停止行驶会产生副作用吗？ 作为一个人为的例子，开始运行可能会导致角色掉落所握持的物品，或者停止可能会导致角色略微滑倒。 在这种情况下，使用单独的方法可能很有意义。

• 也许只有几个预定义状态，但是根据环境的不同，运行速度可能会有所不同。

结果是：哪种从概念上建模角色属性的方法最适合您的游戏逻辑/物理？ 解决这个问题，然后在此基础上创建类的界面。 不要太早就挂在确切的API上，这种东西很容易重构。

Answer 5

当您希望代码可读且避免从另一个类中以错误的方式使用公共类字段时，getter和setter方法很有用。 这个例子说明了如何重要。

A类：

 public class ClassA{     
 // in the body class
 private String fieldClass1;

//classic setter
public void setfieldClass1(String f1)
{
 fieldClass1 = f1;
}

}

B类：

public class ClassB{   
// in the bodyclass
public String fieldClass2;

//classic setter
public void setfieldClass2(String f2)
{
 setfieldClass2 = f2; 
}    

C类：

public class ClassC{

//in the body of the class this method use class a and class b
public void calc()
{
 ClassA aObject = new ClassA();
 ClassB bObject = new ClassB();
 ClassA.fieldClass1 = 5 + 5; // illegal expression for the compiler and costrain the developer to use setters
 ClassB.fieldClass2 = 8 + 8; // legal expression
}
 }

这意味着您必须在进行make setter和getter之前定义“修饰符逻辑”（受保护的，私有的，公共的）。 在修饰符之前定义，在定义setter和getter之后定义。