为什么 i++ 是“无法访问的代码”？

Question

编辑

我把它留在这里，即使它让我看起来很愚蠢，因为如果你在深夜工作并且不注意，它可能会咬你。感谢 Visual Studio 拥有如此智能的解析器。

基本上我错过了我有嵌套循环，所以continue 语句在这里基本上毫无价值，因为它继续foreach 循环，而不是for 循环。

原始问题

我正在对工作簿进行搜索，寻找与所有字符串搜索条件匹配的工作表。在 Visual Studio 编辑器中，i++ 下划线为“无法访问的代码”。

/// <summary>
/// Finds the first sheet that has cells that match all the criteria.
/// </summary>
/// <param name="wb"></param>
/// <param name="searches"></param>
/// <returns></returns>
public static ISheet FindSheet( this IWorkbook wb, params string[] searches )
{
    if( null == wb || null == searches )
        return null;

    for( int i = 0; i < wb.NumberOfSheets; i++ )
    {
        var sheet = wb.GetSheetAt( i );
        foreach( var search in searches )
        {
            var cell = sheet.FindCell( search );
            if( null == cell )
                continue;
        }

        return sheet;
    }

    return null;
}

我认为continue 语句在这里有一个明确的含义：“如果任何搜索条件返回null 单元格，则继续下一次迭代。否则，只需返回在此迭代中找到的工作表。”

没有继续声明的更正代码

/// <summary>
/// Finds the first sheet that has cells that match all the criteria.
/// </summary>
/// <param name="wb"></param>
/// <param name="searches"></param>
/// <returns></returns>
public static ISheet FindSheet( this IWorkbook wb, params string[] searches )
{
    if( null == wb || null == searches )
        return null;

    for( int i = 0; i < wb.NumberOfSheets; i++ )
    {
        var sheet = wb.GetSheetAt( i );
        if( searches.All( s => null != sheet.FindCell( s ) ) )
            return sheet;
    }

    return null;
}

Answer 1

在这里，您的 for 循环完全没用。在第一次迭代之后，无论如何它都会返回。当然你的 foreach 循环有一个continue 语句，但这仅适用于foreach 循环。这就是您的代码无法访问的原因。

因此，将只分析第一张纸。如果这是您想要的，您可以简单地删除 for 循环并将工作表定位到索引 0，或者您需要重新排列循环。

Answer 2

Eric Lippert 有an SO answer 和a blog post 说明当您处于内循环时想要继续外循环时可以做什么。

他标记为“真棒”的技术是在可能的情况下摆脱所有循环，事实上我相信这里是可能的：

public static ISheet FindSheet( this IWorkbook wb, params string[] searches )
{
    if( null == wb || null == searches ) { return null; }

    return wb.FirstOrDefault(sh => searches.All(sr => sh.FindCell(sr) != null));
}