Как правильно написать цикл while внутри while в PHP?

Question

[Федор Павлов]

Всем здравствуйте, как будет правильно: Например вывод статей через while из база данных, в котором изображения внутри статьи тоже через while из базы данных?
Проблема в том, что выводит только одну статью, с одной изображением
структура БД:
db_news (id, news,...)
db_news_img (id, news_id, link_img)
Сам код:

$db->Query("SELECT * FROM db_news WHERE cat_id = '$catid' ORDER BY id DESC");//
          if($db->NumRows() > 0){
               while($news = $db->FetchArray()){
                    $textfull = $news["news"];

<div class="col">
            <div class="accordion" id="accordionNews">
              <div class="card mb-4 shadow-sm" id="heading<?=$news["id"];?>">
                <div class="d-flex justify-content-between align-items-center">
                    <div class="new-title"><?=$news["title"]; ?> </div>
                    <small class="text-muted" style="padding-right: 20px;"><?=date("d.m.Y",$news["date_add"]); ?></small>
                </div>
                <div id="carousel<?=$news["id"];?>" class="carousel slide">
                  <ol class="carousel-indicators">
                    <li data-target="#carousel<?=$news["id"];?>" data-slide-to="0" class="active"></li>
                    <li data-target="#carousel<?=$news["id"];?>" data-slide-to="1"></li>
                    <li data-target="#carousel<?=$news["id"];?>" data-slide-to="2"></li>
                  </ol>
                  <div class="carousel-inner">

$db->Query("SELECT * FROM db_news_img WHERE news_id = '$news[id]' ORDER BY id DESC");//
          if($db->NumRows() > 0){
               while($img = $db->FetchArray()){

<div class="carousel-item active">
         <img src="/<?=$img["link_img"]; ?>" class="d-block w-100 cover" alt="...">
</div>

}

</div>
                  <a class="carousel-control-prev" href="#carousel<?=$news["id"];?>" role="button" data-slide="prev">
                    <span class="carousel-control-prev-icon" aria-hidden="true"></span>
                    <span class="sr-only">Previous</span>
                  </a>
                  <a class="carousel-control-next" href="#carousel<?=$news["id"];?>" role="button" data-slide="next">
                    <span class="carousel-control-next-icon" aria-hidden="true"></span>
                    <span class="sr-only">Next</span>
                  </a>
                </div>
                <div class="card-body">
                  <div id="collapse<?=$news["id"];?>" class="collapse" aria-labelledby="heading<?=$news["id"];?>" data-parent="#accordionNews">
                    <?=$textfull?> 
                  </div>
                  <div class="d-flex justify-content-between align-items-center">
                    <button role="button" class="btn btn-sm btn-primary" data-toggle="collapse" data-target="#collapse<?=$news["id"];?>" aria-expanded="false" aria-controls="collapse<?=$news["id"];?>">Подробно</button>
                    <form action="" method="post">
                      <small class="text-muted"><?=$news["new_like"]; ?>
                        <input type="hidden" name="like" value="<?=$news["id"];?>" />
                        <button class="btn btn-link" style="color: red;" type="submit"><i class='fa fa-heart'></i></button>
                      </small>
                    </form>
                  </div>
                </div>
              </div>
            </div>
          </div>

}

Comments

[Araik]

Заменить на foreach ?

[Федор Павлов]

Пробовал так: первый цикл через while, вторую через foreach. Проблема та же, выводит только одну. Вреда цикл обрывается, при повторном запросе в БД. Не могу решить эту проблему

[Araik]

Федор Павлов,

$news = $db->FetchArray();

foreach ($news as $new) {
var_dump($new);
}

Answer 1

[AUser0]

Всё потому, что вы используете одно и то же подключение к базе данных. Получается так:
1. Запрашиваются записи из db_news, база принимает этот запрос как текущий.
2. Читается первая запись текущего запроса, т.е. из db_news.
3. Через это же подключение запрашиваются записи из db_news_img. База запоминает новый запрос как текущий, готова передавать записи из db_news_img.
4. Читается первая запись текущего запроса, т.е. из db_news_img.
5. Возврат к началу цикла, хочется прочитать из db_news. Но этот запрос уже не является текущим, он "потерян", поэтому из него невозможно получить данные.
6. Данных нет - значит конец цикла.

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

$db_res1 = $db->Query("SELECT * FROM db_news WHERE cat_id = '$catid' ORDER BY id DESC");//
if($db_res1->NumRows() > 0){
       while($news = $db_res1->FetchArray()){
               $textfull = $news["news"];
                $db_res2 = $db->Query("SELECT * FROM db_news_img WHERE news_id = '$news[id]' ORDER BY id DESC");//
                if($db_res2->NumRows() > 0){
                        while($img = $db_res2->FetchArray()){

Answer 2

[Одиночка Айс]

Получать данные заранее(!!!), а не в цикле и не посреди вывода страницы!!!

1. Использовать LEFT/INNER JOIN
или
2. Перебрать массив с картинками так, чтобы ключом был id новости, тогда можно проверять наличие картинки для новости - if(isset($news_img[$news["id"]])) и потребуется всего один цикл

Comments

[Федор Павлов]

Спасибо, но можешь более подробный код второго варианта? если не сложно, было бы прекрасно:)

[Одиночка Айс]

Федор Павлов, эм... самый примитивный код:

//$news - получили заранее
//$news_img - получили заранее

//заполняем массив с картинками
$images = [];
foreach($news_img as $img){
  $images[$img['news_id']] = $img;
  unset($img);
}

//проверяем наличие новости в картинках (что бы это ни значило...)
foreach($news as $item){
  if(!empty($images[$item['id']])){
    //выводим картинку новости
  }
  unset($item);
}