将一个方法中的代码段与大括号分开是不好的做法？

Question

我倾向于将多行代码填充到某个任务中,并在顶部添加一个小注释,如下所示:

public void doSomething(){
    // common variables needed by all blocks

    { // comment for block 1
        ...
        ... about 5 to 30 lines of code
        ...
    }

    { // comment for block 2
        ...
        ... about 5 to 30 lines of code
        ...
    }

}

我这样做是因为在我看来,它易于阅读,一个块所需的变量将无法在另一个块中造成伤害,并且因为我不想为其他地方不需要的块制作单独的方法.

你会说这是不好的做法吗？我编写的很多人不同意这种编码风格.我知道c#中有区域,但它们不会隔离变量.

编辑:因为每个人都建议我从块中创建方法:有时候我会这样做但我不想如果类已经有20多个方法,那么任何其他方法都不需要块,并且所有块的方法都是还小.

Answer 1

如果你可以像这样打破代码,为什么不把它分解成单独的方法呢？然后将您的doSomething()方法更改为只调用那些较小的方法？

那样:

• 很清楚这项工作的每个要素是什么意思
• 阅读顶级方法,很容易看到整体计划并深入到一个特定部分
• 您可以单独对每个小方法进行单元测试(尽管这可能需要将其设置为非私有的,仅用于测试;是否可以将其作为个人偏好,与其他任何内容一样)