0

Есть список матчей, выводится из массива. При клике на матч в $_GET запрос уходит ID матча. Мне надо вывести только этот матч, то есть отфильтровать массив, и оставить только один матч. Сейчас я делаю так, но думаю такой метод не очень хороший

if(isset($_GET['match'])) {
    $matches = getMatches();
    foreach ($matches as $match) { 
        if ($match['id'] == $_GET['match']) {
            $currentMatch = $match;
        }
    }
}
9
  • В каком виде изначально хранятся матчи, что происходит в getMatches()? 12 сен 2018 в 14:38
  • @Alexxosipov getMatches() возвращает массив матчей. 12 сен 2018 в 14:43
  • 1
    Мне опыт подсказывает, что Вы совершили архитектурную ошибку. Что стоит до метода getMatches()? Откуда берутся эти данные? Почему не вынесли объект матча в отдельный класс? 12 сен 2018 в 14:46
  • 1
    Ну и от нас-то вы что хотите? Если есть метод получающий все матчи, значит должен быть метод получающий матч по его ID.
    – u_mulder
    12 сен 2018 в 14:46
  • @u_mulder данные берутся с API, а количество бесплатных запросов ограничено, поэтому делать лишний запрос не вариант 12 сен 2018 в 14:49

1 ответ 1

1

Сам метод поиска нужного элемента в массиве у вас в принципе обычен. Другой вопрос в том, что если данное действие потенциально будет повторяться, то каждый раз писать подобное - не лучшая затея.

Наиболее простой способ для такого поиска, да и в целом, сделать так, чтобы ключами возвращаемого в getMatches массива были id, тогда искать ничего и не понадобится.

Модифицируйте вашу функцию getMatches

function getMatches(){
    .....
    return $result;
}

например, как-то так

function getMatches(){
   ...... 
   return array_combine(array_column($result,'id'), $result);
}

и сможете использовать, условно, так:

if(isset($_GET['match'])){
     $matches = getMatches();
     $current = $matches[ $_GET['match'] ] ;
}

Второй вариант - определите функцию getMatchById($id) или нечто подобное, в которой отфильтруете нужный матч и вернете его.

 function getMatchById($id){
     $matches = getMatchs();

     foreach($matches as $m){
          if( $m['id'] == $id) return $m;
     }
     return null;
 } 

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.