Java 使用引用传递来修改值，这样做好吗？

[zzfer]

今天在代码审核的时候看到这么一段代码，在 setProjectInfo 方法没有返回内容，但在方法体内给 list set 了值，想了下，是 Java 引用类型可以改变参数的值，这样写确实可行

	if (!CollectionUtils.isEmpty(list)){
            cmService.setProjectInfo(list);
        }
    	return list;

但如果是我写同样的需求，肯定是返回类型是 list<>，然后结果 return 给 list 。我不知道哪种写法好，想问问大家

	if (!CollectionUtils.isEmpty(list)){
            list = cmService.setProjectInfo(list);
        }
    	return list;

[Vegetable]

不要使用第二种
说实话这种内部的 list 就不该暴露出来

[Vegetable]

看错了，当我没说哈哈

[VeryZero]

第一种调用栈比较深的时候就很蛋疼了。维护过这种代码，简直折磨

[husher123]

虽然我也写过 1，但是更喜欢 2，可以链式调用 0.0