千万不要把 bool 当成函数参数

justjavac 发布于 2012/11/14 10:23
阅读 947
收藏 3

【开源中国 APP 全新上线】“动弹” 回归、集成大模型对话、畅读技术报告”

我们有很多 Coding Style 或 代码规范。 但这一条可能会经常被我们所遗忘,就是我们 经常会在函数的参数里使用bool参数,这会大大地降低代码的可读性。 不信?我们先来看看下面的代码。

当你读到下面的代码,你会觉得这个代码是什么意思?

widget->repaint(false);

是不要 repaint 吗?还是别的什么意思?

看了文档后,我们才知道这个参数是 immediate,也就是说,false 代表不立即重画,true 代码立即重画。

Windows API 中也有这样一个函数:InvalidateRect,当你看到下面的代码,你会觉得是什么意思?

InvalidateRect(hwnd, lpRect,  false);

我们先不说 InvalidateRect 这个函数名取得有多糟糕,我们先说一下那个 false 参数? invalidate 意为 “让XXX无效”,false 是什么意思? 双重否定? 是肯定的意思?

如果你看到这样的代码,你会相当的费解的。 于是,你要去看一下文档,或是 InvalidateRect 的函数定义, 你会看到那个参数是 BOOL bErase,意思是:“是否要重画背景”。

这样的事情有很多,再看下面的代码,想把 str 中的 ”%USER%” 替换成真实的用户名:

str.replace("%USER%", user, false);   // Qt 3

TNND,那个 false 是什么意思?不替换吗?还是别的什么意思?

看了文档才知道,false 代表: “大小写不敏感的替换”。

其实,如果你使用枚举变量/常量,而不是 bool 变量,你会让你的代码更易读,如:

widget->repaint(PAINT::immediate);
widget->repaint(PAINT::deffer);

InvalidateRect(hwnd, lpRect, !RepantBackground);

str.replace("%USER%", user, Qt::CaseInsensitive); // Qt 4

如果对这个事不以为然的话,我们再来看一些别的示例,你不妨猜猜看看下面的代码:

component.setCentered(true, false);

这什么玩意儿啊?

看了文档你才知道,这原来是 setCentered(centered, autoUpdate);

new Textbox(300, 100, false, true);

这又是什么啊?

看了文档才知道,这是创建一个文本框,第三个参数是:“是否要滚动条”,第四个是:“是否要自动换行”。TNND!

这种情况还不算最差,看看下面的双重否定。

component.setDisabled(false);
filter.setCaseInsensitive(false)

再来一个,如果你读到下面的代码,相信你会和我一样,要么石化了,要么凌乱了。

event.initKeyEvent("keypress", true, true, null, null,
					false, false, false, false, 9, 0);

看完这篇文章,我希望你再也 不要把bool为作为函数参数了。除非两个原因:

  • 你 100% 确认不会带来阅读上的问题,比如 Java 的 setVisible (bool).
  • 你 100% 确认你想去 写出迷一样的代码

如果你想设计一个好的 API,强烈推荐你读一下 Nokia 的 Qt 的《API Design Principles》,本文就是其中的 “Boolean Trap”。

加载中
0
独孤小败
独孤小败
这得看你函数怎么命名了,比如:function 确定这样命名(bool 是/否);
独孤小败
独孤小败
回复 @justjavac : 对,是我看粗心了,刚没看作者,现在看你图标才看到,原来是你啊,不错不错,你的文章一直不错,总是能说到人们平时忽略的地方。
justjavac
justjavac
setVisible (bool)
0
coffeescript
coffeescript
我总结一下: 当函数名是个一般疑问句的时候,可以用bool。如果不是的话,尽量不要用bool。
0
吃土的汉子
吃土的汉子
又见标题党。函数参数用不用bool不是关键,关键是直接把true和false传给了一个看不出意思的函数,声明个bool变量会死么
0
HawkYang
HawkYang

碰到这种情况,传递参数的时候最好用个枚举

上面举的例子,setVisible (bool) 也不是直接传递True/False、

而是XXX.Visible或者XXX.Invisible

0
modernizr
modernizr
期待楼主下一篇文章: 千万不要把 Int 当成函数参数
0
chromeplugin
chromeplugin
期待楼主下一篇文章:  千万不要把 String 当成函数参数
0
coffeescript
coffeescript
期待楼主下一篇文章:    千万不要把 Float 当成函数参数
0
红薯
红薯
期待楼主下一篇文章:    千万不要把 OSChina 当成网站
0
SatSun
SatSun
期待楼主下一篇文章: 千万别把FastCRM当成家喻户晓
0
justjavac
justjavac
下一篇文章: 千万别拿红薯不当干粮@红薯
OSCHINA
登录后可查看更多优质内容
返回顶部
顶部