c++ - 标准::交换()；在 else 语句中使用时不起作用

Question

这是一组条件语句，它根据特定变量使用swap 函数在 vector 中移动一个char:

 if (m == 's')
{
  if (currenti == 5)
  {
    swap(height[currenti][currentj], height[0][currentj]);
  }
  else if (height[currenti++][currentj] == 'T')
  {
    cout << "Game over!" << endl;
    gameLive = 0;
  }
  else
  {
    swap(height[currenti][currentj], height[currenti++][currentj]);
  }
}

由于某些原因，else 部分中的 swap 不会交换两个 vector 部分，即使它被调用也是如此。 (我在 else 部分输入了一个 cout 来检查)。但是，如果我像这样删除其他条件:

if (m == 's')
{
  swap(height[currenti][currentj], height[currenti++][currentj]);
}

它工作得很好。

有谁知道是什么导致它不能以"else" 格式工作？

Answer 1

这一行:

swap(height[currenti][currentj], height[currenti++][currentj]); // Don't do it!!

正在与自身交换一个值!表达式 height[currenti++][current] 将 currenti 的 current 值作为第一个索引，然后递增它。

你应该为你的 else block 使用这样的代码:

else
{
    swap(height[currenti][currentj], height[currenti+1][currentj]);
    ++currenti;
}

注意:对第二个参数中的索引使用预递增运算符很有诱惑力:

swap(height[currenti][currentj], height[++currenti][currentj]); // Don't do it!!

但这是未定义的行为，因为std::swap 的参数求值顺序未指定。两者之一(后增量或前增量)可能在某些平台上工作，某些时间，但您可以永远不要依赖它。