一些java程序代码中的隐患bug

1、

   if(attributes!=null && !attributes.isEmpty()){
    for(String key : attributes.keySet()  ) {
    sb.append(" ").append(entry).append("=").append("'").append(attributes.get(entry)).append("'").append(" ");
    }
   }

用上面的代码会有这样的隐患：

Performance - Inefficient use of keySet iterator instead of entrySet iterator

This method accesses the value of a Map entry, using a key that was retrieved from a keySet iterator. It is more efficient to use an iterator on the entrySet of the map, to avoid the Map.get(key) lookup.

findbugs:WMI_WRONG_MAP_ITERATOR Efficiency > Processor use 

用entrySet来遍历 , 像下面这样 ，就不会有隐患 。

   if(attributes!=null && !attributes.isEmpty()){
    for(Entry<String, Object> entry : attributes.entrySet() ) {
    sb.append(" ").append(entry).append("=").append("'").append(attributes.get(entry)).append("'").append(" ");
    }
   }

2、

SsonAPI  ssonApi =SsonHelper. getNewInstance(  request ,  response  );

            PubAccount pubAccount=ssonApi.getUserFromCookie();

上面的取得pubAccount出现乱码

下面的就对了

SsonAPI  ssonApi =SsonHelper. getNewInstance(  request ,  response  );

          PubAccount  pubAccount  =  ssonApi  .getUser();

注： getUser是封装好的方法。

3、

AccountApp accountApp = accountAppService.getByPubId(pubAccount.getAccountId());

上面的代码可能有null的隐患，改为下面的代码。

AccountApp accountApp = null;

if( pubAccount!=null ){
accountApp = accountAppService.getByPubId(pubAccount.getAccountId());
}

相关链接：https://ci.openmrs.org/sonar/rules_configuration/index/4