吐吐苦水: 收集一些老项目中不合理的设计与编码,给后来开发人员添麻烦的例子

作者：V君 发布于：2018-9-9 2:11 Sunday 分类：填坑经验

从17年下半年开始收集，内容不多就先不放出来占条目

 

供引以为戒

 

1) 返回值是集合的方法,查询无结果时返回null

调用者必须繁琐地检查空引用

应参考Linq的ToArray, 在无结果时返回元素个数为0的集合实例

 

2) 对于WEB请求无论报错还是无结果,返回空字符串,尽管把错误写入日志

无法从浏览器前端直接得知是查询无结果还是服务端报错

应当在无查询结果时返回空集合实例, 在服务端报错时返回经过筛选的友好提示

 

3) RouterMapping.config

在一个相当老的项目中, 出现把所有MVC中Controller的Action全都定义到配置文件的现象，

这和早些年Java的SSH一个鸟养哇！

由于发布版本一般会排除配置文件,因为开发环境和正式环境不同，于是经常出现莫名其妙的Action丢失或404的扯蛋现象。或许是那时候使用的技术过于老旧

 

4) 二进制协议解析,变长数据,长度敏感

在协议中已明确变长的数据段中使用特定长度区分报文内容，

若厂商在日后扩充了协议（即长度变更）将会无法正确解析数据。

我们应该严谨的遵循协议文档实现解析，

遍历所有出现的数据段，只捡出需要内容，以便向后兼容。

 

5) (UintFlaggedEnum)int.Parse(dataReader["UintColumn"].ToString())

这真是2008年左右前刚毕业的人写的代码的味道.

在第31个bit被使用之前,这段代码是好用的, 一旦bit31被使用即爆炸.

在不能用ORM的时候我们应该使用一致的数据类型。

现在可以用Dapper之类的轻量级ORM代替手动邦定数据。

标签: 软件开发 C# Web技术 传输协议

« 搭建自己的NuGet托管服务器,并通过符号源服务器提供带源码的调试体验 | 前端新手历险记之在VUE项目中使用Ace编辑器(brace)和一些TypeScript知识点»

引用地址:

发表评论：

Powered by emlog 去你妹的备案 sitemap