不要为了完成代码而写代码

从我工作到现在，我也参加了很多项目的代码设计和编写工作，也带过了不少的人员进行代码的设计和编写工作。总体上讲，一般刚刚在我带领的团队工作的人都会经历一段相当的痛苦的时期，那段时期他将会被我严重的教育“为了完成代码而写代码”的毛病。
其实这些毛病谁都发生。下面我就讲讲这么多年来，我见过的一些代码编写人员常犯的错误。
（本文的代码都是伪代码）
1.不遵循“先出域”原则
原代码
public void Save(string id)
{
if(string.IsNullOrEmpty(id))
{
//复杂的业务代码
}
else
{
return;
}
}
改进后的代码
public void Save(string id)
{
if(!string.IsNullOrEmpty(id)) return;
//复杂的业务代码
}
大家看明白了吗？改成下面一种以后是不是可读性加强了很多？！不管是在方法体，还是流程语句中，都要尽量把return语句往前靠，这样会增加可读性；
2.方法体过长，复杂度太高
原代码：
public void Save(string id)
{
for()
{
if()
{
}
else
{
}
for()
{
}
}
switch()
{
case:
{
}
case :
{
}
default:
{
}
}
}
改进后的代码：
类似这种的方法只能进行拆分，同志们，这种拆分应该遵循一个原则：一个方法中的业务代码长度尽量不要超过5行（赋值语句除外）。这点虽然比较难，但是努力一下还是可以做到的。
3.将用户控件Id写死
原代码：
<script language="javascript" type="javascript/text">
function Test(string txtId)
{
document.getElementById("UserControls1_"+txtId)
}
</script>
改进代码：
<script language="javascript" type="javascript/text">
function Test(string txtClientID)
{
document.getElementById(txtClientID)
}
</script>
看明白了吗？在javascript的代码中，传递一个客户端Id，而不是服务器端ID就可以了，虽然这样的改动是很小的，但是绝对的让你后患无穷，各位想想看，如果你这段代码正好写在用户控件上，而这个用户控件在一个页面被使用了两次，那么你第一种写法能完成第二个用户控件中的功能吗？