下面这几个,是在实际开发或阅读中发现的一些问题,有些甚至是有很多年开发人员写出的代码,也是很多人经常犯的错误。各位可以看看,你有没有躺着中枪。

第一个,对整型变量进行非null判断。

  1. // a 是int型 (不是int?)
  2. if(a != null){
  3. //操作
  4. }

个人点评:无意义判断,值类型永远不可能为null。

第二个,用static来保持页面回发

  1. static int id;
  2. protected void Page_Load(object sender, EventArgs e)
  3. {
  4. if (Request.QueryString["ID"] != null && Request.QueryString["ID"].ToString() != "")
  5. {
  6. id = Convert.ToInt32(Request.QueryString["ID"].ToString());
  7. }
  8. }

个人点评:这个不解释,不知道怎么说。但最近还真就遇到了,而且也不是什么小项目,WebForm无服务器控件开发。

第三个,用编程方式绑定数据控件时,数据源为DataSet时判断null而不判读DataSet内的Tables数。

  1. DataSet ds = bll.GetList();
  2. if (ds != null)
  3. {
  4. Repeater1.DataSource = ds;
  5. Repeater1.DataBind();
  6. }

个人点评:当bll.GetList()返回的DataSet非null但里面没有包含数据表时,执行DataBind()方法时会报HttpException异常(IListSource 不包含任何数据源)。正确写法应该是

  1. DataSet ds = bll.GetList();
  2. if (ds != null && ds.Tables.Count > )
  3. {
  4. Repeater1.DataSource = ds;
  5. Repeater1.DataBind();
  6. }
  7. //或
  8. DataSet ds = bll.GetList()??new DataSet();
  9. if (ds.Tables.Count > )
  10. {
  11. Repeater1.DataSource = ds;
  12. Repeater1.DataBind();
  13. }

第四个,用编程方式绑定数据控件时,数据源为DataTable或List<T>时判断null。

  1. DataTable dt = bll.GetList();
  2. if (dt!=null)
  3. {
  4. Repeater1.DataSource = dt;
  5. Repeater1.DataBind();
  6. }

个人点评:无意义判断,下面的写法没有任何问题,即使dt=null

  1. DataTable dt = bll.GetList();
  2. Repeater1.DataSource = dt;
  3. Repeater1.DataBind();

第五个

  1. Model m = new Model();
  2. m = bll.GetModel(id);
  3. m.name;

个人点评:以为只要声明时不为null,后面就不需要做非空非null判断了。万一第二步BLL层返回的model就为null呢?

第六个,在Repeater1_ItemDataBound中写这样的代码

  1. Label lblPMID = (Label)e.Item.FindControl("lblPMID");
  2. if (lblPMID.Text != "")
  3. {
  4. //操作
  5. }

个人点评:低效,无意义判断,很可能出现NullReferenceException(未将对象引用设置到对象的实例)异常。
正确写法:

  1. Label lblPMID = e.Item.FindControl("lblPMID") as Label;
  2. if (lblPMID!=null && lblPMID.Text != "") //视里面使用情况决定是否判断lblPMID.Text为“”或空白
  3. {
  4. //操作
  5. }

第七个

  1. string txtName = Request["txtName"] == null ? "" : Request["txtName"].ToString();
  2. string strWhere += "and ID=" + userId + ""; //userId是int
  3. if (txtName != "")
  4. {
  5. strWhere += " and NAME='" + txtName + "'";
  6. }
  7. strWhere += " order by id desc";
  8. //项目本身都是采用参数化查询的,这里是一些暴露给Web层的高级查询条件。

个人点评:1、值类型和字符串拼接会隐式装箱,2、SQL注入危险。正确做法是userId.ToString()并且过滤txtName中特殊字符,限制字符串长度。
注意,拼接SQL时过滤字符串并不能完全防止SQL注入,但很多时候在高级查询时拼接SQL是最简单也是最方便的,这时候过滤不应该只过滤一些指定的特殊字符,
比如只过滤单引号,等号,大于/小于/等于,空格,括号之类的危险字符。应该对除中文字符、英文字母、和数字外的所有字符全部过滤掉(视情况而定)。
并且严格限制字符串长度,一般查询时输入的关键字不会太长,如果用户输入的有空格,就拆分成多个条件。这样能尽可能的减小SQL注入的机会。

最后,给大家分享几个小经验,虽说有些只是语法糖,但却可以帮助我们更高效编写或阅读代码。

一、引用类型的null值很麻烦,因为类型为null时使用点运算符 (.)会报异常,所以经常要做非null判断,可以用?? null 合并运算符减少代码量。例如:

  1. //写法一
  2. int ID;
  3. if (Request.Form["ID"] != null && Request.Form["ID"].ToString() != "")
  4. {
  5. ID = Convert.ToInt32(Request.Form["ID"].ToString());
  6. }
  7.  
  8. //写法二
  9. int id;
  10. if (int.TryParse(Request.Form["ID"]??"",out id))
  11. {
  12.  
  13. }
  14.  
  15. //方法一
  16. string userName2=string.Empty;
  17. if (Session["userName"]!=null)
  18. {
  19. userName2 = Session["userName"].ToString();
  20. }
  21.  
  22. //方法二
  23. string userName1 = Session["userName"] == null ? "" : Session["userName"].ToString();
  24.  
  25. //方法三
  26. string userName = (Session["userName"] ?? "").ToString();

二、Web项目中的所有Session或cookie最好统一放到一个类中管理。最重要的目的是把Session中索引名独立出来管理,也就是除了本类外的所有页面都不要输入Session名。
可能用语言表达不够直白,直接上代码。
看到很多人是这样,包括网上流行的一些很常见的辅助类库。

  1. /// <summary>
  2. /// Session 操作类
  3. /// 1、GetSession(string name)根据session名获取session对象
  4. /// 2、SetSession(string name, object val)设置session
  5. /// </summary>
  6. public class SessionHelper
  7. {
  8. /// <summary>
  9. /// 根据session名获取session对象
  10. /// </summary>
  11. /// <param name="name"></param>
  12. /// <returns></returns>
  13. public static object GetSession(string name)
  14. {
  15. return HttpContext.Current.Session[name];
  16. }
  17. /// <summary>
  18. /// 设置session
  19. /// </summary>
  20. /// <param name="name">session 名</param>
  21. /// <param name="val">session 值</param>
  22. public static void SetSession(string name, object val)
  23. {
  24. HttpContext.Current.Session.Remove(name);
  25. HttpContext.Current.Session.Add(name, val);
  26. }
  27. /// <summary>
  28. /// 检测session是否存在
  29. /// </summary>
  30. /// <param name="name"></param>
  31. /// <returns></returns>
  32. public static bool CheckSession(string name)
  33. {
  34. try
  35. {
  36. if (GetSession(name) == null)
  37. {
  38. return false;
  39. }
  40. else
  41. {
  42. return true;
  43. }
  44. }
  45. catch
  46. {
  47. return false;
  48. }
  49. }
  50. }

我想知道,这样做有什么实际意义么?而且HttpContext.Current还不做null检查。
项目中还到处都是SessionHelper.SetSession("name"),SessionHelper.GetSession("name")这样编译器无法找到具体引用的代码,当你有很多页面用到这个会话后,
你再想更改会话名称或删除这个会话那将是一场灾难,而且这样的代码多了还可能出现多个会话重名造成冲突,名称写错造成会话丢失。
要克服以上问题,Web项目的Session会话你应该这样写

  1. /// <summary>
  2. /// 系统中Session会话管理
  3. ///<para>注意:该类使用Session,若在一般处理程序中调用该类方法必须实现 IRequiresSessionState 接口</para>
  4. /// </summary>
  5. public class SessionManager
  6. {
  7. private const string USER_LOGIN_MARK = "userModel"; //用户登录
  8. private const string USER_REGISTER_MARK = "registerModel"; //用户注册
  9. private const string CHECKSUM_MARK = "checksumModel"; //校验码
  10. private const string CAPTCHA_MARK = "captchaModel"; //验证码
  11.  
  12. #region 用户登录会话
  13. /// <summary>
  14. /// 添加用户登录标识
  15. /// </summary>
  16. /// <param name="cardModel"></param>
  17. internal static void AddUserLoginMark(UserModel model) //UserModel是用户对象
  18. {
  19. if (model == null)
  20. {
  21. throw new ArgumentNullException("参数不能为Null");
  22. }
  23.  
  24. HttpContext context = HttpContext.Current;
  25. if (context != null)
  26. {
  27. context.Session[USER_LOGIN_MARK] = model;
  28. }
  29. }
  30.  
  31. /// <summary>
  32. /// 移除用户登录标识
  33. /// </summary>
  34. internal static void RemoveUserLoginMark()
  35. {
  36. HttpContext context = HttpContext.Current;
  37. if (context != null)
  38. {
  39. context.Session.Remove(USER_LOGIN_MARK);
  40. }
  41. }
  42.  
  43. /// <summary>
  44. /// 获取当前登录用户对象
  45. /// </summary>
  46. /// <returns></returns>
  47. internal static UserModel GetUserLogin()
  48. {
  49. HttpContext context = HttpContext.Current;
  50. if (context != null)
  51. {
  52. return context.Session[USER_LOGIN_MARK] as UserModel;
  53. }
  54.  
  55. return null;
  56. }
  57. #endregion
  58.  
  59. #region 用户注册会话
  60. /// <summary>
  61. /// 添加用户注册会话标识
  62. /// </summary>
  63. /// <param name="cardModel"></param>
  64. internal static void AddUserRegisterMark(RegisterModel model)
  65. {
  66. if (model == null)
  67. {
  68. throw new ArgumentNullException("参数不能为Null");
  69. }
  70.  
  71. HttpContext context = HttpContext.Current;
  72. if (context != null)
  73. {
  74. context.Session[USER_REGISTER_MARK] = model;
  75. }
  76. }
  77.  
  78. /// <summary>
  79. /// 移除用户注册会话标识
  80. /// </summary>
  81. internal static void RemoveUserRegisterMark()
  82. {
  83. HttpContext context = HttpContext.Current;
  84. if (context != null)
  85. {
  86. context.Session.Remove(USER_REGISTER_MARK);
  87. }
  88. }
  89.  
  90. /// <summary>
  91. /// 获取当前注册会话对象
  92. /// </summary>
  93. /// <returns></returns>
  94. internal static RegisterModel GetUserRegister()
  95. {
  96. HttpContext context = HttpContext.Current;
  97. if (context != null)
  98. {
  99. return context.Session[USER_REGISTER_MARK] as RegisterModel;
  100. }
  101.  
  102. return null;
  103. }
  104. #endregion
  105.  
  106. #region 校验码会话(手机和邮件)
  107. /// <summary>
  108. /// 添加一个校验码会话
  109. /// </summary>
  110. internal static void AddChecksumMark(ChecksumModel model)
  111. {
  112. if (model == null)
  113. {
  114. throw new ArgumentNullException("参数不能为Null");
  115. }
  116.  
  117. HttpContext context = HttpContext.Current;
  118. if (context != null)
  119. {
  120. context.Session[CHECKSUM_MARK] = model;
  121. }
  122. }
  123.  
  124. /// <summary>
  125. /// 移除当前用户的校验码会话
  126. /// </summary>
  127. internal static void RemoveChecksumMark()
  128. {
  129. HttpContext context = HttpContext.Current;
  130. if (context != null)
  131. {
  132. context.Session.Remove(CHECKSUM_MARK);
  133. }
  134. }
  135.  
  136. /// <summary>
  137. /// 获取当前用户的校验码会话
  138. /// </summary>
  139. internal static ChecksumModel GetChecksum()
  140. {
  141. HttpContext context = HttpContext.Current;
  142. if (context != null)
  143. {
  144. return context.Session[CHECKSUM_MARK] as ChecksumModel;
  145. }
  146.  
  147. return null;
  148. }
  149. #endregion
  150.  
  151. #region 验证码会话
  152. /// <summary>
  153. /// 添加一个验证码会话
  154. /// </summary>
  155. internal static void AddCaptchaMark(string c)
  156. {
  157. if (c == null)
  158. {
  159. throw new ArgumentNullException("参数不能为Null");
  160. }
  161.  
  162. HttpContext context = HttpContext.Current;
  163. if (context != null)
  164. {
  165. context.Session[CAPTCHA_MARK] = c;
  166. }
  167. }
  168.  
  169. /// <summary>
  170. /// 移除当前用户的验证码会话
  171. /// </summary>
  172. internal static void RemoveCaptchaMark()
  173. {
  174. HttpContext context = HttpContext.Current;
  175. if (context != null)
  176. {
  177. context.Session.Remove(CAPTCHA_MARK);
  178. }
  179. }
  180.  
  181. /// <summary>
  182. /// 获取当前用户的验证码会话
  183. /// </summary>
  184. internal static string GetCaptcha()
  185. {
  186. HttpContext context = HttpContext.Current;
  187. if (context != null)
  188. {
  189. return context.Session[CAPTCHA_MARK] as string;
  190. }
  191.  
  192. return null;
  193. }
  194.  
  195. /// <summary>
  196. /// 检查验证码是否和服务器端一致(不区分大小写)
  197. /// </summary>
  198. /// <param name="code">用户输入的验证码</param>
  199. /// <returns></returns>
  200. public static bool ValidateCaptcha(string code)
  201. {
  202. if (string.IsNullOrWhiteSpace(code))
  203. {
  204. throw new ArgumentNullException("参数不能为Null");
  205. }
  206. string code2 = GetCaptcha() ?? "";
  207. if (code.ToUpper() == code2.ToUpper())
  208. {
  209. return true;
  210. }
  211. return false;
  212. }
  213. #endregion
  214. }

这样写,项目中使用时:
登录成功就添加会话SessionManager.AddUserLoginMark(当前登录用户对象);
页面登录检查时判断 SessionManager.GetUserLogin();返回是否为null就行。
退出登录时SessionManager.RemoveUserLoginMark();即可。
这样就只管调用就行,不用再去管Session名是什么,删除、更改也更方便,当然也不会出现Session重名现象了(如果这样都能整成会话重名的话,那你真成人才了)。

当然,这样写也不是一点缺点都没有,和前一种相比,这种方法可能就不能做到一次书写,到处使用了,需要跟据当前项目具体灵活改动相应代码,但好处是很明显的。这样的方法同样适用于Cache和Cookie。

三、最好不要用Request[]代替Request.Form[]和Request.QueryString[]。

如果页面有很多Request.Form[]、Request.QueryString[]、Session[]最好在页面Page_Load中就把所有值取出来存放在变量中,并转换成需要的类型。
满篇的Request.Form[]、Request.QueryString[]、Session[]编译器没法检查[]中的字符串,容易出错,影响阅读,还可能同一参数需要多次类型转换(这一条针对WebFrom无服务器控件开发时特别有用)。

四、尽量使用 as 代替引用类型间转换(见上面第六个)

这个大家都知道,但还是发现很多人在用强制转换,包括一些优秀的开源项目。

五、RegisterClientScriptBlock、RegisterClientScriptInclude、RegisterStartupScript、RegisterOnSubmitStatement、RegisterClientScriptResource等方法要求前台页面必须有form服务器控件(<form runat="server"></form>)。
也就意味着在WebForm无服务器控件开发时,这几个没什么用了(同样的还有Page.IsPostBack要小心了)。

六、微软不推荐直接在后台使用Response.Write()输出JS,并且有的浏览器的确会造成页面变形。

但发现很多人在用,包括一些很优秀的开源项目。我暂时用在前台加入<%= strMsg %>来接收后台传过来的消息,不知道各位都有什么好的方法。

七、最后向大家分享一段自己写的小代码,为Repeater控件添加EmptyDataTemplate模板(EmptyDataTemplate在FooterTemplate之前)。

原理很简单,默认添加的模板会出现在FooterTemplate之后,在RenderChildren()方法中给它们换下位置就行。
代码如下:

  1. using System;
  2. using System.ComponentModel;
  3. using System.Web.UI;
  4.  
  5. namespace MyRepeater
  6. {
  7. public class Repeater : System.Web.UI.WebControls.Repeater
  8. {
  9. [PersistenceMode(PersistenceMode.InnerProperty), Browsable(false), TemplateContainer(typeof(TemplateControl))]
  10. public ITemplate EmptyDataTemplate { get; set; }
  11.  
  12. protected override void OnDataBinding(EventArgs e)
  13. {
  14. base.OnDataBinding(e);
  15.  
  16. this.Controls.Clear();
  17. this.ClearChildViewState();
  18.  
  19. this.CreateControlHierarchy(true);
  20. this.ChildControlsCreated = true;
  21.  
  22. if (EmptyDataTemplate != null)
  23. {
  24. if (this.Items.Count == )
  25. {
  26. EmptyDataTemplate.InstantiateIn(this);
  27. }
  28. }
  29. }
  30.  
  31. protected override void RenderChildren(HtmlTextWriter output)
  32. {
  33. if (HasControls())
  34. {
  35. for (int i = ; i < Controls.Count; i++)
  36. {
  37. if (this.FooterTemplate != null && this.Items.Count == && EmptyDataTemplate != null)
  38. {
  39. if (i == Controls.Count - )
  40. {
  41. Controls[i + ].RenderControl(output);
  42. continue;
  43. }
  44. if (i == Controls.Count - )
  45. {
  46. Controls[i - ].RenderControl(output);
  47. continue;
  48. }
  49. }
  50. Controls[i].RenderControl(output);
  51. }
  52. }
  53. }
  54.  
  55. protected override void Render(HtmlTextWriter output)
  56. {
  57. RenderChildren(output);
  58. }
  59. }
  60. }

暂时能想到的只有这么多,都是自己的经验之谈,当然也属于一家之言,如果有什么错误或不合理的,可以在下边留言给我或狠狠的踩上几脚。同时也希望大家能把自己的一些开发经验或技巧分享出来,供大家学习学习。

2014-01-04补充

错误修正

第二段、第七个、为Repeater控件添加EmptyDataTemplate模板会导致Repeater的ItemDataBound事件处理执行两次。

请注释掉下面这几行代码

  1. //this.Controls.Clear();
  2. //this.ClearChildViewState();
  3.  
  4. //this.CreateControlHierarchy(true);
  5. //this.ChildControlsCreated = true;

本来我的代码就参考了kdalan的这篇文章,他的文章中本来没有这几句的,但在看见msdn中的示例有这几句,当时也没多想就加上了,也不一在用,没出什么毛病。但前两天在调试的时候发现ItemDataBound事件会执行两次。为了查找为什么ItemDataBound事件为什么会多次执行,费了好大一番功夫,最后才排确定是this.CreateControlHierarchy(true)这句导致的。因为这几句是在MSDN上的示例中看到的,所有一直没怀疑它的正确性,在排错的过程中放到最后,耽误不少时间。其实最后想想,也大概知道了其中的原因,.net中的事件可以多次订,基类和子类都调用this.CreateControlHierarchy(true);会导致多次订阅ItemDataBound事件。为了证实我的猜想,专门建立了项目,对自定义的Repeater再次继承重写,在调用时ItemDataBound事件会被执行三次。测试项目我放到google code上,有兴趣的朋友可以下载看看,http://my-repeater.googlecode.com/svn

分享几个asp.net开发中的小技巧的更多相关文章

  1. ios开发中的小技巧

    在这里总结一些iOS开发中的小技巧,能大大方便我们的开发,持续更新. UITableView的Group样式下顶部空白处理 //分组列表头部空白处理 UIView *view = [[UIViewal ...

  2. iOS开发中调试小技巧

    对于软件开发而言,调试是必须学会的技能,重要性不言而喻.对于调试的技能,基本上是可以迁移的,也就是说你以前在其他平台上掌握的很多调试技巧,很多也是可以用在iOS开发中.不同语言.不同IDE.不同平台的 ...

  3. iOS - 开发中调试小技巧

    对于软件开发而言,调试是必须学会的技能,重要性不言而喻.对于调试的技能,基本上是可以迁移的,也就是说你以前在其他平台上掌握的很多调试技巧,很多也是可以用在iOS开发中.不同语言.不同IDE.不同平台的 ...

  4. Android开发中的小技巧

    转自:http://blog.csdn.net/guxiao1201/article/details/40655661 简单介绍: startActivities (Intent[] intents) ...

  5. Android 开发中常用小技巧

    TextView中的getTextSize返回值是以像素(px)为单位的, 而setTextSize()是以sp为单位的. 所以如果直接用返回的值来设置会出错,解决办法是 用setTextSize() ...

  6. vue开发组件开发中的小技巧

    声明:以下随笔由博主自主编写,也有部分引用网友的,引用部分版权归原作者所有,其他博主原创部分禁止转载.复制全部或部分用以重新发布! vue递归组件事件阻止冒泡 其实这里主要还有递归组件的自定义事件不生 ...

  7. 在这里总结一些iOS开发中的小技巧,能大大方便我们的开发,持续更新。

    UITableView的Group样式下顶部空白处理 //分组列表头部空白处理 UIView *view = [[UIView alloc] initWithFrame:CGRectMake(0, 0 ...

  8. js开发中常用小技巧

    1.获取指定范围内的随机数 function getRadomNum(min,max){ return Math.floor(Math.random() * (max - min + 1)) + mi ...

  9. 一些IOS开发中的小技巧

    1.打包后提交报错误 错误信息:ERROR ITMS-90035: "Invalid Signature. Code object is not signed at all. The bin ...

随机推荐

  1. linux下subversion server安装手册

    linux下subversion server安装手册 安装基于的Linux版本为:Red Hat Enterprise Linux Server release 6.3. 一 准备需要的安装包. ( ...

  2. webform 中使用ajax

    常用的方式有 js –> WebService  , js->*.ashx, js->WebAPI, js->MVC Controller->Action. 前两种就不说 ...

  3. PHP多文件上传(二维数组$_FILES('文件域的名称'),move_uploaded_file(‘临时文件名’,‘新的文件名’))

    <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/ ...

  4. 关于gridview里加按钮事件的总结

    1. onrowcommand="GridView1_RowCommand1" 在gridview申明时的属性里要有,然后找到闪电,双击 还有要把那个按钮行模板化,就是箭头里面的t ...

  5. hosts文件的一个小发现

    今天才发现原来同一个ip可以在hosts文件中配置多个域名.之间老是换一个网站就改一下,现在终于不用这么麻烦了 127.0.0.1 gg.pclady.com.cn 127.0.0.1 gg.pcon ...

  6. 用javascript写Android和iOS naitve应用,实在炫酷。

    关注NativeScript有一段时间了,说好了的三月发第一个Beta版,终于发布了. // declare the extended NativeScriptActivity functionali ...

  7. Quartz.net(调度框架) 使用Mysql作为存储

    最近公司的做的项目中涉及到配置任务地址然后按照配置去目标地址提取相关的数据,所以今天上午在Internet上查看有关定时任务(调度任务)的相关信息,筛选半天然后查找到Quartz.net. Quart ...

  8. 在Linux上以服务的方式运行ASP.NET Core站点

    更新:用supervisor是更好的解决方法,详见 Linux下为 dotnet 创建守护进程 要在生成环境下在Linux服务器上跑ASP.NET Core站点,首先要解决的问题是以服务的方式运行AS ...

  9. GLFW初体验

    GLFW - 很遗憾,没有找到FW的确切含义,Wiki上没有,GLFW主页也没有.猜测F表示for,W表示Window GLFW是干啥用的? 一个轻量级的,开源的,跨平台的library.支持Open ...

  10. .NET Framework中重点类型的继承关系

    继承关系 Object ├─Array │ └─T[] ├─ArrayList ├─List<T> └─String 集合类型的接口 下图展示了集合类型的各种接口的相互关系.注意,下图中所 ...