Question

我的以下代码存在一些问题。

var groupedItems = inputFiles.GroupBy(q => q.Name.ToLower().Split('_').ElementAt(2));
        string currentNo = ////value retreived from someMethod;
        if (string.IsNullOrEmpty(currentNo))
        {
            if (groupedItems.Count() > 1)
            {
                foreach (var group in groupedItems)
                {
                    foreach (var groupedItem in group)
                    {
                        ErrorFile(groupedItem);
                    }
                }
            }
            else if (groupedItems.Count() == 1)
            {
                ProcessFile();
            }
        }
        else
        {
            foreach (var group in groupedItems.Where(x => x.Key != currentNo))
            {
                foreach (var groupedItem in group)
                {
                    ErrorFile(groupedItem);
                }
            }
        }

有嵌套的foreach循环的重复代码。我正在考虑优化的可能性
当Split和ElementAt（2）返回错误时如何处理。即使我无法通过_分割，我仍然需要调用ErrorFile（）方法。

Answer 1

如果没有更多的上下文，很难理解你的代码在做什么，但这应该是正确的：

static void SomeMethod(IEnumerable<File> inputFiles)
{
    var groupedItems = inputFiles.GroupBy
        (f => splitAndFindElement(f.Name, '_', 2, string.Empty));

    string currentNo = //whatever;

    if (string.IsNullOrEmpty(currentNo))
    {
        if (groupedItems.Count() > 1)
        {
            foreach (var item in groupedItems.SelectMany(g => g))
            {
                ErrorFile(item);
            }
        }
        else if (groupedItems.Count() == 1)
        {
            ProcessFile();
        }
    }
    else
    {
        foreach (var item in groupedItems.Where(g => g.Key != currentNo).SelectMany(g => g))
        {
            ErrorFile(item);
        }
    }
}

辅助方法：

static string splitAndFindElement(string input, char splitter, int index, string resultOnFail)
{
    var succesful = false;
    string[] words = null;

    if (input != null)
    {
        words = input.Split(splitter);
        succesful = words.Length > index;
    }

    return succesful ? words[index] : resultOnFail;
}

这里的诀窍是如果文件不能被分割，则将文件与密钥""分组。这将确保使用ErrorFile处理它们，或者因为有多个分组或因为密钥不等于currentNo。我假设Name无法在"_"结束。

此外，SelectMany用于展平枚举的可枚举并避免嵌套循环。

Answer 2

我们可以优化重复循环如下，并处理错误使用try catch

var groupedItems;
try
{
    groupedItems= inputFiles.GroupBy(q => q.Name.ToLower().Split('_').ElementAt(2));

    string currentNo = ////value retreived from someMethod;
    if (string.IsNullOrEmpty(currentNo) && groupedItems.Count() == 1)
    {
        ProcessFile();
    }
    else
    {
        foreach (var group in groupedItems.Where(x => string.IsNullOrEmpty(currentNo) || x.Key != currentNo))
        {
            foreach (var groupedItem in group)
            {
                ErrorFile(groupedItem);
            }
        }
    }
}
catch
{
    ErrorFile(groupedItem);
}

C＃代码优化组

2 个答案: