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); 
}