看一看這段代碼，讓我們來看看主要存在的問題
//設置sql語句
insertstr="insert into userinfo(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命令，那你就慘了，呵呵。
那么，怎樣來寫呢，上面這段代碼可以改成這樣：
string strsql = "insert into sometable (c1 , c2 , c3 , ...) values(@c1 , @c2 , @c3,...)"
sqlcommand mycommand = new sqlcommand(strsql , myconn)
try
{
mycommand.parameters.add(new sqlparameters("@c1" , sqldatatype.varchar , 20)
mycommand.parameters["@c1"].value = this._name ;
....
//有幾個加幾個
....
}
catch(...)
...
這樣呢，既可以避免低效率的字符串連接，又可以利用sqlcommand參數有效性檢測來避免非法字符的出現，并且由于這種parameter方式是預編譯的，效率更高。
一舉數得，何樂而不為呢。
