我正在尝试从代码中删除重复的行。目前,它看起来像这样:
if (FALSE == DoFoo(strFile1, strData1))
{
return (FALSE);
}
if (FALSE == DoFoo(strFile2, strData1))
{
return (FALSE);
}
if (FALSE == DoFoo(strFile3, strData1))
{
return (FALSE);
}
if (FALSE == DoBar(strData1))
{
return (FALSE);
}
if (FALSE == DoFoo(strFile4, strData2))
{
return (FALSE);
}
if (FALSE == DoFoo(strFile5, strData2))
{
return (FALSE);
}
if (FALSE == DoFoo(strData1, strData2))
{
return (FALSE);
}
if (FALSE == DoBar(strData2))
{
return (FALSE);
}
return (TRUE);
我把它改成了这样:
if (DoFoo(strFile1, strData1))
{
if (DoFoo(strFile2, strData1))
{
if (DoFoo(strFile3, strData1))
{
if (DoBar(strData1))
{
if (DoFoo(strFile4, strData2))
{
if (DoFoo(strFile5, strData2))
{
if (DoFoo(strData1, strData2))
{
if (DoBar(strData2))
{
return (TRUE);
}
}
}
}
}
}
}
}
return (FALSE);
是否有嵌套ifs的标准?是否存在普遍约定的限制?通常情况下,它可能会与如此多的嵌套混淆,但由于没有任何复杂的事情发生,没有额外的代码行,它似乎仍然清晰。哪种方式更好?或者还有另一种方法可以做到比其中任何一种更整洁和/或更短的方式吗?
答案 0 :(得分:3)
我会按如下方式实现它:
bool func() {
return DoFoo(strFile1, strData1)
&& DoFoo(strFile2, strData1)
&& DoFoo(strFile3, strData1)
&& DoBar(strData1)
&& DoFoo(strFile4, strData2)
&& DoFoo(strFile5, strData2)
&& DoFoo(strData1, strData2)
&& DoBar(strData2);
}
答案 1 :(得分:2)
没有标准。你可以自由地做你想做的事情,直到你单独处理代码。
但大多数人建议避免它(限制自己1-2个嵌套级别),因为这样的代码很难阅读。实践表明,在大多数情况下,这是可能的。
例如,在您的情况下,您可以使用&&操作者:
if ( DoFoo(strFile1, strData1) &&
DoFoo(strFile2, strData1) &&
DoFoo(strFile3, strData1) &&
DoBar(strData1) &&
DoFoo(strFile4, strData2) &&
DoFoo(strFile5, strData2) &&
DoFoo(strData1, strData2) &&
DoBar(strData2)
) {
return FALSE;
}
见?这使代码更容易。
在其他情况下,您可以使用
如果组成:
if (A) {
} else if (B) {
} else if (C) {
...
函数调用:
void CheckCoor( ... ) {
if ( coor >= ... ) {
CheckAngle(...);
else {
CheckPosibilityToChangeCoor(...);
}
}
通常这4个选项足以避免多个嵌套级别。
答案 2 :(得分:1)
有更好的方法:
if(DoFoo(strFile1, strData1) && DoFoo(strFile2, strData1) && //etc...
return true;
或者:
if(!DoFoo(strFile1, strData1) || !DoFoo(strFile2, strData1) || //etc...
return false;
答案 3 :(得分:1)
也许类似于以下内容,这是摆脱嵌套if语句的不同可能性。
有一个丑陋的C喜欢演员表使表格正确,我们假设有额外的,未使用的参数不会有问题。
在调试器中运行时,这会编译并似乎与Microsoft Visual Studio 2005一起用作Windows控制台应用程序。当然,DoFoo ()
和DoBar()
函数都是本例中的存根,假定的char指针不指向任何东西,只有结构存在。
bool DoFoo (char *s1, char *s2)
{
return true;
}
bool DoBar (char *s1)
{
return true;
}
char *strData1;
char *strData2;
char *strFile1;
char *strFile2;
char *strFile3;
char *strFile4;
char *strFile5;
typedef bool (*xFunc)(char *s1, char *s2);
typedef struct {
xFunc pF;
char *s1;
char *s2;
} TableItem;
bool myFunc(void)
{
TableItem fArray[] = {
{ DoFoo, strFile1, strData1},
{ DoFoo, strFile2, strData1},
{ DoFoo, strFile3, strData1},
{ (xFunc)DoBar, strData1, NULL},
{ DoFoo, strFile4, strData2},
{ DoFoo, strFile5, strData2},
{ DoFoo, strData1, strData2},
{ (xFunc)DoBar, strData2, NULL}
};
bool bRet = true;
int i = 0;
for (i = 0; bRet && i < sizeof(fArray)/sizeof(fArray[0]); i++) {
bRet = fArray[i].pF(fArray[i].s1, fArray[i].s2) && bRet;
}
return bRet;
}
int _tmain(int argc, _TCHAR* argv[])
{
myFunc();
return 0;
}
编辑 - 另一种可能性
另一种方法如下。这不需要C样式演员。
bool DoFoo (char *s1, char *s2)
{
return true;
}
bool DoBar (char *s1)
{
return true;
}
char *strData1;
char *strData2;
char *strFile1;
char *strFile2;
char *strFile3;
char *strFile4;
char *strFile5;
typedef enum {funcType1 = 1, funcType2} funcType;
typedef struct {
funcType fType;
char *s1;
char *s2;
} TableItem;
bool myFunc(void)
{
TableItem fArray[] = {
{ funcType1, strFile1, strData1},
{ funcType1, strFile2, strData1},
{ funcType1, strFile3, strData1},
{ funcType2, strData1, NULL},
{ funcType1, strFile4, strData2},
{ funcType1, strFile5, strData2},
{ funcType1, strData1, strData2},
{ funcType2, strData2, NULL}
};
bool bRet = true;
int i = 0;
for (i = 0; bRet && i < sizeof(fArray)/sizeof(fArray[0]); i++) {
switch (fArray[i].fType) {
case funcType1:
bRet = DoFoo (fArray[i].s1, fArray[i].s2);
break;
case funcType2:
bRet = DoBar (fArray[i].s1);
break;
default:
break;
}
}
return bRet;
}
int _tmain(int argc, _TCHAR* argv[])
{
myFunc();
return 0;
}
编辑 - 还有另一种可能性
另一种方法如下。这是非常直接的C ++。
bool DoFoo (char *s1, char *s2)
{
return true;
}
bool DoBar (char *s1)
{
return true;
}
char *strData1;
char *strData2;
char *strFile1;
char *strFile2;
char *strFile3;
char *strFile4;
char *strFile5;
class funcObj {
public:
funcObj (bool (*pf) (char *s1, char *s2), char *s1, char *s2);
funcObj (bool (*pf) (char *s1), char *s1);
funcObj ();
~funcObj() {}
bool eval ();
private:
bool (*m_dofoo) (char *s1, char *s2);
bool (*m_dobar) (char *s1);
char *m_s1;
char *m_s2;
};
funcObj::funcObj (bool (*pf) (char *s1, char *s2), char *s1, char *s2)
{
m_dofoo = pf;
m_dobar = 0;
m_s1 = s1;
m_s2 = s2;
}
funcObj::funcObj (bool (*pf) (char *s1), char *s1)
{
m_dofoo = 0;
m_dobar = pf;
m_s1 = s1;
m_s2 = 0;
}
funcObj::funcObj ()
{
m_dofoo = 0;
m_dobar = 0;
m_s1 = 0;
m_s2 = 0;
}
bool funcObj::eval ()
{
bool bRet = false;
if (m_dofoo)
bRet = m_dofoo(m_s1, m_s2);
else if (m_dobar)
bRet = m_dobar (m_s1);
return bRet;
}
typedef struct {
funcObj fObj;
} TableItem;
bool myFunc(void)
{
funcObj fArray[] = {
funcObj(DoFoo, strFile1, strData1),
funcObj(DoFoo, strFile2, strData1),
funcObj(DoFoo, strFile3, strData1),
funcObj(DoBar, strData1),
funcObj(DoFoo, strFile4, strData2),
funcObj(DoFoo, strFile5, strData2),
funcObj(DoFoo, strData1, strData2),
funcObj(DoBar, strData2)
};
bool bRet = true;
int i = 0;
for (i = 0; bRet && i < sizeof(fArray)/sizeof(fArray[0]); i++) {
bRet = fArray[i].eval();
}
return bRet;
}
int _tmain(int argc, _TCHAR* argv[])
{
myFunc();
return 0;
}