软件中的大多Bug都与各种低级错误有关，而大多低级错误是靠测试测不出来的，更多的需要Code Review来发现问题。
培养好的习惯，不靠任何假设编程，可以先从写好每个函数开始，我想用注释驱动来提醒自己经常Review每个函数。
以下是我拟定的Code Review Comments：
/* ### Code Review * ### Reviewer: wawa * ### Last Review Date: 2013-01-29 * * - todo: input review * - todo: output review * - todo: exception review * - todo: null reference review * - todo: out of bounds review * * */
每个函数都加上这么一段注释，某项review通过后把todo改成done，如果代码修改后要重新review，并修改Last Review Date。
这么玩肯定会很麻烦，但我的理念就是代码写的越慢越好，通过Check List来强制对代码进行某些思考，从而慢慢养成习惯。
比如下面这段用户登录的代码。
function login(username, password){ var account = accountDAO.getAccount(); if (account['password'] == password){ return {code:200, message:'login success.'} }else{ return {code:403, message:'login faild.'} } }
很明显有很多问题,然后经过review，修改如下
/* ### Code Review * ### Reviewer: wawa * ### Last Review Date: 2013-01-29 * * - done: input review * - done: output review * - done: exception review * - done: null reference review * - done: out of bounds review * * */ function login(username, password){ if (username === null || password === null || username === '' || password === ''){ return {code:400, message: 'input invalid'}; } try{ var account = accountDAO.getAccount(); if (account === null){ return {code:404, message: 'account not found'}; } } catch(err){ return {code:500, message: 'server error'}; } if (!('password' in account)){ return {code:500, message: 'server error'}; } if (account['password'] == password){ return {code:200, message:'login success.'} }else{ return {code:401, message:'login faild.'} } }
第一步就先做这几个基本的检查吧。
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。