Как построить цикл foreach с условием if else?

Question

[ermilovroman]

У меня есть следующий запрос:

public function check($orderId){
		$url = $this->url.'load_adv_adv.php';
		$pageid = 0;
		$result = $this->http($url, array('page' => $pageid), true);
		preg_match('/({[^{}]*?'.$orderId.'.+?})/', $result, $matches);
		if(isset($matches[1])){
			$order = $matches[1];
			echo $order;
		}
	}

Как сделать так, что бы когда условие if(isset($matches[1])) не срабатывало $pageid увеличивалось на один $pageid++ и запрос повторялся заново?

Answer 1

[Алексей Уколов]

public function check($orderId)
{
    $url = $this->url.'load_adv_adv.php';
    $pageid = 0;

    $found = false;

    while (!$found)
    {
        $result = $this->http($url, array('page' => $pageid), true);

        preg_match('/({[^{}]*?'.$orderId.'.+?})/', $result, $matches);

        if (isset($matches[1]))
        {
            $found = true;
            $order = $matches[1];
        }
        else
        {
            $pageid++;
        }
    }
   
    return $order;
}

$pageid лучше, конечно, чем-то ограничить сверху, а то можно долго ждать, если несуществующий заказ скормить.

Кому-то такой вариант больше по душе (хотя, лично мне он не нравится):

public function check($orderId)
{
    $url = $this->url.'load_adv_adv.php';
    $pageid = 0;

    while (true)
    {
        $result = $this->http($url, array('page' => $pageid), true);

        preg_match('/({[^{}]*?'.$orderId.'.+?})/', $result, $matches);

        if (isset($matches[1]))
        {
            return $matches[1];
        }
         
        $pageid++;
    }
}

Comments

[ermilovroman]

Вот такой вариант ограничения будет верным?

public function check($orderId)
{
$url = $this->url.'load_adv_adv.php';
$pageid = 0;
$found = false;
while (!$found)
{
$result = $this->http($url, array('page' => $pageid), true);
preg_match('/({[^{}]*?'.$orderId.'.+?})/', $result, $matches);
if (isset($matches[1]))
{
$found = true;
$order = $matches[1];
} else if ($pageid == 2) {
$found = true;
$order = "Не найдено";
} else {
$pageid++;
}
}
echo $order;
}

[Алексей Уколов]

ermilovroman: вы про ограничение количества страниц? Да, так будет работать. Только мой внутренний параноик поставил бы >= 2, а не ==. Ну и первые два условия лучше разбить на отдельные if для читаемости, а $pageid++ оставить на последней строке цикла без всяких условий.
P.S. Делать echo внутри таких функций - дурная практика, лучше вернуть значение.

Answer 2

[Евгений Бухарев]

public function check($orderId){
    $url = $this->url.'load_adv_adv.php';
    $pageid = 0;
    start:
    $result = $this->http($url, array('page' => $pageid), true);
    preg_match('/({[^{}]*?'.$orderId.'.+?})/', $result, $matches);
    if(isset($matches[1])){
      $order = $matches[1];
      echo $order;
    } else {
      $pageid++;
      goto start;
   }
}

Comments

[Алексей Уколов]

Надеюсь, это стёб :)

[Евгений Бухарев]

Алексей Уколов: что goto напрягает?

[Алексей Уколов]

Не то слово.

[Евгений Бухарев]

Алексей Уколов: ну да олдскул)

[Алексей Уколов]

Тогда уж так:

public function proverka($nomerZakaza){
    $adres = $this->adres.'load_adv_adv.php';
    $stranica = 0;
    nachalo:
    $resultat = $this->http($adres, array('page' => $stranica), true);
    preg_match('/({[^{}]*?'.$nomerZakaza.'.+?})/', $resultat , $sovpadeniya);
    if(isset($sovpadeniya[1])){
      $zakaz= $sovpadeniya[1];
      echo $zakaz;
    } else {
      $stranica++;
      goto nachalo;
   }
}

:)

[Евгений Бухарев]

Алексей Уколов: Хочешь сказать работать не будет?

[Алексей Уколов]

evgenybuckharev: во-первых, мы с вами на "ты" не переходили, а во-вторых, работать, конечно, будет, но зачем?

[Алексей Уколов]

Это очень плохая практика, поскольку нарушается абстракция и код становится сложночитаемым. Не нужно притворяться компилятором, нужно писать выразительные алгоритмы.

[Евгений Бухарев]

Я предложил один из вариантов решения, вполне рабочий, если он Вам не понравился проходите мимо и свой сарказм оставьте при себе, автор сам решит что использовать, и касательно сложночитаемости тут можно поспорить, и не одного внятного аргумента с Вашей стороны я не услышал, хоть сам goto в своей работе не использую

[Алексей Уколов]

evgenybuckharev: нарушение абстракции - внятный аргумент.

[Котик Антон]

evgenybuckharev: По-моему достаточно внятный аргумент php.net/manual/ru/images/0baa1b9fae6aec55bbb73037f...