Java- setter/getter 的最佳实践，单个 setter/getter 或不同变量的多个 setter/getter ？

Question

我是 Android 编程的新手(~ 2 个月)有必要为几十个不同的变量设置 getter 吗？

例如——

//Yes I realise that this isn't 'dozens'
public float getX() {
    return position.x;
}

public float getY() {
    return position.y;
}

public float getWidth() {
    return width;
}

public float getHeight() {
    return height;
}

public float getRotation() {
    return rotation;
}

例如，虽然有必要为 float 和字符串使用不同的 getter 和 setter，但使用 switch 语句之类的东西来返回不同的变量是不是一种不好的做法？如果是这样，为什么会这样？

public float returnSomething(String theThing) {
    switch (theThing) {
        case "height":
            return height;
        case "rotation" :
            return  rotation;
        case "width":
            return  width;

        default:
            return 0;
    }
}

那么，上面的代码是否被认为是错误的？如果是，请解释原因。

感谢您的帮助，这不是真正的问题，因为任何一种方法都可以正常工作，我只是不明白为什么如果没有充分的理由人们会使用几十个 getter 和 setter。

我想同样的问题也适用于 setter

Answer 1

因为当你做类似的事情的时候

public float returnSomething(String theThing) {
    switch (theThing) {
        case "height":
            return height;
        case "rotation" :
            return  rotation;
        case "width":
            return  width;

        default:
            return 0;
    }
}

我能感觉到 Stack Overflow 的下一个问题，“为什么我的高度总是 0？”

然后像这样发布代码

public class GameThingy {
      //...

     private void doStuff(GameObject gameObject) {
         float gravity = 5*gameObject.returnSomething("hieght");
         gameObject.setSomething("velocyty", gravity+50);
     }
}

从技术上讲，一旦您在任何地方打错字，您就很难找到问题的根源。那，你很幸运，这些字段都是 float ，它们不一定是。

编辑: 顺便说一句，这实际上是在某些数据库查询中定义感兴趣字段的典型问题。就像，必须通过 String 指定您要查找的字段.

现实生活中的例子是 RealmQuery<T> .

A RealmQuery<T>看起来像这样:

RealmQuery<User> query = realm.where(User.class);

// Add query conditions:
query.equalTo("name", "John");

// Execute the query:
RealmResults<User> result1 = query.findAll();

他们在哪里上课 User是这样的:

public class User {
    private String name;

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }
}

一个有趣的注意事项是 Realm 创建了一个“代理子类”，在那里他们重新定义了 setName和 getName方法(又名，你需要 getters/setters 来使一些系统工作!他们假设你有它们!)

但这里重要的是你需要提供字段名称 "name" .

稍后，任何人都可能会打错字，或者您不会记住这些字段。在这种特殊情况下，他们倾向于创建一个元模型(为您存储字段名称的东西)，以便您不必使用字符串来引用字段名称。

例如，在 Criteria API 中，而不是

Root<Pet> pet = cq.from(Pet.class);
cq.select(pet.get("name"));

他们有这样的元模型:

Root<Pet> pet = cq.from(Pet.class);
cq.select(pet.get(Pet_.name));

因此，它消除了 String 的需要

同样，我倾向于使用 Realm 来创建一个“元模型”(尽管您现在可以使用 RealmFieldNamesHelper 自动为您生成它):

public class User
        extends RealmObject {
    @PrimaryKey
    private long id;
    private String name;

    public static enum Fields { //this is the "metamodel"
        ID("id"),
        NAME("name");

        private String fieldName;

        Fields(String fieldName) {
            this.fieldName = fieldName;
        }

        public String getField() {
            return fieldName;
        }

        @Override
        public String toString() {
            return getField();
        }
    }
}

因此您可以像这样替换查询

RealmResults<User> result2 = realm.where(User.class)
                                  .equalTo(User.Fields.NAME.getField(), "John")
                                  .or()
                                  .equalTo(User.Fields.NAME.getField(), "Peter")
                                  .findAll();

但是有了 getter 和 setter，您已经拥有了类型安全，并且您已经拥有了无需记住的方法 - 因此，您拥有了编译时错误检查。

所以回答你的问题，在 Java 中通过字符串名称引用变量是一种不好的做法的原因是因为

1.) 拼写错误可能会发生，错误只会发生在运行时，而不是编译时

2.)它类型不安全；你很幸运，你得到float s back 不管在这个例子中是什么，但现在它可以是 String , 你需要返回 Object并转换或使用 public <T> T get(String field) { ... }并且调用该方法的人必须确切地知道他们接收到的是什么 - 也容易出现运行时错误。