Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Смущает повтарение в коде..

 PHP.SU

 Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (1): [1]   

> Без описания
Поиск в теме | Версия для печати
etoYA
Отправлено: 10 Ноября, 2011 - 05:05:42
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)
PHP:
скопировать код в буфер обмена
  1. <?
  2. require_once('config.php');
  3.  
  4. // Переменная айди новости
  5. $id = $_GET['id'];
  6.  
  7. // Если запрос по айди существует, то показать полную новость
  8. if (isset($id)) {
  9.         $result = mysql_query("SELECT * FROM `news` WHERE id='{$id}' LIMIT 1") or die (mysql_error());
  10.         $row = mysql_fetch_array($result);
  11.         echo '<a href="index.php?id='.$row['id'].'">'.$row['title'].'</a>';
  12.         if ($_SESSION['group'] == 'admin') {
  13.                 echo ' Edit    Delete<br />';
  14.         }
  15.         echo $row['full_news'];
  16.         echo '<hr />Добавил: ' . $row['author'] . ' | Дата добавления: ' . $row['add_date'] . ' | Просмотров: ' . $row['reads'] . ' | Комментариев: ' . $row['comments_num'] . '<hr />';
  17. }
  18. else {
  19.  
  20. // Выборка 5ти коротких новостей
  21. $result = mysql_query("SELECT * FROM `news` ORDER BY `id` DESC LIMIT 5") or die (mysql_error());
  22.  
  23. // Иначе вывести 5 коротких новостей
  24. while ($row = mysql_fetch_array($result)) {
  25.         echo '<a href="index.php?id='.$row['id'].'">'.$row['title'].'</a>';
  26.         if ($_SESSION['group'] == 'admin') {
  27.                 echo ' Edit    Delete<br />';
  28.         }
  29.         echo $row['cut_news'] . '<br />';
  30.         echo '<hr />Добавил: ' . $row['author'] . ' | Дата добавления: ' . $row['add_date'] . ' | Просмотров: ' . $row['reads'] . ' | Комментариев: ' . $row['comments_num'] . '<hr />';
  31. }
  32. }
  33. ?>
  34.  


Что-то я тут намудрил, 2 раза повтаряется 1 участок... А именно. Как неповтарять этот участок кода, или это нормально?

PHP:
скопировать код в буфер обмена
  1.         echo '<a href="index.php?id='.$row['id'].'">'.$row['title'].'</a>';
  2.         if ($_SESSION['group'] == 'admin') {
  3.                 echo ' Edit    Delete<br />';
  4.         }
  5.         echo $row['cut_news'] . '<br />';
  6.         echo '<hr />Добавил: ' . $row['author'] . ' | Дата добавления: ' . $row['add_date'] . ' | Просмотров: ' . $row['reads'] . ' | Комментариев: ' . $row['comments_num'] . '<hr />';


P.S Это не готовый код, а только набросок)

(Отредактировано автором: 10 Ноября, 2011 - 05:07:01)

 
 Top
sKaa
Отправлено: 10 Ноября, 2011 - 05:21:19
Post Id



Частый посетитель


Покинул форум
Сообщений всего: 979
Дата рег-ции: Окт. 2011  
Откуда: Россия г. Нижний Новгород


Помог: 25 раз(а)

[+]
Как-то так, но я не уверен в правильности.. потести. у тебя выводится какой запрос сформировался echo 'query : '.$query.'<br>';
PHP:
скопировать код в буфер обмена
  1.  
  2. require_once('config.php');
  3. // Переменная айди новости
  4. $id = 0;
  5. $limit = 5;
  6. $query = "SELECT * FROM `news`";
  7. if(isset($_GET['id']))
  8.         $id = mysql_real_escape_string($_GET['id']);
  9.        
  10. if($id != 0)
  11.         $query .= " WHERE id='{$id}'";
  12. else $query .= "ORDER BY `id` LIMIT 0, $limit";
  13. // Debug
  14. echo 'query : '.$query.'<br>';
  15.  
  16. $result = mysql_query($query) or die (mysql_error());
  17. if(!mysql_num_rows($result))
  18.         echo 'Новость не найдена.';  
  19.         else
  20.         for($k=0;$k<mysql_num_rows($result);$k++){
  21.                 $row = mysql_fetch_array($result);
  22.                 echo '<a href="index.php?id='.$row['id'].'">'.$row['title'].'</a>';
  23.                 if ($_SESSION['group'] == 'admin')
  24.                         echo ' Edit    Delete<br />';
  25.                
  26.                 echo $row['full_news'];
  27.                 echo '<hr />Добавил: ' . $row['author'] . ' | Дата добавления: ' . $row['add_date'] . ' | Просмотров: ' . $row['reads'] . ' | Комментариев: ' . $row['comments_num'] . '<hr />';
  28.         }
  29.  

(Отредактировано автором: 10 Ноября, 2011 - 05:30:14)

 
 Top
DeepVarvar Супермодератор
Отправлено: 10 Ноября, 2011 - 06:23:54
Post Id



Активный участник


Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008  
Откуда: Альфа Центавра


Помог: 353 раз(а)
Еще бы вынести обработку запроса и цикл результата запроса в отдельную ф-цию.
И пусть возвращает либо результат в виде массива, либо false;
Затем вызывать ее одной строчкой в любом участке кода ниже.
(Добавление)
sKaa пусть etoYA сам напишет. Ну или сам скажет что ничо не понял, далее растолкуем. Закатив глазки
 
 Top
etoYA
Отправлено: 10 Ноября, 2011 - 06:44:59
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)
Спасиб, буду пробовать).
 
 Top
sKaa
Отправлено: 10 Ноября, 2011 - 07:05:05
Post Id



Частый посетитель


Покинул форум
Сообщений всего: 979
Дата рег-ции: Окт. 2011  
Откуда: Россия г. Нижний Новгород


Помог: 25 раз(а)

[+]
DeepVarvar, да пусть пишет мне чё, жалко чтоль)) я наоборот усложню задачу ему.. ему бы сразу сюда ещё $page = 0; if(isset($_GET['page'])) $page = $_GET['page'] приделать,
дальше
PHP:
скопировать код в буфер обмена
  1. $news_per_page = 5; // Кол-во новостей на странице.
  2. $total_page_count = mysql_query("SELECT COUNT(*) FROM `news`"); // Получаем общее кол-во новослей из базы
  3. $total_page_count = ceil($total_page_count[0] / 5); // Общее кол-во страниц по $news_per_page на каждой
  4.  
  5. $start_from = $page * $news_per_page; // Откуда начнем читать из базы
  6.  
  7. for($k=0;$k<$total_page_count;$k++){
  8. ?><a href=index.php?page=<? echo $k; ?>><? echo $k; ?> </a> <?
  9. }
  10.  
  11. ...
  12. ну а дальше уже  как-то так
  13. $query .= "ORDER BY `id` LIMIT $start_from , $news_per_page";

и вуаля! Постраничный вывод...
Я опять в коде не уверен я его на коленке можно сказать карандашем написал писал )))
ну а суть в том что всё это уже в функцию выносить.. которая в качестве аргумента будет принимать id новости и page=n номер страницы

(Отредактировано автором: 10 Ноября, 2011 - 07:08:29)

 
 Top
Поиск в теме | Версия для печати
Страниц (1): [1]
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Вопросы новичков »


Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.
  



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB