php中的goto循環問題

Question

我已經用goto命令編寫了一個腳本，但是在我想執行腳本的服務器上有一個以前的PHP版本（< 5.3），所以我必須更改代碼。代碼的結構是這樣的：

for($i = 0; $i < 30; $i++) // print 30 articles 
{ 
    $x = 0; 
    // choose a a feed from the db 
    // parse it 

    a: 
    foreach($feed->get_items($x, 1) as $item) 
    { 
     // create a unique id for the article of the feed 

     if($id == $dbid) 
     { 
      // if this id exists in the db, take the next article of the same feed which is not in the db 
      $x++; 
      goto a; 
     } 
     else 
     { 
      // print the original article you grabbed 
     } 

    } // end of foreach 
} // end of for 

我已經測試了一切。你有什麼想法，我怎麼能不用goto轉換這個代碼，以便正確執行？

Answer 1

如果不知道->get_items()電話的行爲，你可以代替轉到開關的使用窮舉法：

for($i = 0; $i < 30; $i++) 
{ 
    $x = 0; 


    $a = 1; 
    while ($a--) 
    foreach($feed->get_items($x, 1) as $item) 
    { 

     if($id == $dbid) 
     { 
      $x++; 
      $a=1; break; 
     } 
     else 
     { 
     } 

    } // end of foreach 
} // end of for 

標籤被替換爲while和自我實現的停止條件。並且goto變成休息並且重置$a停止條件。

Answer 2

像這樣的東西可能會工作...

function loop(){ 
    foreach($feed->get_items($x,1) as $item){ 
    if($id==$dbid){ 
     $x++; 
     loop(); 
    }else{ 

    } 
    } 
} 
for($i=0;$i<30;$++){ 
    $x=0; 
    loop(); 
} 

對不起，我刪除了所有的評論，他們是討厭。

Answer 3

移動聲明的$ x的外部for循環和休息後替換您的標籤/ GOTO組合，像這樣......

$x=0; 
    for($i=0;$i<30;$++) //print 30 articles 
    { 
    foreach($feed->get_items($x,1) as $item) 
    { 
     // create a unique id for the article of the feed 
     if($id==$dbid) 
     { 
     //if this id exists in the db,take the next article of the same feed which is not in the db 
      $x++; 
      continue; 
     } 
     else 
     { 
     //print the original article you grabbed 
     } 

    } // end of foreach 
    }//end of for 

Answer 4

與未設置同意 - 利用突破將打破，如果循環，並保持通過迭代for循環

Answer 5

這個問題說明了爲什麼要避免goto。它可以讓你在沒有足夠思考算法的情況下離開。

標準的做法是用一個標誌。我想你也不能指望「herezthecode kthxbai」之類的答案，但在這種情況下，解釋它是寫代碼的最佳方式 -

for($i=0;$i<30;$++){ 
    $x=0; 
    do { 
    $found = false; 
    foreach($feed->get_items($x,1) as $item){ 
     // get $id 
     if($id==$dbid){ 
     $found = true; 
     break; 
     }else{ 
     // other things 
     } 
    } 
    $x++; 
    } while($found); 
}