Форум MySQL

Написал скрипт котоый выводит номенклатуру товара из БД, его работу можно посмотреть на http://www.el-eko.ru/State_Standard_Samples.php . Просьба к спецам оценить код и указать на недостатки, вот

<?php //соединяемся с MySQL $db_host = "localhost"; $db_user = "login"; $db_password = "password"; $db_name = "my_db"; $connection = mysql_connect ($db_host, $db_user, $db_password) or die ('error connection'); mysql_select_db($db_name, $connection); //переменные @$id =$_GET['id']; @$name = $_GET['name']; @$group_production =$_GET['group_production']; //checkboxes if (isset($id)) {     $srch='id';} //id - имя столбца в таблице     if (isset($name)) {         $srch='name';} //name - имя столбца в таблице         if (isset($group_production)) {             $srch='group_production';} //group_production - имя столбца в таблице //выводим заголовок таблицы echo "<table border='1' cellspacing='1' cellpadding='1' class='3'>"; echo "<tr><td class='3' align='center'><b>Наименование товара</b></td></tr>"; //запрос $r = mysql_query("SELECT * FROM tovary WHERE group_production = 'ГСО(Государственные Стандартные Образцы)' LIMIT 0,50 "); //выводим таблицу for ($i=0; $i<mysql_num_rows($r); $i++) { $class = ( $i % 2 == 0 ) ? "odd" : "even"; echo '<style>              .odd{background:99FFCC}               .even{background:FFFFCC}               </style><tr class="'.$class.'">'; $f = mysql_fetch_array($r); echo "<td>$f[name]</td>"; echo "</tr>"; } echo "</table>" ?>

я бы

<? //checkboxes  if (isset($id)) {      $srch='id';} //id - имя столбца в таблице      if (isset($name)) {          $srch='name';} //name - имя столбца в таблице          if (isset($group_production)) {              $srch='group_production';} //group_production - имя столбца в таблице ?>

сделал бы так

<? //checkboxes  if (!empty($id)) $srch='id'; //id - имя столбца в таблице  if (!empty($name)) $srch='name'; //name - имя столбца в таблице  if (!empty($group_production)) $srch='group_production'; //group_production - имя столбца в таблице ?>

поскольку после этого

<? @$id =$_GET['id'];  @$name = $_GET['name'];  @$group_production =$_GET['group_production']; ?>

переменные уже существуют

Понял только вывод, остальное зачем так и не просек...
и все же вывод бы сделал так.

Вместо.

//выводим таблицу  for ($i=0; $i<mysql_num_rows($r); $i++)  {  $class = ( $i % 2 == 0 ) ? "odd" : "even";  echo '<style>               .odd{background:99FFCC}                .even{background:FFFFCC}                </style><tr class="'.$class.'">';  $f = mysql_fetch_array($r);  echo "<td>$f[name]</td>";  echo "</tr>";  }

Сделал бы..

//выводим таблицу  echo "<style>               .odd{background:99FFCC}                .even{background:FFFFCC}                </style>"; while($rows=mysql_fetch_array($r)){    $i++;   $class = ( $i % 2 == 0 ) ? "odd" : "even";    echo "<tr class=".$class."><td>".$rows[name]."</td></tr>";  }

Цыклить <style> нет смысла...

Эм... сорри за вопрос, ты перень или девушка?

эм...

<?php //соединяемся с MySQL $db_host         = "localhost"; $db_user         = "login"; $db_password     = "password"; $db_name         = "my_db"; mysql_connect ($db_host, $db_user, $db_password) or die ('error connection'); mysql_select_db($db_name, $connection) or die ('error connection'); /* Вот этот блок вообще нафига? //переменные $id                 = $_GET['id']; $name                 = $_GET['name']; $group_production     = $_GET['group_production']; //checkboxes if (isset($id)) {     $srch='id';} //id - имя столбца в таблице     if (isset($name)) {         $srch='name';} //name - имя столбца в таблице         if (isset($group_production)) {             $srch='group_production';} //group_production - имя столбца в таблице*/ $group_production = ($_GET['group_production'])?$_GET['group_production']:'ГСО(Государственные Стандартные Образцы)'; ?>   <style>      .cell_1{background:99FFCC}       .cell_2{background:FFFFCC}   </style> <table border='1' cellspacing='1' cellpadding='1' class='3'>";     <tr>         <td class='3' align='center'>             <b>Наименование товара</b         </td>     </tr>"; <?php // Почему бы не сделать постраничную навигацию? нафига плодить множество скриптов? // Переменная хранит число сообщений выводимых на станице  $num = 50;  // Извлекаем из URL текущую страницу  $page = $_GET['page'];  // Определяем общее число сообщений в базе данных  $result = mysql_query("SELECT COUNT(*) FROM tovary");  $posts = mysql_fetch_row($result);  // Находим общее число страниц  $total = intval(($posts - 1) / $num) + 1;  // Определяем начало сообщений для текущей страницы  $page = intval($page);  // Если значение $page меньше единицы или отрицательно  // переходим на первую страницу  // А если слишком большое, то переходим на последнюю  if(empty($page) or $page < 0) $page = 1;    if($page > $total) $page = $total;  // Вычисляем начиная к какого номера  // следует выводить сообщения  $start = $page * $num - $num;  // Выбираем $num сообщений начиная с номера $start  $result = mysql_query("SELECT * FROM tovary WHERE group_production = `$group_production` LIMIT $start, $num");  // В цикле переносим результаты запроса в массив $postrow  while ($arResult = mysql_fetch_array($result)):     $i++; ?>     <tr class='cell_<?=($i%2)+1;?>'>         <td>             <?=$arResult['name'];?>         </td>     <tr> <?endwhile;?>  </table> <?php  // Проверяем нужны ли стрелки назад  if ($page != 1) $pervpage = '<a href= ./page?page=1><<</a>                                 <a href= ./page?page='. ($page - 1) .'><</a> ';  // Проверяем нужны ли стрелки вперед  if ($page != $total) $nextpage = ' <a href= ./page?page='. ($page + 1) .'>></a>                                     <a href= ./page?page=' .$total. '>>></a>';  // Находим две ближайшие станицы с обоих краев, если они есть  if($page - 2 > 0) $page2left = ' <a href= ./page?page='. ($page - 2) .'>'. ($page - 2) .'</a> | ';  if($page - 1 > 0) $page1left = '<a href= ./page?page='. ($page - 1) .'>'. ($page - 1) .'</a> | ';  if($page + 2 <= $total) $page2right = ' | <a href= ./page?page='. ($page + 2) .'>'. ($page + 2) .'</a>';  if($page + 1 <= $total) $page1right = ' | <a href= ./page?page='. ($page + 1) .'>'. ($page + 1) .'</a>'; // Вывод меню  echo $pervpage.$page2left.$page1left.'<b>'.$page.'</b>'.$page1right.$page2right.$nextpage;  ?>

вобщем где-то так
Постраничная навигация

Поставил твой вариант скрипта, при открытии страницы пишет: Fatal error: Unsupported operand types in /home/elekoru3/public_html/State_Standard_Samples.php on line 199
у меня вот эта строка:

$total = intval(($posts - 1) / $num) + 1;

Может я не прав, но помойму

>$total = int(($posts - 1) * $num + 1);

хотя int и intval синонимы, но предпочитаю первый вариант

Да по сути ничего не изменилось, просто в сообщении об ошибке изменился №строки:Fatal error: Unsupported operand types in /home/elekoru3/public_html/State_Standard_Samples.php on line 198