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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

这是一个创建于 1170 天前的主题，其中的信息可能已经有所发展或是发生改变。

今天在代码审核的时候看到这么一段代码，在 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;

24 条回复 • 2021-10-17 01:05:36 +08:00

hidemyself

2021-10-15 16:02:11 +08:00

额，我都是用第一种写法。

zydxn

2021-10-15 16:04:39 +08:00

看不出你这么给方法 setProjectInfo 改成加上返回值在你的业务代码里有什么意义

如果你的某个实体里有字段属性是 Collection，你也要把 set 方法改成有返回值的么

crazypig14

2021-10-15 16:05:19 +08:00

第二种好在支持链式调用，第一种方法名起好一点就好，比如叫 append 啥啥好点

liuxey

2021-10-15 16:05:28 +08:00

一般传递给不受信任的方法时要用 ImmutableList，比如：

ImmutableList<XXX> list = ...;
if (!CollectionUtils.isEmpty(list)){
list2 = cmService.setProjectInfo(list);
}
return list2;

但实际操情况中 list 内部变量会因为是 Mutable 而被隐性破坏：list.get(0).setXXX(x);

所以爱咋写咋写

Jooooooooo

2021-10-15 16:09:19 +08:00

没有区别.

AoEiuV020

2021-10-15 16:09:32 +08:00

挺常见的做法，
按我习惯而言，一般用前者，
如果是必须复制一个新的列表，不能动原 list 才会用第二种写法返回个新的，

chendy

2021-10-15 16:14:07 +08:00

要么改原 List，不返回（副作用方法）
要么返回新的 List，不改原 List （无副作用方法）
把参数改了又返回出去真的很迷惑

zzfer

2021-10-15 16:14:44 +08:00

@hidemyself 我一直都是第二种，也不知道怎么养成的代码习惯。

@zydxn 最大的意义就是可读性更好吧。实体字段是给实体内 set 值，这个相当于你在 set 中修改你要 set 的值，不是一个概念

@crazypig14 确实支持链式调用也是一个好处。我感觉可读性比第一种高点

@liuxey 学到了

shellus

2021-10-15 16:31:40 +08:00

```
修手机方法 1：
var 我的手机
修好的手机=维修店.全面修复(我的手机)
```
```
修手机方法 2：
var 我的手机
工具 = 维修店.购买工具(xxx)
技术 = 维修店.学习技术(xxx)
修好的手机=我.修理(我的手机，工具，技术)
```

第一个写法，简单，但是你不知道维修店对你的手机做了啥
第二个方法，复杂，你需要在你的方法里写很多无关的东西

但是无论如何，都是用返回值接收，因为下面这种会看起来很怪异

```
// 我的手机坏的
维修店.全面修复(我的手机)
// 我的手机好了
```