-5

我有下面的代码，它可以满足我的需要，但我有一个想法，它可以做得更快。请让我知道是否可以以任何方式改进此代码...

主要问题是我需要多次查询“数据”。我只需要确保没有我可以使用的快捷方式。

data= GetData()// this return ILookup<Tuple(string, string, string),string>
foreach (var v0 in data)
{
    if (v0.Key.Item3 == string.Empty)
    {
        //Get all related data
        var tr_line = data[v0.Key];
        sb.AppendLine(tr_line.First());

        foreach (var v1 in data)
        {
            if (v1.Key.Item2 == string.Empty && v1.Key.Item1 == v0.Key.Item1)
            {
                var hh_line = data[v1.Key];
                sb.AppendLine(hh_line.First());

                foreach (var v2 in data)
                {
                    if (v2.Key.Item1 == v0.Key.Item1 && v2.Key.Item2 != string.Empty && v2.Key.Item3 != string.Empty)
                    {
                        var hl_sl_lines = data[v2.Key].OrderByDescending(r => r);
                        foreach (var v3 in hl_sl_lines)
                        {
                            sb.AppendLine(v3);
                        }
                    }
                }
            }
        }
    }
 }
4

2 回答 2

1

首先，尽量避免在这种代码中使用元组，因为即使对你来说，几个月后，这段代码也会变得难以理解。使用正确的属性名称创建一个类，甚至更好的不可变结构。如果无法维护，即使是最快的代码也毫无价值。

也就是说，您有三个迭代同一集合的嵌套循环。排序后的集合执行得更快是合理的，因为您只需要与相邻的项目进行比较。

请尝试解释您要完成的工作，以便有人会尝试提供更具体的帮助。

于 2012-09-14T13:10:44.057 回答
1

更整洁，更多 linq：

        var data = GetData();

        foreach (var v0 in data)
        {
            if (v0.Key.Item3 != string.Empty) continue;

            //Get all related data 
            var tr_line = data[v0.Key];
            sb.AppendLine(tr_line.First());

            var hhLines = from v1 in data
                          where v1.Key.Item2 == string.Empty &&
                                v1.Key.Item1 == v0.Key.Item1
                          select data[v1.Key];

            foreach (var hh_line in hhLines)
            {
                sb.AppendLine(hh_line.First());

                var grouping = v0;
                var enumerable = from v2 in data
                                 where v2.Key.Item1 == grouping.Key.Item1 &&
                                       v2.Key.Item2 != string.Empty &&
                                       v2.Key.Item3 != string.Empty
                                 select data[v2.Key].OrderByDescending(r => r)
                                 into hl_sl_lines from v3 in hl_sl_lines select v3;

                foreach (var v3 in enumerable)
                {
                    sb.AppendLine(v3);
                }
            }
        }
于 2012-09-14T13:16:48.237 回答