Как исправить и сделать выборку?

Question

[Hello World]

Привет всем.
Допустим, я сейчас сделал выборку по "AMD" и в адресной строке появился адрес prntscr.com/71fii7 family1 не должен был появиться.
Сделал выбору по intel core i7, то prntscr.com/71fjk4 в адресной строке maker=all. А хотя должно было быть просто .php?family1=Intel+Core+i7
Как сделать, чтобы при выборке в адресной строке появлялся только get адрес выбранного значения? И как избавиться от лишнего get адреса при выборке?
Если я выберу сортировку по семейству, то выборка по производителю не должна сработать.
Вот код:

$_GET['maker']=($_GET['maker']);
  if(isset($_GET['maker']) && $_GET['maker']!="ALL" && $_GET['maker']!=""){
	$query_get = ("SELECT * FROM `Processors` Where Manufacture='".$_GET['maker']."'");
   }
  else  {
  $query_get = ("SELECT * FROM `Processors` ");
   }
    $result_get        = mysql_query($query_get) or die(mysql_error()); 
    while($row_get     = mysql_fetch_assoc($result_get)){  $arr_get[] = $row_get; }

 $_GET['family1'] = $_GET['family1'];
 if(isset($_GET['family1']) && $_GET['family1']!="ALL" && $_GET['family1']!="") {
   $query_get1  = ("SELECT * FROM `Processors` Where Family_processor ='".$_GET['family1']."'");
  }
  else {
    $query_get1  = ("SELECT * FROM `Processors` ");
  }
   $result_get1        = mysql_query($query_get1) or die(mysql_error()); 
    while($row_get1     = mysql_fetch_assoc($result_get1)){  $arr_get[] = $row_get1; }

Answer 1

[Алексей Яхненко]

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

Comments

[Hello World]

ccЫлка jsfiddle.net/557zxc7r тут не показывает весь код

[Алексей Яхненко]

код чудовищный по нескольким причинам, а код формы похоже парсер выпилил.
попробуй <code></code>

[Hello World]

echo "<form action='index.php' method='get'>";
 echo "<div class='newselect'>";
  echo "<select class='chosen' name='maker' onChange='submit()'>";
    echo '<option value="ALL">-- Производитель --</option>';
	echo '<option value="ALL">Любой</option>';
          while ($line = mysql_fetch_array($manufact)){
    echo '<option value="'.$line['Manufacture'].'">'.$line['Manufacture'].'</option>'; }
   echo "</select>";
  echo "</div>";       
        
 echo "<div class='newselect'>";
   echo "<select name='family1' onChange='submit()'>";
     echo '<option value="">-- Семейство процессора --</option>';
     echo '<option value="ALL">Любой</option>';
       while ($line = mysql_fetch_array($family)){
     echo '<option value="'.$line['Family_processor'].'">'.$line['Family_processor'].'</option>';}
      echo "</select>";
   echo "</div>";
echo "</form>";

[Алексей Яхненко]

ну собственно да, форма отправляет все значения что в ней есть, а что ты хотел.

[Алексей Яхненко]

только если с помощью жаваскрипта при сабмите формы выпиливать из нее все пустые значения, но смысл?

[Hello World]

Алексей Яхненко: а как это исправить? Окутывать каждый селект в форму? Или что

[Алексей Яхненко]

для начала стоит определиться, а надо ли оно вообще.
работает ведь?
если смущают гет-параметры в адресе - смени метод формы на пост, и всего делов.

[Hello World]

Алексей Яхненко: поставил сейчас закрытия формы после первого селекта. Также вызывает в адресную строку AMD&family1=&General=&cores=&type_socket=&tyre=

[Алексей Яхненко]

я искренне надеюсь что тебе не платят за это деньги :)

Также вызывает в адресную строку AMD&family1=&General=&cores=&type_socket=&tyre=
нет таких полей в форме же, хто-та урот, как говорит один товарищ.

[Hello World]

Алексей Яхненко: не всё работало. Потому что, когда я выборку делал по семейству, то ничго не происходило. Сейчас я изменил на такое и заработало:

$_GET['maker'] = $_GET['maker'];

    if(isset($_GET['maker']) && $_GET['maker']!="ALL" && $_GET['maker']!="")
    {
      $query_get  = ("SELECT * FROM `Processors` Where Manufacture='".$_GET['maker']."'");
    }
    else
     if(isset($_GET['family1']) && $_GET['family1']!="ALL" && $_GET['family1']!="")
    {
      $query_get  = ("SELECT * FROM `Processors` Where Family_processor ='".$_GET['family1']."'");
    }
    else
    {
      $query_get  = ("SELECT * FROM `Processors` ");
    }
      $result_get        = mysql_query($query_get) or die(mysql_error()); 
      while($row_get     = mysql_fetch_assoc($result_get)){  $arr_get[] = $row_get; }

Остался вопрос, как сделать взаимодействие при выборке? Допустим, я выбрал выборку по Intel. ОСталось 10 результатов, а хочу ещё семейство Intel i5. И должно из 10 мне выбрать 3 к примеру

[Алексей Яхненко]

давай ты сначала почитаешь про PDO, и забудешь как страшный сон такие вот подходы :)

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

[Hello World]

Алексей Яхненко: собираюсь, собираюсь, но не могу дойти. Мне надо сейчас, чтобы работало как говорится, а потом переходить на pdo, mysqli и т.д. Я ж пришёл для получения ответа, а не выслушивания про то, что мой год дерьмо откровенное. Я это и без тебя знаю))

[Алексей Яхненко]

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

[Hello World]

Алексей Яхненко: я хочу изучать с нуля pdo. Чисто с нуля. Вон курс Специалиста по php 2015 есть. Есть чето про pdo, но я не дошёл туда. По примерам потом буду.

[Hello World]

Алексей Яхненко: можно сделать мой запрос через массив(чтобы автоматом подстраивался под выборку), что значительно сократит код, но я не знаю как это сделать.

[Алексей Яхненко]

ну, я вроде тебе объяснил что тут не так, добавить нечего.

[Алексей Яхненко]

что значит не знаю как это сделать?
что именно не знаешь?

[Hello World]

Алексей Яхненко: создать массив типа

$accepted = array( 
        'maker',
        'family1',
    );
  $type = isset($_GET['type'])?$_GET['type'] : null;
   $column = in_array( $type, $accepted ) ? $type : $accepted[0];

и как-то вставлять в запрос, чтобы при клике на соотвествующий select происходила выборка из столбца и т.д

[Hello World]

может и не правильно написал.. Подскажи что ли

[Алексей Яхненко]

$select = ' select * from processors ';

$where = array();

if (!empty($_GET['type'])) {
  $where[] = "type = '{$_GET['type']}'";
}

... и таким макаром все фильтры

if (!empty($where)) {
  $select .= ' where ';
  $select .= implode(' and ', $where);
}

... получаем селект со всеми использованными фильтрами.

[Алексей Яхненко]

это в общих чертах, этот код опасен, но суть ты понял.

[Hello World]

Алексей Яхненко: попробую, но как сделать, чтобы при выборке из 1 селекта можно было сделать выборку из 2-го селекта и результат был сохранен? Типа как я говорил с производителем и семейством. Через Group BY?
А про опасность - мне всё равно сейчас) сайт для локального использования.

[Алексей Яхненко]

один из нас чего-то не понимает.

у тебя один фильтр будет manufacturer=intel, второй будет family=i5

вот и выберешь из базы одним запросом процессоры с такими параметрами.

[Hello World]

Алексей Яхненко: ясно, спасибо. Буду пробовать сейчас