C# 新手问题第 2 部分 - 声明类和方法答案

【问题标题】：C# newbie problem part 2 - declaring class and methodC# 新手问题第 2 部分 - 声明类和方法
【发布时间】：2010-03-26 16:06:11
【问题描述】：

[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Upload(Photo photo)
{
    foreach (string file in Request.Files)
    {

        var path = "~/Uploads/Photos/";

        HttpPostedFileBase posted = (HttpPostedFileBase)Request.Files[file];

        if (posted.ContentLength > 0)
        {
            photo.Date = DateTime.Now;
            photo.AlbumID = 1;
            photo.Title = Path.GetFileName(posted.FileName);

            photoRepository.Add(photo);
            photoRepository.Save();

            posted.SaveAs(Server.MapPath(path + photo.PhotoID + ".jpg"));

            Image img1 = Image.FromFile(Server.MapPath("~/Uploads/Photos/") + photo.PhotoID + ".jpg");

            imageHelper.CreateThumbnail(posted, img1);

            int newWidth = 100;
            int newHeight = 100;
            double ratio = 0;

            if (img1.Width > img1.Height)
            {
                ratio = img1.Width / (double)img1.Height;
                newHeight = (int)(newHeight / ratio);
            }
            else
            {
                ratio = img1.Height / (double)img1.Width;
                newWidth = (int)(newWidth / ratio);
            }

            Image bmp1 = img1.GetThumbnailImage(newWidth, newHeight, null, IntPtr.Zero);
            bmp1.Save(Server.MapPath("~/Uploads/Photos/Thumbnails/") + photo.PhotoID + ".jpg");

            img1.Dispose();
            bmp1.Dispose();
        }
    }

    return RedirectToAction("Index");
}

我想更好地组织这段代码，如下所示：

[AcceptVerbs(HttpVerbs.Post)]
public ActionResult Upload(Photo photo)
{
    foreach (string file in Request.Files)
    {

        var path = "~/Uploads/Photos/";

        HttpPostedFileBase posted = (HttpPostedFileBase)Request.Files[file];
        ImageHelper imageHelper = new ImageHelper();  

        if (posted.ContentLength > 0)
        {
            photo.Date = DateTime.Now;
            photo.AlbumID = 1;
            photo.Title = Path.GetFileName(posted.FileName);

            photoRepository.Add(photo);
            photoRepository.Save();

            posted.SaveAs(Server.MapPath(path + photo.PhotoID + ".jpg"));

            Image img1 = Image.FromFile(Server.MapPath("~/Uploads/Photos/") + photo.PhotoID + ".jpg");

            // Create thumbnail
            imageHelper.CreateThumbnail(posted, img1);                    
        }
    }

    return RedirectToAction("Index");
}

在这里，我在 Helpers 文件夹中创建了处理缩略图的类和方法：

public class ImageHelper
{        
    public void CreateThumbnail(HttpPostedFileBase posted, Image img1)
    {
        int newWidth = 100;
        int newHeight = 100;
        double ratio = 0;

        if (img1.Width > img1.Height)
        {
            ratio = img1.Width / (double)img1.Height;
            newHeight = (int)(newHeight / ratio);
        }
        else
        {
            ratio = img1.Height / (double)img1.Width;
            newWidth = (int)(newWidth / ratio);
        }

        Image bmp1 = img1.GetThumbnailImage(newWidth, newHeight, null, IntPtr.Zero);
        bmp1.Save(Server.MapPath("~/Uploads/Photos/Thumbnails/") + photo.PhotoID + ".jpg");

        img1.Dispose();
        bmp1.Dispose();
    } 
}

但是我得到编译错误 Server（在 bmp1.Save(Server.MapPath... 行的 ImageHelper 类中）在当前上下文中不存在，而如果代码在一个地方它可以正常工作。
我做错了什么，这是声明方法和组织代码的正确方式吗？

提前致谢，
伊莱

【问题讨论】：

标签： c# asp.net-mvc class methods declaration

【解决方案1】：

Server 是 Controller 类的属性。这就是为什么它可以在Update 操作中访问。要从外部使用它，您可以通过HttpContext.Current.Server 获取它。

然而，这将使您的代码更难测试，并且与 ASP.NET 细节的耦合更紧密。而是重写此方法，使其独立于任何与 ASP.NET 相关的东西。

在这种特殊情况下，您可以向该方法添加额外的参数，这将告诉它保存缩略图的位置：

void CreateThumbnail(Image img1, string targetDirectory)

【讨论】：

【解决方案2】：

与其让您的 CreateThumbnail 方法直接从“服务器”调用事物，不如将这些资源传递给您的函数。这允许更好地分离关注点，并最终实现更好的可重用性。此外，它还将纠正您的问题。

【讨论】：