如何通过使用“？”来缩小这个特定的 if 行（混乱）操作员

Question

我正在开发一个项目，我必须查看别人的代码并对其进行修改。 然而，由于我班上的很多同学都是编程新手，所以很多人的组织都很混乱。 我被指派临时编写的代码有很多缺陷和凌乱的冗余行。 我正在尽力清理它们，但是由于我的经验不足，我发现清理它们很难。 诸如此类的线条

if (turnElapsed[1] == 2)
{
    turnElapsed[0] += 1;
    turnElapsed[1] = 0;
}
turnElapsed[1]++;

对我来说看起来相当多余。 我相信，并且一定有更好的方法来编写它的简单版本。 所以我尝试了下面的代码，但它似乎无法正常工作。

turnElapsed[0] += (turnElapsed[1]++ == 2) ? 1 ; 0 ;
turnElapsed[1] = (turnElapsed[1] == 2 ) ? 0; turnElapsed[1];

Answer 1

首先，您使用 ; 作为分隔符，而不是 :，这是一个语法错误。其次，您在第一行增加 turnElapsed[1] ，这意味着当它到达第二行时，它将不再等于 2 - 这与原始逻辑不同。这就是您的版本无法正常工作的原因。

但是，如果您确实修复了这些错误，我认为您的代码不会更容易阅读。原文更具可读性，因为它更清楚地表达了意图。您可以阅读该内容并将其表述为“如果turnElapsed[1] 为2，则...”。您的替代方案需要更少的行数，但更“神秘”。原始代码的另一个优点是，如果您想在 if 条件为 true 时中断，可以在大括号内放置一个断点 - 您不能使用三元运算符 (?) 来做到这一点。

Answer 2

你的问题是它不可读和不可理解。这就是为什么你看不到你的错误。
错误在于您不会在第二个版本中增加。试试这个

turnElapsed[0] += (turnElapsed[1]++ == 2) ? 1 : 0 ;
turnElapsed[1] = ((turnElapsed[1] == 2 ) ? 0; turnElapsed[1]) + 1;

Answer 3

难道你不只是在turnElapsed中保留一个二进制数，而最高有效位在turnElapsed[0]中吗？代码的更好版本是：

long turnElapsed;

...

turnElapsed++;

或者，如果您真的不想开始搞乱位：

turnElapsed[0] += turnElapsed[1]
turnElapsed[1] = (1 + turnElapsed[1]) % 2

编辑： 显然，turnElapsed[1] 是 0 或 1，并且每次turnElapsed 从 1 变化时，turnElapsed[0] 都会递增到 0。所以你有以下内容：

long number;

...

number++;
turnElapsed[0] = number/2;
turnElapsed[1] = number%2;

不需要 ifs、?:'s 或其他任何东西。事实上，您甚至不需要该数组。