为什么 ，方法参数/捕获的异常/foreach的变量不应该被重新赋值？

Question

最近使用，sonar代码审查，报错，“ Method parameters, caught exceptions and foreach variables should not be reassigned”

官方提供的例子如下：

1. Noncompliant Code Example：不规范代码示例

`MyClass {

public String name;

public MyClass(String name) {

name = name; // Noncompliant - useless identity assignment

}

public int add(int a, int b) {

a = a + b; // Noncompliant

return a; // Seems like the parameter is returned as is, what is the point?

}

public static void main(String[] args) {

MyClass foo = new MyClass();

int a = 40;

int b = 2;

foo.add(a, b); // Variable "a" will still hold 40 after this call

}

}`

2. Compliant Solution：规范代码示例

`MyClass {

public String name;

public MyClass(String name) {

this.name = name; // Compliant

}

public int add(int a, int b) {

return a + b; // Compliant

}

public static void main(String[] args) {

MyClass foo = new MyClass();

int a = 40;

int b = 2;

foo.add(a, b);

}

}`

我觉得官方给的示例，后面规范的代码并没有高明很多？ 请大神解惑。

Answer 1

第一个，this.name 指的是你上面定义的，类里面的一个属性，name指的是构造函数的入参，把入参赋给类的属性，要用 this.name = name

第二个，单纯的想返回两个入参的和的话，直接 return a + b就行了，没必要设个中间变量。而且就算设中间变量了也应该新建一个 int temp = 0;，不要对入参做修改。

第三个看起来跟形参和实参的关系。