将 IEnumerable 转换为 List 以符合方法签名 [关闭]

Question

我有一个下面的代码，它返回所有文件的列表。它正在从文件夹和子文件夹中读取所有文件并将其作为列表返回给用户。

我在下面的代码中有两种情况。

案例1：

• 如果IS_PROC 为真，那么我会以不同的方式获取files，然后将其返回给用户。
• 在这种情况下，我有一个IEnumerable 查询，我将其转换回列表，因为我的方法签名是IList。不确定将IEnumerable 转换为List 是否是个好主意。

案例2：

• 如果IS_PROC 为假，我会以另一种方式获取files，然后将其返回给用户。
• 在这种情况下，我只有一个列表，所以我将其返回。

下面是我的代码：

private IList<string> ReadFiles(string path)
{
    var files = new List<string>();
    try
    {
        if (IS_PROC)
        {
            // case 1:
            IEnumerable<string> query =
                from directory in System.IO.Directory.EnumerateDirectories(path)
                let dev = System.IO.Path.Combine(directory, "dev")
                from file in System.IO.Directory.EnumerateFiles(directory)
                let fi = new System.IO.FileInfo(file)
                let dev_file = System.IO.Path.Combine(dev, fi.Name)
                select System.IO.File.Exists(dev_file) ? dev_file : file;
            // is this the right way to do it?
            return (!query.Any()) ? files : query.ToList();
        }
        // case 2:
        var lclJsonFiles = Directory.GetFiles(path, "*.json", SearchOption.AllDirectories);
        var lclTxtFiles = Directory.GetFiles(path, "*.txt", SearchOption.AllDirectories);
        files.AddRange(lclJsonFiles);
        files.AddRange(lclJsonFiles);
    }
    catch (Exception ex)
    {
        // log error here
    }
    return files;
}

所以我的问题是-

• 考虑到我在这两种情况下的操作，从我的方法中返回 IList 是否有意义？
• 我的一个案例返回IEnumerable，其他案例返回List，为了使其与我的方法签名兼容，我将IEnumerable 转换为Case1 中的列表？这是将IEnumerable 转换为List 的正确方法吗？
• 上面的方法有没有更好的写法？

Answer 1

你可以用任何一种方式编写方法，这真的取决于你想做什么。如果你想返回一个IEnumerable，你可以放弃ToList() 调用，它应该可以工作。您也可以将GetFiles 替换为Directory.EnumerateFiles，因为它还会返回IEnumerable：

private IEnumerable<string> ReadFiles(string path)
{
    try
    {
        return IS_PROC
            ? (from directory in Directory.EnumerateDirectories(path)
                let dev = Path.Combine(directory, "dev")
                from file in Directory.EnumerateFiles(directory)
                let fi = new FileInfo(file)
                let devFile = Path.Combine(dev, fi.Name)
                select File.Exists(devFile) ? devFile : file)
            : Directory
                .EnumerateFiles(path, "*.json", SearchOption.AllDirectories)
                .Union(Directory.EnumerateFiles(path, "*.txt", SearchOption.AllDirectories));
    }
    catch (Exception ex)
    {
        // log error here
        return new List<string>();
    }
}

否则，您可以将.ToList() 添加到返回值的末尾并返回IList。请注意，List 实现了 IEnumerable，因此无论哪种情况，您都可以只返回一个 List。