贴段java代码

最近在做一个项目时对有个地方一直有点迷惑，背景是这样：
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类可能更清晰一些。