Форум PHP программистов > Посмотрите код. Как упростить?

[ Поиск ] - [ Пользователи ] - [ Календарь ]

Полная Версия: Посмотрите код. Как упростить?

asdf27

23.07.2012 - 04:22

В этом фрагменте я вывожу все сообщения темы, а в углу каждого вывожу ссылку на удаление сообщения для админа.

Мне нужно проверить роль пользователя, и если у него нужные права, ссылка на удаление появляется. Проблема в том, что если я выдергиваю id до цикла, проставляется id темы, а не сообщений. Я это понимаю.

Поэтому зашил проверку прав в цикл. В принципе, на странице будет не более 20 сообщений, но все же, это же лишняя нагрузка? Как вытащить id сообщения, не включив проверку в цикл?


do {
    if ($_SESSION[role] = 3) {
    $del = '<a href="messages.php?forum=' . $_GET[forum] . '&topic=' . $_GET[topic] . '&delete=' . $myrow[id] . '">x</a>';
}
    echo '<tr><td  rowspan=2 valign="top" align="center" id="autor"><img src="123.png"><br>' . $myrow[autor] . '</td>
            <td id="messdate"> <a href="messages.php?forum=' . $_GET[forum] . '&topic=' . $_GET[topic] . '#' . $myrow[id] . '">Сообщение #' . $myrow[id] . '</a><a name=' . $myrow[id] . '>   ' . $myrow[timet] . '
     '.$del.'</td></tr>
            <tr><td valign=top id="message">' . $myrow[text] . '</td></tr>';
} while ($myrow = mysql_fetch_array($result));

Спустя 2 часа, 14 минут, 56 секунд (23.07.2012 - 05:37) alex12060 написал(а):

Скажу одно - забудь Попова как страшный сон.
А во вторых, твой код супер огромной нагрузки не дает.

Спустя 7 минут, 34 секунды (23.07.2012 - 05:44) NierRa написал(а):

Попова выдает do while

Спустя 1 минута, 58 секунд (23.07.2012 - 05:46) alex12060 написал(а):

NierRa

Не только

Спустя 11 минут, 4 секунды (23.07.2012 - 05:57) NierRa написал(а):

Ну ок проявлю себя знатоком ошибок господина попова.

В этом куске кода его выдает
1. do while
2. mysql_fetch_array
3. Длиннючие цепочки кода
4. Точно не помню, но кажется вывод html в php коде

Спустя 3 часа, 53 минуты, 28 секунд (23.07.2012 - 09:51) nikolasline написал(а):

Сильной нагрузки не будет но код я написал бы так:


while ($myrow = mysql_fetch_array($result)){
    if ($_SESSION[role] = 3) {
      $del = '<a href="messages.php?forum=' . $_GET[forum] . '&topic=' . $_GET[topic] . '&delete=' . $myrow[id] . '">x</a>';
   }
    echo '<tr><td  rowspan=2 valign="top" align="center" id="autor"><img src="123.png"><br>' . $myrow[autor] . '</td>      <td id="messdate"> <a href="messages.php?forum=' . $_GET[forum] . '&topic=' . $_GET[topic] . '#' . $myrow[id] . '">Сообщение #' . $myrow[id] . '</a> <a name=' . $myrow[id] . '>   ' . $myrow[timet] . '
     '.$del.'</td></tr> <tr><td valign=top id="message">' . $myrow[text] . '</td></tr>';
}

Спустя 9 часов, 23 минуты, 51 секунда (23.07.2012 - 19:14) asdf27 написал(а):

Всем спасибо. Попова не читал, видимо его последователей. Будем переучиваться

Быстрый ответ:

Здесь расположена полная версия этой страницы.