又有一段时间没有发表文章了,感觉在同一个公司待久了,能写出来的东西就少了,呵呵,大家是不是有一样的感觉。
今天来总结一下开发常见的易引发错误或影响效率的东西。
开发对于懂开发的人来说其实很简单,做开发这项工作简直就像日常吃饭一样熟练,但是,开发过程中,由于各种各样的问题,例如:业务逻辑不清晰、开发人员粗心、开发人员经验不够、生产环境不同等等,这些问题导致各种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是因为人粗心或思维不严谨、遗漏等原因造成的,那就太不应该了。
所以平日里,我们开发时必须要考虑仔细,提交前至少保证先编译通过,那样问题就会减少很多。