这段代码是否有内存溢出的情况，如果有怎么修改呢？

0 0

这段代码是否有内存溢出的情况，如果有怎么修改呢？0

	public List<ModuleDefineInfo> findUserModuleList(long userID) {
		try{
		Set<String> roleIDs=new HashSet<String>();
		roleIDs.addAll(loginDao.findRoleIDByUser(userID));
		List<String> groupIDs=loginDao.findGroupIDByUser(userID);		
		if(groupIDs!=null && !groupIDs.isEmpty()){
			roleIDs.addAll(loginDao.findRoleIDByGroup(userID));
			roleIDs.addAll(loginDao.findRoleIDByUserAndGroup(userID));
		}
		String roleIDCondStr=getCondStr(roleIDs);
		List<String> moduleIDs=loginDao.findModuleIDByRoleIDs(roleIDCondStr);
		String moduleIDCondStr=getCondStr(moduleIDs);
		List<ModuleDefineInfo> list=loginDao.findParentModuleList(moduleIDCondStr);
		List<RoleModuleFunctionVo> functions=loginDao.findModuleFunctions(roleIDCondStr, moduleIDCondStr);
		setFunctionToModule(list,functions);
		return ModuleUtil.constructTree(list,ModuleUtil.DESC);
		}catch(Exception e){
			logger.error(e,e);
			throw new RuntimeException(e);
		}
	}	
	
	private void setFunctionToModule(List<ModuleDefineInfo> modules,List<RoleModuleFunctionVo> functions){
		for(ModuleDefineInfo m:modules){
			if(!m.isHasURL()) continue;
			for(Iterator<RoleModuleFunctionVo> it=functions.iterator();it.hasNext();){
				RoleModuleFunctionVo f=it.next();
				if(m.getModule_id().equals(f.getModule_id())){
					
					List<RoleModuleFunctionVo> moduleFuncs = m.getFunctions();
					Boolean bExist = false;
					if (moduleFuncs != null) {
						for(Iterator<RoleModuleFunctionVo> itt=moduleFuncs.iterator();itt.hasNext();) {
							RoleModuleFunctionVo ff=itt.next();
							if (ff.getFunction_id().equals(f.getFunction_id())) {
								bExist = true;
								break;
							}
						}
					}
					if (bExist)
						continue;
					
					m.addFunction(f);
					it.remove();
				}
			}
		}
	}

溢出

2014年2月21日 08:46

newstong1
0
0 0 8

1个答案按时间排序按投票排序

0 0

代码和业务有一定的相关性，没仔细看：

如果Service涉及到的操作只有局部变量，是不会有内存泄露的（局部变量调用完以后会在下次Minor GC的时候被回收）；

一个会内存泄露的例子：
。类有一个Map之类的属性（类是单例的或者是Spring的Bean）
。只有Add操作，没有对称的remove操作。

2014年2月25日 11:18

鳄鱼旭
248
0 0 17