IT牛人博客聚合

好的代码应该是直观的, 简单的. 直观就是"所思就所写", 想的是什么样就要把代码写成什么样子, 不要七拐八绕.

例如, 在做结构设计和流程设计时, 我们分析出某个功能流程应该这样做:

先做步骤1, 然后做步骤2.

什么是程序设计? 程序设计就是流程, 是串行化, 是先后顺序. 所以, 文档设计完毕之后, 必须写下这样的代码:

step1();
step2();

没错, 就是非常直观的两个函数调用语句, 一眼就能看出有先后顺序, 先 1 后 2. 但是, 初学者往往会把对step2()的调用, 放到step1()里面, 这就非常不直观了. 所以, 在做代码 review 时, 必须明确提出这个问题:

哪一段代码直观地体现了你的设计?

如果代码和设计思路没有在外在形式上一一对应, 那么那样的代码就是不直观的, 不好的, 因为看代码的人除非先看文档, 否则无法快速知晓流程上的先后顺序. 即使先看了文档, 也要费很大工夫才能从几百行代码中找到先后顺序关系的线索.

下面的代码, 你能一眼看出先后顺序吗?

func main() {
	// 启动 10 个步骤1的线程
	for i := 0; i < 10; i ++ {
		go step1()
	}

	// 启动 1 个步骤2 的线程
	go step2()
}

// 步骤1
func step1() {
	for {
		// do something...
	}
	wg.Done() // 等同于调用 step2()
}

// 步骤2
func step2() {
	wg.Wait() // 等待步骤1执行完毕
	// do something...
}

直观的代码应该是这样的:

func main() {
	// 启动主线程
	go func() {
		// 先执行 step1, 再执行 step2
		step1()
		step2()
	}()
}

// 步骤1
func step1() {
	// 启动工作线程
	for i := 0; i < 10; i ++ {
		go sub1()
	}
	wg.Wait() // 等待工作线程结束
}

// 工作线程函数
func sub1() {
	for {
		// do something...
	}
	wg.Done()
}

// 步骤2
func step2() {
	// do something...
}

虽然两者的代码在初学者眼中是一样的, 甚至有可能根据代码行数就错误地认为直观的写法太复杂, 但是, 直观的写法是自顶向下 的, 代码本身就直接对应了设计文档, 即使文档丢失, 也能一眼看出来流程.

Related posts:

1. 生产者消费者编程模式
2. 小心递归次数限制
3. 必须放在循环中的pthread_cond_wait
4. 接口与实现分离
5. 数据通信与传输协议基础