- 在线时间
- 31 小时
- 最后登录
- 2014-1-6
- 注册时间
- 2012-11-27
- 听众数
- 11
- 收听数
- 0
- 能力
- 0 分
- 体力
- 391 点
- 威望
- 0 点
- 阅读权限
- 30
- 积分
- 151
- 相册
- 1
- 日志
- 1
- 记录
- 2
- 帖子
- 78
- 主题
- 10
- 精华
- 0
- 分享
- 1
- 好友
- 13
升级   25.5% TA的每日心情 | 擦汗 2014-1-6 09:21 |
|---|
签到天数: 29 天 [LV.4]偶尔看看III
- 自我介绍
- calm down
 群组: web前端开发交流 |
3 r8 u7 N& K: H; F1 l1 _9 ]7 u6 S
作者: Paul Firth 来源: 外刊IT评论 , C F, o% x5 L* r% L
/ p( w$ v. j f2 y6 Y
英文原文:10 steps to becoming a better programmer! ~4 F7 B5 M; U6 g% O9 S1 o) _7 z
这篇文章要介绍的,是我作为专业程序员这些年来学到的能真正提高我的代码质量和整体工作效率的 10 件事情。4 F9 \6 A. h- G- n
1. 永远不要复制代码
. k( i1 {4 Q% V# y d$ D) e) d( x 不惜任何代价避免重复的代码。如果一个常用的代码片段出现在了程序中的几个不同地方,重构它,把它放到一个自己的函数里。重复的代码会导致你的同事在读你的代码时产生困惑。而重复的代码如果在一个地方修改,在另外一个地方忘记修改,就会产生到处是 bug,它还会使你的代码体积变得臃肿。现代的编程语言提供了很好的方法来解决这些问题,例如,下面这个问题在以前很难解决,而如今使用 lambda 却很好实现:
9 T: s4 _+ }# |/ R/// <summary>
; ?0 m% s' w! _' m0 U/// 一些函数含有部分重复代码1 k2 J: ?) I& v7 ?; k
/// </summary>3 \4 [: O5 W7 W4 O6 X6 N
void OriginalA()
6 E j- L+ @& E" a# e( G$ D1 w# V{
$ M' R( Y, x; }. l: G+ p DoThingsA();9 s1 i1 m. K& p$ Z- F
// unique code- _0 M( P( A3 y5 j- f
DoThingsB();% ^* v( v- V) y; G/ `
}
& `; g# M# e9 S% ?: v; p/// <summary>6 P) D' }9 U% P$ n) t& W
/// 另外一个含有部分重复代码的函数
0 T5 Q2 I g h$ h( E; G* u/// </summary>& Y* c3 a$ O4 ?; O6 Q, R
void OriginalB()% I5 f0 p% `# L( b4 i8 k/ b+ e" N
{3 r& k ^) X0 D+ Q
DoThingsA();
; b) I& l: J1 J // 没有重复的代码
0 ~9 ~7 N4 c% @3 |6 Q DoThingsB();
' G1 x- F4 m; I) S& j7 K$ W% z/ _4 ^}! X- e9 g0 W9 f$ t/ C
现在我们重构含有部分相同代码的函数,用 delegate 模式重写它们:$ _( @8 D+ r/ Q3 I) A6 L
/// <summary>$ i2 B. h# E/ A/ [. {+ S8 W Y
/// Encapsulate shared functionality: m0 q% K' x* f9 y6 J* Z
/// </summary>
: p! l/ o, O; S# ]/// <param name="action">User defined action</param>
1 q0 D+ q0 Q" x* ^$ |8 Rvoid UniqueWrapper(Action action)
% W8 X: \6 `6 }{" l8 J0 h: b( b+ Z7 ^$ S' ?3 ]
DoThingsA();
: e" Z5 R9 r" U$ l f c# G3 `9 V action();
) p& r; L& J b DoThingsB();
5 M' i) B7 ]3 s0 @0 X}9 V, M+ h# ^9 Z8 j
/// <summary>
$ m- g- @3 H0 G& r: y/// New implmentation of A t# u2 m! f5 m" u
/// </summary>
1 Y- Q5 E, t7 K3 W, `* t* Svoid NewA()
4 S, @% s5 x# K* F7 o{0 B7 T H R4 w% ~( v( @5 D1 b
UniqueWrapper(() =>; s3 _& R9 p0 \4 G: Z; w
{; M3 l4 D3 I2 r/ g" c
// unique code
/ _+ C) u% u3 q' A* m& l8 ]1 R });
7 n/ p! L9 [/ W}
4 y \2 b \( ~; Y. e L/// <summary>9 ]+ Q% h1 E& E- q
/// New implementation of B
- z5 S; T/ Z/ u, l' t/// </summary>* ^8 }) W; ?; n. n/ V
void NewB()
/ z/ g3 O1 G7 L9 ?- u2 Q) k{& X3 i6 `; s- W7 l$ M3 w
UniqueWrapper(() =>
- {0 G- i5 m, H/ k {
5 c: V+ L2 ^3 ~4 H' S // unique code
0 X7 S0 `$ j& A3 N1 ^' \ });& Y$ B5 m2 b; S( j0 j* `5 z8 X
}4 j- Q0 @' M) f, D$ I
2. 留意你开始分心的时候
! ]9 n. H% H: D; T6 | 当你发现自己在浏览 facebook 或微博,而不是在解决问题,这通常是一种你需要短暂休息的信号。离开办公桌,去喝一杯咖啡,或去跟同事聊 5 分钟。尽管这样做看起来有点反直觉,但长久去看,它会提高你的工作效率。6 Z0 D; M8 Y& S& n2 }" x
3. 不要匆忙赶任务而放弃原则
: u2 K; ?) j) s- J. R: H& I 当带着压力去解决一个问题或修改一个 bug,你很容易失去自制,发现自己匆匆忙忙,甚至完全忘了一直坚持的重要的测试过程。这通常会导致更多的问题,会让你在老板或同事眼里显得很不专业。
# u/ m6 R% C% H1 j 4. 测试你完成的代码
; w( A1 j7 {% `1 B. B. V 你知道你的代码能做什么,而且试了一下,它确实好用,但你实际上需要充分的验证它。分析所有可能的边界情况,测试在所有可能的条件下它都能如期的工作。如果有参数,传递一些预期范围外的值。传递一个 null 值。如果可能,让同事看看你的代码,问他们能否弄坏它。单元测试是到达这种目的的常规方法。
3 k5 d. a: d/ ~ 5. 代码审查
& N1 p/ ~9 |) k# p' t8 K" r 提交你的代码之前,找个同事一起坐下来,向他解释你做了哪些修改。通常,这样做的过程中你就能发现代码中的错误,而不需要同事说一句话。这比自己审查自己的代码要有效的多得多。( t- c, ]& Q5 S v
6. 让代码更少
) g) B. D- G! U! R; ]1 N8 r 如果你发现写了大量的代码来解决一个简单的问题,你很可能做错了。下面的 boolean 用法是一个很好的例子:- [' w& V/ M9 y8 p0 X. C
if (numMines > 0)
$ z3 Z( V0 a" _; ~- \6 L{
1 t4 m7 b5 v( W% O# |! F+ d enabled=true;
$ c. I2 F3 ?; v' F$ g' O! j}
; r- Z5 Q f0 C, B; \else2 L+ [& _- y ~3 D6 n8 l! ^
{
' @6 ?2 R# U& z4 X' M enabled=false;
* k& Y- Y- H, E7 V8 g" z0 Q! d} ' w8 R0 o0 D3 K
这时你应该写成这样:
; B; H! i. R+ @enabled = numMines > 0; ' \5 b! B" s# t) K
代码越少越好。这会使 bug 更少,重构可能性更小,出错的几率更小。要适度。可读性同等重要,你可不能这样做而使代码丧失可读性。
4 N6 l" Z3 J. _0 m7 o 7. 为优雅的代码而努力
8 q% x' L' d$ X+ [3 h* O 优雅的代码非常的易读,只用手边很少的代码、让机器做很少的运算就能解决问题。在各种环境中都做到代码优雅是很难的,但经过一段时间的编程,你会对优雅的代码是个什么样子有个初步的感觉。优雅的代码不会通过重构来获得。当你看到优雅的代码是会很高兴。你会为它自豪。例如,下面就是一个我认为是优雅的方式来计算多边形面积的方法:+ ^) J2 t. s- G/ @4 n: z" |+ i
static public double GetConvexPolygonArea (Vector2[] vertices)0 S8 p4 d, G; n, A; ]$ Y7 w
{
+ E- N o( w1 ]! T double area = 0;( A6 a) A7 C/ I' h! _
for (int i = 0; i < vertices.Length; i++)9 K' P4 {* F# t8 m6 ]2 I
{: f, b" L5 D5 F8 X
Vector2 P0 = vertices[i];6 C) f/ i0 H1 H! ?9 _/ r: L/ b
Vector2 P1 = vertices[(i + 1) % vertices.Length];
{: Z7 R( k' B" @0 M7 M: y9 U area += P0.Wedge (P1);
6 M( s# a( { H% u* G2 u }
$ m4 X) J' i$ {# o& \) x& A return area / 2;5 w/ u4 G, P' ` K# @+ k1 f
}
3 R9 m+ p( N# Q7 f; ]8 z+ m) W 8. 编写不言自明的代码& g% Q o# v8 K6 {5 Q, O
勿庸置疑,注释是编程中很重要的一部分,但能够不言自明的代码更胜一筹,因为它能让你在看代码时就能理解它。函数名变量名要慎重选择,好的变量/方法名字放到语言语义环境中时,不懂编程的人都能看懂。例如:
, q) K7 Y: P& e2 ^4 }void DamagePlayer (Player player, int damageAmount)
9 D3 P c7 J' r D0 q/ I. a{
, X D, ~8 \+ v) o4 }" b3 v9 [ if (!player.m_IsInvincible && !player.m_IsDead)' Q' ?2 l6 I/ g5 p" v7 I, q9 [
{
( n& u) P2 q6 F' F# x) `( C X, Q player.InflictDamage ( damageAmount );' D1 r0 j$ G" t- o
}
z3 \- l/ i9 l3 ?% C} 8 ~ g6 T1 C) C, W9 s/ N
能自我说明的代码不能代替注释。注释是用来解释“为什么”的,而自我说明的代码是来描述“是什么”的。
7 x4 s6 _3 x+ D 9. 不要使用纯数字
4 k) z8 U5 _+ [ 直接把数字嵌入代码中是一种恶习,因为无法说明它们是代表什么的。当有重复时更糟糕——相同的数字在代码的多个地方出现。如果只修改了一个,而忘记了其它的。这就导致 bug。一定要用一个命名常量来代表你要表达的数字,即使它在代码里只出现一次。
( r* K/ y3 H% x9 \ 10. 不要做手工劳动0 t% S8 X( F- y- O
当做一系列动作时,人类总是喜欢犯错误。如果你在做部署工作,并且不是一步能完成的,那你就是在做错事。尽量的让工作能自动化的完成,减少人为错误。当做工作量很大的任务时,这尤其重要。
6 O0 |: H u" c+ U 11. 避免过早优化
7 y4 s' W) g0 U4 t 当你要去优化一个已经好用的功能代码时,你很有可能会改坏它。优化只能发生在有性能分析报告指示需要优化的时候,通常是在一个项目开发的最后阶段。性能分析之前的优化活动纯属浪费时间,并且会导致 bug 出现。7 i1 D) [/ h- a0 W, I: I
好吧,我说是 10 个,但你却得到了额外赠送的一个!4 s$ e( E) s) r- j9 J
这些就是我要说的,我希望它们能帮助你改进编程开发过程。/ G# ~5 W; p5 W8 A( c' c; r% C
下次再见!祝快乐!
2 S ~- ]: z7 b B% S3 R9 j* L Cheers, Paul. |
zan
|