Java三元运算符和设置循环索引值

Question

我有一个循环遍历 ArrayList 的 for 循环。

如果 for 循环中满足条件：

• 我从 ArrayList 中删除当前元素
• 减少 ArrayList 局部变量的大小 减少
• for 循环的索引，如下所示，检查确保它们永远不会低于零。

我们刚刚删除了 ArrayList 的最后一个元素的情况：

i = (i > 0) ? i-- : i;

我的问题是，当 i > 时，上面的方法不会将 i 减少 1。 0.我已经无数次使用三元运算符，但从未见过这种行为。我测试过 i 确实是 > 0 并且 i-- 部分正在被调用。它根本没有减少 i 的值。删除 0 值检查并简单地运行 i--; 确实会按预期减少 i 。

EDIT2：好吧，有人编辑了我的上一次编辑，我在其中提到我在这种情况下特别不使用 ListIterator，因为循环本身对性能敏感，位于 Android 代码的关键部分。

Answer 1

i-- 递减 i，但返回原始值。

i = i-- 会递减 i，然后将其赋给其原始值。

您应该使用i - 1。

Answer 2

最小修复：

您可能会遇到这样的问题：

for (int i = 0; i < l.size(); i++) {         <--------------------------------.
    if (cond) {                                                               |
        l.remove(i);                                                          |
        i--;              // even if i == -1, it will be set back to 0 here --'
    }
}

---

向后迭代：

另一个常见的解决方案是向后迭代，如下所示：

for (int i = l.size() - 1; i >= 0; i--) {
    if (cond) {
        l.remove(i);
    }
}

---

使用 ListIterator（除非对性能至关重要）：

但是，使用 ListIterator 会更好：

Iterator<String> iter = l.iterator();
while (iter.hasNext()) {
    if (shouldRemove(iter.next())
        iter.remove();
}

Answer 3

你尝试过吗？：

i = (i > 0) ? --i : i;

通过预减，你应该解决所有问题。

Answer 4

像 i = i-- 这样的赋值（对于 i > 0 会发生这种情况）根本没有意义：您到底在这里期望什么？使用以下内容代替：

i = (i > 0) ? i - 1 : i;

或者更好：

if (i > 0)
    i--;

Answer 5

做正确的事情：使用迭代器循环数组列表并在迭代时安全地删除项目。

Answer 6

您所描述的算法对于您尝试执行的操作来说是一个糟糕的策略：从 ArrayList 中删除元素是一个 O(n) 操作，因此在一般情况下将其应用于整个 ArrayList 是 O(n^2) 。

更好的策略：

• 创建一个新的空 ArrayList
• 循环遍历原始 ArrayList 一次，然后仅将要保留的元素添加到新的 ArrayList

这是 O(n)，并且实际上会是更简单的代码（您不必担心摆弄循环边界等）

Answer 7

考虑不要将三元运算符与 noop 一起使用。

如果（i>0）
{
- 我
}

准确描述了您想要的功能（基于您的示例）。

Answer 8

正如其他人所说，i-- 递减 i 并返回原始值，这意味着您的代码将执行递减，但随后立即将其设置回原始值。

您可以将其更改为 --i，在这种情况下，将返回递减的值，但无论哪种方式，您都在使用 -- 进行不必要的工作，因为它必须设置 i，返回新值，然后再次将 i 设置为新值。

你最好只做 i-1 ，它不会两次设置 i 的值：

i = (i > 0) ? i-1 : i;

话虽如此，因为三元的 false 选项不会'不做任何事情，您可能最好使用简单的 if()：

if(i > 0) { i--; }

它更容易阅读，无论 -- 的方向如何，并且无论结果如何，都不进行任何不必要的处理。