For-Loop 很奇怪答案

【问题标题】：For-Loops being weirdFor-Loop 很奇怪
【发布时间】：2013-10-19 14:23:52
【问题描述】：

由于某种原因，在这个 for 循环中，i 达到 1，并导致 index out of range 错误。 Items.Count 等于 4，我检查了使用断点，StockList.Count 也等于 4。我似乎无法弄清楚为什么我会达到一个，知道吗？

for (int i = 0; i <= (Items.Count / 4) - 1; i++)
{
    for (int ii = 0;ii <= Program.StockList.Count - 1;i++)
    {
        if (Items[(i * 4) + 3] == Program.StockList[ii].ID) //Crash here
        {
            MessageBox.Show(Program.StockList[ii].Name + " Match!");
        }
    }
}

【问题讨论】：

您在第二个 for 循环中递增 i。当这种情况发生时，我讨厌它。
选择好的变量名是这场战斗的 90%，i 和 ii 不是好名字。
只是为了添加到变量命名之战中，使用循环的一个很好的例子是它使用 i 表示第一维，j 表示第二维，k 表示第三维。

标签： c# .net winforms for-loop

【解决方案1】：

这个（第二个循环）：

for (int ii = 0;ii <= Program.StockList.Count - 1;i++)

应该是这样的：

for (int ii = 0;ii <= Program.StockList.Count - 1;ii++)

我确信在这里很难发现差异，因此在您的代码中更难发现这一点也就不足为奇了。考虑将j 用于内部循环，并将您的代码划分为更小的函数以避免此类错误。

正如kenny 在下面的 cmets 中所指出的，您可以将第二个循环替换为 foreach 循环：

foreach (var stock in Program.StockList)
{
    if (Items[(i * 4) + 3] == stock.ID)
    {
        //...
    }
}

【讨论】：

啊，就是这样。在我使用的以前的语言中，我不必做 i++ 的事情，所以我必须在嵌套循环中注意这一点。谢谢！
@Nathan 你知道 foreach 可以遍历集合吗？