炮轰 杨丰盛的《Android应用开发揭秘》

看到javaeye很多人在吹嘘、推荐杨丰盛的《Android应用开发揭秘》，个人就买了一本。

首先拿到书时，再次感受到机工出版社的书的纸张独有的“薄如蝉翼、黄如粪便”的特征，不过想着javaeye一众人对书中内容的推荐，纸张的质量我忍了。

接着看书，不过我慢慢发现书中一堆晦涩不清的表述——这种表述可能是我自己的中文理解问题，算了不说了。

但书中还充斥着一堆垃圾代码，看下面书中代码（P270，免得说我诬陷）：

	private Runnable _discoveryWorkder = new Runnable() {
		public void run() 
		{
			/* 开始搜索 */
			_bluetooth.startDiscovery();
			for (;;) 
			{
				if (_discoveryFinished) 
				{
					break;
				}
				try 
				{
					Thread.sleep(100);
				} 
				catch (InterruptedException e){}
			}
		}
	};

上面_bluetooth.startDiscovery();就是开始进行蓝牙搜索了，

接下来那段死循环就纯属多余！！代码判断_discoveryFinished为true时结束死循环，但这个死循环什么都没做啊。
事实上，完全可以把这段垃圾代码删除。

我不确定杨丰盛到底会不会Android，但我觉得这java代码写得也太垃圾了。

书中大量这种的代码，让人不堪入目。前几章，看过也就算了，后来发现越来越多
上来发发牢骚。

评论

10 楼 cczlg 2012-03-28  

再补充一句，既然对技术要求这么严谨，那么，严谨的说：BroadcastRecevier不是监听蓝牙搜索结束的，而是每发现一个设备就调用一次。但是可以在这里判断action是否等于BluetoothAdapter.ACTION_DISCOVERY_FINISHED来得知搜索过程是否结束。前提还得是注册过BluetoothAdapter.ACTION_DISCOVERY_FINISHED的IntentFilter。
我猜_discoveryFinished就是通过这种方法设置为true的。

9 楼 cczlg 2012-03-28  

这本书我没看过，不能随便说好坏。但你提到的这个例子我觉得它有可取之处。且不考虑CPU和耗电量，就从实现功能上说。我在做一个功能，发现蓝牙设备之后才能进行下一步操作，典型的单线程同步思维。当初Android团队可能是考虑为了不影响UI主线程与用户交互，把startDiscovery做成一个异步方法。那么我这个需求，怎么实现呢？可能还有其它方法能实现，但我觉得杨先生这个方法简单、粗暴、有效。
另外，别有事没事就“炮轰”，人家就写本书而已，有那么大仇么。

8 楼 zhaopian16 2011-04-02  

呵呵，这兄弟挺有意思！上来就炮轰！android垃圾书多了去了，为什么不去轰那些直接翻译API，粘贴API里Demo凑页数的书？非要挑一本写的不错的的来轰？想出名？搞炒作？赚点击率？
人尚且无完人，人家写的是“书”不是宪法，你搞搞清楚！！！

cloixio您老人家晚上好好睡一觉，酝酿一下，明早早起去炮轰红楼、三国、水浒……这样来的快些.

7 楼 cloixio 2011-03-31  

我们再来看杨丰盛这个案例中SamplesUtils类中的代码：
Java代码 
    private static void indeterminateInternal(final Context context, final Handler handler, String message, final Runnable runnable,  
        OnDismissListener dismissListener, boolean cancelable)  
    {  
 
        final ProgressDialog dialog = createProgressDialog(context, message);  
        dialog.setCancelable(cancelable);  
 
        if (dismissListener != null)  
        {  
 
            dialog.setOnDismissListener(dismissListener);  
        }  
        dialog.show();  
 
        new Thread() {  
 
        @Override  
        public void run()  
        {  
         runnable.run();  
 
            handler.post(new Runnable() {  
 
                public void run()  
                {  
 
                    try  
                    {  
 
                        dialog.dismiss();  
                    }  
                    catch (Exception e)  
                    {  
 
                        ; // nop.  
                    }  
 
                }  
            });  
        };  
        }.start();  
    }  


注意粗体字代码部分，这行代码仿佛“神来一笔”，杨丰盛执行线程的方法居然是调用Runnable对象的run方法。

接着再看dialog.dismiss(); 这行代码，就是这行代码，它会让“scanning”对话框消息消失。这就是典型的“用单线程思维编写多线程代码”～
正确的做法：这里直接调用new Thread(runnable).start();启动多线程——把dialog.dismiss();交给监听蓝牙搜索结束的BroadcastRecevier去做。

6 楼 yarin 2011-03-31  

我不知道何种原因，cloixio兄现在不出来回帖了！
因为cloixio兄始终对这个问题纠结着，我30日找到以前的代码运行测试了N次，结果和kangfu兄说的情况一样。
鉴于此次cloixio兄的“炮轰”，对作者及出版社影响甚大，在我得测试结果之后，我并没有上来回帖，而是联系了出版社的编辑，希望编辑能先私下联系cloixio兄本人，同时（编辑也另外又找了几个Android方面比较有经验的朋友进行了测试，结果都是不能按照cloixio兄的做法来去掉那个所谓的“多余的循环”），再上来解决此问题，但是到目前为止未果。
我一直关注着此贴的最新消息，现在是在忍受不了了，加上kangfu兄也再次验证了正确无误的结果，所以我得上来回复了，希望cloixio兄能够出现回答此问题，否则会误导其他读者。
另外，正如编辑所说的，本书并不一定是完美的，但也不至于完全没有任何价值，也不是cloixio兄所谓的，javaeye上大部分都是人云亦云的“鹦鹉”，我一直知道javaeye上有着很多国内外知名的大师。
最后感谢kangfu兄以及所有为正确结果而讨论验证的师兄弟，包括对技术非常认真严肃的cloixio兄。

现在已经越来越多的人测试出结果，如果按你说的去掉循环，搜索对话框根本就弹不出来，或者是只弹出很短的时间。

5 楼 cloixio 2011-03-28  

yarin 写道

只能说明这位仁兄喜欢自己的设备的CPU使用率永远都是100%的。
最大限度的发挥设备的能力。
帮助用户消耗一部分电量。
非常抱歉，本书没有满足你上诉要求！

既然作者用这么NB哄哄的口吻来解释，
我就再发表一下我的看法：

看来作者认为这个死循环是可以让_bluetooth.startDiscovery();这个耗时操作暂停下来，让CPU空闲出来。

看来有必要补充一下多线程的知识了：

这个程序定义了一个线程来执行蓝牙搜索，这条线程只做了2件事情：
1. _bluetooth.startDiscovery();    ——这条代码是个耗时操作。
2. 死循环。

第一种假设：假如_bluetooth.startDiscovery(); 是同步执行的，也就是程序必须等这条代码执行完成后，才会去执行死循环，那死循环能暂停_bluetooth.startDiscovery();这个耗时操作吗？能让CPU空闲出来吗？
 
幸好Android没有这么SB！因为这个操作完全可能会搜索1～3分钟。这里是异步执行的——也就是说死循环与_bluetooth.startDiscovery();其实是并发执行的。那么根据多线程的“基础知识”：当死循环线程
执行Thread.sleep(100)时，系统完全可能会被调度到去执行 _bluetooth.startDiscovery();这个耗时操作，那么死循环有意义吗？

4 楼 yarin 2011-03-24  

你要“污蔑”作者一个人无关紧要，javaeye上如此多高手总不能被你一竿子解决了吧。

3 楼 yarin 2011-03-24  

就在javaeye上给你转一段过来你看看：

你把sleep删掉在Windows上跑跑然后打开任务管理器看看CPU是不是100%，加上sleep()是不是立刻变成0%-1%
不需要Android，随便写个Java程序就可以测试

2 楼 yarin 2011-03-24  

只能说明这位仁兄喜欢自己的设备的CPU使用率永远都是100%的。
最大限度的发挥设备的能力。
帮助用户消耗一部分电量。
非常抱歉，本书没有满足你上诉要求！

1 楼 wdjezym 2011-03-23  

没看过书,不懂蓝牙的菜鸟飘过

那段是等待蓝牙连接完成吧
连接完成前一直sleep