oma1989
2012-01-11 13:28 阅读 247
已采纳

代码规范 那种更好一些?

两个方法做的同样的工作,那种写法更好一些?有没有更好的写法呢?

 

private String sixOperaBaseChk(String code,String status,String[]modelId,String account)throws Exception{
     // 0 失败  1 成功  2 基本型利润率不达标  3基本型利润率未维护  4 plm编码未返回  6 ddp利润率未维护
     TargetSetDAO_S daos = new TargetSetDAO_S();
     TargetSetDAO dao = new TargetSetDAO();
     String msg = "3";
     FeeRatioVO vo = daos.baseInTarget(code, true);
     if(vo!=null&&!"".equals(vo.getFeeValue())){
      BigDecimal bt = new BigDecimal(vo.getFeeValue());
      List<ProjItemDetialVO> ls = dao.checkModelVerson(modelId, status);
      if (ls.size() == 0){
                boolean tp = true;
                if ("3".equals(status)||"2".equals(status)){
                    ls = dao.chkPlmIsNull(modelId);
                    tp = ls.size() > 0 ? false : true;
                }
                if (tp){   
                    boolean ratioChk = true;
                    for(int i=0;i<modelId.length;i++){
                        String basePlan = daos.sixOperaInPlan(modelId[i]);
                        BigDecimal bp = new BigDecimal(basePlan);
                        if(bp.compareTo(bt)==-1){
                            ratioChk = false;
                            msg = "2";
                        }
                    }
                    if(ratioChk){
                        int flag = dao.toFina(status,modelId,account);
                        msg = flag + "";
                    }
                }
                else{
                     msg = "4";
                }
            }else{
                msg = "";
                for (ProjItemDetialVO pidv : ls){
                     msg += "[" + pidv.getModelName() + "]";
                }
            }
     }
     
     return msg;
    } 

----------------------------

 

private String sixOperaBaseChk(String code,String status,String[]modelId,String account)throws Exception{
     // 0 失败  1 成功  2 基本型利润率不达标  3基本型利润率未维护  4 plm编码未返回  6 ddp利润率未维护
     //检查基本型目标利润率是否维护
     DefFeeRatioDAO defRatioDao = new DefFeeRatioDAO();
     List<BrandVO> brands = defRatioDao.getProjectBrands(code);
     Map<String,FeeRatioVO> baseRatioTarget = defRatioDao.getBaseRatioTarget(code, brands, true);
     for(BrandVO vo:brands){
          FeeRatioVO feeRatio = baseRatioTarget.get(vo.getToBrand());
          if(feeRatio == null || "".equals(feeRatio.getFeeValue())){
              return "3";
          }
     }
     //检查型号是否都保存过
     TargetSetDAO dao = new TargetSetDAO();
     List<ProjItemDetialVO> ls = dao.checkModelVerson(modelId, status);
     if(ls.size()>0){
      String msg = "";
            for (ProjItemDetialVO pidv : ls){
                msg += "[" + pidv.getModelName() + "]";
            }
            return msg;
     }
     //检查PLM编码是否回传
        if ("3".equals(status)||"2".equals(status)){
            ls = dao.chkPlmIsNull(modelId);
            if(ls.size()>0){
             return "4";
            }
        }
        //基本型利润率闸口
        for(int i=0;i<modelId.length;i++){
           ProjItemDetialVO vo = defRatioDao.getModelBaseRatioPlan(modelId[i]);
           BigDecimal ratioTarget = new BigDecimal(baseRatioTarget.get(modelId[i]).getFeeValue());
           BigDecimal modelRatioPlan = new BigDecimal("0");
           if(vo!=null){
               ProgramService ps = new ProgramService();
               modelRatioPlan = new BigDecimal(ps.div(vo.getProfit(), vo.getIncome(), 4));
           }
           if(modelRatioPlan.compareTo(ratioTarget)==-1){
               return "2";
           }
         }
        return String.valueOf(dao.toFina(status,modelId,account));
    }

 

 

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享

5条回答 默认 最新

  • 已采纳
    gulufather gulufather 2012-01-12 09:44

    用设计模式进行优化 此代码不好维护 职责链模式可供参考

    点赞 评论 复制链接分享
  • ll89308839 ll89308839 2012-01-11 13:59

    MyEclipse中
    alt+shift+j

    点赞 评论 复制链接分享
  • xcc258 xcc258 2012-01-11 14:20

    第二种好点。。第一种我第一次读的话。。我先要确定最外层的if判断是什么意思再看对应的else然后还要分别看if和else里面的代码是什么意思。。来回反复。。。。比较麻烦。。。第二种我可以从上往下顺利的看下去。

    点赞 评论 复制链接分享
  • qiemengdao qiemengdao 2012-01-11 17:29
    在团队开发过程中,代码规范非常重要,我们项目组一般是这样子做的:制定一个统一的template,要求每个开发人员导入和使用这个template。
    这样最大的好处:svn提交的时候你格式化的代码和别人格式化后的代码风格一致,不会因为代码风格问题比较出差异;在svn上可以很方便的合并代码。
    
    点赞 评论 复制链接分享
  • liukaihandsome liukaihandsome 2012-01-12 16:39

    return 最好用枚举来表示.
    一个小小的建议.

    点赞 评论 复制链接分享

相关推荐