Java:MVC 和 Controller 的匿名类

Question

我有一个关于 MVC 模式的问题。我正在尝试将它用于桌面项目(即不适用于网络应用程序)，在 IntelliJ IDEA 中使用 Java。我正在尝试根据 MVC 模式创建模型、 View 和 Controller 。我的问题是:从 Java 的角度来看，标记方法已经充当了 Controller ，实际上不需要创建一个显式的 Controller 类作为单独的类，这是否安全？像这样:

或者我是否必须创建一个显式的 CalculatorController 类，如下所示:

请记住，我的任务是保留 MVC 模式，其中 M、V 和 C 的任务被明确划分。在这 2 个选项中，哪一个保留了 MVC 模式？只有第二个，还是两者都一样？谢谢!

Answer 1

第二个选项

如果您使用 Controller 作为匿名内部类，您将如何测试您的 Controller 层？如果您不分离 Controller ，您将无法出于测试目的进行模拟。

即使您不想编写测试，第一个选项也会在 View 和 Controller 之间创建真正的硬耦合。根本没有分离!

最好在实现之前显式创建 Controller 并创建接口(interface)。它会让 TDD 变得更加容易，同时你也会在你的项目中做出更好的代码设计。

但请记住，这只是我的意见。始终进行研究，以确保您的项目决策是正确的并且适合您。

看看这篇文章，它有一个非常简单的模式实现:https://www.tutorialspoint.com/design_pattern/mvc_pattern.htm

编辑

让我们比较一下这两种方法。

1. 可测试性

内部类:您无法测试控制层。您的所有 Controller 都是匿名内部类。所以你无权访问 then，从而无法进行测试。在 MCV 架构中，测试 Controller 非常重要。结论:对于测试来说确实很糟糕。

显式类:您可以模拟 View 和模型，以便测试 Controller 。测试 View 和模型时遵循相同的逻辑。结论:易于测试。

可维护性

内部类:所有 Controller 逻辑都位于匿名内部类(或 lambda)内。相信我，你不会想要这样的。您的应用程序的所有复杂性都将位于一个不应该变大的类中。一个简单的例子:

//I need to get the people older than 50 years old, present to the user and wait for a click. If the list is empty, do something, if it's not do something else.

      buttonRefresh.onClickListener(() -> {
          List<Person> people = model.requestPeople();

          people = people.stream()
                      .filter(p -> p.getAge > 50) 
                      .collect(Collectors.toList())

          [code to create an adapter]

          if(people.isEmpty()){
             [do something...]
          } else{
             [do something else...]
          }

      })

这是一个非常简单的代码，但它已经变得令人困惑。如果我的 if 和 else 中有另一个 lambda 怎么办？匿名内部类中的代码越大，情况就越糟。结论:这不利于可维护性，因为您的代码将在错误的位置增长。

显式类:您的代码可以在类内扩展。您可以在那里添加依赖项、创建方法、拥有内部类...

职责分离

内部类:确实没有 Controller 层。所有 Controller 代码都位于 View 类内部，因此 View 和 Controller 基本上是相同的类。因此，您将拥有一个非常庞大的 View 类，其中包含大量困惑的代码。结论:不好，应用程序的几乎所有代码都将位于 View 类中。

显式类: Controller 和 View 有非常清晰的分离。您的 View 将只有用于更新 View 的方法。

(我知道您将在 View 类中使用基本上 1-3 行代码创建很多方法。但这比 View 和控制逻辑位于同一位置的庞大方法要好。)

我是这么想的。但您始终可以尝试这两种方法并比较结果!