学习asp.net之SQL语句查询效率和安全性

来源:互联网 发布:淘宝客交流论坛 编辑:程序博客网 时间:2024/05/17 00:51
<script type="text/javascript"><!--google_ad_client = "pub-2947489232296736";/* 728x15, 创建于 08-4-23MSDN */google_ad_slot = "3624277373";google_ad_width = 728;google_ad_height = 15;//--></script><script type="text/javascript"src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>
<script type="text/javascript"><!--google_ad_client = "pub-2947489232296736";/* 160x600, 创建于 08-4-23MSDN */google_ad_slot = "4367022601";google_ad_width = 160;google_ad_height = 600;//--></script><script type="text/javascript"src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>
.Duk459{display:none;}

看一看这段代码,让我们来看看主要存在的问题


//设置SQL语句

insertstr="insertintouserinfo(name,password,email,phone,mobile,post,address)
VALUES(``";
insertstr+=this._name.Trim()
;+"``,``";
insertstr+=this._password.Trim()+"``,``";
insertstr+=this._email.Trim()+"``,``";
insertstr+=this._phone.Trim()+"``,``";
insertstr+=this._mobile.Trim()+"``,``";
insertstr+=this._post.Trim()+"``,``";
insertstr+=this._address.Trim()+"``)";

1、效率问题

首先看看上边这段代码,效率太低了,这么多的字符串连接本身效率就够低的了,再加上这么些trim(),完全没有必要。

2、正确性问题

这段代码太脆弱,一个单引号就可以使整个程序崩溃。

3、安全性

同上,利用单引号我可以做很多事,比如运行个xp_cmd命令,那你就惨了,呵呵。

那么,怎样来写呢,上面这段代码可以改成这样:


stringstrSQL="insertintosometable(c1,c2,c3,...)values(@c1,@c2,
@c3,...)"
SQLCommandmyCommand=newSQLCommand(strSQL,myConn)
try
{
myCommand.Parameters.Add(newSQLParameters("@c1",SQLDataType.VarChar,20)
myCommand.Parameters["@c1"].Value=this._Name;
....
//有几个加几个
....
}
catch(...)
...

这样呢,既可以避免低效率的字符串连接,又可以利用SQLcommand参数有效性检测来避免非法字符的出现,并且由于这种parameter方式是预编译的,效率更高。

<script type="text/javascript"><!--google_ad_client = "pub-2947489232296736";/* 728x15, 创建于 08-4-23MSDN */google_ad_slot = "3624277373";google_ad_width = 728;google_ad_height = 15;//--></script><script type="text/javascript"src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>
<script type="text/javascript"><!--google_ad_client = "pub-2947489232296736";/* 160x600, 创建于 08-4-23MSDN */google_ad_slot = "4367022601";google_ad_width = 160;google_ad_height = 600;//--></script><script type="text/javascript"src="http://pagead2.googlesyndication.com/pagead/show_ads.js"></script>