程小雨1992 2013-12-18 07:35 采纳率: 0%
浏览 1764

C#代码优化,请大家提提意见。

请看下面的代码,从代码的可读性、减少冗余考虑,大家有没有优化的方法了。谢谢!
以下两个方法,我想来想去也没找出可以简化的地方了,请大家指教。
private void getRecordCount()
{
var recordCount = "0";
var cid = Session["cid"].ToString();
var sql = "SELECT COUNT(*)RCOUNT FROM (SELECT DONO FROM D1030 WHERE DOABBR = '" + cid + "' GROUP BY DONO) A";
var dataSet = SqlHelper.dataSet(sql);
if (dataSet != null && dataSet.Tables.Count > 0 && dataSet.Tables[0].Rows.Count > 0)
{
recordCount = dataSet.Tables[0].Rows[0][0].ToString();
}
Response.Write(recordCount);
}

  • 写回答

2条回答 默认 最新

  • 黑子大哥 2014-02-17 03:17
    关注

    dataSet != null && dataSet.Tables.Count > 0 && dataSet.Tables[0].Rows.Count > 0
    修改成
    !(dataSet == null || dataSet.Tables.Count == 0 || dataSet.Tables[0].Rows.Count == 0)

    因为这样 当dataset为空的话就不会去判断dataset.table.count==0了,同样会不会-判断后面的语句
    而你的三条语句在任何情况下都要满足执行

    评论

报告相同问题?

悬赏问题

  • ¥100 求数学坐标画圆以及直线的算法
  • ¥100 c语言,请帮蒟蒻写一个题的范例作参考
  • ¥15 名为“Product”的列已属于此 DataTable
  • ¥15 安卓adb backup备份应用数据失败
  • ¥15 eclipse运行项目时遇到的问题
  • ¥15 关于#c##的问题:最近需要用CAT工具Trados进行一些开发
  • ¥15 南大pa1 小游戏没有界面,并且报了如下错误,尝试过换显卡驱动,但是好像不行
  • ¥15 自己瞎改改,结果现在又运行不了了
  • ¥15 链式存储应该如何解决
  • ¥15 没有证书,nginx怎么反向代理到只能接受https的公网网站