Question

我刚开始学习C ++，我试图使用嵌套循环来输出美国国旗。我非常接近，但我似乎无法修复我的代码中的这个逻辑错误，并希望有人可以指出它，因为我已经搜索了几个小时，它让我发疯。问题在于最后三行。

#include <iostream>
using namespace std;

int main()
{
int rows = 7;
int columns = 24;


int max = rows * columns;

int value = 1;
for (int r=1; r<=7; r++ ){
  for (int c=1; c<=24; c++ ){
      if (c <= 7 && r <= 4)
      cout << "* ";

      if(r % 2 != 0 && c > 7 && r > 0)
          cout << "= ";

      if(r % 2 == 0 && c > 7 && r > 0)
          cout << "- ";
}
cout << endl;
}

return 0;
}

此代码将输出：https://i.imgur.com/wpsG90q.png

Answer 1

你在前四行中画出了你不会考虑的星星。当你不画星星时，你必须画一些东西。

    for (int c=1; c<=24; c++ ){
        if (c <= 7 && r <= 4)
            cout << "* ";
        else {
            if(r % 2 != 0)
                cout << "= ";
            else
                cout << "- ";
        }

Answer 2

这是一个很好的问题，可以了解组合多个逻辑条件的基础知识。这个问题确定了三个不同的条件：

：写＆＃39; *＆＃39;
不在左上角，在奇数行中：写＆＃39; =＆＃39;
不在左上角，在偶数行中：写＆＃39; - ＆＃39;

这些条件在代码中的明显音译如下：

if (c <= 7 && r <= 4)
    std::cout << "* ";

if (!(c <= 7 && r <= 4) && r %2 != 0)
    std::cout << "= ";

if (!(c <= 7 && r <= 4) && r %2 == 0)
    std::cout << "- ";

当简化!(c <= 7 && r <= 4)时，问题中的代码会误入歧途。根据德摩根定律，该表达式相当于!(c <= 7) || !(r <= 4)。反过来，这可以简化为c > 7 || r > 4。问题中的错误是它使用&&而不是||。我这样做时讨厌它。另请注意，&&绑定比||更紧密，因此您必须将其放在括号中。所以简化版看起来像这样：

if (c <= 7 && r <= 4)
    std::cout << "* ";

if ((c > 7 || r > 4) && r %2 != 0)
    std::cout << "= ";

if ((c > 7 && r > 4) && r %2 == 0)
    std::cout << "- ";

并不简单。： - （

但是，一旦你开始用C ++思考，那么版本会更好：不要重复左上角测试而是否定它，使用else子句。所以

if (c <= 7 && r <= 4)
    std::cout << "* ";
else { /* get here when we're not in the upper left corner */
    if (r % 2 != 0)
        std::cout << "= ";
    if (r % 2 == 0);
        std::cout << "- ";
}

但现在请注意，有两个测试涉及r % 2，一个检查不 0，另一个检查 0那也应该立即对你说if ... else ...。所以：

if (c <= 7 && r <= 4)
    std::cout << "* ";
else { /* get here when we're not in the upper left corner */
    if (r % 2 != 0)
        std::cout << "= ";
    else
        std::cout << "- ";
}

现在，正如@Manuel指出的那样，我们在外层有两个逻辑案例：在左上角和左上角之外。这并非不合理，但我更倾向于将此视为三个逻辑案例，因为我开始提出问题。为了反映这种方法，摆脱大括号：

if (c <= 7 && r <= 4)
    std::cout << "* ";
else if (r % 2 != 0)
    std::cout << "= ";
else
    std::cout << "- ";

嵌套循环创建美国国旗

2 个答案: