重构 for 循环，每次迭代设置不同属性

Question

作为代码审查的一部分，我看到了以下 Java 代码，坦率地说，它非常可怕。但是，我不知道重写它的最佳方法是什么。

我们有一个始终以相同顺序 (0-5) 排列的列表，每个索引对应于需要设置的不同属性。使用 groovy 会很容易，因为它是 myObject."setLine${i}" 或类似的但我们如何才能在普通的 Java?

中实现相同的效果

    SomeObject myObject = new SomeObject();
    List<String> lines = new ArrayList<>(){{
    add("Line 1");
    add("Line 2");
    add("Line 3");
    add("Line 4");
    add("Line 5");
    add("Line 6");
}};

for(int i=0; i< lines.size(); i++){
    if(!StringUtils.isBlank(lines.get(i))){
        String line = lines.get(i);
        // line1
        if(i==0){
            myObject.setLine1(line);
        }
        // line2
        if(i==1){
            myObject.setLine2(line);
        }
        // line3
        if(i==2){
            myObject.setLine3(line);
        }
        // line4
        if(i==3){
            myObject.setLine4(line);
        }
        // line5
        if(i==4){
            myObject.setLine5(line);
        }
        //line6
        if(i==5){
            myObject.setLine6(line);
        }
    }
}

我知道 for:each 可以为我们提供索引，但我查询的是 "dynamic" 方法调用。欢迎任何建议——我正在考虑 MyObject class 应该有一个方法来接收地址字符串列表，这样它就不会通过向调用者公开行数来紧密耦合（即如果我们 add/remove 一个 "setLineX" 方法，我们不想更新调用者）。

Answer 1

第一期在 myObject。如您所见，为每一行都设置方法是乏味的。像这样的怎么样？

class MyObject{
    private Map<Integer,String> lines = new HashMap<>();

    public void setLine(int lineNumber, String line){
        lines.put(lineNumber,line);
    }

    public String getLine(int lineNumber){
        lines.get(lineNumber);
    }

}

现在你可以简单地做

for(int i=0;i<lineListSize;i++){
    myOjbect.setLine(i,lineList.get(i));
}

编辑：正如 Fabian 刚刚指出的那样，您可以对 List.

做同样的事情

class MyObject{
    private List<String> lines = new ArrayList<>();

    public void setLine(int lineNumber, String line){
        lines.add(lineNumber,line);
    }

    public String getLine(int lineNumber){
        lines.get(lineNumber);
    }

}

EDIT2：我永远不能独自离开。您可以使用

使传输更加清晰

class MyObject{
    ... 
    // Collection, getter and setter omitted
    ...
    public void addAllLines(List<String> lines){
       // implementation varies depending on collection,
       // but List would simply be "this.lines.addAll(lines);" 
    }

    public List<String> getAllLines(){
       // Again, depends on the type of Collection used by 
       // this instance.
       // Probably want to return a copy if thread safety is an issue.
    }
}

Answer 2

如果您将方法调用包装在消费者中（如果您不使用 Java 8，请定义您自己的方法）并传入 myObject 和 String 的元组。由于消费者是无状态的，您可以将它们保存在静态列表中。

private static List<Consumer<Tuple<SomeObject, String>> consumers = Arrays.asList(
    new Consumer<>() {
        public void consume(Tuple<SomeObject, String> tuple) {
            tuple._1.setLine1(tuple._2);
        }
    },
    new Consumer<>() {
        public void consume(Tuple<SomeObject, String> tuple) {
            tuple._1.setLine2(tuple._2);
        }
    },
    ... and so on
);

你的循环就变成了

final int numConsumers = consumers.size();
for(int i=0; i < numConsumers && i < lines.size(); i++){
    String line = lines.get(i);
    if(!StringUtils.isBlank(line)){
        consumers.get(i).consume(new Tuple<>(myObject, line));
    }
}

作为参考，元组只是一个class来包装多个值

public class Tuple<A, B> {
    public final A _1;
    public final B _2;

    public Tuple(A a, B a) {
        this._1 = a;
        this._2 = b;
    }
}

如果你有 Java 8，消费者定义会急剧缩小：

private static final List<BiConsumer<MyObject, String>> CONSUMERS = Arrays.asList(
        (o, s) -> o.setLine1(s),
        (o, s) -> o.setLine2(s),
        (o, s) -> o.setLine3(s)

);

并且消费者的使用不再需要元组：

consumers.get(i).consume(myObject, line);

整个事情现在看起来像这样：

public class ExampleClass {

    private static final List<BiConsumer<MyObject, String>> CONSUMERS = Arrays.asList(
        (o, s) -> o.setLine1(s),
        (o, s) -> o.setLine2(s),
        (o, s) -> o.setLine3(s)
        // and so on
    );

    public void exampleMethod() {
        SomeObject myObject = new SomeObject();
        List<String> lines = Arrays.asList("Line 1", "Line 2", "Line 3", "Line 4", "Line 5", "Line 6");

        for(int i = 0, numConsumers = consumers.size(), numLines = lines.size(); i < numConsumers && i < numLines; i++){
            String line = lines.get(i);
            if(!StringUtils.isBlank(line)){
                consumers.get(i).consume(myObject, line);
            }
        }
    }
}

重构 for 循环，每次迭代设置不同属性

Refactoring for loop with each iteration setting a different property

java

refactoring

coding-style

重构 for 循环，每次迭代设置不同 属性

Refactoring for loop with each iteration setting a different property

java

refactoring

coding-style

重构 for 循环，每次迭代设置不同属性