嵌套ifs与串行逻辑块

时间:2013-11-06 19:42:56

标签: c++ if-statement

我正在尝试从代码中删除重复的行。目前,它看起来像这样:

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的标准?是否存在普遍约定的限制?通常情况下,它可能会与如此多的嵌套混淆,但由于没有任何复杂的事情发生,没有额外的代码行,它似乎仍然清晰。哪种方式更好?或者还有另一种方法可以做到比其中任何一种更整洁和/或更短的方式吗?

4 个答案:

答案 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;
}

见?这使代码更容易。

在其他情况下,您可以使用

  1. switch(...)运算符。
  2. 如果组成:

    if (A) {
    
    } else if (B) {
    
    } else if (C) {
    ...
    
  3. 函数调用:

    void CheckCoor( ... ) {
      if ( coor >= ... ) {
         CheckAngle(...);
      else {
         CheckPosibilityToChangeCoor(...);
      }
    }
    
  4. 通常这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;
    }