Почему если цикл в цикле, то идёт дублирование?

Question

[vladislav997]

Подскажите, почему если цикл в цикле, то идёт дублирование постов?

Пример:

foreach($users as $user)
{
    foreach($posts as $post)
    {
        if($post->author_id == $user->id)
        {
                $var .= $post->name;
        }
    }
    print_r($var);
}

Результат:
пост1
пост2
пост1
пост2
пост1
пост2

Comments

[Vitsliputsli]

Например, потому что вы для каждого пользователя выводите переменную $var, но нигде ее не обнуляете. Соответственно для каждого последующего будут выводиться значения и для предыдущих.

[sl0]

Потому что переменная не обнуляется. Судя по всему у вас 2 поста и 3 юзера.

[vladislav997]

Vitsliputsli sl0
так в том то и проблема, как ее обнулить?

[sl0]

vladislav997, Так а чего вы хотите? Вывести все посты для юзера или что?

[vladislav997]

sl0, чтобы в print_r($var); не дублировались посты

[sl0]

vladislav997, Ну так запустите цикл по постам отдельно, а не внутри юзерского цикла.

[vladislav997]

sl0, прикaлист? там нужно просто break поставить после print_r, причём тут цикл отдельно

[sl0]

vladislav997, Если там поставить break, то выведутся посты для первого юзера. Для этого цикл по юзерам вообще не нужен.

[vladislav997]

sl0, цикл с юзерами нужен для условий

Answer 1

[Gennady S]

vladislav997, чтобы в print_r($var); не дублировались посты нужно воспользоваться инъекцией зависимостей и обязательно подключить пару серьезных библиотек, например, AR/ORM.

Если серьезно, путей решения того, что хотите сделать, очень много. Проще всего — группировать, что в каком-то смысле и делают ORM, где сразу доступно $user->posts. Группировать можно и на уровне запроса к базе, и на уровне кода. Например.

$vars = [];
foreach($posts as $post)
  $vars[$post->author_id][] = $post->name;
print_r($vars); // теперь у Вас записи группированы по пользователю, делайте что хочется,
// например:
foreach ($users as $user)
  if (isset($vars[$user->id]))
    print_r(implode(', ', $vars[$user->id]));

— вообще, есть правило, старайтесь избегать двойного обхода массивов. А в данном случае Вы дважды обходили $posts внутри перебора $users.