Копируется 2 раза. Как исправить?

Question

[parger]

Возникла такая ситуация, использую foreach код отображается 2 раза

<?php
R::selectDatabase('default');
$server = R::getAll( 'SELECT * FROM user WHERE server = 1' );
foreach ($server as $servers):
?>

<div class="box">

<?php
R::selectDatabase('db1');
$server = 'Phoenix';
$admin = R::count( 'accounts' );
$admin = R::count( 'accounts', "online >= 0 AND admin >= 1" );
$online = R::count( 'accounts' );
$online = R::count( 'accounts', 'online >= 0');
?>

<div class="boxtitle"><?php echo $server?> Online: <?=$online?> | Админов: <?=$admin?></div>
<ol>

<?php
R::selectDatabase('db1');
$users = R::getAll( 'SELECT * FROM accounts WHERE `online` >=0 AND `admin` >= 1' );
foreach ($users as $user)
{
$id = $user['id'];
$user = R::load('accounts', $id);
echo('<a href="nickname.php?iddb='.$user['id'].'">');
echo('<li type="1">[LVL '.$user['admin'].'] '.$user['Name'].'(ONLINE: '.$user['online'].')</li>');
echo('</a>');
}
?>

</ol>
<ol>

<?php
R::selectDatabase('default');
$users =  R::getAll( 'SELECT * FROM user WHERE cont = 1 AND server = 1' );						
foreach ($users as $user)
{
$id = $user['id'];
$user = R::load('user', $id);
if(''.$user['server1'].'' == '0' ) $server = ''; 
else $server = 'Phoenix';
echo('<a href="user.php?id='.$user[id].'">');
echo('<li type="1">'.$user[username].' ('.$server.') '.$user[lastdate].'</li>');
echo('</a>');
}
?>

</ol>
</div>

<?php endforeach; ?>

Comments

[Модератор]

parger забудьте о теге pdo, пока не поймёте для чего он.

[Daria Motorina]

Отформатируйте код, это просто нечитаемо. Из вашего вопроса непонятно, что именно печатается два раза. Как минимум два раза видно
foreach ($users as $user)
и не видно закрытия одного из них

[parger]

Daria Motorina, обновил, php в php, html в html. Копируется весь код
от
foreach ($server as $servers):
до
<?php endforeach; ?>

[Araik]

Вероятно массив $server содержит 2 элемента и потому код внутри foreach отрабатывает 2 раза?
https://www.php.net/manual/ru/control-structures.f...

p.s. с точки зрения именования переменных, наверное правильней все же было бы foreach ( $servers as $server)

[parger]

NinjaNickName, то что два элемента вполне возможно, но как бы пофиксить

[Araik]

parger, уберите foreach, вроде он тут вообще ни к чему.

p.s. var_dump($server), чтобы узнать кол-во элементов и как к ним обращаться.

[parger]

NinjaNickName, Да, спасибо огромное, нужно было добавить проверку по ID, по коду он выводил всех пользователей из бд, тем самым и происходило отображение 2 раза.

[Araik]

parger, Не за что, но попробуйте все же удалить foreach совсем, судя по коду он там вообще ни к чему, что есть, что нет, разницы никакой не должно быть.

[parger]

NinjaNickName, foreach я использую в качестве вывода кода, когда значение 0 весь код пропадает за счет использования foreach
Взято с сайта примера запросов

//$books = R::getAll('SELECT `title` FROM `book`');
$id = 1;
$books = R::getAll('SELECT `title` FROM `book` WHERE `id` > ?', [$id]);
 
foreach ($books as $book){
  echo $book['title'].'<br>';
}

[FanatPHP]

NinjaNickName, он из всей библиотеки знает только одно слово getAll. Которое возвращает массив. Поэтому убрать форич не может.

Поэтому надо или садиться и с ложечки обьяснять азы программирования, или махнуть рукой и оставить как есть :)

[Araik]

parger,

foreach я использую в качестве вывода кода, когда значение 0 весь код пропадает за счет использования foreach

Правильней будет использовать для этого if :

if (!empty($server) {
   // если $server не пустой, то выполнить код в этой секции
} else {
  // а если пустой, то в этой
}

Или без фигурных скобок:

<?php if (!emtpy($server)): ?>

тут HTML который будет выводиться, когда $server не пустой

<?php endif ?>

https://www.php.net/manual/ru/control-structures.if.php
https://www.php.net/manual/ru/function.empty.php

FanatPHP, Будет стараться, мало-помалу научиться:)

Answer 1

[FanatPHP]

Очередная жертва видеокурсов.

Всем задающим вопрос про тухлую фасоль надо насильно отключать Ютуб, давать книгу Котерова, и не включать пока не прочитают.

Потому что это садизм - показывать несчастным детям два приема использования устаревшей в прошлом веке библиотеки и говорить что они освоили программирование и дальше могут сами