Question

我一直得到“只允许图片”，我试过“file.PostedFile.FileName”也行不通!!

此代码是在一个单独的类中编写的。

public static String UploadFile(FileUpload file, String type, out String filename)
{
    String ext = System.IO.Path.GetExtension(file.FileName);
    filename = "";
    if (file.PostedFile.ContentLength > 2000000)
    {
        return "File is larger than 2 MB";
    }
    else if (type != "File")
    {
        if (ext.ToLower() != ".jpg" || ext.ToLower() != ".png" || ext.ToLower() != ".gif" || ext.ToLower() != ".jpeg")
        {
            return "Only images are allowed";
        }
        else
        {
            filename = System.IO.Path.GetRandomFileName() + "_" + file.PostedFile.FileName;
            String root = HttpContext.Current.Server.MapPath("~/Images/");
            file.SaveAs(root + type + "/" + filename);
            return "Success";
        }
    }
    else
    {
        filename = System.IO.Path.GetRandomFileName() + "_" + file.PostedFile.FileName;
        String root = HttpContext.Current.Server.MapPath("~/Files/");
        file.SaveAs(root + filename);
        return "Success";
    }
}

Answer 1

你的情况有误，应该如下：

if (ext.ToLower() != ".jpg" && ext.ToLower() != ".png" && ext.ToLower() != ".gif" && ext.ToLower() != ".jpeg")
{
return "Only images are allowed";
}
else
{
///statement
}

OR

   if (ext.ToLower() == ".jpg" || ext.ToLower() == ".png" || ext.ToLower() == ".gif" || ext.ToLower() == ".jpeg")
    {

    ///statement
    }
    else
    {
return "Only images are allowed";
    }

Answer 2

@ volpav的答案将解决您的问题，但是大if并不是解决问题的最简单方法。

更优雅的是定义接受的扩展名列表，并检查列表中是否有ext。这样做的好处是，如果您以后必须更改有效类型，则更容易维护，并且如果需要，您可以使用户可以定义扩展名。

在下面的示例中，我为我的类定义了一个常量（well readonly变量），其中包含一个包含所有异常的数组，并使用Contains()扩展方法来测试是否存在ext在UploadFile

中验证时

public static readonly string[] VALID_EXTENSIONS = 
    new string[4] { ".png", ".jpg", ".gif", ".jpeg" };

// in UploadFile ...
    if (!VALID_EXTENSIONS.Contains(ext.ToLower())) { 
         return "Only images are allowed";
    }

通过在上面的代码中使其静态，我可以在UI中使用此列表来指示什么是例外扩展，而不是让用户猜测什么是有效的图像类型（毕竟，除了其他图像类型之外你所包括的那些）。

Answer 3

检查有效扩展名的条件在逻辑上是不正确的（始终评估为true）。它应该是这样的（||被&&替换）：

if (ext.ToLower() != ".jpg" && ext.ToLower() != ".png" && ext.ToLower() != ".gif" && ext.ToLower() != ".jpeg")
{
    return "Only images are allowed";
}

获取发布的文件扩展名asp.net

3 个答案: