翻译:杨贺宏
谁都会写代码!几个月的编程经验可以让你写出“可运行应用程序”。让它可运行容易,但是以最有效率的方式编码就需要下更多的功夫!要知道,大多数程序员在写”可运行代码,“而不是”高效代码“。我们在这个指南课程前面提到,你想成为你们公司”最尊贵的专业人员“吗?写”高效代码“是一项艺术,你必须学习和实践它。
变量和方法参数使用Camel 大小写形式
public class HelloWorld{ int totalCount = 0 ; void SayHello( string name) { string fullMessage = " Hello " + name; ... }}
不要使用匈牙利方法来命名变量 以前,多数程序员喜欢它-把数据类型作为变量名的前缀而m_作为成员变量的前缀。例如:
string m_sName; int nAge;然而,这种方式在.NET编码规范中是不推荐的。所有变量都用camel 大小写形式,而不是用数据类型和m_来作前缀。 用有意义的,描述性的词语来命名变量 - 别用缩写。用name, address, salary等代替 nam, addr, sal - 别使用单个字母的变量象i, n, x 等. 使用 index, temp等 用于循环迭代的变量例外: 如果变量只用于迭代计数,没有在循环的其他地方出现,许多人还是喜欢用单个字母的变量(i) ,而不是另外取名。 - 变量名中不使用下划线 (_) 。 - 命名空间需按照标准的模式命名 . . . 文件名要和类名匹配 例如,对于类HelloWorld, 相应的文件名应为 helloworld.cs (或, helloworld.vb)
这段代码看起来比上面的好::
在一个类中,各个方法需用一空行,也只能是一行分开。 花括弧需独立一行,而不象if, for 等可以跟括号在同一行。. 好: if ( ... ) { // Do something } if ( ... ) { // Do something } 不好: 在每个运算符和括号的前后都空一格。. 好: if ( showResult == true ) { for ( int i = 0 ; i < 10 ; i ++ ) { // } } 不好:bool SayHello ( string name ) { string fullMessage = " Hello " + name; DateTime currentTime = DateTime.Now; string message = fullMessage + " , the time is : " + currentTime.ToShortTimeString(); MessageBox.Show ( message ); if ( ... ) { // Do something // ... return false; } return true ; } 避免使用大文件。如果一个文件里的代码超过300~400行,必须考虑将代码分开到不同类中。 避免写太长的方法。一个典型的方法代码在1~25行之间。如果一个方法发代码超过25行,应该考虑将其分解为不同的方法。 方法名需能看出它作什么。别使用会引起误解的名字。如果名字一目了然,就无需用文档来解释方法的功能了。好: void SavePhoneNumber ( string phoneNumber ) { // Save the phone number. } 不好: 一个方法只完成一个任务。不要把多个任务组合到一个方法中,即使那些任务非常小。 // This method will save the phone number. void SaveData ( string phoneNumber ) { // Save the phone number. } 好: 不好: 使用C# 或 VB.NET的特有类型,而不是System命名空间中定义的别名类型。好: int age; string name; object contactInfo; 不好: Int16 age; String name; Object contactInfo; 别在程序中使用固定数值,用常量代替。 别用字符串常数。用资源文件。 避免使用很多成员变量。声明局部变量,并传递给方法。不要在方法间共享成员变量。如果在几个方法间共享一个成员变量,那就很难知道是哪个方法在什么时候修改了它的值。 必要时使用enum 。别用数字或字符串来指示离散值。 好: enum MailType { Html, PlainText, Attachment } void SendMail ( string message, MailType mailType) { switch ( mailType ) { case MailType.Html: // Do something break ; case MailType.PlainText: // Do something break ; case MailType.Attachment: // Do something break ; default : // Do something break ; } } 不好: void SendMail ( string message, string mailType) { switch ( mailType ) { case " Html " : // Do something break ; case " PlainText " : // Do something break ; case " Attachment " : // Do something break ; default : // Do something break ; } } 别把成员变量声明为 public 或 protected。都声明为 private 而使用 public/protected 的Properties. 不在代码中使用具体的路径和驱动器名。 使用相对路径,并使路径可编程。 永远别设想你的代码是在“C:”盘运行。你不会知道,一些用户在网络或“Z:”盘运行程序。 应用程序启动时作些“自检”并确保所需文件和附件在指定的位置。必要时检查数据库连接。出现任何问题给用户一个友好的提示。 如果需要的配置文件找不到,应用程序需能自己创建使用默认值的一份。 如果在配置文件中发现错误值,应用程序要抛出错误,给出提示消息告诉用户正确值。 错误消息需能帮助用户解决问题。永远别用象"应用程序出错", "发现一个错误" 等错误消息。而应给出象 "更新数据库失败。请确保登陆id和密码正确。" 的具体消息。 显示错误消息时,除了说哪里错了,还应提示用户如何解决问题。不要用 象 "更新数据库失败。"这样的,要提示用户怎么做:"更新数据库失败。请确保登陆id和密码正确。" 显示给用户的消息要简短而友好。但要把所有可能的信息都记录下来,以助诊断问题。
// Save address and send an email to the supervisor to inform that the address is updated. SaveAddress ( address, email ); void SaveAddress ( string address, string email ) { // Job 1. // Save the address. // ... // Job 2. // Send an email to inform the supervisor that the address is changed. // ... }