【异常问题总结】参数如果在方法中重新初始化，则不会影响主线程对象

迷南。 2023-01-13 01:46 16阅读 0赞

问题案例：  
例1：

public static void main(String[] args) {
            Jedis jedis = null;
            try {
                doSomething(jedis);
            } catch (Exception e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            } finally {
                //这里jedis依然为null，会导致diSomething方法中创建的连接没有断开
                if (jedis != null) {
                    try {
                        jedis.close();
                    } catch (Exception e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
                }
            }
        }
        private static void doSomething(Jedis jedis) {
            if (jedis == null) {
                jedis = MobileRedisUtils.getInstance().getJedis();
            }
            /**
             * 业务处理
             */
        }

例2：

public static void main(String[] args) {
    		Map<String, String> map = new HashMap<>();
    		map.put("1", "1");
    		doSomething(map);
    		System.out.println(map);
    	}
    	private static void doSomething(Map<String, String> map) {
    		map = new HashMap<>();
    		map.put("2", "2");
    	}

输出结果： \{1=1\}  
      
问题分析：  
    在将对象作为参数传到方法时，是将主线程里的对象地址传递给方法的参数，如果在方法内重新初始化则仅是修改了方法中参数的引用地址，并不会影响主线程中的对象  
      
优化方式：  
    将新创建的对象作为返回结果回传回去，覆盖掉原对象

public static void main(String[] args) {
            Jedis jedis = null;
            try {
                jedis = doSomething(jedis);
            } catch (Exception e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            } finally {
                //这里jedis依然为null，会导致diSomething方法中创建的连接没有断开
                if (jedis != null) {
                    try {
                        jedis.close();
                    } catch (Exception e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
                }
            }
        }
        private static Jedis doSomething(Jedis jedis) {
            if (jedis == null) {
                jedis = MobileRedisUtils.getInstance().getJedis();
            }
            /**
             * 业务处理
             */
            return jedis;
        }

public static void main(String[] args) {
    		Map<String, String> map = new HashMap<>();
    		map.put("1", "1");
    		map = doSomething(map);
    		System.out.println(map);
    	}
    	private static map doSomething(Map<String, String> map) {
    		map = new HashMap<>();
    		map.put("2", "2");
    	}