帮忙优化下代码、

[著名天才___]

  if (file == "YaopinDetails.aspx" || file == "YaopinDetails_1.aspx" || file == "YaopinDetails_2.aspx" || file == "YaopinDetails_3.aspx" || file == "YaopinDetails_4.aspx" || file == "YaopinDetails_5.aspx" || file == "BaoJianDetails.aspx" || file == "BaoJianDetails_1.aspx" || file == "BaoJianDetails_2.aspx" || file == "BaoJianDetails_3.aspx" || file == "BaoJianDetails_4.aspx" || file == "BaoJianDetails_5.aspx" || file == "QixieDetails.aspx" || file == "QixieDetails_1.aspx" || file == "QixieDetails_2.aspx" || file == "QixieDetails_3.aspx" || file == "QixieDetails_4.aspx" || file == "QAReply.aspx" || file == "BaiKeDetails.aspx" || file == "BaiKeArticleDetails.aspx" || file == "NewsDetails.aspx" || file == "YaoChangDetails.aspx" || file == "YaoChangIntroduce.aspx" || file == "YaoChangMessage.aspx" || file == "YaoChangQAReply.aspx" || file == "YaoChangConsult.aspx" || file == "YaoChangProduct.aspx" || file == "YaoChangNews.aspx" || file == "YaoChangWork.aspx")

[暖枫无敌]

这么多啊？我倒
你干脆将这些页面文件名称放在一个数组里，然后通过遍历数组并和你的这个file进行比较，加入你的file属于该数组集合中的一个子集，那么条件就成立。

[著名天才___]

[Quote=引用 1 楼 taomanman 的回复:]

这么多啊？我倒
你干脆将这些页面文件名称放在一个数组里，然后通过遍历数组并和你的这个file进行比较，加入你的file属于该数组集合中的一个子集，那么条件就成立。
[/Quote]
是个不错的选择、应该会比 || 好、还有其它方法么

[人生难得一只鸡]

其实无论怎样 都是一样的 只不过是爽与不爽的问题 如果想独立一点，可以放置一个外部配置文件，以后如果还需要加，那么直接在配置里多加个节点

[逐风]

var x = "YaopinDetails.aspx|B.aspx";
List<string> xx = new List(x.Split('|'));
if(xx .Contains(file ))

[baitoclus]

除了数组想不到更好的解决方案了

[人生难得一只鸡]

其实都一样 只不过是爽与不爽的问题，如果更加的独立，可以加一个配置文件，专门配置这些需要判断的节点，以后如有新需求，直接增加配置节点，而无需去改动代码了

[昵称不能为空噢噢噢噢]

[Quote=引用楼主 zhumingtiancai 的回复:]
C# code


if (file == "YaopinDetails.aspx" || file == "YaopinDetails_1.aspx" || file == "YaopinDetails_2.aspx" || file == "YaopinDetails_3.aspx" || file == "YaopinDetails_4.aspx" || file == "Yaop……
[/Quote]
写个方法

private bool check(string [] ar0,string file)
{
  List<string>list=new List<string>();
  for(int i=0;i<ar0.Length;i++)
    list.add(ar0[i]);
  if(list.Contains(file))
     return true;
  return false;

}
...........
if(check(arr,flie))
{
//xxoo
}

[king122085]

void m(xx)
string x = "YaopinDetails.aspx|B.aspx"(可以做一个config配置来控制);

if(x .Contains(xx))
{

}

[vrhero]

不算多，如果较固定不怎么变改成switch即可...

switch(file){
case "YaopinDetails.aspx":
case "YaopinDetails_1.aspx":
...
default:
...
}

如果经常变改成字典...

但是...从这你罗嗦的命名看，你只需要一个文件传递参数而已...可能是你的设计本身就有问题，去找病根而不是头疼去医脚...

[油泼白菜]

建议用配置文件处理，修改也方便，，

[988kook]

用File本身啊 你上传文件 不是要比较是否本地有那几个文件么 如果有就的话就就执行someing~~~~ 都不用那么多的名称了 让微软替你搞定

[988kook]

 string filePath = Server.MapPath(fileName);
        if (File.Exists(filePath))
        {
                  // 存在就执行你if里的代码了
         }