Question

I have programmed a script with the goto command but on the server where I want to execute the script there is a previous PHP version (<5.3), so I have to change the code. The structure of the code goes like that:

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

I have tested everything. Do you have any ideas how can I retransform this code without goto in order to be executed properly???

Answer 1

Without knowing how the ->get_items() call behaves you could use this brute-force method in lieu of the goto-switch:

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

The label gets replaced by a while and a self-fulfulling stop condition. And the goto becomes a break and resets the $a stop condition.