Блог

У меня есть этот цикл, который должен возвращать номер ключа, соответствующий определенным критериям (где md5 нескольких внутренних ключей соответствует строке, сообщаемой клиентом).

 foreach ( $response->posts as $key => $element ) {
    if ( $postData['sign'] == md5($api_secret.$postData['date'].$response->count.$element->text) ) {
        break;
    }
    elseif ( $postData['sign'] == md5($api_secret.$postData['date'].$postData['num'].$element->text) ) {
        $log_msg = date('H:i:s').' Comment "'.$postData['last_comment'].'" saved with less accuracy (another comment added before it was saved)';
        $this->wh_log($log_msg);
        break;
    }
    else {
        $log_msg = date('H:i:s').' Couldn't save the comment "'.$postData['last_comment'].'" sent from '.$ip.' because it failed the integrity check';
        $this->wh_log($log_msg);
        return $this->save_level2($access_token, $response, $referer);
    }
}
echo 'level 1 $key is '.$key.' || ';
  

Проблема в том, что во время дальнейшего тестирования выяснилось, что если он не находит условие соответствия в нулевом ключе ( $response->posts[0] ), то он немедленно переходит к последнему условию (определенному в else ) и создает файл журнала с ошибкой, даже не пытаясь найти условие соответствия в других ключах, таких как 1, 2 и т.д. И моя функция echo сообщает level 1 $key is 0 , в то время как предполагается, что она отображает ключ, который соответствует условию. Но, похоже, по какой-то причине это не выходит за пределы нулевого ключа.

// Я отредактировал код так, чтобы он не содержал переменных, которые люди не признали бы.

Вы не перебираете значения ($elements)… Он всегда завершается после одной итерации:

• Если условие номер один верно -> разорвет цикл = нет итерации.
• Если верно второе условие -> разорвет цикл = нет итерации.
• Если оба имеют значение false -> вернет = прерывает цикл и завершает работу функции = без итерации.

Я не уверен, что вы пытаетесь сделать — исходя из вашего вопроса, вы ожидаете, что итерация прервется, когда один из хэшей будет равен, в противном случае продолжайте итерацию -> Повторно подумайте об использовании этих разрывов и возврата, я думаю, вы хотите использовать что-то еще вроде CONTINUE .

Пример:

 //Consider this example inputs based on your question:
$secret = "bla";
$date = "00:00:0000";
$num  = 1;
$postData = [
    "example sign1" => md5($secret.$date."postdatahere with date"),
    "example sign2" => md5($secret.$num."postdatahere with num"),
    "date"  => $date,
    "num"   => $num
];
$posts = [
    "key1" => ["text" => "postdatahere with date"],
    "key2" => ["text" => "postdatahere with num"],
    "key3" => ["text" => "will exit the loop"]
];

//Run -> check_postdata is the function containing the logic:
echo check_postdata($posts, $postData, $secret, "example sign1")."n";
echo check_postdata($posts, $postData, $secret, "example sign2")."n";
  

Упрощенный пример 1 с использованием вашей методологии:

 //Using this logic with the BREAKs:
function check_postdata($posts, $postData, $secret, $example) {
    foreach ( $posts as $key => $element ) {

        //Same as the md5 part:
        $correct_value_date = md5($secret.$postData["date"].$element["text"]);
        $correct_value_num = md5($secret.$postData["num"].$element["text"]); 

        if ( $postData[$example] === $correct_value_date ) {
            echo "  -  ".$key." -> break in IF block!n";
            break;
        }
        elseif ( $postData[$example] === $correct_value_num ) {
            echo "  -  ".$key." -> break in ELSEIF block!n";
            break;
        } else {
            echo "  -  ".$key." -> continue in ELSE block!n";
            return 0;
        }

    }
    return 1;
}
/*  The Result is:
 *
 *  -  key1 -> break in IF block!
 *  1
 *  -  key1 -> continue in ELSE block!
 *  0
 */
  

Упрощенный пример 2 с использованием CONTINUE :

 //Using this logic with the CONTINUE:
function check_postdata($posts, $postData, $secret, $example) {
    foreach ( $posts as $key => $element ) {

        //Same as the md5 part:
        $correct_value_date = md5($secret.$postData["date"].$element["text"]);
        $correct_value_num = md5($secret.$postData["num"].$element["text"]); 

        if ( $postData[$example] === $correct_value_date ) {
            echo "  -  ".$key." -> break in IF block!n";
            break;
        }
        elseif ( $postData[$example] === $correct_value_num ) {
            echo "  -  ".$key." -> break in ELSEIF block!n";
            break;
        } else {
            echo "  -  ".$key." -> return in ELSE block!n";
            continue;
        }

    }
    return "Returned key is: ".$key;
}
/*  The Result is:
 *
 *    -  key1 -> break in IF block!
 *  Returned key is: key1
 *    -  key1 -> continue in ELSE block!
 *    -  key2 -> break in ELSEIF block!
 *  Returned key is: key2
 */
  

Как вы можете видеть, использование вашей логики не имеет смысла, поскольку она всегда будет повторяться один раз и останавливаться.
Использование CONTINUE метода будет повторяться до тех пор, пока одно из условий не будет выполнено.

Надеюсь, я помог..

Наконец-то решена проблема. Мне пришлось создать переменную, которая устанавливается только в том случае, если найдено совпадение. Кроме того, мне приходилось продолжать цикл каждый раз, когда совпадение не находилось. Я думал, что это будет продолжаться в любом случае, пока не будет выполнено условие, но я ошибался. Вот рабочее решение:

 foreach ( $response->posts as $key => $element ) {
    if ( $postData['sign'] == md5($api_secret.$postData['date'].$response->count.$element->text) ) {
        $key_obtained = $key;
        break;
    }
    else continue;
    if ( $postData['sign'] == md5($api_secret.$postData['date'].$postData['num'].$element->text) ) {
        $key_obtained = $key;
        $log_msg = date('H:i:s').' Comment "'.$postData['last_comment'].'" saved with less accuracy (another comment added before it was saved)';
        $this->wh_log($log_msg);
        break;
    }
    else continue;
    if ( !isset ($key_obtained) ) {
        $log_msg = date('H:i:s').' Couldn't save the comment "'.$postData['last_comment'].'" sent from '.$ip.' because it failed the integrity check';
        $this->wh_log($log_msg);
        return $this->save_level2($access_token, $response, $referer);
    }
}
echo 'level 1 $key is '.$key.' || ';
  

во-первых, для корректности, ответ на исходный вопрос «почему рассматривается только первый ключ» (перефразированный) по-прежнему:

return немедленно завершит работу функции (если внутри функции) или скрипта / файла (в противном случае). и он будет возвращен независимо от того, находится он в цикле или нет. таким образом, при первом выполнении вашей else ветви цикл завершается из-за return , во всех других ветвях он завершается из-за break

однако для решения основной проблемы:

Проблема, которую нужно решить: найдите первый ключ, соответствующий условию 1, если его нет, найдите первый ключ, соответствующий условию 2, если его нет, выведите какой-нибудь общий материал (вывод ошибки).

Существует много способов решения этой проблемы, где основное различие заключается в том, выполняется ли цикл один или два раза. Поскольку вы уже определили свои условия, я просто назову их $cond1 и $cond2 соответственно:

 $cond1key = $cond2key = null; // <-- this stores what we'll find
foreach($response->posts as $key => $element) {
    if($cond1) {
         // if we find a key, for which $cond1 is true, we can abort
         $cond1key = $key;
         break;
    } elseif(!$cond2key amp;amp; $cond2) {
         // if we find a key, for which $cond2 is true, there still
         // may be a key, for which $cond1 is true, so no break.
         // also notice the !$cond2key in the if statement, this 
         // prevents overwriting
         $cond2key = $key;
    }
 }
 // now we look at the results of our "search"
 if($cond1key) {
     // key found by condition 1
     echo 'level 1 $key is '.$cond1key.' || ';
 } elseif($cond2key) {
     // key found by condition 2
     $log_msg = date('H:i:s').' Comment "'.$postData['last_comment'].'" saved with less accuracy (another comment added before it was saved)';
     $this->wh_log($log_msg);
 } else {
     // no key found
     $log_msg = date('H:i:s').' Couldn't save the comment "'.$postData['last_comment'].'" sent from '.$ip.' because it failed the integrity check';
     $this->wh_log($log_msg);
     return $this->save_level2($access_token, $response, $referer);
 }
  

теперь, я использовал там только один цикл, повторный цикл также был бы возможен (теперь код очень упрощен):

  foreach($response->posts as $key => $value) {
     if($cond1) {
          $cond1key = $key;
          break;
     }
 }
 if(!$cond1key) {
     foreach($response->posts as $key => $value) {
         if($cond2) { // no danger of overwriting, since we break ...
             $cond2key = $key;
             break; // since we're only looking for a cond2 key now.
         }
     }
 }
 // conditional code execution as before
  

Теперь этот код настолько прост, насколько я могу его сформулировать. Существуют несколько более элегантные решения, которые не дают реальной выгоды с точки зрения понимания, это абсолютно не предназначено для победы в каких-либо конкурсах;o)

Примечание: $cond1key / $cond2key явно неправильные имена, если у вас есть имена для этих переменных, которые семантически более подходят, используйте их.

Надеюсь, это поможет;o)