最近在做一个项目时对有个地方一直有点迷惑,背景是这样:
model类:Doctor,PriceStandard
Doctor字段:
PriceStandard字段:
.....
目的:判断当前doctor所收费用是否为0?(不代扣表示收费为0).
代扣费用条件:recoupWay=="1"&&isSystem=="N"
不代扣条件:recoupWay=="2"&&isSystem=="N"
系统费用: recoupWay=="1"||"2"+isSystem=="Y"即isSystem=="Y"
(其中代扣与不代扣只会出现其一)
总费用:代扣费或不代扣费+系统费用
代码:
这段代码看起来非常的别扭,那个doctor.getRecoupWay()明明是外部的变量却不得不放在for循环内部来比较,因此很多人一看见这段代码就有想改一改的冲动,我也不例外:
个人觉得这个recoupWay放在PriceStandard类可能更清晰一些。
model类:Doctor,PriceStandard
Doctor字段:
List<PriceStandard> feeList;
//扣费方式,1表示代扣,2表示不代扣
String recoupWay;
.....
PriceStandard字段:
//Y表示系统收费,N非系统收费
String isSystem;
//费用
double priceAmount;
.....
目的:判断当前doctor所收费用是否为0?(不代扣表示收费为0).
代扣费用条件:recoupWay=="1"&&isSystem=="N"
不代扣条件:recoupWay=="2"&&isSystem=="N"
系统费用: recoupWay=="1"||"2"+isSystem=="Y"即isSystem=="Y"
(其中代扣与不代扣只会出现其一)
总费用:代扣费或不代扣费+系统费用
代码:
//计算挂号实际收取费用
List<PriceStandard> feeList = doctor.getFeeList();
// 费用总数
double hs_price = 0;
double ad_price = 0;
//不代扣费用
double site_price = 0;
for (PriceStandard ps:feeList) {
double priceTemp = ps.getPriceAmount();
//代扣费用
if(doctor.getRecoupWay().equals("1")&&ps.getIsSystem().equals("N")){
hs_price += priceTemp;
}
//不代扣费用
if(doctor.getRecoupWay().equals("2")&&ps.getIsSystem().equals("N")){
site_price += priceTemp;
}
if(ps.getIsSystem().equals("Y")){
ad_price += priceTemp;
}
}
这段代码看起来非常的别扭,那个doctor.getRecoupWay()明明是外部的变量却不得不放在for循环内部来比较,因此很多人一看见这段代码就有想改一改的冲动,我也不例外:
//计算挂号实际收取费用
List<PriceStandard> feeList = doctor.getFeeList();
// 费用总数
double hs_price = 0;
double ad_price = 0;
//不代扣费用
double site_price = 0;
for (PriceStandard ps:feeList) {
if(ps.getIsSystem().equals("Y")){//系统费用
ad_price += ps.getPriceAmount();
}else if(ps.getIsSystem().equals("N")){//非系统收费
if(doctor.getRecoupWay().equals("1")){//代扣费用
hs_price += ps.getPriceAmount();
}else if(doctor.getRecoupWay().equals("2")){//不代扣费用
site_price += ps.getPriceAmount();
}
}
}
个人觉得这个recoupWay放在PriceStandard类可能更清晰一些。