纠结才能写出好代码

程序员的代码修炼应该有两个目标,一个是代码的执行效率,另一个是代码的可读性。朝着这两个目标努力的人很多,但是能够达到目标的人很少。

以前部门老大曾经说过一句话,大概意思是,想要写出好的代码,就要在写的时候纠结一会儿。

其实编程本身就是一个寻找最优解的哲学问题。解决一个问题,有时候我们要适度纠结,来使我们的代码更加可读,效率更高。

以下以一个简单的例子,列举一下一个小纠结的问题。我们这里尽量将重点放在如何改善代码等问题上。

方法作用

从WebView中读取Favicon,并且返回,如果webview为null,或者web.getFavicon()为null,则返回默认的一个图标。

原始的代码

1 2 3 4 5 6 7

public Bitmap getFavicon(final WebView webview) { if (null != webview && null != webview.getFavicon()) { return webview.getFavicon(); } else { return BitmapFactory.decodeResource(getResources(), R.drawable.default_favicon); } }

上面存在一种情况,即null != webview && null != webview.getFavicon() 这个条件成立的时候,其实还是有改善的地方,因为这种情况下webview.getFavicon()会存在两次调用,一次作为判断条件需要,另一次是返回引用需要。

这样改好么

1 2 3 4 5 6 7 8 9 10 11 12

public Bitmap getFavicon(final WebView webview) { if (null != webview) { Bitmap favicon = webview.getFavicon(); if (null != favicon) { return favicon; } else { return BitmapFactory.decodeResource(getResources(), R.drawable.default_favicon); } } else { return BitmapFactory.decodeResource(getResources(), R.drawable.default_favicon); } }

这种情况下,没有了上面存在的两次调用的问题了,但是关于返回默认值是存在两处相同的代码,如果需要修改默认的图标时,有可能需要修改两次。还是不够好。

这个总可以了吧

1 2 3 4 5 6 7 8 9 10 11 12 13

public Bitmap getFavicon(final WebView webview) { Bitmap defaultFavicon = BitmapFactory.decodeResource(getResources(), R.drawable.default_favicon); if (null != webview) { Bitmap favicon = webview.getFavicon(); if (null != favicon) { return favicon; } else { return defaultFavicon; } } else { return defaultFavicon; } }

这个确实没有上面的两个问题了,但是还是有不完美的地方,就是如果webview不为null,并且webview.getFavicon()也不为null,那么Bitmap defaultFavicon = BitmapFactory.decodeResource(getResources(), R.drawable.default_favicon);实际上是多余的。

终于改好了

1 2 3 4 5 6 7 8

public Bitmap getFavicon(final WebView webview) { Bitmap favicon = null; if (null != webview && (favicon = webview.getFavicon()) != null) { return favicon; } else { return BitmapFactory.decodeResource(getResources(), R.drawable.default_favicon); } }

对,上面的代码没有一处多余,并且也是最简单的。

更好的代码

这个实现来自评论,相比其他实现更好。

1 2 3 4 5 6 7 8

public Bitmap getFavicon(final WebView webview) { Bitmap favicon = null; if (null != webview) { favicon = webview.getFavicon(); } return null != favicon ? favicon : BitmapFactory.decodeResource(getResources(), R.drawable.default_icon); }

从现在开始,写代码的时候纠结吧。不要仅仅为了实现,更不要追求代码的数量,培养你的代码洁癖,做个代码艺术家。

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

发表于

我来说两句

0 条评论
登录 后参与评论

相关文章

来自专栏GIS讲堂

Arcgis For Android之GPS定位实现

2762
来自专栏Android开发指南

11.菜单界面、详情界面

2957
来自专栏Phoenix的Android之旅

关于ListView的那些坑

做过Android的同学应该都有适配安卓兼容性的问题,今天我们来说一个常见却又经常被忽略的问题。

731
来自专栏Sorrower的专栏

界面无小事(二): 让RecyclerView展示更多不同视图

1322
来自专栏何俊林

Android Multimedia框架总结(二十三)MediaCodec补充及MediaMuxer引入(附案例)

前言:前面几章都是分析MediaCodec相关源码,有收到提问,说MediaCodec到底是硬解码还是软解码?看下今天的Agenda: MediaCodec到...

33110
来自专栏向治洪

android wheelview实现三级城市选择

很早之前看淘宝就有了ios那种的城市选择控件,当时也看到网友有分享,不过那个写的很烂,后来(大概是去年吧),我们公司有这么一个项目,当时用的还是网上比较流行的那...

4296
来自专栏肖蕾的博客

自定义BaseAdapter完美解决ListView异常:java.lang.IllegalStateException这是代码使用方法原理另外

1458
来自专栏向治洪

仿微信聊天表情发送

如题,这是公司项目的一个功能模块,先上个效果图: ? 其次大致说说原理: 1,首先判断输入的字符,是否包含表情的文字,比如   ?  这个表情对应的文件名为 e...

5697
来自专栏Phoenix的Android之旅

关于UI的容易弄错的入门知识

写过 UI的同学一定知道 layout_gravity和 gravity这两个属性,

682
来自专栏Android干货

关于Activity销毁,而绘制UI的子线程未销毁出现的问题

3716

扫码关注云+社区

领取腾讯云代金券