如果变量只用于迭代计数,没有在循环的其他地方出现,一般使用单个字母的变量(i),而不是另外取名,这是一个约定俗成的习惯。
至少到现在为止,还没有发现这样做有什么不好。
5、变量名中不使用下划线(_),成员变量除外。
在.net编程中,有时候为了区分局部变量和成员变量,会在成员变量前面加上(_),而且在重构的时候,VS也认可这种规则。
6、命名空间需按照标准的模式命名
namespaceCompanyName.AppName
{
}
文件名要和类名匹配
例如,对于类HelloWorld,相应的文件名应为helloworld.cs(或,helloworld.vb),尽量一个文件一个类
缩进和间隔
缩进用TAB不用SPACES。
注释需和代码对齐。
花括弧({})需和括号外的代码对齐。
用一个空行来分开代码的逻辑分组。
boolSayHello(stringname)
{
stringfullMessage="Hello"+name;
DateTimecurrentTime=DateTime.Now;
stringmessage=fullMessage+",thetimeis:
"+currentTime.ToString();
if(...)
{
//Dosomething
returnfalse;
}
returntrue;
}
在一个类中,各个方法需用一空行,也只能是一行分开。
花括弧需独立一行,而不象if,for等可以跟括号在同一行。
好:
if(...)
{
//Dosomething
}
不好:
if(...){ //Dosomething }
在每个运算符和括号的前后都空一格,否则代码显得拥挤。
好:
if(showResult==true)
{
for(inti=0;i<10;i++)
{
//
}
}
不好:
if(showResult==true)
{
for(inti=0;i<10;i++)
{
//
}
}
良好的编程习惯
遵从以下良好的习惯以写出好程序
usingSystem.Data;
namespaceCompanyName.AppName
{
///
///HelloWord的摘要说明
///
publicclassHelloWord
{
///
///方法说明
///
///姓名
///成功与否
boolSayHello(stringname)
{
stringfullMessage="Hello"+name;
DateTimecurrentTime=DateTime.Now;
stringmessage=fullMessage+","+currentTime.ToString();
if(message!
=string.Empty)
{
//Dosomething
MessageBox.Show(message);
}
returntrue;
}
}
}
避免使用大文件。
如果一个文件里的代码超过300~400行,必须考虑将代码分开到不同类中,因为没有哪个程序员喜欢阅读一大堆眼花缭乱的代码。
避免写太长的方法,一个方法只完成一个任务。
一个典型的方法代码在1~25行之间。
如果一个方法发代码超过25行,应该考虑将其分解为不同的方法。
方法名需能看出它作什么。
别使用会引起误解的名字。
如果名字一目了然,就无需用注释来解释方法的功能了。
好:
voidSavePhoneNumber(stringphoneNumber)
{
//Savethephonenumber.
}
不好:
//Thismethodwillsavethephonenumber.
voidSaveData(stringphoneNumber)//含义不明确的方法
{
//Savethephonenumber.
}
一个方法只完成一个任务。
不要把多个任务组合到一个方法中,即使那些任务非常小。
好:
//Savetheaddress.
SaveAddress(address);
//Sendanemailtothesupervisortoinformthattheaddressisupdated.
SendEmail(address,email);
voidSaveAddress(stringaddress)
{
//Savetheaddress.
//...
}
voidSendEmail(stringaddress,stringemail)
{
//Sendanemailtoinformthesupervisorthattheaddressischanged.
//...
}
不好:
//Saveaddressandsendanemailtothesupervisortoinformthattheaddressisupdated.
SaveAddress(address,email);
voidSaveAddress(stringaddress,stringemail)
{
//Job1.
//Savetheaddress.
//Job2.
//Sendanemailtoinformthesupervisorthattheaddressischanged.
//...
}
使用C#或VB.NET的特有类型,而不是System命名空间中定义的别名类型。
好:
intage;
stringname;
objectcontactInfo;
不好:
Int32age;
Stringname;
ObjectcontactInfo;
别在程序中使用固定数值,用常量代替。
别用字符串常数,用资源文件。
避免使用很多成员变量。
声明局部变量,并传递给方法。
不要在方法间共享成员变量。
如果在几个方法间共享一个成员变量,那就很难知道是哪个方法在什么时候修改了它的值。
使用enum,别用数字或字符串来指示一组值,例如:
星期,部门列表,文本类型。
没有人能保证自己可以精确的记住所有的值,尤其是后期维护的时候,用enum功能更方便,因为VS可以给出智能提示。
好:
enumMailType
{
Html, PlainText, Attachment
}
voidSendMail(stringmessage,MailTypemailType)
{
switch(mailType)
{
caseMailType.Html:
//Dosomething
break;
caseMailType.PlainText:
//Dosomething
break;
caseMailType.Attachment:
//Dosomething
break;
default:
//Dosomething
break;
}
}
不好(你能记住所有mailType类型并保证每个单词都敲对吗?
):
voidSendMail(stringmessage,stringmailType)
{
switch(mailType)
{
case"Html":
//Dosomething
break;
case"PlainText":
//Dosomething
break;
case"Attachment":
//Dosomething
break;
default:
//Dosomething
break;
}
}
别把成员变量声明为public或protected。
都声明为private而使用public/protected的Properties。
不要把本属于数据层的代码搬到页面中实现
使用sql语句,数据库中读取和写入数据,SqlDataReader,DataTable,DataSet,本应该属于底层的代码,不要将这些代码放到页面中,这不符合分层开发的模式。
不在代码中使用具体的路径和驱动器名。
使用相对路径,并使路径可编程。
永远别设想你的代码是在“C:
”盘运行。
你不会知道,一些用户在网络或“Z:
”盘运行程序。
应用程序启动时作些“自检”并确保所需文件和附件在指定的位置。
必要时检查数据库连接。
出现任何问题给用户一个友好的提示。
如果需要的配置文件找不到,应用程序需能自己创建使用默认值的一份。
如果在配置文件中发现错误值,应用程序要抛出错误,给出提示消息告诉用户正确值。
错误消息需能帮助用户解决问题。
永远别用象“应用程序出错”,“发现一个错误”等错误消息。
而应给出象“更新数据库失败。
请确保登陆id和密码正确。
”的具体消息。
显示错误消息时,除了说哪里错了,还应提示用户如何解决问题。
不要用象“更新数据库失败。
”这样的,要提示用户怎么做:
“更新数据库失败。
请确保登陆id和密码正确。
”
要把所有可能的出错信息都记录下来,以助诊断问题。
注释
别每行代码,每个声明的变量都做注释。
在需要的地方注释。
可读性强的代码需要很少的注释。
如果所有的变量和方法的命名都很有意义,会使代码可读性很强并无需太多注释。
行数不多的注释会使代码看起来优雅。
但如果代码不清晰,可读性差,那就糟糕。
如果应为某种原因使用了复杂艰涩的原理,为程序配备良好的文档和重分的注释。
对一个数值变量采用不是0,-1等的数值初始化,给出选择该值的理由。
简言之,要写清晰,可读的代码以致无须什么注释就能理解。
对注释做拼写检查,保证语法和标点符号的正确使用。
异常处理
不要“捕捉了异常却什么也不做”。
如果隐藏了一个异常,你将永远不知道异常到底发生了没有。
voidReadFromFile(stringfileName)
{
try
{
//readfromfile.
//这样写catch,会让你永远都不知道这里发生了什么异常
}
catch() { }
}
发生异常时,给出友好的消息给用户,但要精确记录错误的所有可能细节,包括发生的时间,和相关方法,类名等,便于以后解决问题。
只捕捉特定的异常,而不是一般的异常。
好:
voidReadFromFile(stringfileName)
{
try
{
//readfromfile.
}
catch(FileIOExceptionex)
{
//logerror.
// re-throwexceptiondependingonyourcase.
throw;
}
}
不好:
voidReadFromFile(stringfileName)
{
try
{
//readfromfile.
}
catch(Exceptionex)
{
//Catchinggeneralexceptionisbad...
//wewillneverknowwhetherit
//wasafileerrororsomeothererror.
//Hereyouarehidinganexception.
//Inthiscasenoonewilleverknowthatanexceptionhappened.
return"";
}
}
不必在所有方法中捕捉一般异常。
不管它,让程序崩溃。
这将帮助你在开发周期发现大多数的错误。
你可以用应用程序级(线程级)错误处理器处理所有一般的异常。
遇到”以外的一般性错误“时,此错误处理器应该捕捉异常,给用户提示消息,在应用程序关闭或用户选择”忽略并继续“之前记录错误信息。
不必每个方法都用try-catch。
当特定的异常可能发生时才使用。
比如,当你写文件时,处理异常FileIOException.
别写太大的try-catch模块。
如果需要,为每个执行的任务编写单独的try-catch模块。
这将帮你找出哪一段代码产生异常,并给用户发出特定的错误消息。