请看下面的代码,从代码的可读性、减少冗余考虑,大家有没有优化的方法了。谢谢!
以下两个方法,我想来想去也没找出可以简化的地方了,请大家指教。
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);
}
C#代码优化,请大家提提意见。
- 写回答
- 好问题 0 提建议
- 追加酬金
- 关注问题
- 邀请回答
-
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了,同样会不会-判断后面的语句
而你的三条语句在任何情况下都要满足执行解决 无用评论 打赏 举报
悬赏问题
- ¥50 comfyui下连接animatediff节点生成视频质量非常差的原因
- ¥20 有关区间dp的问题求解
- ¥15 多电路系统共用电源的串扰问题
- ¥15 slam rangenet++配置
- ¥15 有没有研究水声通信方面的帮我改俩matlab代码
- ¥15 对于相关问题的求解与代码
- ¥15 ubuntu子系统密码忘记
- ¥15 信号傅里叶变换在matlab上遇到的小问题请求帮助
- ¥15 保护模式-系统加载-段寄存器
- ¥15 电脑桌面设定一个区域禁止鼠标操作