상수를 사용하는 것이 올바른 상황입니까?

그래서 교수님은 제가 진행하고있는 프로젝트에 대한 피드백을 제공하고있었습니다. 그는이 코드에 몇 가지 마크를 도킹했습니다.

if (comboVendor.SelectedIndex == 0) {
  createVendor cv = new createVendor();
  cv.ShowDialog();
  loadVendors();
}

콤보 박스 "인덱스 변경"핸들러에 있습니다. 사용자가 새 공급 업체를 만들려고 할 때 사용되는 최상위 옵션 (인덱스 0, 변경되지 않음)은 "새 공급 업체 생성"대화 상자를 엽니 다. 따라서 내 콤보 상자의 내용은 다음과 같습니다.

Create New Vendor...
Existing Vendor
Existing Vendor 2
Existing Vendor 3

그의 문제는 첫 번째 라인 코드입니다.

if (comboVendor.SelectedIndex == 0)

그는 0이 일정해야한다고 주장하고 실제로 그로 인해 마크를 도킹했습니다. 그는 코드에서 리터럴을 전혀 사용해서는 안된다고 주장합니다.

문제는 그 상황에서 해당 코드를 상수로 만들고 싶은 이유를 이해하지 못한다는 것입니다. 그 지수는 절대 바뀌지 않으며 조정해야 할 것이 아닙니다. 매우 구체적인 상황에 사용되며 절대 변경되지 않는 메모리에 단일 0을 유지하는 것은 메모리 낭비처럼 보입니다.

C #에서이를 수행하는 실제 올바른 방법은 ComboItems의 순서에 전혀 의존하지 않는 것 입니다.

public partial class MyForm : Form
{
    private readonly object VENDOR_NEW = new object();

    public MyForm()
    {
        InitializeComponents();
        comboVendor.Items.Insert(0, VENDOR_NEW);
    }

    private void comboVendor_Format(object sender, ListControlConvertEventArgs e)
    {
        e.Value = (e.ListItem == VENDOR_NEW ? "Create New Vendor" : e.ListItem);
    }

    private void comboVendor_SelectedIndexChanged(object sender, EventArgs e)
    {
        if(comboVendor.SelectedItem == VENDOR_NEW)
        {
            //Special logic for selecting "create new vendor"
        }
        else
        {
            //Usual logic
        }
    }
}

콤보 상자의 순서가 변경 될 수 있습니다. "새 벤더 만들기 ..."앞에 "특별 공급 업체 만들기 ..."와 같은 다른 옵션을 추가하면 어떻게됩니까?

상수를 사용하는 이점은 콤보 상자의 순서에 따라 많은 메서드가있는 경우 상수가 변경되면 모든 메서드가 아니라 상수 만 변경하면된다는 것입니다.

상수를 사용하는 것도 리터럴보다 읽기 쉽습니다.

if (comboVendor.SelectedIndex == NewVendorIndex)

대부분의 컴파일 된 언어는 컴파일 타임에 상수를 대체하므로 성능 저하가 없습니다.

당신이 묘사하는이 상황은 판단 요청이며, 개인적으로 한 번만 사용하고 이미 읽을 수 있다면 개인적으로는 사용하지 않을 것입니다.

진짜 대답은 그가 당신에게 교훈을 가르치기 위해 이것을 골랐다는 것입니다.

그가 교수라는 것을 잊지 마십시오. 그의 일은 코딩과 모범 사례를 가르치는 것입니다.

나는 그가 실제로 꽤 잘하고 있다고 말하고 싶습니다.

그가 조금 절대적으로 벗어날 수도 있지만 마법 번호를 사용하기 전에 다시 생각할 것이라고 확신합니다.

또한이 상황에서 모범 사례로 간주되는 것을 찾기 위해 프로그래머에 대해 온라인 커뮤니티에 참여할 수있을 정도로 귀하의 피부 아래에있었습니다.

교수님의 의견

[...] 최상위 옵션 (인덱스 0, 절대 변경되지 않음)은 "새 공급 업체 생성"대화창을 엽니 다.

설명해야했던 사실은 상수를 사용해야하는 이유를 증명합니다. 와 같은 상수를 도입하면 NEW_VENDOR_DIALOG코드가 더 자명합니다. 또한, 컴파일러는 상수를 최적화하므로 성능에는 변화가 없습니다.

컴파일러가 아닌 프로그래머를위한 프로그램을 작성하십시오. 마이크로 최적화를 구체적으로 시도하지 않는 한, 그것은 당신처럼 보이지 않습니다.

그는 0이 일정해야한다고 주장하고 실제로 그로 인해 마크를 도킹했습니다.

동의합니다. 여기서 0을 사용하는 것은 "마법"입니다. 이 코드를 처음 읽는다고 상상해보십시오. 왜 0이 특별한 지 알지 못하고 리터럴은 0이 특별한 이유에 대해 아무 것도 알려주지 않습니다. 대신에 당신이 말했다 if(comboVendor.SelectedIndex == CreateNewVendorIndex)면 코드가 의미하는 것이 처음 독자에게 매우 분명해집니다.

그는 코드에서 리터럴을 전혀 사용해서는 안된다고 주장합니다.

그것은 극단적 인 입장입니다. 현실적인 위치는 리터럴 사용이 코드가 명확하지 않을 수 있음을 나타내는 빨간색 플래그라고 말할 수 있습니다. 때로는 적절합니다.

문제는 그 상황에서 해당 코드를 상수로 만들고 싶은 이유를 이해하지 못한다는 것입니다. 그 지수는 절대 변하지 않을 것입니다

그것이 결코 변하지 않을 것이라는 것이 그것을 일정하게 만드는 훌륭한 이유 입니다. 이것이 상수를 상수라고하는 이유입니다. 그들은 결코 변하지 않기 때문입니다.

또한 조정해야 할 것이 아닙니다.

정말? 당신이 볼 수 있는 사람이 콤보 상자에 사물의 순서를 변경할 수 있습니다있는 상황을?

미래에 이것이 변경 될 수있는 이유를 알 수 있다는 사실은 그것이 일정 하지 않게 하는 좋은 이유 입니다. 오히려 상수가 아닌 읽기 전용 정적 정수 필드 여야합니다. 상수는 동일하게 유지 보장되는 수량해야합니다 모든 시간을 . Pi와 금의 원자 수는 좋은 상수입니다. 버전 번호는 없습니다. 그들은 모든 버전을 바꿉니다. 금의 가격은 분명히 끔찍한 상수입니다. 매 초마다 바뀝니다. 결코 변하지 않는 일정한 것을 만드십시오 .

매우 구체적인 상황에 사용되며 절대 변경되지 않는 메모리에 단일 0을 유지하는 것은 메모리 낭비처럼 보입니다.

이제 우리는 문제의 핵심에 왔습니다.

이것은 아마도 (1) ​​메모리 및 (2) 최적화에 대한 깊은 결함이 있음을 나타 내기 때문에 아마도 가장 중요한 질문 일 것입니다. 당신은 학교에서 배우고 있으며, 이제 기초에 대한 올바른 이해를 얻을 수있는 좋은 시간입니다. 왜 "0을 메모리에 유지하는 것은 메모리 낭비"라고 믿는 이유를 자세히 설명 할 수 있습니까? 우선, 최소 20 억 바이트의 사용자 주소 지정 스토리지가있는 프로세스에서 4 바이트 메모리 사용을 최적화하는 것이 적절하다고 생각하는 이유는 무엇입니까? 둘째, 정확히 어떤 자원이 여기에서 소비되고 있다고 생각 하십니까? "메모리"가 소비된다는 것은 무엇을 의미합니까?

이러한 질문에 대한 답변은 먼저 최적화 및 메모리 관리에 대한 이해가 어떻게 틀린지 배울 수있는 기회이기 때문에 관심이 있으며, 둘째는 초보자가 왜 기괴한 것을 믿는지 알고 싶어서 설계 할 수 있기 때문입니다. 올바른 신념을 갖도록하는 더 나은 도구.

그가 맞아. 네가 옳아. 네가 틀렸어.

그는 개념적으로 매직 넘버를 피해야한다는 것이 맞습니다. 상수는 숫자의 의미에 컨텍스트를 추가하여 코드를 더 읽기 쉽게 만듭니다. 나중에 누군가가 코드를 읽을 때 특정 숫자가 사용 된 이유를 알게됩니다. 그리고 줄 아래로 값을 변경해야하는 경우 특정 숫자가 사용되는 모든 곳을 찾아 보는 것보다 한 곳에서 변경하는 것이 훨씬 좋습니다.

그렇습니다, 당신 말이 맞아요. 이 특정 경우에는 상수가 보장된다고 생각하지 않습니다. 목록에서 첫 번째 항목을 찾고 있는데 항상 0입니다. 절대 23이되지 않습니다. 또는 -pi. 구체적으로 0을 찾고 있습니다. 나는 코드를 일정하게 만들어서 혼란스럽게해야한다고 생각하지 않습니다.

그러나 상수가 변수를 '메모리 사용'으로 전달한다고 가정하면 잘못되었습니다. 인간과 컴파일러에는 상수가 있습니다. 그것은 컴파일러가 컴파일 도중 그 자리에 그 값을 넣도록 지시합니다. 그렇지 않으면 리터럴 숫자를 넣었을 것입니다. 그리고 가장 까다로운 응용 분야를 제외하고 메모리에 상수를 전달하더라도 효율성 손실은 측정 할 수 없습니다. 단일 정수의 메모리 사용에 대한 걱정은 확실히 '조기 최적화'에 해당합니다.

0와 같은 의미를 명확하게하기 위해를 상수 로 대체했을 것 NewVendorIndex입니다. 주문이 변경되는지 알 수 없습니다.

그것이 교수님의 총체적인 선호입니다. 일반적으로 리터럴을 여러 번 사용할 경우 상수를 사용하고, 라인의 목적이 무엇인지 독자에게 명확하게 알리거나 나중에 리터럴이 변경 될 수 있으며 변경하려는 경우에만 상수를 사용하십시오. 한 곳에서. 그러나 이번 학기에는 교수가 사장이므로 지금부터는 그 수업에서하겠습니다.

기업 세계를위한 좋은 훈련? 아마 가능합니다.

솔직히 말해서, 귀하의 코드가 모범 사례라고 생각하지는 않지만 그의 제안은 솔직히 조금 괴상합니다.

.NET 콤보 박스에 대한보다 일반적인 방법은 "Select .."항목에 빈 값을 제공하고 실제 항목에는 의미있는 값을 부여한 다음 수행하는 것입니다.

if (string.IsNullOrEmpty(comboVendor.SelectedValue))

오히려

if (comboVendor.SelectedIndex == 0)

그는 상수 사용의 가치를 강조하는 데 잘못이 아니며 리터럴을 사용하는 데 잘못이 없습니다. 그가 이것이 예상되는 코딩 스타일이라고 강조하지 않는 한, 리터럴은 해롭지 않기 때문에 사용에 대한 마크를 잃어서는 안됩니다. 상용 코드에서 리터럴이 여러 번 사용되는 것을 보았습니다.

그의 요점은 좋다. 이것은 상수의 이점을 인식하게하는 그의 방법 일 수 있습니다 .

1-실수로 조작되지 않도록 코드를 어느 정도 보호합니다

@DeadMG가 2-As에 따르면, 동일한 문자 값이 여러 곳에서 사용되면 실수로 다른 값으로 나타날 수 있으므로 상수는 일관성을 유지합니다.

3 상수는 유형을 유지하므로 0을 의미하기 위해 0F와 같은 것을 사용할 필요가 없습니다.

4의 경우 읽기의 용이성, COBOL의 값은 제로에 대한 예약 된 단어로 ZERO를 사용 (하지만 당신은 문자 그대로 제로 사용할 수 있습니다) - 그래서, 예를 들어, 이름은 때때로 도움이되는 값을 제공 : (출처 : MS-상수

class CalendarCalc
{
    const int months = 12;
    const int weeks = 52; //This is not the best way to initialize weeks see comment
    const int days = 365;

    const double daysPerWeek = (double) days / (double) weeks;
    const double daysPerMonth = (double) days / (double) months;
}

또는 귀하의 경우 (@Michael Krussel 답변에 표시된)

복잡한 파생이 있거나 자주 반복되는 경우에만 상수로 저장하면됩니다. 그렇지 않으면 리터럴이 좋습니다. 모든 것을 일정하게 유지하는 것은 완전히 과잉입니다.

실제로 언급했듯이 위치가 변경되면 어떻게됩니까? 할 수 있거나해야 할 일은 색인에 의존하기보다는 코드를 사용하는 것입니다.

따라서 선택 목록을 만들면 html과 같이 끝납니다.

<select>
    <option value='CREATE'>Create New Vendor...</option>
    <option value='1'>Existing Vendor</option>
    <option value='2'>Existing Vendor 2</option>
    <option value='3'>Existing Vendor 3</option>
</select>

그런 다음을 확인하는 대신 selectedIndex === 0값이 CREATECODE일정하며이 테스트와 선택 목록을 작성할 때 사용 된 값인지 확인 하십시오.

나는 그것을 완전히 제거 할 것입니다. 콤보 상자 목록 옆에 새 만들기 버튼을 넣으십시오. 목록에서 항목을 두 번 클릭하여 편집하거나 단추를 클릭하십시오. 콤보 박스에 새로운 기능이 내장되어 있지 않습니다. 그런 다음 마법 번호가 모두 제거됩니다.

일반적으로 코드의 리터럴 숫자는 상수를 정의하여 숫자 주위에 컨텍스트를 배치해야합니다. 제로 란 무엇입니까? 이 경우 0 = NEW_VENDOR입니다. 다른 경우에는 다른 의미가있을 수 있으므로 가독성과 유지 관리 성이 컨텍스트를 둘러싼 것이 좋습니다.

다른 사람들이 말했듯이 인덱스 번호 이외의 다른 방법을 사용하여 주어진 동작에 해당하는 콤보 상자 항목을 식별해야합니다. 또는 프로그래밍 논리가있는 색인을 찾아 변수에 저장할 수 있습니다.

내가 쓰고있는 이유는 "메모리 사용"에 대한 귀하의 의견을 제시하기 위해서입니다. C #에서 대부분의 언어에서와 같이 상수는 컴파일러에 의해 "접 힙니다". 예를 들어, 다음 프로그램을 컴파일하고 IL을 검사하십시오. 컴퓨터 메모리는 물론 모든 숫자가 IL에 들어 가지 않는다는 것을 알 수 있습니다.

public class Program
{
    public static int Main()
    {
        const int a = 1000;
        const int b = a + a;
        const int c = b + 42;
        const int d = 7928345;
        return (a + b + c + d) / (-a - b - c - d);
    }
}

결과 IL :

.method public hidebysig static 
    int32 Main () cil managed 
{
    .maxstack 1
    .locals init (
        [0] int32 CS$1$0000
    )

    IL_0000: nop
    IL_0001: ldc.i4.m1  // the constant value -1 to be returned.
    IL_0002: stloc.0
    IL_0003: br.s IL_0005

    IL_0005: ldloc.0
    IL_0006: ret
}

따라서 상수 산술을 사용하여 상수, 리터럴 또는 킬로바이트 코드를 사용하더라도 값은 문자 그대로 IL에서 처리됩니다.

관련 사항 : 상수 접기는 문자열 리터럴에 적용됩니다. 많은 사람들은 이와 같은 호출이 너무 불필요하고 비효율적 인 문자열 연결을 초래한다고 생각합니다.

public class Program
{
    public static int Main()
    {
        const string a = "a";
        const string b = a + a;
        const string c = "C";
        const string d = "Dee";
        return (a + b + c + d).Length;
    }
}

그러나 IL을 확인하십시오.

IL_0000: nop
IL_0001: ldstr "aaaCDee"
IL_0006: callvirt instance int32 [mscorlib]System.String::get_Length()
IL_000b: stloc.0
IL_000c: br.s IL_000e
IL_000e: ldloc.0
IL_000f: ret

결론 : 상수 표현식의 연산자는 상수 표현식을 생성하며 컴파일러는 모든 계산을 수행합니다. 런타임 성능에는 영향을 미치지 않습니다.