Java/Android：匿名本地类与命名类

Question

我想问使用匿名类与命名内部类的良好实践是什么？

我正在编写一个 Android 应用程序，其中包含许多 UI 元素（按钮、文本字段等）。对于他们中的许多人，我需要某种类型的侦听器，因此在应用程序的 onCreate 中，我有一堆非常小的匿名类，例如：

someButton.setOnClickListener(
    new View.OnClickListener() {
        public void onClick(View v) {
            // do something...
        }
    }
);

每个这样的匿名类都有 5 - 20 行大 - 足够小并且非常适合《Java™ in a Nutshell》一书的推荐：

一般来说，如果出现以下情况，您应该考虑使用匿名类而不是本地类：

• 该类的主体非常短。
• 只需要该类的一个实例。
• 该类在定义后立即使用。
• 类的名称并不会让您的代码更容易理解。

但问题是，在我看来，onCreate 变得相当大，并且通过快速查看代码，代码变得更加难以阅读和理解。它仍然很容易理解，但太大了。

那么在这种情况下更好的做法是什么 - 有一堆小的内部子类，其中每个子类都很好地分开，但只使用一次，或者更好地继续使用匿名类？

Answer 1

我认为无论哪种方式都没有明确的答案。两种风格都很好用，这正是您喜欢的。

另一种选择是通过单个函数调用来获取

每个 onClick 的内容，这将使匿名类非常简短。 IE：

someButton.setOnClickListener(
    new View.OnClickListener() {
        public void onClick(View v) {
            doSomeButtonClick();
        }
    }
);


private void doSomeButtonClick() {
  // do something
}

Answer 2

将 onCreate() 重构为按常见功能分组的单独方法，以便您拥有逻辑单元。如果 GUI 很复杂，那么它稍后会对您有所帮助。

编辑：

另外，由于默认情况下，在匿名类中，代码格式化程序会缩进更多，因此您的行需要更短，以避免格式化程序将其分成几行，从而使其变长。这通常表明现在是提取类并为其命名的好时机。

Answer 3

我做桌面/Swing 应用程序，但我认为概念是相同的。

我更喜欢在一个类中处理所有事件，因此我创建一个 Controller 类并在该类中实现我需要的所有侦听器。然后，我在面板上为每种类型的侦听器创建一个方法。它将侦听器添加到面板中需要侦听的每个组件。我将面板作为第一个参数传递给 Controller 类的构造函数，并让它调用面板上的每个添加侦听器方法。然后我在实例化面板时实例化控制器。

这给我带来了几个优点：

• 所有事件处理代码都在一个类中。
• 事件之间的任何功能交叉都很容易捕获和处理。
• 任何受影响的状态都可以在一个类中存储和控制。
• 所有事件处理代码都与组件布局代码分开。
• View层（Panel）对Controller层一无所知。

话虽这么说，如果您挂接的事件都做简单的事情，并且这些事情不与其他事件冲突，那么使用匿名类并没有错。

Answer 4

我不知道社区是否认为这是最佳实践，但是当这种类变得太大时，我创建一个包侦听器并在其中创建我的类。如果内部类或匿名类在稍后的代码中妨碍您，为什么要使用它呢？

但大多数时候，如果这种类变得很大，那是因为你没有足够的委派。也许其他类应该有帮助你的监听器变得更轻的方法。

Answer 5

正如引文所暗示的，这是主观的。您需要自己弄清楚什么才算是“非常短”和“更容易理解”，以确定代码大小在什么时候跨越了界限，从作为匿名类有意义的东西到作为它自己的类有意义的东西单元。

任何人对此给出的任何答案都是基于他们自己对“短”是什么以及不再“短”需要多少行代码的主观衡量。

Answer 6

将这些项目放入命名类中有几个优点。

第一个是你提到的——它将使 onCreate() 方法更加简洁和易于理解。

第二是名称要能够快速识别哪个逻辑对应哪个按钮，这将使代码清晰

。第三是更容易职责分离。如果您决定使用 IoC 模型，那么您将可以更轻松地注入侦听器的命名实现。