Java中传入对象作为方法参数并修改其值是否属于不良编程行为？

首先明确：Java里将引用类型（比如数组、自定义对象）作为方法参数传入，在方法内部修改其状态的做法本身不是不良编程行为，关键看这种修改是否符合代码的可读性、可维护性要求，以及调用方是否对行为有预期。

先看你的例子本质 #

你写的getChildPar方法是并查集的路径压缩实现，这是并查集这种数据结构的核心优化手段——通过修改par数组，把查询路径上的节点直接指向根节点，大幅降低后续查询的时间复杂度。这种修改是有明确业务目的的，属于完全合理的用法，Java社区里也普遍认可这种实现方式。

什么时候是合理的？ #

• 职责明确的工具方法：方法的核心职责就是修改对象状态（比如集合的add、sort方法，或者这里的并查集路径压缩），调用方调用前就清楚会修改输入对象。
• 性能优化需求：像并查集的路径压缩，这种修改能带来显著的性能提升，且没有更优的替代方案。
• 行为可预期：方法命名或文档明确告知调用方会修改参数（比如把方法名改成findAndCompressParent，比getChildPar更直观）。

什么时候会变成不良行为？ #

• 隐式修改参数：方法本该只返回计算结果，却悄悄修改了输入对象，调用方完全不知情，容易引发难以排查的bug。
• 职责混乱：一个方法同时承担“查询结果”和“修改状态”两个职责，违背单一职责原则，降低代码可读性。
• 线程安全问题：在多线程场景下，无同步机制的参数修改会导致数据不一致，这种情况要格外注意。

和C++的对比 #

你提到这种做法在C中常见，其实Java里的引用类型参数传递（传递引用的副本）和C的引用传递本质类似，都是允许修改原对象状态的。Java并没有禁止这种做法，只是强调代码的可读性和可维护性——只要调用方对行为有预期，这种实现就是合理的。

总结建议 #

你的这段代码是合理的，不属于不良编程行为。如果要进一步优化，可以：

• 给方法起更具描述性的名字，让调用方一眼就知道会修改par数组
• 必要时添加注释说明路径压缩的作用，方便后续维护

内容的提问来源于stack exchange，提问作者lightrek