•  06-09-2019
  •  | 
  •  

문제

다음 사이에 큰 성능 차이가 있습니까?

if (this.chkSelectAll.Checked)
    for (int i = 0; i < this.listBoxColumns.Items.Count; i++)
        this.listBoxColumns.SetSelected(i, true);
else
    for (int i = 0; i < this.listBoxColumns.Items.Count; i++)
        this.listBoxColumns.SetSelected(i, false);

vs.

for (int i = 0; i < this.listBoxColumns.Items.Count; i++)
    this.listBoxColumns.SetSelected(i, this.chkSelectAll.Checked);

어느 것이 권장되는지. 간결한 코딩 대 성능 게인?

도움이 되었습니까?

해결책

나는 많은 성능 차이를 기대하지 않을 것이며, 더 읽기 쉬운 것처럼 후자와 함께 갈 것입니다. (그래도 교정기를 둥글게 뿌렸습니다.)

루프를 변경해야 할 상황을 상상하기는 매우 쉽고 첫 번째 예제로 실수로 둘 다 대신 하나만 변경할 수 있습니다. 만약 너라면 진짜 전화를 피하고 싶습니다 Checked 모든 반복의 속성은 항상 할 수 있습니다.

bool checked = this.chkSelectAll.Checked;
for (int i = 0; i < this.listBoxColumns.Items.Count; i++)
{
    this.listBoxColumns.SetSelected(i, checked);
}

그 어느 때보다도 가장 읽기 쉬운 코드를 먼저 작성하고 성능을 위해 디자인/코드를 구부리기 전에 성능 차이를 측정/프로파일로 측정하십시오.

다른 팁

성능 차이가 거의 눈에 띄지 않을 것이라고 생각합니다. 그러나 다음은 효율적이고 읽기 쉬운 변형입니다.

bool isChecked = this.chkSelectAll.Checked;
for (int i = 0; i < this.listBoxColumns.Items.Count; i++) {
    this.listBoxColumns.SetSelected(i, isChecked);
}

실제 최적화를 한 경우 각 반복에서 "this.listboxcolumns"에 액세스하는 오버 헤드가 처음에 두 번 존재하고주의를 기울여야하는지 여부에주의를 기울여야합니다. 그것이 프로파일 링의 것입니다.

첫 번째 예제에 추가 부울 점검이 있습니다. 그러나 성능 차이가 무시할 수있는 것 외에는 성능 차이가 될 것이라고 상상할 수 없습니다. 특정 시나리오에서 이것을 측정해 보셨습니까?

두 번째 예제는 루프 코드를 반복하지 않기 때문에 바람직합니다.

나는 둘 사이에 상당한 성능 차이가 있음을 알 수 없습니다. 그것을 확인하는 방법은 벤치 마크와 시간을 1000 초 이상의 다른 알고리즘을 설정하는 것입니다.

그러나 UI 코드이므로 사용자가 대화 상자를 읽고 다음에 수행 할 작업을 결정하기를 기다릴 때 성능 게인은 의미가 없습니다.

개인적으로 나는 매번 두 번째 접근 방식을 가질 것입니다. 유지해야 할 루프가 하나 뿐이며 코드는 명확합니다.

성능 차이는 무시할 수 있습니다.

주요 관심사는 코드 가독성 및 유지 관리 가능성입니다.

이와 같은 미세 최적화는 종종 잘못 배치됩니다. 성능에 관심이 있기 전에 항상 프로필.

무시할 가능성이 높습니다. 그러나 더 중요한 것은 다음을 인용해야한다고 생각합니다.

"조기 최적화는 모든 악의 근원입니다"

두 번째는 쉽게 읽을 수 있으므로 나중에 최적화 할 필요가 없다면 (내 의견으로는 가능성이 거의 없음).

System.diagnostics.stopwatch를 사용하고 두 사람을 직접 비교해보십시오. 그러나 나는 실제 성능 차이가있을 것이라고 생각하지 않습니다. 첫 번째 예제는 chkselectall.checked 만 액세스하기 때문에 더 빠를 수 있습니다. 둘 다 쉽게 읽을 수 있습니다.

라이센스 : CC-BY-SA ~와 함께 속성
제휴하지 않습니다 StackOverflow
scroll top