如何写健壮的代码

 1  private   static   void  ParseArgs( string [] arguments) 
 2  { 
 3       if  (_parsedArgs) 
 4      { 
 5           return ; 
 6      } 
 7       if  (arguments  ==   null ) 
 8      { 
 9          arguments  =  CommandLineArgs; 
10      } 
11       // 命令行格式： 
12       const   string  usage  =   " [-initial 5] [-level 2] " ; 
13       for  ( int  i  =   0 ; i  <  arguments.Length; i ++ ) 
14      { 
15           // 匹配前导符 
16           if  (arguments[i].CompareTo(InitialStartLevelLead)  ==   0 ) 
17          { 
18               // step forward 
19               if  ( ++ i < arguments.Length) 
20              { 
21                   // 取值 
22                  _initialBundleStartLevel  =  Convert.ToInt32(arguments[i]); 
23                   continue ; 
24              } 
25               else  
26              { 
27                   goto  fail; 
28              } 
29                  
30          } 
31           if  (arguments[i].CompareTo(StartLevelLead)  ==   0 ) 
32          { 
33               // step forward 
34               if  ( ++ i  <  arguments.Length) 
35              { 
36                   // 取值 
37                  _startLevel  =  Convert.ToInt32(arguments[i]); 
38                   continue ; 
39              } 
40               else  
41              { 
42                   goto  fail; 
43              }
44          } 
45       } 
46      _parsedArgs  =   true ; 
47       return ; 
48  fail: 
49       throw   new  Exception( " Invalid args,usage: " +  usage); 
50  }

private   static   void  ParseArgs( string [] arguments) 
{ 
     if  (_parsedArgs)   // 1 这个参数的命名不符合规则，正确的命名应该是_isArgsParsed，如果想让参数可读性更强，可以再将参数的限定词声明一下，变成_is××ArgsParsed。 
    { 
         return ; 
    } 
     if  (arguments  ==   null ) 
    { 
        arguments  =  GetCommandLineArgs(); 
    } 
     // 命令行格式： 
     const   string  usage  =   " [-initial 5] [-level 2] " ; 

// 2 硬编码的字符串。
     for  ( int  i  =   0 ; i  <  arguments.Length; i ++ )   // 3 如果GetCommandLineArgs的返回值为空，则这行代码会出现NullReferenceException；此外，如果arguments.Length大于4个，我们需要给用户提示；如果参数出现重复或者不符合要求，我们也需要给用户提示。 
    { 
         // 匹配前导符 
         if  (arguments[i].CompareTo(InitialStartLevelLead)  ==   0 )  // 4 如果arguments[i]允许带空格，则这行代码会产生错误的执行结果 
        { 
             // step forward 
             if  ( ++ i < arguments.Length) 
            { 
                 // 取值 
                _initialBundleStartLevel  =  Convert.ToInt32(arguments[i]);   // 5 如果arguments[i]不是合法的整数，则这行代码会产生转换失败的异常。 
                 continue ; 
            } 
             else  
            { 
                 goto  fail; 
            } 
                
        } 
         if  (arguments[i].CompareTo(StartLevelLead)  ==   0 )  // 6 问题同3 
        { 
             // step forward 
             if  ( ++ i  <  arguments.Length) 
            { 
                 // 取值 
                _startLevel  =  Convert.ToInt32(arguments[i]);   // 7 问题同4 
                 continue ; 
            } 
             else  
            { 
                 goto  fail; 
            }
        } 
     } 
    _parsedArgs  =   true ; 
     return ; 
fail: 
     throw   new  Exception( " Invalid args,usage: " +  usage);  // 8 异常信息不友好，没有清楚告诉用户是哪一个参数输入有问题，此外在英文文法中，“args”和“usage”中间要空格，另外，给用户的信息不能够随便使用缩写，必须使用完整的词语；硬编码字符串需要放在一个统一的资源文件，并考虑I18N和L10N支持；问题2中的字符串可以直接和这个Exception的字符串直接合并；如果有必要的话，最好重新定义个Exception类型。
}