2 qq 31071631 qq_31071631 于 2016.01.18 11:24 提问

求大神帮忙看看有什么洁简的方法

List listzhuanhuan=maps.get(entryt.getKey());
List listleixing=new ArrayList();
for(int m=0; m<listzhuanhuan.size();m++){
listleixing.add(listzhuanhuan.get(m).getLeibie());
}
boolean a=false;
for (String tempa : listleixing) {
if (tempa.equals("0")) {
a=true;
break;
}
a=false;

}
if (!a) {
Mapshuju mapshuju= new Mapshuju();
mapshuju.setLeibie("0");
mapshuju.setOut_dpt(listzhuanhuan.get(0).out_dpt);
mapshuju.setRenshu("0");
listzhuanhuan.add(mapshuju) ;
}
for (String tempa : listleixing) {
if (tempa.equals("1")) {
a=true;
break;
}
a=false;

}
if (!a) {
Mapshuju mapshuju= new Mapshuju();
mapshuju.setLeibie("1");
mapshuju.setOut_dpt(listzhuanhuan.get(0).out_dpt);
mapshuju.setRenshu("0");
listzhuanhuan.add(mapshuju) ;
}

               for (String tempa : listleixing) {
                    if (tempa.equals("2")) {
                    a=true;
                    break;
                    }
                    a=false;    
               }
               if (!a) {
                    Mapshuju mapshuju= new Mapshuju();
                    mapshuju.setLeibie("2");
                    mapshuju.setOut_dpt(listzhuanhuan.get(0).out_dpt);
                    mapshuju.setRenshu("0");
                    listzhuanhuan.add(mapshuju) ;
               }
               for (String tempa : listleixing) {
                    if (tempa.equals("3")) {
                    a=true;
                    break;
                    }
                    a=false;    
               }
               if (!a) {
                    Mapshuju mapshuju= new Mapshuju();
                    mapshuju.setLeibie("3");
                    mapshuju.setOut_dpt(listzhuanhuan.get(0).out_dpt);
                    mapshuju.setRenshu("0");
                    listzhuanhuan.add(mapshuju) ;
               }
          ComparatorUser comparator=new ComparatorUser();
          Collections.sort(listzhuanhuan, comparator); 

6个回答

hefeng1995
hefeng1995   2016.01.18 11:32

just a test

caozhy
caozhy 回复qq_31071631: 你该学学英文了
接近 2 年之前 回复
qq_31071631
qq_31071631 不懂
接近 2 年之前 回复
qq_31071631
qq_31071631 不懂
接近 2 年之前 回复
rui888
rui888   Ds   Rxr 2016.01.18 13:01

if (tempa.equals("2")) {
a=true;
break;
}
a=false;

                                    这里的 equals 2 或者3 为什么不写在一起。
baokx
baokx   2016.01.18 13:17

判断字符串集合中是否有某个字符串为什么不用contains方法呢。

 if(!listleixing.contains("0")){

        }

还有就是new一个Mapshuju对象的时候只有leibie属性不一样,这段代码提到一个方法里不行么,接收leibie参数,返回一个Mapshuju对象

baokx
baokx   2016.01.18 13:24

或者提取到一个方法里,这些都是可以自己决定的。但是重复代码肯定是不好的。

 List listzhuanhuan = maps.get(entryt.getKey());
        funName(listzhuanhuan,new String[]{"0","1","2","3"});

    }

    public static Mapshuju funName(List listzhuanhuan,String [] leibies){
        List listleixing = new ArrayList();
        for(int m=0; m<listzhuanhuan.size();m++){
            listleixing.add(listzhuanhuan.get(m).getLeibie());
        }
        for(int i = 0 ; i < leibies.length ; i++){
            if(!listleixing.contains(leibies[i])){
                Mapshuju mapshuju= new Mapshuju();
                mapshuju.setLeibie(leibies[i]);
                mapshuju.setOut_dpt(listzhuanhuan.get(0).out_dpt);
                mapshuju.setRenshu("0");
                listzhuanhuan.add(mapshuju) ;
            }
        }
    }
wojiushiwo945you
wojiushiwo945you   Ds   Rxr 2016.01.18 13:46

存在的问题:

 1 变量命名不妥,通常变量都是有意义的英文单词,你这英文加拼音的命名方式很奇怪啊。boolean a=false;这个变量a看不出变量的作用和含义,可读性差。
2 代码不严谨, listzhuanhuan.get(m).getLeibie());这里get(m)如果返回为null,那么久可能抛出空指针异常了。
3 一个方法中代码量太多,有些代码可以抽取成私有方法,如判断是否存在类型为1的记录的代码,可以抽取出来的。
qq_31071631
qq_31071631 刚入门,只会这种请问怎么抽取出来
接近 2 年之前 回复
cuiwei1026522829
cuiwei1026522829   Ds   Rxr 2016.01.20 00:47

就是单独写在一个类里,写一个方法,用到判断就来调用,应该是这样意思。。。

Csdn user default icon
上传中...
上传图片
插入图片
准确详细的回答,更有利于被提问者采纳,从而获得C币。复制、灌水、广告等回答会被删除,是时候展现真正的技术了!