지금까지이 Q & A에서 이상한 점을 발견 한 사람은 실제로 아무도 “하드 코드”또는 더 중요한 대안을 명확하게 정의하려고 시도하지 않았다는 것입니다.

TL; DR : 예, 그것은 이다 하드 코드 값에 좋은 생각이 가끔 있지만,에 관해서는 간단한 규칙이 없다 경우는 , 그것은 상황에 전적으로 달려 있습니다.

질문은 값을 좁히는 데 , 이는 마법의 숫자 를 의미 하지만, 그들이 좋은 아이디어인지 아닌지에 대한 대답 은 실제로 사용되는 것과 관련이 있습니다 !

“하드 코딩 된”값의 몇 가지 예는 다음과 같습니다.

• 구성 값

  나는 같은 문장을 볼 때마다 울었다 command.Timeout = 600. 왜 600입니까? 누가 결정 했습니까? 시간이 초과되어 누군가가 기본 성능 문제를 해결하는 대신 해킹으로 시간 초과를 제기 했습니까? 아니면 실제로 처리 시간에 대한 일부 알려진 문서화 된 기대입니까?

  이들은 마법의 숫자 나 상수 가 아니 어야합니다. 최적의 값은 응용 프로그램이 실행되는 환경에 의해 대부분 또는 전적으로 결정되므로 의미있는 이름으로 구성 파일이나 데이터베이스에서 외부화해야합니다.

• 수학 공식

  공식은 일반적으로 정적 인 경향이 있으므로 내부의 상수 값의 특성이 실제로 중요하지 않습니다. 피라미드의 부피는 (1/3) b * h입니다. 우리는 1 또는 3이 어디에서 왔는지 걱정합니까? 실제로는 아닙니다. 이전의 주석가는 diameter = radius * 2아마 그보다 더 낫다고 지적 diameter = radius * RADIUS_TO_DIAMETER_CONVERSION_FACTOR했지만 그것은 거짓 이분법입니다.

  이 유형의 시나리오에서 수행해야 할 일은 함수를 작성하는 것 입니다. 나는 당신 이 공식을 어떻게 만들 었는지 알 필요는 없지만 여전히 그것이 무엇인지 알아야 합니다 . 위에서 쓴 넌센스 대신에 volume = GetVolumeOfPyramid(base, height)갑자기 모든 것이 훨씬 명확 해지 며 함수 안에 마법의 숫자 ( return base * height / 3)가 있으면 수식의 일부라는 것이 분명하기 때문에 완벽하게 괜찮습니다 .

  열쇠는 여기에 가지고 물론이고 짧은 및 간단한 기능을. 10 개의 인수와 30 개의 계산 라인이있는 함수에서는 작동하지 않습니다. 이 경우 함수 구성 또는 상수를 사용하십시오.

• 도메인 / 비즈니스 규칙

  이것은 값이 정확히 무엇인지에 따라 다르므로 항상 회색 영역입니다. 대부분 의 경우 상수로 변하는 후보가되는 이러한 특정 매직 번호입니다. 프로그램 논리를 복잡하게하지 않고도 프로그램을 더 쉽게 이해할 수 있기 때문입니다. 테스트 if Age < 19대 if Age < LegalDrinkingAge.; 당신은 아마도 상수없이 무슨 일이 일어나고 있는지 알아낼 수 있지만 설명적인 제목으로 더 쉽습니다.

  이것들은 또한 예를 들어 함수 추상화의 후보가 될 수 있습니다function isLegalDrinkingAge(age) { return age >= 19 } . 유일한 것은 비즈니스 로직이 그보다 훨씬 복잡하다는 것입니다. 각각 20-30 개의 매개 변수로 수십 개의 함수를 작성하는 것이 이치에 맞지 않을 수 있습니다. 객체 및 / 또는 함수를 기반으로 명확한 추상화가 없으면 상수를 사용하는 것이 좋습니다.

  주의해야 할 점은 세무 부서에서 일하는 경우 실제로 부담스럽고 ​​솔직하게 쓸 수 없게됩니다 AttachForm(FORM_CODE_FOR_SINGLE_TAXPAYER_FILING_JOINTLY_FOR_DEPRECIATION_ON_ARMPIT_HAIR). 당신은 그렇게하지 않을 것입니다. 당신 AttachForm("B-46")은 일을했거나 일할 모든 개발자가 “B-46″이 단일 납세자 신고를위한 형식 코드라는 것을 알게 될 것입니다. 양식 코드는 도메인 자체의 일부이므로 절대 변경되지 않으므로 실제로 마법의 숫자가 아닙니다.

  따라서 비즈니스 로직에서 상수를 조금만 사용해야합니다. 기본적으로 “매직 넘버”가 실제로 매직 넘버인지 또는 잘 알려진 도메인인지 이해해야합니다. 도메인 인 경우 변경 될 가능성이없는 한 소프트 코딩하지 않습니다.

• 오류 코드 및 상태 플래그

  혹시라도 맞은 가난한 나쁜 놈이 당신에게 말할 수 있듯이, 이것들은 하드 코딩하기에는 결코 괜찮지 않습니다Previous action failed due to error code 46 . 언어가 지원하는 경우 열거 형을 사용해야합니다. 그렇지 않으면 일반적으로 특정 오류 유형에 유효한 값을 지정하는 상수로 가득 찬 전체 파일 / 모듈이 있습니다.

  return 42오류 처리기에서 보지 못하게 하시겠습니까? 변명하지.

아마 몇 가지 시나리오를 생략했지만 그 중 대부분을 다루고 있다고 생각합니다.

그렇습니다. 때로는 하드 코딩하는 것이 좋습니다. 그것에 대해 게으르지 마십시오. 평범한 엉성한 코드보다는 의식적인 결정이어야합니다.

답변

const string server_var="server_var";

const string MySelectQuery="select * from mytable;";

const int 8th_bit=255; //or some other obscure naming scheme that equates to 255.

const int AllowGlobalRead=255;

public static final int HTTP_OK = 200;