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
Форумы портала PHP.SU :: Версия для печати :: Смущает повтарение в коде..
Форумы портала PHP.SU » » Вопросы новичков » Смущает повтарение в коде..

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

1. etoYA - 10 Ноября, 2011 - 05:05:42 - перейти к сообщению
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 Это не готовый код, а только набросок)
2. sKaa - 10 Ноября, 2011 - 05:21:19 - перейти к сообщению
Как-то так, но я не уверен в правильности.. потести. у тебя выводится какой запрос сформировался 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.  
3. DeepVarvar - 10 Ноября, 2011 - 06:23:54 - перейти к сообщению
Еще бы вынести обработку запроса и цикл результата запроса в отдельную ф-цию.
И пусть возвращает либо результат в виде массива, либо false;
Затем вызывать ее одной строчкой в любом участке кода ниже.
(Добавление)
sKaa пусть etoYA сам напишет. Ну или сам скажет что ничо не понял, далее растолкуем. Закатив глазки
4. etoYA - 10 Ноября, 2011 - 06:44:59 - перейти к сообщению
Спасиб, буду пробовать).
5. sKaa - 10 Ноября, 2011 - 07:05:05 - перейти к сообщению
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 номер страницы

 

Powered by ExBB FM 1.0 RC1