zoukankan      html  css  js  c++  java
  • 纠结才能写出好代码

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

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

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

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

    方法作用

    从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);
      }
    }
    

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

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

    (本文来自技术小黑屋)

  • 相关阅读:
    bzoj 2038 [2009国家集训队]小Z的袜子(hose)
    【NOIP2014模拟11.1B组】吴传之火烧连营
    【NOIP2014模拟11.1B组】蜀传之单刀赴会
    phpmystudy:mysql启动失败
    英文漏洞报告解读(一)——PHP 5.4.x < 5.4.32 Multiple Vulnerabilities
    brupsuit Compare 模块及其应用场景
    Android实现Banner界面广告图片循环轮播(包括实现手动滑动循环)
    android ViewPager实现的轮播图广告自定义视图,网络获取图片和数据
    Android首页轮播图直接拿来用
    java 调用webservice的各种方法总结
  • 原文地址:https://www.cnblogs.com/aikaifa/p/4664357.html
Copyright © 2011-2022 走看看