我参与的代码审查经常出现的一件事是代码中的"魔术数字"内联是一件坏事.
首选项是在某处分配符号.
现在我正在使用的是QT sdk.类成员QButtonGroup.checkedId()
将返回-1
一个幻数,表示所选组中没有单选按钮.
如果我写的东西像
if ( buttongroup->checkedId() == -1 )
{
//yadda yadda
}
Run Code Online (Sandbox Code Playgroud)
它将出现在代码审查中.
虽然我当然可以定义一个const static int
符号来给我这个-1
,但Qt
如果它有一个相关的常量定义,我宁愿在命名空间中使用一些东西.
是否已经定义了这样的符号?