52

我有以下代码：

int M = 3; 
int C = 5; 
int match = 3;
for ( int k =0; k < C; k ++ )
{
    match --; 
    if ( match == 0 && k = M )
    {
         std::cout << " equals" << std::endl;
    }
}

但它给出了一个错误说：

错误：表达式必须是可修改的值

在那条“如果”线上。我不是想在这里修改“匹配”或“k”值，但为什么会出现这个错误？如果我只这样写：

if ( match == 0 )

没关系。有人可以向我解释吗？

4

4 回答 4

63

赋值运算符的优先级低于&&，因此您的条件相当于：

if ((match == 0 && k) = m)

但是它的左边是一个右值，即从 subexpression 的评估得到的布尔值match == 0 && k，所以你不能分配给它。

相比之下，比较具有更高的优先级，因此match == 0 && k == m等价于：

if ((match == 0) && (k == m))
于 2012-10-05T11:49:23.713 回答
42

在 C 中，如果您声明 a，您也会遇到相同的错误：

char array[size];

而不是尝试在不指定索引位置的情况下分配值：

array = '\0';

通过做：

array[index] = '0\';

您正在指定先前声明的可访问/可修改地址。

于 2015-04-17T14:20:57.733 回答
5

你测试k = M而不是k == M.
也许这是你想做的，在这种情况下，写if (match == 0 && (k = M))

于 2012-10-05T11:48:23.260 回答
4

请记住，单曲=始终是 C 或 C++ 中的赋值。

你的测试应该是if ( match == 0 && k == M )你在测试中犯了一个错字k == M。

如果您的意思是k=M（即测试中的副作用分配），您应该出于可读性原因编写代码if (match == 0 && (k=m) != 0)，但大多数编码规则建议不要编写该代码。

顺便说一句，您的错误建议询问所有警告（例如，-Wall选项g++），并升级到最近的编译器。下一个 GCC 4.8 将为您提供：

 % g++-trunk -Wall -c ederman.cc
 ederman.cc: In function ‘void foo()’:
 ederman.cc:9:30: error: lvalue required as left operand of assignment
          if ( match == 0 && k = M )
                               ^

而 Clang 3.1 也告诉你ederman.cc:9:30: error: expression is not assignable

因此，请使用最新版本的免费编译器，并在使用它们时启用所有警告。

于 2012-10-05T11:48:06.193 回答