正确处理Ordered Broadcasts「建议收藏」

大家好，又见面了，我是你们的朋友全栈君。

原文：http://aoandroid.com/node/1414

最近，Android官方开发员博客（Developers Blog）登载了Google巴西分布的一名工程师（
Belo Horizonte ）的文章，介绍了正确使用Broadcast的概念和实例分析。该文章在概念上纠正了很多开发人员对Broadcast处理中存在的严重问题，并指出这反映了开发文档存在缺陷，没有解释清楚。

Broadcast是Android的一个很有用的概念。当系统产生某些事件的时候，apps可以对某些事件（ACTION）注册（register）后，通过BroadcastReceiver捕捉到这些Broadcasts，从而获得一些很重要的系统信息。电池状态就是一个例子，比如，ACTION_BATTERY_LOW。

Belo注意到有些开发人员没有使用正确方法调用BroadcastReveiver。比较有代表性的概念混淆是

Not ordered 相对于 Ordered Broadcasts

在Not-ordered模式，系统的Broadcasts是同时发布出来，所有receiver都可以得到该事件信息，互相并不影响。即，一个receiver不能影响其它receivers是否能获得该事件信息。上面电池的 ACTION_BATTERY_LOW 就是属于not-ordered事件。

在ordered模式，broadcasts是通过优先设置传达给不同receivers的，优先度在AndroidManifest文件中通过android:priority的intent-filter设置控制的。某个receiver可以通过BroadcastReceiver取消broadcast，取消后，比它优先度底的receivers将不再能得到该broadcast。一个典型的例子是，ACTION_NEW_OUTGOING_CALL。
 
每当用户拨打电话的时候，系统会发布（broadcast）这个 ACTION_NEW_OUTGOING_CALL 的事件。由于这是个ordered broadcast，app的不同receiver可以对它进行必要的控制。Belo给出一段暗含问题的代码
（注意，CallReceiver 必须首先在AndroidManifest里面关联ACTION_NEW_OUTGOING_CALL）

public class CallReceiver extends BroadcastReceiver {<br /> @Override<br /> public void onReceive(Context context, Intent intent) {<br /> // Original phone number is in the EXTRA_PHONE_NUMBER Intent extra.<br /> String phoneNumber = intent.getStringExtra(Intent.EXTRA_PHONE_NUMBER);<br /> if (shouldCancel(phoneNumber)) {<br /> // Cancel our call.<br /> setResultData(null);<br /> } else {<br /> // Use rewritten number as the result data.<br /> setResultData(reformatNumber(phoneNumber));<br /> }<br />} 

这段代码试图在获得
ACTION_NEW_OUTGOING_CALL后根据条件，或者拒绝拨打电话，或者让用户修改电话号码。

但是，只有在没有其它具备更高优先度的Receiver的时候，这段代码才会如预期的正确执行。如果有一个更高优先的receiver也关联了ACTION_NEW_OUTGOING_CALL，并且先于这段代码执行了，那么，有可能他们会用一样的原始号码，而没有正确的使用修改后的号码。

正确的做法应该是如下列代码那样：

public class CallReceiver extends BroadcastReceiver {<br /> @Override<br /> public void onReceive(Context context, Intent intent) {<br /> // Try to read the phone number from previous receivers.<br /> String phoneNumber = getResultData();<br /> if (phoneNumber == null) {<br /> // We could not find any previous data. Use the original phone number in this case.<br /> phoneNumber = intent.getStringExtra(Intent.EXTRA_PHONE_NUMBER);<br /> }<br /> if (shouldCancel(phoneNumber)) {<br /> // Cancel our call.<br /> setResultData(null);<br /> } else {<br /> // Use rewritten number as the result data.<br /> setResultData(reformatNumber(phoneNumber));<br /> }<br />} 

这段代码事先判断了是否有另外receiver可能已经生成了一个拨号号码，没有的话，才调用intent extra的
EXTRA_PHONE_NUMBER。

这个问题有多严重？

Belo声称，很多apps被发现，使用priority 0的NEW_OUTGOING_CALL的receivers，所有这些receivers都如上述第一个代码例子那样忽略了可能的高优先receivers已经先期执行，并同样关联了NEW_OUTGOING_CALL。当然，你也可以把所有关联了NEW_OUTGOING_CALL的receivers都设在priority 0上，但是这又违反了Android拨打电话的代码处理原则，如果不能访问这个链接，可以参考SDK带的文档。

该处理原则是：

‘为了保持一致性，任何意在阻止向外拨打电话的receiver必须设0优先 （priority 0），以确保该receiver获得最后输入的电话号码。任何意在修改拨打号码的receiver应该设正数优先度，即大于零的正数。负数优先度是给系统保留的，任何apps用负数优先度可能引发问题’

Belo的结论是，很多应用软件存在这个问题，呼吁大家抓紧检查，即时解决这类问题，确保Android应用软件的整体质量。