Remove duplicates from foreach - php

Question

I've got this foreach script and if there is no result found it does show this message "No articles yet" twice, how can i fix this? I've tried to move it out of the foreach but with no success.
Many thanks
$query_show_groupmsg = mysqli_prepare($conn, "SELECT group_message_id, group_message_date, group_message_subject, group_message_body FROM group_messages WHERE group_message_refid = ? ORDER BY group_message_date DESC LIMIT 5");
mysqli_stmt_bind_param($query_show_groupmsg, 'i', $data);
foreach ($id_gruppo as $data) {
mysqli_stmt_execute($query_show_groupmsg);
mysqli_stmt_bind_result($query_show_groupmsg, $group_message_id, $group_message_date, $group_message_subject, $group_message_body);
mysqli_stmt_store_result($query_show_groupmsg);
$groupmsg = mysqli_stmt_num_rows($query_show_groupmsg);
if ($groupmsg > 0) {
while (mysqli_stmt_fetch($query_show_groupmsg)) {
$new_mgsgroup_date = date('d/m/Y, \a\l\l\e h:i a', strtotime($group_message_date));
?>
<div class="item">
<div class="item-head">
<div class="item-details">
<?php
echo ' <a class="item-name primary-link" href="client_blog_app/client_group_post_view.php?id='.$group_message_id.'"> '.ucfirst(strip_tags(mb_strimwidth($group_message_subject, 0, 350, "..."))).'</a>
';
?>
<br><span class="item-label font-blue small">Pubblicato il <?php echo "$new_mgsgroup_date"; ?></span>
</div>
</div>
<div class="item-body"> <?php echo strip_tags(mb_strimwidth($group_message_body, 0, 350,"...")); ?> </div>
</div>
<?php
}
}else{
echo '
<div class="alert alert-info">No articles yet</div>
';
}
}

Answer 1

Add this outside of the for loop and remove the else clause you already have.
<?
if ($groupmsg < 1)
{
?>
<div class="alert alert-info">No articles yet</div>
<?
}

Answer 2

Put the IF ELSE statement outside the FOREACH like this:
$query_show_groupmsg = mysqli_prepare($conn, "
SELECT group_message_id, group_message_date, group_message_subject, group_message_body FROM group_messages
WHERE group_message_refid = ?
ORDER BY group_message_date
DESC LIMIT 5
");
$article_count = 0;
mysqli_stmt_bind_param($query_show_groupmsg, 'i', $data);
foreach ($id_gruppo as $data) {
mysqli_stmt_execute($query_show_groupmsg);
mysqli_stmt_bind_result($query_show_groupmsg, $group_message_id, $group_message_date, $group_message_subject, $group_message_body);
mysqli_stmt_store_result($query_show_groupmsg);
$groupmsg = mysqli_stmt_num_rows($query_show_groupmsg);
if ($groupmsg > 0) {
while (mysqli_stmt_fetch($query_show_groupmsg)) {
$new_mgsgroup_date = date('d/m/Y, \a\l\l\e h:i a', strtotime($group_message_date));
$article_count++;
?>
<div class="item">
<div class="item-head">
<div class="item-details">
<?php echo ' <a class="item-name primary-link" href="client_blog_app/client_group_post_view.php?id='.$group_message_id.'"> '.ucfirst(strip_tags(mb_strimwidth($group_message_subject, 0, 350, "..."))).'</a>'; ?>
<br>
<span class="item-label font-blue small">Pubblicato il <?php echo "$new_mgsgroup_date"; ?></span>
</div>
</div>
<div class="item-body"> <?php echo strip_tags(mb_strimwidth($group_message_body, 0, 350,"...")); ?> </div>
</div>
<?php
}
}
}
if ($article_count == 0){
echo '<div class="alert alert-info">No articles yet</div>';
}

Answer 3

Since your if ($groupmsg > 0) condition is inside the foreach cycle, the else branch can be executed up to count($data) times. Moving the condition out of the foreach loop should fix this. You can keep count of articles inside foreach and after the foreach check if there were any.
However this code is really terrible to read (no comments, random indentation). You should make a minimal working example, remove the useless parts and comment your code, if needed.