Загрузите изображение, переименуйте его, сделайте эскиз и замените оригинал. (оптимизация)

Я создал эти функции, которые я описал в вопросе. Однако я считаю, что то, как я это сделал, не является оптимальным способом.

        [HttpPost]
        public ActionResult Create(FormCollection collection, string schooljaarparam, FlatONASAanbieder foa) {

        if (ModelState.IsValid) {

            // var r = new List<ViewDataUploadFilesResult>();

            foreach (string file in Request.Files) {
                HttpPostedFileBase hpf = Request.Files[file] as HttpPostedFileBase;
                if (hpf.ContentLength == 0)
                    continue;

                //extensie nakijken. jpg, png, jpeg, of GIF. 
                if (MvcApplication.isImage(hpf.FileName)) {
                    //Image img = new Image();


                    string savedFileName = Path.Combine(
                       AppDomain.CurrentDomain.BaseDirectory + "uploads\\ONAS\\",
                       Path.GetFileName(hpf.FileName));
                    FileInfo fi = new FileInfo(savedFileName);

                    int i = 1;
                    while (fi.Exists) {
                        fi = new FileInfo(savedFileName.Substring(0, savedFileName.Length - Path.GetFileName(savedFileName).Length) + Path.GetFileNameWithoutExtension(savedFileName) + " (" + i++ + ") " + Path.GetExtension(savedFileName));
                    }
                    savedFileName = fi.DirectoryName + "\\" + fi.Name;
                    hpf.SaveAs(savedFileName);

                    using (Image Img = Image.FromFile(savedFileName)) {
                        //Size ThumbNailSize = NewImageSize(Img.Height, Img.Width, 79);
                        Size NewSize = VerkleinMaxHoogte(Img.Size, 79);

                        using (Image ImgThnail = new Bitmap(Img, NewSize.Width, NewSize.Height)) {
                            //string ss = savedFileName.Substring(0, savedFileName.Length - Path.GetFileName(savedFileName).Length) + Path.GetFileNameWithoutExtension(savedFileName) + "-thumb" + Path.GetExtension(savedFileName);
                            ImgThnail.Save(savedFileName + ".tmp", Img.RawFormat);
                            ImgThnail.Dispose();
                        }
                        Img.Dispose();
                    }
                    System.IO.File.Delete(savedFileName);
                    FileInfo f = new FileInfo(savedFileName + ".tmp");
                    f.MoveTo(savedFileName);


                } else {
                    ModelState.AddModelError("ONAS_Logo", "Het geuploadde bestand is geen afbeelding. ");

                }

                //r.Add(new ViewDataUploadFilesResult() {
                //    Name = savedFileName,
                //    Length = hpf.ContentLength
                //});
            }
        }

        // return View("UploadedFiles", r);

        return View();
    }


    [NonAction]
    public Size VerkleinMaxHoogte(Size orig, double height) {
        double tempval = height / orig.Height;

        return new Size(Convert.ToInt32(tempval * orig.Width), Convert.ToInt32(height));
    }

в global.asax

    public static bool isImage(string s) {
        if (s.EndsWith(".jpg", true, null) || s.EndsWith(".jpeg", true, null) || s.EndsWith(".gif", true, null) || s.EndsWith(".png", true, null)) {
            return true;
        }
        return false;
    }

так, как я это делаю:

  1. Я получаю файл из браузера
  2. Я проверяю, есть ли это изображение
  3. Я проверяю, существует ли файл, и если да, меняю имя файла соответствующим образом
  4. Я сохраняю файл на диске (ввод-вывод, медленно)
  5. Я открываю файл как изображение
  6. I вычислить ширину и высоту с помощью метода VerkleinMaxHoogte
  7. Я создаю эскиз и сохраняю его с расширением tmp
  8. Я удаляю исходный файл
  9. Я переименовываю эскиз в имя исходного файла (это то, что я хочу)

Как сделать это быстрее?

1
задан Stefanvds 31 August 2010 в 14:20
поделиться