코드의 냄새 : 함수 이름에 있는 암묵적 인자

• 비슷한 함수가 많다
• 프로퍼티를 결정하는 문자열이 함수 이름에 있다는 것 (setTaxByName, setQuantityByName)
• 함수 이름에 있는 일부가 인자처럼 동작하고 있다.

<aside> 💡 함수 이름에 있는 암묵적 인자이다.

</aside>

• 함수 이름에 있는 암묵적 인자 냄새는 두 가지 특징을 보인다. (setTaxByName, setQuantityByName)
  1. 함수 구현이 거의 똑같다.
  2. 함후 이름이 구현의 차이를 만든다.

암묵적인 인자를 드러내기 (리팩토링)

• 암묵적인 인자를 명시적인 인자로 바꾸는 것이다.
  1. 함수 이름에 있는 암묵적 인자를 확인한다.
  2. 명시적인 인자를 추가한다.
  3. 함수 본문에 하드 코딩된 값을 새로운 인자로 바꾼다.
  4. 함수를 부르는 곳을 고친다.

[
{},
{},
{},
]

// 객체를 필드명에 따라 최신화 시켜주는 함수
function setFieldByName(cart, name, field, value){
	const item = cart[name];
	// item = shoes: { name: 'shoes', price: 10 }
	// 객체에 기존 필드가 있으면 수정, 없으면 동적으로 필드 추가된다.
	const newItem = objectSet(item, field, value);
	// 객체가 최신화 된다.
	const newCart = objectSet(cart, name, newItem);
	return newCart;
}

cart = setFieldByName(cart, "shoe", 'price', 13)
cart = setFieldByName(cart, "shoe", 'quantity', 3)
cart = setFieldByName(cart, "shoe", 'shipping', 0)
cart = setFieldByName(cart, "shoe", 'tax', 2.34)

• 이제 기존에 함수 자체에 붙어있던 암묵적인 이름은 인자로 넘길 수 있는 값이 되었다.
• 값은 변수나 배열에 담을 수 있다.

<aside> 💡 식으로 쓸 수 있는 값을 “일급”이라고 한다. 혹은 변수에 담을 수 있는 것은 “일급”이라고 할 수 있다.

</aside>

• 그래서 일급 값이라고 부른다.

• 일급 이라고 부를 수 없는 것들

  • • 연산자
  • • 연산자
  • if 키워드 → 조건문
  • for 키워드 → 반복문

  <aside> 💡 모두다 변수에 담을 수 없다.

  </aside>

일급이 아닌 것을 일급으로 바꾸는 작업을 해보자 🤔

• 일급으로 바꾸는 기술은 함수형 프로그래밍에서 중요하다.