※ 引述《david830317 (dd810)》之铭言： : 开发平台(Platform): (Ex: VC++, GCC, Linux, ...) : Xcode : 额外使用到的函数库(Library Used): (Ex: OpenGL, ...) : 问题(Question)： : 在xcode时用void以外不能用reture : 错误结果(Wrong Output)： : Control may reach end of non-void function : 程式码(Code)：(请善用置底文网页, 记得排版) : http://ideone.com/T36LJH : 补充说明(Supplement)： 关於这个 function 的写法 float Quantity::calcValue() const { if (unit >= 1 && unit <= 10) { return unit * 3.00; } if (unit >= 11 && unit <= 50) { return unit * 2.50; } if (unit >= 51) { return unit * 2.00; } } 我比较建议用 if-elseif-elseif 的结构来写: float Quantity::calcValue() const { if (unit >= 1.0f && unit <= 10.0f) { return unit * 3.00f; } else if (unit > 1.0f && unit <= 50.0f) { return unit * 2.50f; } else if (unit >= 50.0f) { return unit * 2.00f; } else { /* default */ return unit; } /* Unreached */ } 原因: 1. 这样才看得清楚这三条 if 其实是负责同一件事，条件排列整齐後，漏网之鱼也 在 default 那一区块有处理，不管是 error handling 还是 return default， 而且所有的状况在 if-else 结构内都会 return，能轻易判断函式终结点。 2. float 运算时，不管是比大小还是运算，最好两边都是 float (10.0f)， 你有些写 int，问题倒还不大反正会转成 float，但有些写 double (2.50) 就 会两边都转成 double 运算，运算完再转 float 再 return，浪费效能。 另一个问题点，你大概一开始把 unit 定义为 int 後来才改 float， 所以 if 里面完全没有处理到 10.0f<unit<11.0f, 50.0f<unit<51.0f 的状况， 如果习惯把常数的 data type 也写对，遇到可能有问题的状况，心中自然会产生 违和感，觉得这样写怪怪的。你写 10.0f 时就会考虑 10.5f 会发生什麽事 3. 最後注解 Unreached 其实很有必要，照我的写法所有状况都应该在 if-else 的 结构内 return，所以不会跑到 Unreached 那边。但就是有些人在修改 code 时 会呆呆加在後面，以为 if 跑完就会跑他加的东西，所以要注解提醒。 实务上有很多地方需要这样的防呆措施，不只是合作写程式防别人耍呆，很多 时候自己也会发蠢， * * * 举另一个经典例子: if (unit > 1.0f) return unit*3.0f; 这样写在 C/C++ 是允许的，省略大括号。但是就有人这样插一行... if (unit > 1.0f) printf("Using 3.0 ratio.\n"); return unit*3.0f; 结果变成不管 unit 值多少，有没有 print 那行字，都变成 return unit*3.0f 了 所以我会严格要求如果 if 的内容分成两行写，一定要加大括号 if (unit > 1.0f) { return unit*3.0f; } 或这样也行 if (unit > 1.0f) { return unit*3.0f; } 写成同一行的话随便你加不加 if (unit > 1.0f) return unit*3.0f; 大家应该会觉得是插入新行却不长眼看清楚的那个人的错，我也觉得是他的错， 但是这种状况实在太多太多了，大家都是用 Ctrl-C & Ctrl-V 在写程式的样子， 熬夜拼命贴的时候就常常贴错地方，只好设法定一些规则防呆。 这样至少你 code review 时就可以检查大括号来预防出错，而不是每次 release 的 前一天晚上才在抓几百个这种无脑错误。 -- 桃乐丝: 可是, 如果你没有头脑, 为什麽会说话? 稻草人: ㄝ, 我也不知... 但是有些人没有头脑也能说超～多话呢。 --

※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 1.164.198.178 ※ 文章网址: https://webptt.com/cn.aspx?n=bbs/C_and_CPP/M.1461828217.A.029.html