函数中内部作用域块的使用是否不好？

在某些（非常罕见）的情况下，存在以下风险：

• 重用不打算重用的变量（请参见示例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`.
}

看起来不对吗？您会避免使用这种语法吗？初学者很难理解吗？

如果您的函数太长，以至于您再也无法识别任何不必要的副作用或变量的非法重用，那么现在是时候将其拆分为较小的函数了，这使内部作用域变得毫无意义。

为了得到一些个人经验的支持：几年前，我继承了一个C ++遗留项目，包含约150K行代码，并且其中包含一些正是使用此技术的方法。猜猜看-所有这些方法都太长了。当我们重构大部分代码时，方法变得越来越小，我很确定不再有剩余的“内部作用域”方法。根本不需要它们。

对于初学者（和任何程序员）而言，考虑以下内容要容易得多：

• 不使用不相关的变量

而不是想到：

• 添加旨在防止自己不使用无关紧要的变量的代码结构。

而且，从读者的角度来看，此类块没有明显的作用：删除它们不会对执行产生影响。读者可能会scratch头，试图猜测编码人员想要实现的目标。

使用示波器在技术上是正确的。但是，如果要使代码更具可读性，则应使用另一种方法提取该部分，并赋予其含义全名。这样，您可以给出变量的范围，也可以给出任务的名称。

我同意重用变量通常不是代码异味。可能应将此类代码重构为更小的独立单元。

在C＃中，有一种特殊的情况，即OTOH，当它们倾向于弹出时-即使用切换用例构造时。在以下情况中很好地解释了这种情况：带/不带大括号的C＃Switch语句……有什么区别？