可能引發性能問題的幾個代碼寫法總結
可能引發性能問題的幾個代碼寫法,看看你占哪一個.
1. int.Parse() VS int.TryParse()
你是不是正在這樣寫
- int id = 0;
- try{ id = int.Parse(Request["id"]); }
- catch{ id = 0; }
如果是可以這樣試試
- int id = 0;
- int.TryParse(Request["id"], out id);
前一種寫法,一旦 Request["id"] 是非數值型的,將引發一個異常,引發異常的開銷是非常巨大的,而后一種則不會引發任何異常。
2.string.IndexOf()
你正在這樣寫嗎?
- string s = "aaa,bb";
- int pos = s.IndexOf(",");
其實對于單字符的查找,這樣寫會更好
- string s = "aaa,bb";
- int pos = s.IndexOf(',');
有人會問如果我要找多個字符呢,那可以試試下面的
- string s = "aaa,bb";
- int pos = s.IndexOf("bb", StringComparison.OrdinalIgnoreCase);
至于 StringComparison 的具體用法可以google或者baidu得到。
3. RegexOptions.Compiled
如果你正在使用正則并且用到了這個參數,那請你慎重,根據個人經驗使用這個參數在訪問量比較大的情況下可能會引發性能問題,比如cpu偏高。如果你表示懷疑,可以嘗試比較使用和不用這個參數的情況下哪個性能會更好。
4.忘記關閉數據庫連接
數據庫連接是非常好資源的,所以從打開到關閉應該越短越好。想看看有沒有忘記關閉,可以通過性能監視器的 .net Data provider for SqlClient ( 假設你用的是sqlserver ) 來查看,具體的參數說明可以通過google和baidu得到。
5.頻繁的Response.Write()
你正在這樣做嗎?
- Response.Write("這是第1行.< br/>");
- Response.Write("這是第2行.< br/>");
- Response.Write("這是第3行.< br/>");
此種寫法頻繁調用Response.Write() ,根據經驗,這是相當的耗cpu,改成下面的試試
- StringBuilder sb = new StringBuilder();
- sb.Append("這是第1行.< br/>");
- sb.Append("這是第2行.< br/>");
- sb.Append("這是第3行.< br/>");
- Response.Write(sb.ToString());
6. 不必要的重復操作
- List< TopicInfo> list = new List< TopicInfo>();
- //從數據庫得到數據
- list = GetDataFromDB();
- for(int i = 0;i < list.Count; i++ )
- {
- TopicInfo item = list[i];
- }
上面的代碼估計誰都看的出來有什么不好,那下面這個代碼呢
- using System;
- using System.Collections.Generic;
- using System.Text;
- using System.Text.RegularExpressions;
- using System.Web;
- public class UrlUBB
- {
- /// < summary>
- /// 替換UBB里面的url
- /// < /summary>
- /// < param name="content">< /param>
- /// < returns>< /returns>
- public static string RegularUrl(string content)
- {
- if (string.IsNullOrEmpty(content))
- return string.Empty;
- if (content.IndexOf(@"(url=", StringComparison.OrdinalIgnoreCase) == -1 || content.IndexOf(@"(/url)", StringComparison.OrdinalIgnoreCase) == -1)
- return content;
- Regex reg = new Regex(@"\(url=(?< url>.[^\)]*)\)(?< name>.[^\(]*)\(/url\)");
- string url = string.Empty;
- string name = string.Empty;
- string href = string.Empty;
- MatchCollection matches = reg.Matches(content);
- foreach (Match m in matches)
- {
- if (m.Success)
- {
- url = regexUrl(m.Groups["url"].ToString());
- name = m.Groups["name"].ToString();
- href = string.Format("< a href=\"redirect.aspx?goto={0}\">{1}< /a>", url, name);
- content = content.Replace(m.ToString(), href);
- }
- }
- return content;
- }
- }
你是否考慮過這樣的寫法
- using System;
- using System.Collections.Generic;
- using System.Text;
- using System.Text.RegularExpressions;
- using System.Web;
- public class UrlUBB
- {
- private static Regex reg = new Regex(@"\(url=(?< url>.[^\)]*)\)(?< name>.[^\(]*)\(/url\)");
- /// < summary>
- /// 替換UBB里面的url
- /// < /summary>
- /// < param name="content">< /param>
- /// < returns>< /returns>
- public static string RegularUrl(string content)
- {
- if (string.IsNullOrEmpty(content))
- return string.Empty;
- if (content.IndexOf(@"(url=", StringComparison.OrdinalIgnoreCase) == -1 || content.IndexOf(@"(/url)", StringComparison.OrdinalIgnoreCase) == -1)
- return content;
- string url = string.Empty;
- string name = string.Empty;
- string href = string.Empty;
- MatchCollection matches = reg.Matches(content);
- foreach (Match m in matches)
- {
- if (m.Success)
- {
- url = regexUrl(m.Groups["url"].ToString());
- name = m.Groups["name"].ToString();
- href = string.Format("< a href=\"redirect.aspx?goto={0}\">{1}< /a>", url, name);
- content = content.Replace(m.ToString(), href);
- }
- }
- return content;
- }
- }
如果你的代碼不幸也占了那么一、兩個,那么修改并對比一下性能試試看,如果你很幸運的一個也沒占,那么恭喜你,你的程序性能應該還不錯。
【編輯推薦】