设计之美

到处都在谈论 UI 的美感，仿佛 “美” 在软件工程中的定义就要落到界面上面。实际美的存在是广义的，包括架构设计，包括代码建设，包括接口定义，不妨在更多的场合引入对美的评审。软件本身就是一种艺术品，而程序员，应当是被赋予创造力的艺术工作者。

 

仅看这两张图，你觉得哪一张会更美一些？

我相信大多数人会选择第一张，因为后面那张图显得头重脚轻，事实上，后者也确实是一个短命的版本，只存活了不到半年的时间。这两张图，正出自淘宝发展的一个阶段（来自淘宝赵超的博客）。而且，进一步观察发现，对于许多设计图来说，狭窄的汇聚点往往成为性能的瓶颈。

另一个设计上典型的丑陋是混乱，如下面的设计图：

我不相信看到了这样的关系图的人会觉得这是一个好的设计。有人说 “简单就是美”，兴许有人不同意，但是太过复杂一定是丑陋的。

代码层面的美亦然。先来看这样一段代码：

#i nclude <stdio.h>
main(t,_,a)char *a;{return!0<t?t<3?main(-79,-13,a+main(-87,1-_,
main(-86,0,a+1)+a)):1,t<_?main(t+1,_,a):3,main(-94,-27+t,a)&&t==2?_<13?
main(2,_+1,"%s %d %d\n"):9:16:t<0?t<-72?main(_,t,
"@n'+,#'/*{}w+/w#cdnr/+,{}r/*de}+,/*{*+,/w{%+,/w#q#n+,/#{l+,/n{n+,/+#n+,/#\
;#q#n+,/+k#;*+,/'r :'d*'3,}{w+K w'K:'+}e#';dq#'l \
q#'+d'K#!/+k#;q#'r}eKK#}w'r}eKK{nl]'/#;#q#n'){)#}w'){){nl]'/+#n';d}rw' i;# \
){nl]!/n{n#'; r{#w'r nc{nl]'/#{l,+'K {rw' iK{;[{nl]'/w#q#n'wk nw' \
iwk{KK{nl]!/w{%'l##w#' i; :{nl]'/*{q#'ld;r'}{nlwb!/*de}'c \
;;{nl'-{}rw]'/+,}##'*}#nc,',#nw]'/+kd'+e}+;#'rdq#w! nr'/ ') }+}{rl#'{n' ')# \
}'+}##(!!/")
:t<-50?_==*a?putchar(31[a]):main(-65,_,a+1):main((*a=='/')+t,_,a+1)
  :0<t?main(2,2,"%s"):*a=='/'||main(0,main(-61,*a,
"!ek;dc i@bK'(q)-[w]*%n+r3#l,{}:\nuwloca-O;m.vpbks,fxntdCeghiry"),a+1);}

我不相信你能一下看懂。但是它给你的第一感觉是什么？“极其混乱”，对不对？混乱而且无比丑陋。事实上，它正是国际混乱 C 语言大赛 1988 年的获奖作品。

再来看看 OSCache 中的一段代码：

if (this.isStale(cacheEntry, refreshPeriod, cronExpiry)) {
    EntryUpdateState updateState = getUpdateState(key);
    try {
        synchronized (updateState) {
             if (updateState.isAwaitingUpdate() || updateState.isCancelled()) {
             updateState.startUpdate();
             if (cacheEntry.isNew()) {
                 accessEventType = CacheMapAccessEventType.MISS;
             } else {
                 accessEventType = CacheMapAccessEventType.STALE_HIT;
             }
             } else if (updateState.isUpdating()) {
                 if (cacheEntry.isNew() || blocking) {
                     do {
                         try {
                             updateState.wait();
                         } catch (InterruptedException e) {
                         }
                     } while (updateState.isUpdating());
                     if (updateState.isCancelled()) {
                         updateState.startUpdate();
                         if (cacheEntry.isNew()) {
                             accessEventType = CacheMapAccessEventType.MISS;
                         } else {
                             accessEventType = CacheMapAccessEventType.STALE_HIT;
                         }
                      } else if (updateState.isComplete()) {
                         reload = true;
                      } else {
                         log.error("Invalid update state for cache entry " + key);
                      }
                 }
             } else {
                 reload = true;
             }
         }
     } finally {
         releaseUpdateState(updateState, key);
     }
}

这样的代码片段虽然没有那么混乱，但是远谈不上美，看起来还是很辛苦，其中复杂的状态变量依然让人觉得很难理解。这段代码片段正是来自 OSCache 的 Cache.java，也是 OSCache 中 bug 很多，也遭人非议最多的 NRE（NeedsRefreshException）机制中的核心代码，事实上，代码的 contributor 们已经意识到其中的坏味道了。

有意思的是，OSCache 代码中，经常可以看得到程序员的自我反省，比如有这样的注释：

        // TODO - Fix me please!
        // Hack! This is nasty - if two sessions are created within a short ...

不管是设计、编码还是接口定义，追求清晰、简洁，追求其中之美，似乎正是给追求卓越的程序员以精益求精之导向的方式。

文章未经特殊标明皆为本人原创，未经许可不得用于任何商业用途，转载请保持完整性并注明来源链接 《四火的唠叨》