java重构学习6：移除对参数的赋值(Remove Assignments to Parameters)

本系列来自《java 重构改善既有代码的设计》一书

如：需要对一个参数进行赋值。

可以利用一个临时变量取代该参数的位置。

private int discount(int inputValue, int quantity) {
	if (inputValue > 50) {
		inputValue -= 2;
	}
	//do something...
	return 0;
}

请写成：

private int discount(int inputValue, int quantity) {
	int result = inputValue;
	if (inputValue > 50) {
		result -= 2;
	}
	//do something...
	return 0;
}

主要原因是这样处理降低了代码的清晰度，且混淆了传值(pass by value)和传址(pass by reference)两种传递方式。java中只存在传值方式。

因为在传值的方式下，对参数的修改不会对调用端产生任何影响，但是对于使用过传址方式的人而言就很容易混淆(类似C++中的传指针)。且如果你只以参数表示被传递进来的东西的话代码会清晰很多。

在Java中不要对参数赋值，如果你自己有写过，请使用这种方式修改过来。这也是为什么有些人喜欢将参数写成final的主要原因，避免对参数赋值。

做法略：主要是利用一个临时变量，将待处理的参数值赋予给它。

延伸：java传值pass by value

例：