IEnumerable<Request.Files> 上的 ForEach 扩展方法的问题

Question

在对 ASP.Net (4.0) Request.Files（上传）集合进行一些基本验证时，我决定尝试使用 LINQ。

集合是IEnumerable<T>，因此不提供 ForEach。愚蠢地我决定建立一个可以完成这项工作的扩展方法。很抱歉说没有那么成功...

运行扩展方法（如下）引发下一个错误：

无法将“System.String”类型的对象转换为“System.Web.HttpPostedFile”类型

显然有些东西我没有得到，但我看不到它是什么，所以冒着看起来像个白痴的风险（不会是第一次）这里是 3 块代码，以及一个承诺感谢任何帮助。

一、带有Action参数的扩展方法：

//Extend ForEach to IEnumerated Files
public static IEnumerable<HttpPostedFileWrapper> ForEach<T>(this IEnumerable<HttpPostedFileWrapper> source, Action<HttpPostedFileWrapper> action)
{
   //breaks on first 'item' init
   foreach (HttpPostedFileWrapper item in source)
        action(item);
    return source;
}

当内部 foreach 循环命中 'source' 中的 'item' 时会发生错误。

这是调用代码（变量 MaxFileTries 和 attachPath 之前已正确设置）：

var files = Request.Files.Cast<HttpPostedFile>()
    .Select(file => new HttpPostedFileWrapper(file))
    .Where(file => file.ContentLength > 0
        && file.ContentLength <= MaxFileSize
        && file.FileName.Length > 0)
    .ForEach<HttpPostedFileWrapper>(f => f.SaveUpload(attachPath, MaxFileTries));

最后，Action 目标，保存上传文件 - 我们似乎从来没有到过这里，但以防万一，这里是：

public static HttpPostedFileWrapper SaveUpload(this HttpPostedFileWrapper f, string attachPath, int MaxFileTries)
{
    // we can only upload the same file MaxTries times in one session
    int tries = 0;
    string saveName = f.FileName.Substring(f.FileName.LastIndexOf("\\") + 1);   //strip any local
    string path = attachPath + saveName;
    while (File.Exists(path) && tries <= MaxFileTries)
    {
        tries++;
        path = attachPath + " (" + tries.ToString() + ")" + saveName;
    }
    if (tries <= MaxFileTries)
    {
        if (!Directory.Exists(attachPath)) Directory.CreateDirectory(attachPath);
        f.SaveAs(path);
    }
    return f;
}

我承认上面的一些内容是“发现的一些东西”的拼凑，所以我可能会得到我应得的，但如果有人对此有很好的理解（或至少经历过），也许我可以学习一些东西。

谢谢。

Answer 1

你为什么不直接在原来的IEnumerable<T>上调用ToList().ForEach()。

Answer 2

我想这就是你想要的

你的扩展 HttpFileCollection 的类应该是

  public static class HttpPostedFileExtension
  {
    //Extend ForEach to IEnumerated Files
    public static void ProcessPostedFiles(this HttpFileCollection source, Func<HttpPostedFile, bool> predicate, Action<HttpPostedFile> action)
    {
      foreach (var item in source.AllKeys)
      {
        var httpPostedFile = source[item];
        if (predicate(httpPostedFile))
          action(httpPostedFile);
      }
    }  
  }

然后你可以像这样使用它：

  Request.Files.ProcessPostedFiles(
  postedFile =>
  {
    return (postedFile.ContentLength > 0 && postedFile.FileName.Length > 0);      
  },
  pFile =>
  {
    //Do something with pFile which is an Instance of HttpPosteFile
  });

Answer 3

首先，写一个常用的扩展方法：

public static IEnumerable<T> ForEach<T>(this IEnumerable<T> source, Action<T> action)
{
   foreach (T item in source)
        action(item);
    return source; // or void, without return
}

那么，Request.Files 可以转换为IEnumerable<string>，而不是IEnumerable<HttpPostedFile>：

IEnumerable<string> requestFiles = this.Request.Files.Cast<string>();

但实际上System.Web.UI.WebControls.FileUpload.PostedFile 是HttpPostedFile：

HttpPostedFile file = fileUpload.PostedFile;
HttpPostedFileWrapper wrapper = new HttpPostedFileWrapper(file);

但它是单一的，而不是集合。你的收藏是从哪里得到的？

---

另一种扩展方法：

public static IEnumerable<HttpPostedFile> ToEnumerable(this HttpFileCollection collection)
{
    foreach (var item in collection.AllKeys)
    {
        yield return collection[item];
    }
}

用法：

IEnumerable<HttpPostedFile> files = this.Request.Files.ToEnumerable();
IEnumerable<HttpPostedFileWrapper> wrappers = files.Select(f => new HttpPostedFileWrapper(f));