是否可以循环设置器和获取器？

Question

我相当有信心这不可能起作用，但我还是想问一下，以防万一我错了：

我多次听说，每当你在一批中有一定数量的非常相似的代码行时，你应该总是循环遍历它们。

所以说我有类似下面的东西。

setPos1(getCard1());
setPos2(getCard2());
setPos3(getCard3());
setPos4(getCard4());
setPos5(getCard5());
setPos6(getCard6());
setPos7(getCard7());
setPos8(getCard8());
setPos9(getCard9());
setPos10(getCard10());
setPos11(getCard11());
setPos12(getCard12());

没有办法减少代码行数，如下所示，对吗？

for (i = 0; i < 12; i++) {
setPos + i(getCard + i)());
}

我确信以前有人问过这个问题，但谷歌和 SO 搜索都没有给出否定的证据。

感谢您快速确认这一点！

Answer 1

如果没有反射，就没有办法专门在 Java 中做到这一点，而且我认为这是不值得的。这看起来更像是一个提示，表明您应该重构 getcard 函数以采用整数参数。然后你就可以循环了。

Answer 2

这是一个简单的代码片段，展示了如何使用反射循环访问某个对象的 getter 来检查返回值是否为 null：

for (Method m : myObj.getClass().getMethods()) {
    // The getter should start with "get" 
    // I ignore getClass() method because it never returns null
    if (m.getName().startsWith("get") && !m.getName().equals("getClass")) {
            // These getters have no arguments
            if (m.invoke(myObj) == null) {
                // Do something
            }
    }
}

就像其他人所说的那样，这可能不是一个优雅的实现。这只是为了完整性。

Answer 3

你可以通过反射来做到这一点，但这会很麻烦。更好的方法可能是创建通用的 setPos() 和 getCard() 方法，您可以将当前项目的索引传递到其中。

Answer 4

您需要放弃 getter/setter 对，并使用 List 来存储您的对象，而不是尝试将所有内容填充到一个 God 对象中。

这是一个人为的例子：

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;

public class Foo {
    public static class Card {
        int val;

        public Card(int val) {
            this.val = val;
        }

        public int getVal() {
            return val;
        }

    }

    public static class Position {
        int value;

        public Position(Card card) {
            this.value = card.getVal();
        }
    }

    public static void main(String[] args) {
        List<Card> cards = new ArrayList<Card>(Arrays.asList(new Card(1), new Card(2), new Card(3)));
        List<Position> positions = new ArrayList<Position>();
        for (Card card : cards) {
            positions.add(new Position(card));
        }
    }

}

Answer 5

您无法动态构造方法名称然后调用它（无需反射）。即使有反射，它也会有点脆弱。

一种选择是将所有这些操作集中到一个方法中，例如 setAllPositions ，然后只调用该方法。

或者，您可以有一个位置数组，然后循环遍历该数组，在每个索引处设置值。

Card[] cardsAtPosition = new Card[12];

然后类似

public void setCardsAtEachPosition(Card[] valuesToSet) {
   // check to make sure valuesToSet has the required number of cards
   for (i = 0; i < cardsAtPosition.length; i++) {
       cardsAtPosition[i] = valuesToSet[i];
   }
}

Answer 6

对于您的示例案例，反射将是您的唯一选择。