举几个例子说一下：
1. /// <summary>
/// 判断输入是否为日期类型
/// </summary>
/// <param name="s">待检查数据</param>
/// <returns></returns>
public static bool IsDate(string s) { try { DateTime d = DateTime.Parse(s); return true; } catch { return false; } }这什么？ 判断日期？ 难道不知道有DateTime.TryParse ,用trycatch控制流程 可想而知代码的效率了 虽然后台控制不注重效率
2. 项目NetCMS.Common，NetCMS.Config
NetCMS.Common依赖于NetCMS.Config config顾名思义提供配置管理 那common干嘛的呢 为什么也会直接对配置和xml进行访问和读写呢
//NetCMS.Config.BaseConfig的某一个方法
static internal string GetConfigValue(string Target) { string path = HttpContext.Current.Server.MapPath("~/xml/sys/netcms.config"); return GetConfigValue(Target, path); }//NetCMS.Common.Public的某一方法
public static string readparamConfig(string strTarget,string type) { string rstr = ""; if (type != null && type != string.Empty) { string xmlPath = HttpContext.Current.Server.MapPath("~/xml/sys/" + type + ".config"); FileInfo finfo = new FileInfo(xmlPath); System.Xml.XmlDocument xdoc = new XmlDocument(); xdoc.Load(xmlPath); XmlElement root = xdoc.DocumentElement; XmlNodeList elemList = root.GetElementsByTagName(strTarget); rstr += elemList[0].InnerXml; } else { rstr = readparamConfig(strTarget); } return rstr; }光看方法名就知道不是一个人写的 不是一个人写的不要紧 至少有统一的规范吧 大小写都不一样 方法名首字母大写 写过.net的人应该知道的吧
包括页面文件的命名 都是有大写 有小写
3. News_add.aspx 中有个方法
/// <summary>
/// 保存数据
/// </summary>
protected void Buttonsave_Click(object sender, EventArgs e)
接近有800行代码 你以为是写C呢 面向过程 一个方法接近800行 里面到处充斥着变量和判断 变量名没有实际意义 并且也没多少重要的注释 可读性差到极点了 试问这种软件 开源了 跟不开源有什么区别
4.再找个方法给大家看看
#region 得到当前新闻的上一条记录自动编号ID
int _IDStr = 0; DataTable dts = rd.getClassContent(getDataLibStr,7); if (dts != null && dts.Rows.Count > 0) { _IDStr = int.Parse(dts.Rows[0]["Id"].ToString()); dts.Clear(); dts.Dispose(); } else { _IDStr = int.Parse(NetCMS.Common.Rand.Number(8)); } #endregion 结束
返回一个int字段 需要用DataTable 太奢侈了点吧 数据访问层我也不去说了 一个项目 说到底 就是对数据的操作 这个重要的东西都做出这样 更别提什么框架了
说实话 我也不是故意看代码的 只因为发了半天新闻 结果还是看不到 只能来读代码了 顺便说一句 帮助也很差
写了好多操作有什么用呢 至少有个指导吧 如何可以建一个站 如何可以发布显示出新闻