GoTo 的使用是否可接受？

Question

我目前正在.Net Framework 2.0 中用C# 重写一个旧的VB6 程序（不是我的选择，这是公司决定的）。在大多数情况下，事情进展顺利。该程序测量来自精密磨床的传入数据，并显示图表和刻度盘以显示精度。

然而，最初的程序员是一名机械工程师，而不是软件工程师。该程序可以运行，但到处都有一些草率的代码。最值得注意的是，我遇到了一些 GoTo 语句。在必要时将事物放入循环中并从中获得相同的功能非常容易。

然而，我在原始代码中遇到了一个情况，GoTo 似乎不仅仅只是模拟循环。它有几个不同的退出条件。它看起来像这样（不是实际的代码，只是我为了演示而编写的一些简短内容）：

VB6 代码

Public Sub Tick()
    Dim condition1 As Boolean
    Dim condition2 As Boolean
    Dim testNumber As Integer

    beginning:    'The GoTo label'

    ' (... Some Other Code Here ...)'

    If condition1 = True Then
        goto beginning
    Else
        ' (... Do some calculation ...)'
    End If

    If condition2 = True Then
        ' (... Do some calculation ...)'
        goto beginning
    End If

    Select Case testNumber
        Case 1: '(... Some code ...)'
        Case 2: '(... Some code ...)'
        Case 3: '(... Some code ...)'
        Case 4: goto beginning
    End Select
End Sub

实际的代码可能比这少一些条件，但基本思想是有一些不同的事情导致它自行循环。对于这种情况，是否有一个好的方法来编写循环，或者在这种情况下 goto 语句是否可以接受？ （诚​​然，非 goto 解决方案是首选）。

感谢您的时间和考虑。

注意：我尝试使用带有中断的 while(true) 循环；语句，但它导致程序陷入无限循环并锁定。是否更建议编写一个包含多个条件（使用和/或等）的长 while 循环？

Answer 1

while(true) 循环应该没问题，如果你在它的末尾有一个中断并且在之前有 goto 的地方继续。然而，这绝对应该只是第一步——听起来需要进行大力的重构。

Answer 2

首先将该循环的主体放入一个单独的函数中，并用“return”替换“goto”——或者预先准备几个单独的函数：

If condition1 = True Then 
    goto beginning 
Else 
    ' (... Do some calculation ...)' 
End If 

应该很快就会成为

If not condition1
      DoSomeCalculation()
End If

关于何时循环和循环的逻辑。何时退出将会出现。当这种情况发生时，重构此代码应该变得像您已经完成的那样微不足道。

Answer 3

虽然这种情况对于 do/while true 循环来说看起来足够好，但我也见过一些情况并非如此。

在词法分析器或其他 FSA 机制之外，我的判断是，每 2000 行超过 1 个 goto 意味着您做错了什么。

当然，如果您有一个带有 goto 的重复出现的习语，那就是另一回事了，因为重复出现的习语会覆盖样式规则。习语=一致，一致=可读。

Answer 4

我认为您的第一步应该是将所有 '(do some code)' 提取到它们自己的方法中。完成此操作后，实际的代码流程将变得更加清晰。

根据嵌套的方式，有几种可能的方法可以实现此目的（没有实际代码很难实现）。

（我是一名 C# 编码员，我不懂 VB，请原谅）

递归的

Public Sub Tick()
    Dim condition1 As Boolean
    Dim condition2 As Boolean
    Dim testNumber As Integer

    If basecase = True Then
       return;
    EndIf

    ExecuteInitialzerStuff();

    If intialized = False Then
        Tick();
        return;
    Else
        ExecuteAffirmationStuff();
    End If

    If affirmed = True Then
        ExecutePostAffirm();
        Tick();
        return;
    End If

    Select Case testNumber
        Case 4: Tick();
    End Select
End Sub

另一个选择是将每个选项分解为离散的代码流

Public Sub Tick()
    Dim condition1 As Boolean
    Dim condition2 As Boolean
    Dim testNumber As Integer

    If condition1 = true Then
       Tick_Condition1();
       return;
    EndIf

    If condition2 = true Then
       Tick_Condition2();
       return;
    EndIf

    Tick_Switch(testNumber);

一旦分解了每个代码部分试图执行的每个单独的任务完成后，很可能应该清楚该方法应该被完全删除，并分成几个单独的 Tick() 方法，每个方法调用 TickInit() TickDestroy()、TickSkyFalling(); 或其他，视情况而定。

我认为尝试重构这个函数是错误的决定。但在没有看到实际代码的情况下我无法确定。

Answer 5

在 switch 语句中：（

switch (groupMembershipStatus)
{
    case SocialGroupMembershipStatus.Banned:
        return redirect();
    case SocialGroupMembershipStatus.MembershipRequestDenied:
        Abc();
        goto case SocialGroupMembershipStatus.Banned;
}   

如您所见，我刚刚在生产代码中编写了一个 goto，我想知道是否存在关于 goto 的使用的 C# 问题！）

Answer 6

我将其包装在单元测试中，并通过它触发各种值并记录结果。

然后，当您将代码重构为 C# 时，您可以使用测试结果来验证您的操作。