Day02 - jeremy0405/Codesquad_CS GitHub Wiki

MagicNumber

MagicNumber란 소스 코드 안에 작성된 구체적인 숫자값을 의미한다. 동일한 기능을 하는 매직 넘버의 개수가 많다면 나중에 소스코드를 변경해야 할 때 동일한 기능을 하는 매직 넘버를 모두 같은 값으로 변경해야 한다는 단점이 있다. 또한 구체적인 숫자값이 무엇을 의미하는지 알기 힘든 경우가 많다. 이럴 때 심볼릭 정수를 이용하여 동일한 기능을 하는 매직 넘버를 모두 심볼릭 정수로 치환하여 코드를 작성하면 가독성이 좋아지고 코드 변경에도 유연하게 대처할 수 있게 된다.

미션을 진행 중 carry와 sum을 길이가 2인 배열 인덱스의 0번째는 carry, 1번째는 sum을 저장하고 지속적으로 꺼내서 사용하게 되었다.

이전의 코드로는 계속 아래와 같이 작성하였지만 매직 넘버를 통해 보기에 좋도록, 또 실수가 발생하지 않도록 리팩토링을 진행했다.

public boolean[] halfadder(boolean bitA, boolean bitB) {
    boolean[] answer = new boolean[2];
    answer[0] = //계산한 carry 값
    answer[1] = //계산한 sum 값
    return answer;
}

boolean tmpCarry = halfadder(bitA, bitB)[0];
boolean tmpSum = halfadder(bitA, bitB)[1];

static final int CARRY = 0;
static final int SUM = 1;

public boolean[] halfadder(boolean bitA, boolean bitB) {
    boolean[] answer = new boolean[2];
    answer[CARRY] = //계산한 carry 값
    answer[SUM] = //계산한 sum 값
    return answer;
}

boolean tmpCarry = halfadder(bitA, bitB)[CARRY];
boolean tmpSum = halfadder(bitA, bitB)[SUM];

변경 결과 코드의 가독성이 좋아졌다. 앞으로도 매직 넘버를 남발하는 코드는 지양해야겠다.