魔方函数 C++

Question

这是我的魔方的最后一个函数，由于某种原因，它给了我一个错误，数组下标有“'[int]'”，但我不知道这意味着什么，如果有人可以帮助解释我必须做什么做。

bool Square::is_magic()
{
    for (i = 0; i < size-1; i++)
    {
        if (sum_row[i] != sum_row[i+1])
            return false;

    if (sum_col[i] != sum_col[i+1])
        return false;
        }

    if (sum_row[0] != sum_col[0])
        return false;

    if (sum_row[0] != sum_maindiag[0])
        return false;

    if (sum_row[0] != sum_other[0])
        return false;

    return true;
}

Answer 1

好吧，每个人都曾经是初学者。我强烈推荐你阅读一两本关于 C++ 的书。 （我个人是通过“21天学习c++”学习编程的，很多人抱怨，但这对我来说是一个好的开始）。

以及代码。不确定这是否是您需要的，它应该是这样的：

bool Square::is_magic()
{
   int i;
   for (i = 0; i < size-1; i++)
   {
       if (sum_row[i] != sum_row[i+1])
          return false;

       if (sum_col[i] != sum_col[i+1])
          return false;
   }

   if (sum_row[0] != sum_col[0])
         return false;

   if (sum_row[0] != sum_maindiag[0])
         return false;

   if (sum_row[0] != sum_other[0])
         return false;

   return true;

}

一些评论：

1. 在 if、for、while 语句之后不需要括号为 1 个命令

   在 if、for、while 语句

2. 建议使用 if ->否则如果 ->别的。在这里，这并不重要，因为一旦发现不正确的地方，您就会跳出函数，但如果您继续代码，即使没有必要，您也会检查其他语句。

3. 习惯某种风格，创造自己的风格或复制别人的风格。我个人是这样使用括号的：

   如果（某事！=某事Else）{
   doSomeNastyThings();
   doEvenMore();
   }

好运..

编辑：添加变量声明 int for 语句，更新括号（聪明的想法，因为最后 3 个 if-s 不使用索引）

Answer 2

如果语句的格式如下：

if (condition) {
  do_this()
}

不是这样：

{
if (condition)
  do_this()
}

它们是您格式化代码的方式，您在两行后关闭 for 循环，我想这不是您想要做的（因为您指的是 var我后来）。