Загрузка данных из бд в select

жесть. а зачем делать Select *, если вы используете только $product[name]?. Думаю ограничение выборки уже снизит нагрузку в несколько раз.

можно выбрать все требуемые данные джоином в одну таблицу, и уже во всех ваших селектах проходить по этой таблице, но это уже имхо изврат. хотя можете попробовать.

 

[sql]SELECT name FROM $table ORDER BY name[/sql]

а что, из одной таблицы 2 раза выбирается одно и тоже? Или где-то ещё пропущен кусок где $table принимает другое значение?

 

creage
смените аватар, по правилам разрешены только личные фото

 

Это правильнее, чем делать несколько запросов.

 

PHP:

1. <?php
2. $sql = mysql_query('SELECT `name` FROM `'.$table.'` ORDER BY `name`' ,$db);
3. $select1 = '<select name="s1" >';
4. $select2 = '<select name="s2" >';
5. $select = '';
6. while($product = mysql_fetch_array($sql))
7.     $select .= '<option value='.$product['name'].'>'.$product['name'].'</option>';
8. $select1 .= $select.'</select>';
9. $select2 .= $select.'</select>';
10. // первый селект
11. echo $select1;
12. // ну и где-то второй
13. echo $select2;
14. ?>

ну или вместо переменных можно в массив загонять

 

Нисколько не критикую метод Luge, хотя я бы предложил что-то типа такого:

PHP:

1. <?php
2.  
3. function select_get_data($table)
4. {
5.     static $prod = array();
6.     if ( !isset($prod[$table]) )
7.     {
8.          $sql = mysql_query('SELECT `name` FROM `'.$table.'` ORDER BY `name`';
9.          while($product = mysql_fetch_array($sql))
10.          {
11.             $prod[$table][] = $product;
12.          }
13.     }
14.     return $prod[$table];
15. }
16. function select_draw($table, $name)
17. {
18.     $data = select_get_data($table);
19.     echo "<select name='$name'>";
20.     foreach ($data as $product)
21.     {
22.         echo "<option value='". $product['name'] ."'>" . $product['name'] . "</option>";
23.     }
24.     echo "</select>";
25. }
26.  
27.  
28. // select_draw('user_table', 'user_form_name1');
29. // select_draw('user_table', 'user_form_name2');
30. ?>

 

в смысле

PHP:

1. <?php
2. $selected = 'какой-то иам продукт';
3. while($product = mysql_fetch_array($sql))
4.     $select .= '<option value='.$product['name'].' '.(($product['name']==$selected)?'selected':'').'>'.$product['name'].'</option>';
5. ?>

?

 

Я всегда загоняю необходимые СЕЛЕКТу данные в массив, а затем испольлзую такой класс :

PHP:

1. <?php
2. //Класс работы с select
3. class select
4. {
5. protected $name;       // имя select-a (списка или комбобокса)
6. protected $options;    // асоциативный массив "значение=>текст"
7. protected $value;       // значение select-a (списка или комбобокса), может быть массивом
8. protected $attributes; // доп. атрибуты тега
9. protected $disabled;  //Метка в конце имени опции, указывающая на её недоступность
10. protected $show_all; //Если true -- ингорирует метки недоступности и вычистит их
11. protected $ignore;    //Метка в конце имени опции, указывающая, что эту опцию показывать не надо
12.  
13. function __construct($name,$options,$value=NULL,$attributes="",$disabled="%%%",$show_all=false,$ignore="@@@")
14.     {
15.     $this->name=$name;
16.     $this->options=$options;
17.     $this->value=$value;
18.     if (!is_array($this->value))
19.         settype($this->value,"string");
20.     $this->attributes=$attributes;
21.     $this->disabled=$disabled;
22.     $this->show_all=$show_all;
23.     $this->ignore=$ignore;
24.     }
25.  
26. //---------------------------------
27.  
28. function html()
29.     {
30.     $select="\n<select name=\"".$this->name."\" ".$this->attributes.">";
31.  
32.     $options="";
33.  
34.     $len_ignore=strlen($this->ignore);
35.     $len_disabled=strlen($this->disabled); 
36.    
37.     foreach ($this->options as $value=>$text)
38.         {
39.  
40.         if (substr($text,strlen($text)-$len_ignore,$len_ignore)==$this->ignore)
41.             {
42.             continue;
43.             }
44.  
45.         $options.="\n<option value=\"$value\"";
46.  
47.         if (is_array($this->value)) //Для списка с Multiple
48.             {
49.             if (array_key_exists ( $value, $this->value ))
50.                 $options.=" selected=\"selected\"";
51.             }
52.         else //Т.е. значение -- "скалярная" велоичина (не массив)
53.             {
54.             settype($value,"string");
55.             if ($value==$this->value)
56.                 {
57.                 $options.=" selected=\"selected\"";
58.                 }
59.             }
60.  
61.         if (substr($text,strlen($text)-$len_disabled,$len_disabled)==$this->disabled)
62.                 {
63.                 if ($this->show_all==false)
64.                     {
65.                     $options.=" disabled=\"disabled\"";
66.                     }
67.                 $text1=substr($text,0,strlen($text)-$len_disabled);
68.                 }
69.         else
70.                 $text1=$text;
71.  
72.         $options.=">$text1</option>";
73.         }
74.    
75.     $select.=$options."\n</select>\n";
76.     return $select;
77.     }
78.  
79. //---------------------------------
80.  
81.  
82.  
83. }//end class
84. ?>