一个工作5年的程序员竟然可以这样说

资深项目经理

没有new说明new肯定是在if{}里的，而if判断的条件根本没有满足。

当初的模块设计怎么约定的，参数没有判断是否符合公司的编程规范？

gtkmm

回复 21# w_anthony
赞

我经常纠结是否判断参数，失败了是否打印消息，还是弄日志文件，还是抛异常，
每次都觉得不知所措。

是返回-1还是错误码，或是含有消息的结构体。。还是直接在程序里面写好几个变量，调试时候给gdb留着。。
这是无比痛苦的。

我的程序本来是简单的，最后都是看着就觉得恶心了。。

x5miao

关键看你们公司的类型吧。如果是外包型和项目型，只要功能到位了这些一般都不管的。

A.com

大家讨论这么热烈。。。我也说说我们的处理原则吧。

因为NULL是一个非常特殊的值，所以，我们需要先确定NULL的合法性。
如果产生NULL是合法的，那么使用这个数据的模块必然是要对NULL做出特殊处理的。
如果产生的NULL是非法的，或者说是意外的，那么使用数据的模块是不对这种意外做容错的，正如shan_ghost所说，这样做的唯一结果就是让你的代码增加越来越多的莫名其妙的容错。正确的做法是消除意外数据，谁出的错谁来处理，错哪儿改哪儿。只有这样，才能避免1个bug影响8个模块，最后在一个八竿子打不着的地方爆了。这时候，你上哪去找这个bug去？

wwbwscn

我以前做的项目发生过类似的问题，正确的做法是在数据源头就判断出来，楼主说的程序员我认为他的做法也不算太错。

fayewangfans

程序出错了早点挂掉好，好调试，好检查。

Kallawa

把分析过程写清楚，邮件抄全组~~
这是分享的好机会，也是催使当事人修复bug的信号，感觉这是双赢的方式
我们组一直都是这么搞的，气氛还不错~~

smalloc

现在不到5年经验的都这么叼？

smalloc

我又查看了一个此人的代码，发现他的代码几乎所有的函数都没有对函数的输入进行判断！
----l
kernel里很多也没有做判断.

shang2010

smalloc 发表于 2014-01-01 15:45:37
我又查看了一个此人的代码，发现他的代码几乎所有的函数都没有对函数的输入进行判断！
----l
kernel里很多

死毛，一个发呵呵恢复