[리팩토링] 매개변수로의 값 대입 제거

Parameter(매개변수)는 Method 호출 시 argument 로 넘어간 변수를 칭한다.

여기서 매개변수로의 값 대입은 Parameter로 받은 변수의 값을 호출한 Method에서 어떠한 값을 대입 하는 경우를 말한다.

전달받은 매개변수에 특정 Object나 Variable등을 참조 대입시에 코드의 명료성과
코드를 보는 이에게 Call by value 인지 Call by reference 인지 혼동을 불러 일으킬수 있다.

따라서, 매개변수로의 값 대입을 코드 작성할 시 지양해야 하며, 발견시에 해당 내용을 수정할 수 있도록 하자.

 

변경 전

int discount (int inputVal)
{
	if (inputVal > 50)
		inputVal -= 10;
	// below code blah blah
}

 

변경 후

int discount (int inputVal)
{
	int nInputVal = inputVal;

	if (nInputVal > 50)
		nInputVal -= 10;
	// below code blah blah
}