Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Сохранение выбора в select (https://javascript.ru/forum/misc/58840-sokhranenie-vybora-v-select.html)

steepfox 13.10.2015 19:53

Выводится ошибка:

(!) Notice: undefined index:_GET

laimas 13.10.2015 20:01

$_GET - это суперглобальный массив, и даже при первом открытии этой страницы, когда запроса на сортировку не было, условие $_GET['keyname'] никогда не вернет ошибки ибо $_GET['keyname'] в этом случае вернет что и положено - null, если выключать вывод предупреждений на экран, что на удаленном сервере обязательно нужно делать, в том числе и не выводить ошибки, а писать их в лог. файл.

Notice: - это не ошибка, а предупреждение. Делайте дополнительную проверку isset($_GET['keyname']), если не уверены в правоте своей. И я уже говорил, где-то до вывода уже должна быть проверка пришедших параметров (ключей) сортировки, используйте их.

steepfox 13.10.2015 23:46

А можно по другому это сделать?

laimas 14.10.2015 00:05

По другому это как? Ну так по уму списки выгодно выводить в цикле, дабы не писать по много раз одно и то же, да если еще и в значения опций помещать только изменяемое, а не все скопом, то все упрощается. Кроме того, два ключа определяют 4 набора сортировки, которые с успехом можно заменить на 0-3, где четные значения определяют сортировку tsort по возрастанию, нечетные по убыванию, а деленное по модулю 2 это значение определит поле сортировки sort.

В цикле получаем код списка сравнивая итерацию с единственным GET параметром (0-3), и если равны, то выбираем опцию.

Еще по другому, это на клиенте всю портянку значений опций сравнивать параметром url, но я даже и писать такого не буду, такое можно советовать только ленивым. ;)

steepfox 14.10.2015 00:59

Наконец нашел код, где формируются код тегов option, нужно только определиться, какую из них использовать, а как это сделать не понимаю. Сам код:

<?php


function formation_html_select_from_mas($mas,$on_choose_value=false,$choose_value="",$on_add_begin=false,$value_add_begin=-1,$name_add_begin="Выберите...",$return_only_html = false)
  {
   
   $html="";
   $selected=false;
   
   foreach ($mas as $key => $value)
     {
      $element_choose="";
      if ($on_choose_value && $choose_value == $key)
        {
         $element_choose="selected='selected'";
         $selected=true;
        }
      $html.="<option $element_choose value='$key'>$value</option>";
     }
   if ($on_add_begin)
     {
      $html="<option value='$value_add_begin'>$name_add_begin</option>".$html;
     }
   if ( !$return_only_html )
     $rez=array("html"=>$html,"selected"=>$selected);
    else
      $rez = $html;
   return ($rez);
  }


function formation_html_select_from_mas_rows($mas,$key_value,$key_name,$on_choose_value=false,$choose_value="",$on_add_begin=false,$value_add_begin=-1,$name_add_begin="Выберите...",$return_only_html = false)
  {
   // если $key_value равняется false, то для значения option будет браться ключ соответствующей строки массива
   $html="";
   $selected=false;
   
   foreach ($mas as $key_row => $one_row)
     {
      $key= ( ( $key_value === false ) ? $key_row : $one_row[$key_value] );
      $value=$one_row[$key_name];
      $element_choose="";
      if ($on_choose_value && $choose_value == $key)
        {
         $element_choose="selected='selected'";
         $selected=true;
        }
      $html.="<option $element_choose value='$key'>$value</option>";
     }
   if ($on_add_begin)
     {
      $html="<option value='$value_add_begin'>$name_add_begin</option>".$html;
     }
   if ( !$return_only_html )
     $rez=array("html"=>$html,"selected"=>$selected);
    else
      $rez = $html;
   return ($rez);
  }

laimas 14.10.2015 05:05

Из серии "найди пять отличий"? :) Нет бы проверять аргументы и соответственно выполнение, плодятся близнецы именами разные.

Какая используется - так по имени функции, не хватает на это опыта, так на время выяснения предварите вывод списка произвольным текстом, по которому и будет найдено, например в первой функции строку 7 заменить на $html="Ищу 1";, а во второй строку 35 на $html="Ищу 2";

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

steepfox 14.10.2015 11:15

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

steepfox 14.10.2015 11:17

Как связать все это дело с предыдущим кодом? Так как используется при выборе фильтрации 5 пунктов.

laimas 14.10.2015 11:37

Цитата:

Сообщение от steepfox
Разработчик движка сказал,что именно эти функции используются.

Разработчик? Так третируйте его, что же вы маетесь, с бедламом вашего разработчика даже с коньяком не охота разбираться. :)

Вы задайте своему разработчику вопрос - как может код на физически существующей странице, в котором не чисто html-результат некой функции, а монотонно прописанная строка каждой опции списка с php-вставками, может иметь отношение к некой функции?

laimas 14.10.2015 11:42

Цитата:

Сообщение от steepfox
Как связать все это дело с предыдущим кодом?


Что именно связать - функции разработчика с кодом, который вы ранее показывали? Так это с вопросом к разработчику.

Цитата:

Сообщение от steepfox
Так как используется при выборе фильтрации 5 пунктов.


Да хоть 55, и если не описывать каждый набор "портянками", то хоть 255, все равно будет компактно. И эту компактность (наборы) запросто можно описать единственным числом.


Часовой пояс GMT +3, время: 04:09.