如何重构这种类型的switch-case语句？

Question

基于minIndex，我设置了不同的视图可见性。 我认为可以将其重构为简单的方法。

switch (minIndex) {
    case 0:
        viewOne.setVisibility(View.VISIBLE);
        break;
    case 1:
        viewTwo.setVisibility(View.VISIBLE);
        break;
    case 2:
        viewThree.setVisibility(View.VISIBLE);
        break;
    case 3:
        viewFour.setVisibility(View.VISIBLE);
        break;
    case 4:
        viewFive.setVisibility(View.VISIBLE);
        break;
    case 5:
        viewSix.setVisibility(View.VISIBLE);
        break;
}

如何将该代码重构为更具可读性的代码？

Answer 1

如果数字与实际视图很好地匹配，则可以使用数组。

View[] views = new View[] {viewOne, viewTwo, viewThree, ...};
...
views[minIndex].setVisibility(View.VISIBLE);

Answer 2

我认为您已经对该网站的代码进行了匿名处理。 我本来建议@Sotirios Delimanolis回答了什么，但我认为您实际上做得太多了也太少了。

您已将控制器与视图的绑定过多，并且已将视图彼此的绑定过多。 为什么它们都应该在同一switch语句中？

如果添加其他视图，则需要更改多少？

相反，您应该让每个视图向持有minindex的对象注册一个PropertyChangeListener 。 顺便说一句，这真是个坏名字。 当对象更改minindex ，应将PropertyChangeEvent发送给所有侦听器。 每个视图的侦听器都应检查事件是否希望该视图可见。 如果是这样，则视图应自动唤醒。

class ViewController {
    private PropertyChangeSupport pcs = new PropertyChangeSupport();

    // delegate methods to add and remove listeners to pcs variable.

    private int viewIndex; // Changed for documentation.  Use String instead?

    public void setViewIndex(final int viewIndex) {
        int oldIndex = this.viewIndex;
        this.viewIndex = viewIndex; 
        pcs.firePropertyChange("viewIndex", oldIndex, this.viewIndex);
    }
}

class ViewOne {
    private ViewController vc;
    private final Integer myIndex = 1;

    // Constructor

    public void init() {
        // Never add this or an inner class to another object in a constructor.
        vc.addPropertyChangeListener("viewIndex",
            new PropertyChangeListener() {
                public propertyChange(final PropertyChangeEvent evt) {
                    if (myIndex.equals(evt.getNewValue()) {
                         setVisibility(View.VISIBLE);
                    }
                }
            });
    }
}

有关构造函数的警告是，如果你暴露this还是一个内部类的this在构造函数中的另一个对象，外部对象可以与交互this之前，它是完全构造。 您可以在构造函数中构建PCL。 使用另一种方法将其添加到控制器。