매직넘버는 사용되서는 안됩니다.

 

매직넘버는 반복문의 반복 횟수를 제한하거나 특정 속성의 값을 테스트하는 등 뜬금없이 사용되는 숫자로, 코드의 구문 내에서 직접적으로 사용됩니다. 매직 넘버를 사용하는 것은 코드를 작성할 때 명확하고 간단해보일 수 있지만, 디버깅할 때에는 덜 명확하고 간단하지 않습니다. 그렇기에 매직 넘버를 사용하기 전에 이를 명확하게 명명된 변수에 할당하여 이해할 수 있게 만들어야 합니다. -1,0 그리고 1은 매직넘버로 간주되지 않습니다.

규칙을 어긴 코드

public static void doSomething() {
	for(int i = 0; i < 4; i++) { // 규칙을 어긴 코드, 4 는 매직넘버입니다/
		...
	}
}

규칙을 준수한 해결책

public static final int NUMBER_OF_CYCLES = 4;
public static void doSomething() {
  for(int i = 0; i < NUMBER_OF_CYCLES ; i++){
    ...
  }
}

If you like SONARKUBE, don’t forget to give me a star. :star2:

원문으로 바로가기

Star This Project