主题：【原创】乱侃软件工程师的素养 1 -- poorfat
- 复 不见得，讨论一下
家园 另外这种情况的性能优化是没有必要的。 性能优化的3个规律
1.往往80％的性能问题出现在20％的代码中；
2.往往我们认为是性能瓶颈的地方，偏偏不是瓶颈；
3.往往细节上的代码优化会导致更多的微妙问题出现。
所以，根本意见是：[SIZE=3]不要优化，或者不要过早优化。[/SIZE]
- 复 不见得，讨论一下
家园 有点不同一点。 对于一个稳定的商业软件，必须考虑系统维护成本，系统运行中的细节应该能够通过日志等技术手段追溯。尤其是一些关键分支的堆栈情况。
在我的团队中，我是要求通过单元测试保证这种分支的情况能被覆盖到，虽然实际上也很难保证达到要求。
- 复 不见得，讨论一下
家园 你说得很对 不过对我来说，由我的职业地位决定，我必须把可测性放在第一位。没法测的东西我可不敢签发布许可。
- 复 你说得很对
家园 应当将错误码报告放在函数中 很重要的原则,应当将错误码报告放在函数中,即发现错误的第一时间.
有几点好处:
1. 更容易定位错误,例如function1()返回失败到底是参数错,还是检索不到结果或结果不匹配?只有在function1()内部才能给出.
2.上层模块可以从具体的错误返回中抽象出来,用楼主的例子举个具体的例,假设是个密码检测函数,那么:function1() / function2() / function3()之一返回失败,上层模块只要知道"密码错"就够了,至于是密码长度不同,大小写不匹配或是密码不同那是下层函数的责任.如故写成:
const bool OMG_WE_FAILED = false;
bool ret = ture;
if (!function1()) {log("Length Unmatched"); ret = OMG_WE_FAILED;}
else if (!function2()) {log("Passwrod Unmatched"); ret = OMG_WE_FAILED;}
else if (!function3()) {log("Case Unmatched"); ret = OMG_WE_FAILED;}
return ret;
那才UGLY
3.程序有更好的可读性,更符合人的思维逻辑
家园 这个问题需要分成两种情况 一种function这个函数是用来检测状态的，那么状态不对返回值为FALSE，这种情况下，在function中不作日志提示，而在调用function的函数中应该给出错误提示。
另一种是function函数本身就是一个过程动作，在过程中出错，在function函数内部就应该有日志提示出错原因，并且不同的错误有不同的错误码返回值。而在调用function的函数中可以有错误日志也可以没有。
其实在任何情况下，将几个函数联到一条语句中都会造成检测的困难，并且降低程序的可读性。虽然将函数写到一个判定语句中确实能够提高效率，但是就现在的系统性能而言，这个效率不算也罢。
- 复 不见得，讨论一下
家园 中肯，花一个
家园 没看懂 用户拿到的是什么东西？如果是已经编译过的机器码，写成一行和写成三行没有区别吧。 如果是源代码，用户自然知道怎么跟踪。
而且，既然这三个函数是你的实现，你当然可以把它们封装起来，用户连到底和几个函数相关都用不着知道。用户知道出错了就可以了。就像你说的，
客户只会给你打电话抱怨说你的产品又坏了我不认为
如果你把三个函数调用分别放在三个IF语句里，里有任何好作用。
唯一可能有作用的是
分别用不同的错误代码可是这个完全是接口设计的责任，而不是程序员的责任。为了便于排错，正规的做法是使用不同级别的log.
家园 我也跟个最近遇到的错误 DateTime yesterday = new DateTime(DateTime.Now.Year, DateTime.Now.Month, DateTime.Now.Day - 1);
这个错误躲过了两个测试的围剿，终于在某月1日爆发了。
家园 哈哈，这么搞笑的代码 你们没有code review吗？