二重循环优化

Posted 2020-10-19 纯丿乱

今天做了个任务，用的是二重循环，结果老大来审查代码的时候被骂了。

为什么被骂？

以为代码写的low。

废话不多说，直接看代码。

错误的代码：

	/**
	 * 给每一个新增柜员添加 bankId属性
	 */
	private void setBankId4PbUser(List<PbUser> userList) throws Exception{
		List<Network> allNetWorks = netWorkService.loadAllNetwork("");
		
		for(Network network : allNetWorks){
			for(PbUser pbUser : userList){
				String bankCode = pbUser.getBank_code();

				if(network.getCode().equals(bankCode)){
					pbUser.setBank_id(network.getId());
					pbUser.setBank_name(network.getName());
					break;
				}
			}
		}
	}

 

正确的代码：

	/**
	 * 给每一个新增柜员添加 bankId属性
	 */
	private void setBankId4PbUser(List<PbUser> userList) throws Exception{
		List<Network> allNetWorks = netWorkService.loadAllNetwork("");
		Map<String,Network> allNetWorksMap = new HashMap<String, Network>();
		for(Network network : allNetWorks){
			allNetWorksMap.put(network.getCode(), network);
		}
		for(PbUser pbUser : userList){
			Network network = allNetWorksMap.get(pbUser.getBank_code());
			if(network == null){
				throw new PbException("未找到柜员"+pbUser.getUser_code()+"对应的网点");
			}
			pbUser.setBank_id(network.getId());
			pbUser.setBank_name(network.getName());

		}
	}

测试可得： 100用户100网点的条件下，正确代码比错误代码速度快70%。

分析：这个效率提升在不再需要使用遍历去匹配，而是新创建一个Map，将code作为键。提高了效率，还增加了阅读性。