使用GOTO語句只能跳轉出現的問題？

Question

僅使用GOTO語句跳下來會有什麼壞處嗎？或者我們完全安全？

我的意思是覺得這是我的代碼，

Some code ... 
... 

GOTO whereToJump 

Some code ... 
... 

whereToJump: 

Some code ... 

當whereToJump點總是GOTO語句下面，有沒有安全問題？

Answer 1

轉到不會讓代碼不那麼安全，不管你跳哪種方式。

您以任何喜歡的方式構建您的代碼。這是你的代碼決定代碼是好還是壞。 goto也許在某些情況下是適當的（不，我可以展示一個例子:-)）

goto的問題是它可能會隱藏程序中的執行流程並使代碼混淆。但是，如果你創建容易閱讀/維護代碼，那麼這不是一個問題。

Answer 2

這是最着名的計算機科學論文之一的主題。 Dijkstra的Go To Statement Considered Harmful。它基本上表明，沒有人需要轉到（我知道有一些例外情況一如既往）。

它是1968年，但今天仍然非常可讀。

Answer 3

它是有道理的，只有向下的跳躍使得意大利麪條代碼的潛力大大降低。 （根據我的經驗，在調試遺留的基於GOTO的代碼時，75％的頭痛來自於這種情況，當向上的gotos導致混亂的循環時）

但是，考慮到你只使用向下跳轉，應該很容易轉換爲不基於代碼的代碼。我不確定有多少改進你的gotos會提供。

Answer 4

在一個更復雜的例子中，當然如果你決定只使用GOTO來跳轉在相同的範圍，你可以使用if語句。 （如果它不在同一個範圍內，它不是真的「只是向前跳躍」，是嗎？）

（當然，如果你的真實代碼不是那麼複雜，那麼你可以擺脫那個第二塊「部分代碼...」）

Answer 5

好吧，這裏真的沒有問題。該代碼相當於：

Some code ... 
... 

if (false) 
{ 
Some code ... 
... 
} 

Some code ... 

這就是說，我不會這樣做。 goto的好用例非常少見，而其他流控構造在C＃中更具慣用性。

Answer 6

現在應該避免GOTO的主要原因是程序員一般不再習慣GOTO（這是一件好事）。因此，如果您編寫廣泛使用GOTO的代碼，那麼您的同行程序員很可能難以理解並擴展它。

更糟的是，這裏和那裏的GOTO可能會導致破窗綜合症，因爲同事們開始使用越來越多的GOTO，直到您剩下一大碗意大利麪。

xkcd說得最好...

Answer 7

我使用goto語句時需要它JUMB從向下許多地方的代碼塊。通常沒有goto語句，這可能需要編寫另一個函數（以便上面的代碼塊在第一個函數中不重複）並在第一個函數內部調用。 goto的效率不及if或switch語句，因爲它還利用if，switch等使用的jumb語句之一（jz，jnz等）。

Answer 8

有了這個評論，我不鼓勵使用的GOTO，但只是爲了表明在某些特定情況下它可能有用。到目前爲止，在實踐中我還沒有看到任何使用goto語句的代碼向上或向下跳轉，但有一個特定的用例，我發現goto更具可讀性。這裏是一個代碼示例，我認爲goto更有意義。

void fun_test() 
{ 
    .code.. 
    allocate dynamic memory. 
    acquire_lock(); 

    //do some action, call api. 
    if (some error) { 
     print error; 
     free memory; 
     unlock(); 
     return FAIL; 
    } else { 
     // some code. 
    } 

    if (some_other error) { 
     print error; 
     free memory; 
     unlock(); 
     return FAIL; 
    } 
    : 
    : 
    // more error condition checks. 
    : 
    return SUCCESS 

} 

void fun_test_withgoto() 
{ 
    error_code rc = SUCCESS; 
    .code.. 
    allocate dynamic memory. 
    acquire_lock(); 

    //do some action, call api. 
    if (some error) { 
     rc = <Failure id>; 
     goto function_cleanup; 
    } else { 
     // some code. 
    } 

    if (some_other error) { 
     rc = <failure id>; 
     goto function_cleanup;  
    } 
: 
: 
// more error conditions. 
: 

function_cleanup: 
    if (rc == SUCCESS) { 
     print "Success.."; 
    } else { 
     print "Error : <failure id>"; 
    } 
    free_memory(); 
    unlock(); 
    return(rc); 
}