在java中将动作存储为数据的最佳方法是什么？

Question

我在一个 android 应用程序中有一个类（活动），上面有一堆示例数学问题。您可以通过按下更改问题计数器并显示新问题的按钮来移动到不同的问题。您可以通过单击“显示工作”按钮来查看如何解决问题，该按钮会在屏幕上显示大量信息。我的问题是我有一堆看起来像这样的方法：

public void showWorkButtonClicked()
{
    if (questionCounter == 1)
        showWork1();
    else if (questionCounter == 2)
        showWork2();
    else if (questionCounter == 3)
        showWork3();
    //for how ever many questions are available
}

显然，这些 if 语句是糟糕的设计。我可以将每个函数所需的数据存储在一个类中，这在这种情况下可能会起作用（但可能只会让事情变得混乱），但是如果每个 showWork 方法都足够独特以至于不切实际怎么办。我知道如果我是在 C# 中制作应用程序，我可以简单地将代表放入一个列表中，并且会有一个优雅的解决方案。如果有人有更好的解决方案（理想情况下使用更少的代码），我很想听听。

Answer 1

使用Map<Integer, Runnable> 存储您的操作：

private static Map<Integer, Runnable> actions = new HashMap<Integer, Runnable>() {{
    put(1, () -> showWork1());
    put(2, () -> showWork2());
    put(3, () -> showWork3());
}};

然后查找它们：

public void showWorkButtonClicked() {
    actions.getOrDefault(questionCounter, () -> {}).run();
}

在这里，如果没有对数字执行任何操作，我将使用可运行的“什么都不做”来避免 NPE。或者，您可以：

public void showWorkButtonClicked() {
    Optional.of(questionCounter)
       .map(actions::get)
       .orElseThrow(IllegalArgumentException::new)
       .run();
}

Answer 2

您不应在代码中放入任何数据，也不应使用专用函数来显示不同的内容。如果您想根据计数器显示数据，请使用 List 或 Map 来存储它并在单个函数中读取它以进行显示。