Question

对于我的Android项目，我准备了一个专门的类来监听onClick()，onTouch()和onItemSelected()事件（它是MVC的控制器），屏幕上有几个Spinner对象。在这种情况下，当触发事件时，算法必须知道1）选择了Spinner和2）选择了Item。因此，它会产生如下所示的嵌套开关。

@Override
public void onItemSelected(AdapterView<?> parent, View v, int position, long id) {
    switch(parent.getId()) {
    case R.id.spn1:
        switch (position) {
        case 0:
            // code
            break;
        case 1:
            // code
            break;
        case 2:
            // code
            break;
        }
    break;
    case R.id.spn2:
        switch (position) {
        case 0:
            // code
            break;
        case 1:
            // code
            break;
        case 2:
            // code
            break;
        }
    break;
    case R.id.spn3:
        switch (position) {
        case 0:
            // code
            break;
        case 1:
            // code
            break;
        case 2:
            // code
            break;
        }
    break;
    default:
    }
}

下面发布的样本让我很头疼。我一定很想知道是否有某些方法（使用OOP，也许）来避免这个大混乱。

Answer 1

我会为每个Listener创建一个单独的View类。

public class MyClass extends Activity{

    @Override
    public void onCreate(Bundle savedInstanceState){
        super.onCreate(savedInstanceState);

        /* Necessary View fetching code omitted */
        mySpinner.setOnItemSelectedListener(new MySpinnerOnItemSelectedListener());
    }

    private class MySpinnerOnItemSelectedListener implements OnItemSelectedListener{

        @Override
        public void onItemSelected(AdapterView<?> parent, View v, int position, long id){ 
            /* Here you only have your inner switch statement */
        }
    }
}

Answer 2

您可以为每个微调器使用单独的OnItemSelectedListener。

如果你不想这样，因为他们一起共享一些代码，我仍然会考虑在onItemSelected方法中使用一个开关来确定单击了哪个微调器，然后为每个微调器调用一个单独的方法。旋转器，用于实际切换物品。

但只要他们不共享任何代码，我建议为每个微调器使用不同的侦听器。我也会将这些监听器创建为内部类，尽管它会使你的类膨胀一些。

您也可以使用id参数，只要您的适配器使用所有微调器之间不同的stable ids。大多数情况并非如此，因此您仅限于以上两种可能性。

Answer 3

这取决于你想对这些微调器位置做什么。如果你对每个微调器位置做了完全不同的事情，那么你就无法进一步优化。

在这种情况下，您可以做的最好的事情是为每个项目创建并调用一个私有函数（或者对于每个微调器，如果一个微调器在所有位置中执行类似的操作）。

不要认为嵌套开关是坏的，它们执行起来快，易于阅读和维护。你应该避免的是在每个case块中放入太多代码。你应该改为私人职能。

onItemSelected（）与嵌套开关...如何避免它？

3 个答案: