如何使用一致的语法重写此LINQ表达式?

时间:2013-10-22 20:00:58

标签: c# linq

这个单一语句查询整齐地说“给我一个裸文件名列表,该文件是一个包含特定文件结构的ZIP存储库。”

但我同时使用.Where()扩展方法(流利语法)和select查询,因为我尝试的其他任何东西都无法编译。如果我将“.Where(file ==>< statement>)”更改为“where< statement>”,我会收到一条错误,即匿名方法代码没有返回bool,如果我更改了“select&lt” ;子句>”中“.Select(< clause>)”,错误是“没有使用select子句。”

我对查询或流利的语法感到满意,但我想解决其中一个问题。任何人都可以解释为什么这不起作用,以及我需要做些什么来解决一个一致的语法?

return (from file in Directory.EnumerateFiles(
                    Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.CommonApplicationData), Globals.CompanyName, ProjectName, FolderName),
                    imageExtension,
                    SearchOption.TopDirectoryOnly)
    .Where(file =>
    {
        try
        {
            string relativePath = ClassFru.Station + "/";   // Inside ZIPs, paths use a single forward slash
            var zip = new ZipFile();
            zip.ZipError += (s, o) => { throw new Exception(); };
            using (zip = ZipFile.Read(file))
            {
                /// <todo>if (zip.Comment != Globals.CompanyName) { return false; }</todo>
                foreach (var fru in this.gFrus)
                {
                    var fruPath = relativePath + fru.Id + '.';
                    if (!(from e in zip where !e.IsDirectory && e.FileName.StartsWith(fruPath) select true).Any()) { return false; }
                }
                return true;
            }
        }
        catch (Exception)
        {
            return false;
        }
    })
    select Path.GetFileNameWithoutExtension(file)).ToArray();

3 个答案:

答案 0 :(得分:3)

因为我没有你在这个表达式中使用的所有类型,所以很难编译它,但我想我应该这样工作:

            return (Directory.EnumerateFiles(
            Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.CommonApplicationData),
                Globals.CompanyName, ProjectName, FolderName),
            imageExtension,
            SearchOption.TopDirectoryOnly)
            .Where(file => {
                try
                {
                    string relativePath = ClassFru.Station + "/"; // Inside ZIPs, paths use a single forward slash
                    var zip = new ZipFile();
                    zip.ZipError += (s, o) => {
                        throw new Exception();
                    };
                    using (zip = ZipFile.Read(file))
                    {
                        /// <todo>if (zip.Comment != Globals.CompanyName) { return false; }</todo>
                        foreach (var fru in this.gFrus)
                        {
                            var fruPath = relativePath + fru.Id + '.';
                            if(zip.Any(e=> !e.IsDirectory && e.FileName.StartsWith(fruPath))
                                    .Any())
                            {
                                return false;
                            }
                        }
                        return true;
                    }
                } catch (Exception)
                {
                    return false;
                }
            }).Select(Path.GetFileNameWithoutExtension).ToArray());

答案 1 :(得分:1)

除了将select的使用更改为Select(file =>之外,您还需要在开始时删除from file in。然后,您将删除该查询语法select子句的使用。正是那个导致你看到的错误的from条款。每个from [...] in子句都需要匹配select

答案 2 :(得分:0)

  

我对查询或流利的语法感到满意,但我想解决其中一个问题。任何人都可以解释为什么这不起作用,以及我需要做些什么来解决一个一致的语法?

它对您来说效果不佳,因为LINQ并不是真正用来处理大块复杂逻辑作为标准。特别是LINQ表达式语法假设您将提供表达式(而不是块),因此它不直接支持多行语句。

如果您只是删除查询的from部分,则可以轻松地将方法语法设置为ISun shows you how to do

另一方面,如果您只是简单地提取匿名方法,那么您的代码将更容易理解和模块化,正如@Servy在评论中所建议的那样。在这种情况下,您可以决定说where FileHasMatchingZipStructure(file)。或.Where(FileHasMatchingZipStructure),如你所愿。