| « | August 2025 | » | | 日 | 一 | 二 | 三 | 四 | 五 | 六 | | | | | | 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9 | 10 | 11 | 12 | 13 | 14 | 15 | 16 | 17 | 18 | 19 | 20 | 21 | 22 | 23 | 24 | 25 | 26 | 27 | 28 | 29 | 30 | 31 | | | | | | | |
|
公告 |
戒除浮躁，读好书，交益友 |
| Blog信息 |
|
blog名称:邢红瑞的blog
日志总数:523
评论数量:1142
留言数量:0
访问次数:9693616
建立时间:2004年12月20日 |
| |
|
 [编程感想]程序员写防弹程序的问题 
原创空间, 软件技术, 电脑与网络
邢红瑞 发表于 2006/7/29 18:55:09 |
|
这是一个老问题，也是大问题。记得去年，一个朋友招几名资深程序员，我做面试官，问到一个有10年经验的程序员，这位仁兄说道，写程序也就是到处copy代码，不能运行的我改改就能让它运行。这句话我听了很恐怖，建议这人还是不要的好，会害死人。但是还是招聘了，造成了项目的bug满天飞。根据我写代码的经验，代码分为3种，可以运行的代码，正确的代码，和好的代码。写大多数时间能够运行的代码很easy的，使用几种常用的输入，得到几种常用的输出，一旦给一个不常用输入数据，程序就over了。正确的代码不会over的，所有可能的输入数据，都是正确的，一般来说，所有可能的输入是测试不到的。但是正确的代码并不一定是好的代码，正确的代码可能逻辑混乱，或者垃圾代码很多。编写好的代码是我们的目标，好的代码具有鲁棒性，运行起来高效，即使不常见的输入也不导致当机或错误的输出，当然也必须满足其他的要求，线程安全，数据库的事务处理等等。坏的程序，我总结了一下，出了问题大家的说法1.函数不能这样调用，它的参数必须合法。2.这段代码一直工作，它不会产生任何错误。3.我的文档说明了这个全局变量是内部用的，只有我可以用。经验告诉我们，你的代码在某个时间，某种情况下，会出错，很多人解释就是，不正确的输入，他当然会出问题。当我们写程序的时候，我们不能做任何假设，我们不能想它不能发生。当你写的代码越来越多，工作也越做越快，你没有时间验证每一种情况，你的程序鲁棒性下降，你的代码的bug也越来越多。举个我遇到的例子这个一个给停车场的读卡器设置频率的程序，因为这种使用日立SuperH系列芯片的读卡器，可以设置各种读卡频率，使用函数setFreq，原型为 int setFreq(int array[],int arraylenth);传入参数有两个，一个int数组，后面的数组长度，返回的设置成功的频率个数，这位仁兄把这函数封装一下，认为后面的那个arraylenth不需要，因为sizeof(a)就是得到数组长度了。函数写成这样int SetFrequency(int a[]){ int retcode=setFreq(a,sizeof(a)/sizeof(int));); return retcode;}因为是交叉编译平台，公司没有购买相应的调试设备，无法经行调试，关于调试的问题，我以后会说的。当时公司只有一张频率的卡用于测试，我们测试通过，就ok了。到了客户那里，不久就有投诉，很多频率的卡读不到，当时感到很是奇怪，后来发现问题。因为SuperH是32位的RISC CPU，a是指针，sizeof(a)的长度为4，setFreq只是设置1个频率，恰巧我的那张卡，就是这个频率。马上修改了这段代码，事后这位仁兄喋喋不休到，int a[]={1,2,3,4,5,6,7}; printf("%d",sizeof(a)/sizeof(int));得到就是数组长度吗？注意你一旦传递数据，被调用的函数得到是个指针。 |
|
|