함수 내에서 내부 스코프 블록 사용이 잘못된 스타일입니까?

다음과 같은 위험이있는 경우가 있습니다.

• 재사용하지 않는 변수 재사용 (예 1 참조)

• 또는 다른 변수 대신 변수를 사용하여 의미 적으로 닫습니다 (예 2 참조).

예 1 :

var data = this.InitializeData();
if (this.IsConsistent(data, this.state))
{
    this.ETL.Process(data); // Alters original data in a way it couldn't be used any longer.
}

// ...

foreach (var flow in data.Flows)
{
    // This shouldn't happen: given that ETL possibly altered the contents of `data`, it is
    // not longer reliable to use `data.Flows`.
}

예 2 :

var userSettingsFile = SettingsFiles.LoadForUser();
var appSettingsFile = SettingsFiles.LoadForApp();

if (someCondition)
{
    userSettingsFile.Destroy();
}

userSettingsFile.ParseAndApply(); // There is a mistake here: `userSettingsFile` was maybe
                                  // destroyed. It's `appSettingsFile` which should have
                                  // been used instead.

범위를 도입하면이 위험을 완화 할 수 있습니다.

예 1 :

// There is no `foreach`, `if` or anything like this before `{`.

{
    var data = this.InitializeData();
    if (this.IsConsistent(data, this.state))
    {
        this.ETL.Process(data);
    }
}

// ...

// A few lines later, we can't use `data.Flows`, because it doesn't exist in this scope.

예 2 :

{
    var userSettingsFile = SettingsFiles.LoadForUser();

    if (someCondition)
    {
        userSettingsFile.Destroy();
    }
}

{
    var appSettingsFile = SettingsFiles.LoadForApp();

    // `userSettingsFile` is out of scope. There is no risk to use it instead of
    // `appSettingsFile`.
}

잘못 보입니까? 그런 구문을 피하겠습니까? 초보자가 이해하기 어렵습니까?

함수가 너무 길어서 원하지 않는 부작용이나 변수의 불법 재사용을 더 이상 인식 할 수 없다면, 더 작은 함수로 분할하여 내부 범위를 무의미하게 만들어야합니다.

개인적인 경험으로 이것을 뒷받침하기 위해 : 몇 년 전 ~ 150K 줄의 코드로 C ++ 레거시 프로젝트를 상속했으며 정확히이 기술을 사용하는 몇 가지 방법이 포함되었습니다. 그리고 그 모든 방법이 너무 길다는 것을 추측하십시오. 우리가 그 코드의 대부분을 리팩토링 할 때, 메소드는 점점 작아지고 더 이상 남아있는 "내부 범위"메소드가 더 이상 없다고 확신합니다. 그들은 단순히 필요하지 않습니다.

초보자 (및 프로그래머)는 다음을 생각하기가 훨씬 쉽습니다.

• 관련이없는 변수를 사용하지 않는 것

생각보다 :

• 관련이없는 변수를 사용하지 못하게하는 코드 구조 추가

또한 독자의 관점에서 볼 때 이러한 블록은 명백한 역할이 없습니다. 블록을 제거해도 실행에 영향을 미치지 않습니다. 독자는 코더가 달성하고자하는 것을 추측하려고 머리를 긁을 수 있습니다.

범위를 사용하는 것은 기술적으로 옳습니다. 그러나 코드를 더 읽기 쉽게하려면 다른 방법으로 해당 부분을 추출하고 그 의미를 전체 이름으로 지정해야합니다. 이런 식으로 변수 범위를 지정하고 작업 이름을 지정할 수도 있습니다.

변수 재사용이 코드 냄새가 아닌 것보다 더 자주 동의합니다. 아마도 이러한 코드는 더 작고 독립적 인 블록으로 리팩토링되어야합니다.

C #에서 팝업되는 경향이있는 OTOH라는 특정 시나리오가 있습니다. 스위치 케이스 구문을 사용할 때입니다. 상황은 다음과 같이 매우 잘 설명되어 있습니다. 중괄호가 있거나없는 C # 스위치 문… 차이점은 무엇입니까?