【代码优化】学会利用“配置文件”优化我们的代码结构

假设有这样一个场景: 某系统会产生三类错误, 分别是:ERROR_TYPE_ONE, ERROR_TYPE_TWO, ERROR_TYPE_THREE. 但是呢, 各省份有不同的需求, 比如:

      1. 安徽省要求遇到上述三种错误后分别显示:101, 102, 103

      2. 湖北省要求遇到上述三种错误后分别显示:1001, 1002, 1003

      3.广东省要求遇到上述三种错误后分别显示:10001, 10002, 10003

      4.黑龙江要求遇到上述三种错误后分别显示:201, 202, 203

      5.浙江省要求遇到上述三种错误后分别显示:2001, 2002, 2003


      面对这些省份的不同要求, 我们该怎么搞呢? “最勤奋”的程序猿写出来的代码是:

[cpp]  view plain  copy
  1. #include <iostream>  
  2. using namespace std;  
  3.   
  4. enum ErrorType  
  5. {  
  6.     UNDEFINED,  
  7.     ERROR_TYPE_ONE,  
  8.     ERROR_TYPE_TWO,  
  9.     ERROR_TYPE_THREE,     
  10. };  
  11.   
  12. int main()  
  13. {  
  14.     char szProvince[100] = {0};  
  15.     cin >> szProvince;  
  16.       
  17.     int error = -1;  
  18.     cin >> error;  
  19.   
  20.     if(0 == strcmp("anhui", szProvince))  
  21.     {  
  22.         if(ERROR_TYPE_ONE == error)  
  23.         {  
  24.             cout << 101 << endl;  
  25.         }  
  26.         else if(ERROR_TYPE_TWO == error)  
  27.         {  
  28.             cout << 102 << endl;  
  29.         }  
  30.         else if(ERROR_TYPE_THREE == error)  
  31.         {  
  32.             cout << 103 << endl;  
  33.         }  
  34.     }  
  35.   
  36.     if(0 == strcmp("hubei", szProvince))  
  37.     {  
  38.         if(ERROR_TYPE_ONE == error)  
  39.         {  
  40.             cout << 1001 << endl;  
  41.         }  
  42.         else if(ERROR_TYPE_TWO == error)  
  43.         {  
  44.             cout << 1002 << endl;  
  45.         }  
  46.         else if(ERROR_TYPE_THREE == error)  
  47.         {  
  48.             cout << 1003 << endl;  
  49.         }  
  50.     }  
  51.   
  52.     if(0 == strcmp("guangdong", szProvince))  
  53.     {  
  54.         if(ERROR_TYPE_ONE == error)  
  55.         {  
  56.             cout << 10001 << endl;  
  57.         }  
  58.         else if(ERROR_TYPE_TWO == error)  
  59.         {  
  60.             cout << 10002 << endl;  
  61.         }  
  62.         else if(ERROR_TYPE_THREE == error)  
  63.         {  
  64.             cout << 10003 << endl;  
  65.         }  
  66.     }  
  67.   
  68.     if(0 == strcmp("heilongjiang", szProvince))  
  69.     {  
  70.         if(ERROR_TYPE_ONE == error)  
  71.         {  
  72.             cout << 201 << endl;  
  73.         }  
  74.         else if(ERROR_TYPE_TWO == error)  
  75.         {  
  76.             cout << 202 << endl;  
  77.         }  
  78.         else if(ERROR_TYPE_THREE == error)  
  79.         {  
  80.             cout << 203 << endl;  
  81.         }  
  82.     }  
  83.   
  84.     if(0 == strcmp("zhejiang", szProvince))  
  85.     {  
  86.         if(ERROR_TYPE_ONE == error)  
  87.         {  
  88.             cout << 2001 << endl;  
  89.         }  
  90.         else if(ERROR_TYPE_TWO == error)  
  91.         {  
  92.             cout << 2002 << endl;  
  93.         }  
  94.         else if(ERROR_TYPE_THREE == error)  
  95.         {  
  96.             cout << 2003 << endl;  
  97.         }  
  98.     }  
  99.   
  100.     return 0;  
  101. }  
       我们看看上面的代码结果, 一看就感觉是重复啰嗦加累赘, 可能有的初学者说, 这很自然啊, 如果优化的话, 可以用switch来优化啊.  其实, 用switch又怎样呢? 还不是一样的垃圾代码?


      我们之前说过用数组优化if/else,   我们可以考虑一下数组。 但此处我自问一下, 为什么数组可以优化? 因为数组的本质就是映射! 废话少说, 直接上代码:

[cpp]  view plain  copy
  1. #include <iostream>  
  2. using namespace std;  
  3.   
  4. int a[][3] =   
  5. {  
  6.     {101, 102, 103},  
  7.     {1001, 1002, 1003},  
  8.     {10001, 10002, 10003},  
  9.     {201, 202, 203},  
  10.     {2001, 2002, 2003}  
  11. };  
  12.   
  13. int main()  
  14. {  
  15.     char szProvince[100] = {0};  
  16.     cin >> szProvince;  
  17.       
  18.     int error = -1;  
  19.     cin >> error;  
  20.   
  21.     if(error < 1 || error > 3)  
  22.     {  
  23.         return 1;  
  24.     }  
  25.   
  26.     if(0 == strcmp("anhui", szProvince))  
  27.     {  
  28.         cout << a[0][error - 1] << endl;  
  29.     }  
  30.   
  31.     if(0 == strcmp("hubei", szProvince))  
  32.     {  
  33.         cout << a[1][error - 1] << endl;  
  34.     }  
  35.   
  36.     if(0 == strcmp("guangdong", szProvince))  
  37.     {  
  38.         cout << a[2][error - 1] << endl;  
  39.     }  
  40.   
  41.     if(0 == strcmp("heilongjiang", szProvince))  
  42.     {  
  43.         cout << a[3][error - 1] << endl;  
  44.     }  
  45.   
  46.     if(0 == strcmp("zhejiang", szProvince))  
  47.     {  
  48.         cout << a[4][error - 1] << endl;  
  49.     }  
  50.   
  51.     return 0;  
  52. }  
      看看, 是不是简洁多了。 其实上述代码还可以优化, 我们只需要建立字符串到整数的映射即可, 如下:

[cpp]  view plain  copy
  1. #pragma warning(disable : 4786)     
  2. #include <iostream>    
  3. #include <string>    
  4. #include <map>    
  5. using namespace std;    
  6.   
  7. int a[][3] =   
  8. {  
  9.     {101, 102, 103},  
  10.     {1001, 1002, 1003},  
  11.     {10001, 10002, 10003},  
  12.     {201, 202, 203},  
  13.     {2001, 2002, 2003}  
  14. };  
  15.   
  16. typedef map<string, int> strMapInt;  
  17. typedef map<string, int> :: iterator It;    
  18. strMapInt g_map;  
  19.   
  20. void buildMap()  
  21. {  
  22.     g_map.insert(make_pair(string("anhui"), 1));  
  23.     g_map.insert(make_pair(string("hubei"), 2));  
  24.     g_map.insert(make_pair(string("guangdong"), 3));  
  25.     g_map.insert(make_pair(string("heilongjiang"), 4));  
  26.     g_map.insert(make_pair(string("zhejiang"), 5));  
  27. }  
  28.   
  29. int main()  
  30. {  
  31.     buildMap();  
  32.   
  33.     char szProvince[100] = {0};  
  34.     cin >> szProvince;  
  35.       
  36.     int error = -1;  
  37.     cin >> error;  
  38.   
  39.     if(error < 1 || error > 3)  
  40.     {  
  41.         return 1;  
  42.     }  
  43.   
  44.     It it = g_map.find(string(szProvince));    
  45.     if(g_map.end() != it)    
  46.     {    
  47.         cout << a[it->second - 1][error - 1] << endl;  
  48.         return 0;  
  49.     }  
  50.   
  51.     return 1;  
  52. }  
       但是呢, 上述程序有两个问题: 1. 如果某个省份比较讨厌, 它突然要改变对应的错误码显示, 比如安徽想把101变为1000001. 这个时候, 程序猿不得不动代码了。 2. 如果要增加一些省份怎么办呢? 这个时候, 程序猿也需要动代码了。

       下面, 我们考虑利用配置文件来进行优化, 通常来讲, 在这种情况下, .xml文件做配置文件是非常适合的, 但为了简便起见, 我们用.txt文件来代替。 我们在config.txt文件中制作如下内容(以安徽为例):

1
101
2
102
3
103

       然后改程序如下:

[cpp]  view plain  copy
  1. #include <iostream>    
  2. #include <string>    
  3. #include <fstream>  
  4. using namespace std;    
  5.   
  6. int main()  
  7. {  
  8.     int error = -1;  
  9.     cin >> error;  
  10.   
  11.     ifstream infile("config.txt");  
  12.     string line;  
  13.     int n = 0;  
  14.     if(infile)  
  15.     {  
  16.         while(getline(infile, line))  
  17.         {  
  18.             if(atoi(line.c_str()) == error)  
  19.             {  
  20.                 getline(infile, line); // 取下一行  
  21.                 n = atoi(line.c_str());  
  22.   
  23.                 cout << n << endl;  
  24.                 return 0;  
  25.             }  
  26.         }  
  27.     }  
  28.   
  29.     return 1;  
  30. }  
       我们可以看到, 这个程序不怕省份增多, 也不怕省份的错误码变更。 我们需要做的是, 对每一个省份, 搞个配置文件即可。

       不得不说, 说面的程序还是很fragile的。需要在程序中增加更多异常的判断, 比如检验配置文件格式的合法性。 但是, 在本文中, 我们仅作示例, 所以有很多异常情况并没有考虑。


      总结一下:配置文件的引入, 使得程序对于需求变化不敏感, 也就是说, 程序不怕需求变化, 体现了很好的可扩展性, 回味一下, 确实如此。 实际的软件开发中, 有很多类似这样的地方, 建议大家注意程序结构的合理性

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值