zl程序教程

您现在的位置是:首页 >  其他

当前栏目

bool当成函数参数错误理解

错误 理解 函数参数 bool 当成
2023-06-13 09:14:41 时间
我们有很多CodingStyle或代码规范。但这一条可能会经常被我们所遗忘,就是我们经常会在函数的参数里使用bool参数,这会大大地降低代码的可读性。不信?我们先来看看下面的代码。
当你读到下面的代码,你会觉得这个代码是什么意思?
widget->repaint(false);是不要repaint吗?还是别的什么意思?
看了文档后,我们才知道这个参数是immediate,也就是说,false代表不立即重画,true代码立即重画。
WindowsAPI中也有这样一个函数:InvalidateRect,当你看到下面的代码,你会觉得是什么意思?
InvalidateRect(hwnd,lpRect,false);我们先不说InvalidateRect这个函数名取得有多糟糕,我们先说一下那个false参数?invalidate意为“让XXX无效”,false是什么意思?双重否定?是肯定的意思?
如果你看到这样的代码,你会相当的费解的。于是,你要去看一下文档,或是InvalidateRect的函数定义,你会看到那个参数是BOOLbErase,意思是:“是否要重画背景”。
这样的事情有很多,再看下面的代码,想把str中的”%USER%”替换成真实的用户名:
str.replace("%USER%",user,false);//Qt3TNND,那个false是什么意思?不替换吗?还是别的什么意思?
看了文档才知道,false代表:“大小写不敏感的替换”。
其实,如果你使用枚举变量/常量,而不是bool变量,你会让你的代码更易读,如:
复制代码代码如下:

widget->repaint(PAINT::immediate);
widget->repaint(PAINT::deffer);
InvalidateRect(hwnd,lpRect,!RepantBackground);
str.replace("%USER%",user,Qt::CaseInsensitive);//Qt4如果对这个事不以为然的话,我们再来看一些别的示例,你不妨猜猜看看下面的代码:
component.setCentered(true,false);

这什么玩意儿啊?
看了文档你才知道,这原来是setCentered(centered,autoUpdate);
newTextbox(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的《APIDesignPrinciples》,本文就是其中的“BooleanTrap”。