孤好梦中杀人 2014-06-19 15:33
浏览 1234
已采纳

两层for循环判断,如何优化

[code="java"]

List myRoles = new ArrayList();
List hasRoles = new ArrayList();

public boolean hasRole()
{
for (String my : myRoles)
{
for (String role : hasRoles)
{
if(my.equals(role))
{
return true;
}
}
}

return false;

}
[/code]

上面方面的逻辑是:myRoles集合中的字符串,只要有一个在hasRoles集合中存在,就返回true
但是,这个方法嵌套了2层for循环,效率似乎很低

求改进的方法

  • 写回答

12条回答 默认 最新

  • luhaixun 2014-06-19 21:12
    关注

    用Set做一个myRoles和hasRoles的并集,然后判断这个并集的大小是不是等于myRoles和hasRoles的大小相加,如果不是的话,可定有重复了。

    [code="java"]Set totalRoles = new HashSet();
    List myRoles = new ArrayList();

    List hasRoles = new ArrayList();

    public boolean hasRole()

    {

    // you can use addAll()
    for (String my : myRoles)

    {

    totalRoles.add(my);
    }
    // you can use addAll()
    for (String role : hasRoles)

    totalRoles.add(role);

    }

    return totalRoles.size()==(myRoles.size()+hasRoles.size());
    } [/code]

    查看下ArrayList.retainAll 和removeAll本质上都做了两个for循环

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论
查看更多回答(11条)

报告相同问题?

悬赏问题

  • ¥15 关于#matlab#的问题:在模糊控制器中选出线路信息,在simulink中根据线路信息生成速度时间目标曲线(初速度为20m/s,15秒后减为0的速度时间图像)我想问线路信息是什么
  • ¥15 banner广告展示设置多少时间不怎么会消耗用户价值
  • ¥16 mybatis的代理对象无法通过@Autowired装填
  • ¥15 可见光定位matlab仿真
  • ¥15 arduino 四自由度机械臂
  • ¥15 wordpress 产品图片 GIF 没法显示
  • ¥15 求三国群英传pl国战时间的修改方法
  • ¥15 matlab代码代写,需写出详细代码,代价私
  • ¥15 ROS系统搭建请教(跨境电商用途)
  • ¥15 AIC3204的示例代码有吗,想用AIC3204测量血氧,找不到相关的代码。