Question

I’ve got an image upload page that works just fine when I only upload the files.

I added a ‘Create Thumbnail’ function. It looks like the file system has a handle on the images when the thumbnail process starts.

I get the ‘unspecified GDI+ error’ only when the image is over about 250K. When the files are below 250K, thumbnails are created as expected.

What are my options? Is there an elegant solution here? I want something not hacky.
Also, I am using HttpFileCollection so we can upload multiple images at one time. I’ve tried to use .Dispose on the Thumbnail creation, but it fails before we get to this point.

public void Upload_Click(object Sender, EventArgs e)
{
 string directory = Server.MapPath(@"~\images\");
  HttpFileCollection hfc = Request.Files;
  for (int i = 0; i < hfc.Count; i++)
  {
    HttpPostedFile hpf = hfc[i];

    if (hpf.ContentLength > 0)
    {
      string fileName = hpf.FileName;
      fileName = fileName.Replace(" ", "");
      hpf.SaveAs(fileName);
      createThumbnail(fileName);
    }
  }
}

private void createThumbnail(string filename)
{
  Image image = Image.FromFile(filename);
  Image thumb = image.GetThumbnailImage(100,100, () => false, IntPtr.Zero);
  thumb.Save(filename);
  image.Dispose();
  thumb.Dispose();
 }

Answer 1

Please let me know if this works any better:

    public string ImageDirectory { get { return Server.MapPath(@"~\images\"); } }

    public void OnUploadClick(object sender, EventArgs e)
    {
        var files = HttpContext.Request.Files.AllKeys.AsEnumerable()
            .Select(k =>HttpContext.Request.Files[k]);

        foreach(var file in files)
        {
            if(file.ContentLength <= 0)
                continue;

            string savePath = GetFullSavePath(file);
            var dimensions = new Size(100, 100);

            CreateThumbnail(file,savePath,dimensions);
        }
    }

    private void CreateThumbnail(HttpPostedFile file,string savePath, Size dimensions)
    {
        using (var image = Image.FromStream(file.InputStream))
        {
            using (var thumb = image.GetThumbnailImage(dimensions.Width, dimensions.Height, () => false, IntPtr.Zero))
            {
                thumb.Save(savePath);
            }
        }
    }

    private string GetFullSavePath(HttpPostedFile file)
    {
        string fileName = System.IO.Path.GetFileName(file.FileName).Replace(" ", "");
        string savePath = System.IO.Path.Combine(this.ImageDirectory, fileName);
        return savePath;
    }

Edit –

The foreach should have followed more to this pattern:

var files = HttpContext.Request.Files.AllKeys.AsEnumerable()
                .Select(k =>HttpContext.Request.Files[k]);

foreach(var file in files)
{

}