5

在我的 Android 项目中,我定义了一些回调来操作按钮单击、连接事件或 UI 事件,例如 Dilaog.onShow()。出于演示目的,我选择了一个必须从一些 Activity 代码启动的 Runnable 接口。使用 Java,我有不同的方式来表达自己。

一种模式是使用匿名类

runOnUiThread(new Runnable() { 
    public void run() { 
        doSomething(); 
    }
});

private void doSomething() {
}

另一个 - 定义一个内部私有类,即

private DoSomething implements Runnable {
    public void run() { 
        // do something; 
    }
}
...
runOnUiThread(new DoSomething());

还有一个 - 使用私人成员,如下所示:

private final Runnable doSomething = new Runnable() {
    public void run() { 
        // do something; 
    }
}
...
runOnUiThread(doSomething);

这是我最喜欢的另一个,因为一方面它实际上不会构造对象,除非有人真正使用它,因为它避免了额外的类,因为它可以在需要时接受参数。

private Runnable doSomething() { 
    return new Runnable() {
        public void run() { 
            // do something; 
        }
    }
}
...
runOnUiThread(doSomething());

我不是在寻找品味或宗教信仰的论据,而是代码的可维护性和性能。我希望收到可以帮助我发展自己的偏好的提示和建议,可能 - 根据给定情况的不同偏好。

剧透:

Java 的进步已经使这个问题过时了,​​请参阅接受的答案。

4

5 回答 5

2

像@Manuel Silva 和@Toby Champion 一样,我不喜欢匿名的内部课程。它们有点难以阅读,不是很“OO”,因为它们不能扩展,不能有 DIP、setter 或任何调整行为的东西,等等……而且它们往往最终违反了 DRY 原则当您在 27 个不同的地方添加相同的代码时。

我倾向于使用私有成员(您的选项#3),或通常命名为 getAsRunnable() 的私有函数(您的第 4 种样式)。

于 2011-12-12T18:45:20.833 回答
2

我不相信有任何惯用的方式来处理回调。

我通常首先内联一个匿名类。当方法变得太大时,我将类创建提取到一个单独的函数中。当类变得太大时,我提取到它自己的文件。

如果您使用像 Eclipse 这样的 IDE,您可以自动安全地执行所有这些重构。

于 2011-12-12T18:30:22.683 回答
1

我对 Android 很陌生,但是匿名课程让我感到恶心,而且似乎你仍然可以替代 runOnUiThread:AsyncTask,在这里讨论: runOnUIThread 问题

于 2011-12-12T17:28:12.010 回答
0

从我的角度来看,匿名类确实降低了可读性。因为 Ui 代码通常非常冗长,为每个按钮添加匿名回调可能会导致非常大的类。因此,我使用的是内部私有类。

于 2010-09-15T14:52:27.993 回答
0

今天,当 Java 8 几乎可用于 Android 时,Android Studio自动假装 lambdas 已被支持,匿名类(#1)解决方案似乎是显而易见的选择:

倒塌

展开

于 2016-01-07T04:53:10.623 回答