总结：开发容易出Bug的代码或操作

Posted 2022-07-31 真爱无限

又有一段时间没有发表文章了，感觉在同一个公司待久了，能写出来的东西就少了，呵呵，大家是不是有一样的感觉。

 

今天来总结一下开发常见的易引发错误或影响效率的东西。

开发对于懂开发的人来说其实很简单，做开发这项工作简直就像日常吃饭一样熟练，但是，开发过程中，由于各种各样的问题，例如：业务逻辑不清晰、开发人员粗心、开发人员经验不够、生产环境不同等等，这些问题导致各种BUG，是需要特别重视的，下面整理了一些情况来说明问题。

svn文件漏提交影响他人开发，这个在实际开发中出现比较频繁.

A:漏提交文件（自己不知道，他本地代码编译没报错）

B:获取编译报错，在群里大叫“谁的代码，编译不过”

…………过了半天没有回答，去查下svn提交记录，查出是A提交后就出问题了

B: 在QQ群里，“@A，你提交的有问题，自己去看下”

A: 看下自己果然有文件没提交，赶紧把漏提交的文件提交了，在群里回复“可以了，再获取下”

……然后，就ok了。

 

 

非空判断

B2bDistributorOrderMapping distributorOrderMap = allDistributorOrder.Where(o => o.OrderId == order.OrderId).FirstOrDefault();

var amount =  distributorOrderMap.CreditAmount.GetValueOrDefault(0M);//如果distributorOrderMap为空，则必定报错

循环内多次连续读取数据库-最大可能导致数据库服务器cpu高

while (hasData)//每天订单数10000的情况下

var orderListQuery = OrdersRepository.Entities.Where(o => o.CheckInDate <= usedDate && (o.FinanceReceiptDate >= usedDate || o.CheckOutDate >= usedDate) && o.CancelStatus != (int)OrderCancelStatusEnum.OrderCanceled && o.OrderStatusId == audited);

if (loopCount == 0) order = orderListQuery.OrderBy(o => o.OrderId).FirstOrDefault();

else order = orderListQuery.OrderBy(o => o.OrderId).Skip(loopCount).FirstOrDefault();

if (order == null) hasData = false;

else hasData = true;

loopCount++;

逻辑不严谨-粗心导致

//正确代码

if(a==1) return 100;

else return 0;

//错误代码

var retVal;

if(a==1) retVal=100;

retVal = 0;

return retVal;

测试代码写死临时本地配置（如:192.168.9.220）被误提交，本地测试没问题，一发到线上就报错

 

索引超出范围报错

如：

var a=new int[2]0,1;//两个元素，下标为0，1

Var b=a[2];//取下标为2的元素，报错

 

数据库基础数据或网站配置不同

比如：

测试环境数据库加了个字段线上忘记加，或本地config配置文件加、改配置，线上忘记改，造成各种报错，也是最常见的错误。

 

 

高级问题：

环境冲突，未考虑测试环境和正式环境

比如：

测试环境上传文件用cdn传文件：http://cdn.xxxx.com/2016-01-01.txt

正式环境文件也是：http://cdn.xxxx.com/2016-01-01.txt

有可能测试环境刚生成文件，造成文件覆盖，正式环境正好在获取，就获取了测试环境的同名文件内容，造成数据错误。

环境不同导致

比如，财务事务补偿正常的操作流程图：

出错情况的正常情况分析：

 

 

总结：

其实有BUG也是正常的，发现后及时修复就行了，但是如果大量的BUG是因为人粗心或思维不严谨、遗漏等原因造成的，那就太不应该了。

所以平日里，我们开发时必须要考虑仔细，提交前至少保证先编译通过，那样问题就会减少很多。