是否可以循环设置器和吸气剂？

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)());
}

我确信在某个地方之前会有人问这个问题，但Google和SO Search都没有提出否定证据。

感谢您快速确认！

Answer 1

没有反思就没办法在Java中专门做到这一点，我认为这不值得。 这看起来更像是一个提示，您应该重构getcard函数以获取整数参数。 然后你可以循环。

Answer 2

你可以通过反射来做到这一点，但这会很麻烦。 更好的方法可能是制作通用的setPos（）和getCard（）方法，您可以在其中传递当前项的索引。

Answer 3

这是一个简单的代码片段，演示如何使用反射循环访问某个对象的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 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

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