Работа с формой

@Gcom · Регистрация: 03.02.2016

Author24 — интернет-сервис помощи студентам

Есть код:

Кликните здесь для просмотра всего текста

PHP/HTML

<?php
header ("Content-Type:text/html; charset=UTF-8");
ini_set ('display_errors', 1); 
error_reporting (E_ALL);
 
$sQValue  = '';
$sStrict  = '0'; // точный поиск (1-да/0-нет) "0"
$sCross   = '0'; // поиск с кросами (1-да/0-нет) "0"
$sReplace = '0'; // поиск с заменами (1-да/0-нет) "0"
 
if ($sStrict == '0')  $sStrictStatus  = ''; else $sStrictStatus  = 'checked';
if ($sCross == '0')   $sCrossStatus  = ''; else $sCrossStatus  = 'checked';
if ($sReplace == '0') $sReplaceStatus  = ''; else $sReplaceStatus  = 'checked';
 
if(isset($_REQUEST['q'])) { // Обработка кнопки "Найти"
 
    $sQValue = $_REQUEST['q'];
    
    if (isset($_REQUEST['strict']))  {
        $sStrict = '1';
        $sStrictStatus  = 'checked';
    }   else {
        $sStrict = '0';
        $sStrictStatus  = '';
    };
    
    if (isset($_REQUEST['cross']))  {
        $sCross = '1';
        $sCrossStatus  = 'checked';
    }   else {
        $sCross = '0';
        $sCrossStatus  = '';
    };
 
    if (isset($_REQUEST['replace']))  {
        $sReplace = '1';
        $sReplaceStatus  = 'checked';
    }   else {
        $sReplace = '0';
        $sReplaceStatus  = '';
    };  
};
 
echo ('/strict '.$sStrict.'/cross '.$sCross.'/replace '.$sReplace.'<br>');
 
$sFindFormHTML = '
            <form>
                <input type = "hidden" name = "type" value = "article">
                <p>
                введите артикул: <input type = "text" name = "q" value = "'.$sQValue.'" placeholder = "48510-29176" pattern=".{0,15}" required title="">
                <input type = "submit" value = "Найти">
                </p>
                <hr>
                <p>
                <input type = "checkbox" name = "strict" value = "'.$sStrict.'" '.$sStrictStatus.'> строгое соответствие *
                <input type = "checkbox" name = "cross" value = "'.$sCross.'" '.$sCrossStatus.'> с кроссами *
                <input type = "checkbox" name = "replace" value = "'.$sReplace.'" '.$sReplaceStatus.'> с заменами
                </p>
            </form>
    ';
 
$sPageHTML = '
    '.$sFindFormHTML.'
    ';
echo (''.$sPageHTML);
?>

Без запроса выводит: /strict 0/cross 0/replace 0

Вводим число "123", ставим флажок "строгое соответствие" нажимаем на кнопку, выводит:
/strict 1/cross 0/replace 0 (вроде как надо)
но в адресной строке:

?type=article&q=123&strict=0

Нажимаем еще раз на кнопку и в адресной строке:

?type=article&q=123&strict=1

И так с каждым флажком, почему так происходит? Подскажите где я намудрил в коде?

@Kerry_Jr · 27.02.2016, 15:04

Gcom, у вас сначала все считается, а потом по результатам расчетов рисуется форма. Т.е. чтобы единичка появилась в строке запроса, нужно поставить галочку, нажать на кнопку, а потом ещё раз нажать на кнопку. Можете даже перед вторым нажатием снять эту галочку и/или поставить другие галочки.

@pav1uxa · 27.02.2016, 21:20

Сообщение от Gcom

Подскажите где я намудрил в коде?

Намудрили много где

Но основное - зачем-то меняете value у checkbox'ов. Это бессмыслица.

PHP/HTML

<?php
 
$sQValue  = isset($_GET['q']) ? $_GET['q'] : '';
 
$sStrict  = isset($_GET['strict']) ? '1' : '0';
$sCross   = isset($_GET['cross']) ? '1' : '0'; 
$sReplace = isset($_GET['replace']) ? '1' : '0';
 
$sStrictStatus  = isset($_GET['strict']) ? ' checked' : '';
$sCrossStatus   = isset($_GET['cross']) ? ' checked' : ''; 
$sReplaceStatus = isset($_GET['replace']) ? ' checked' : '';
 
echo ('/strict '.$sStrict.'/cross '.$sCross.'/replace '.$sReplace.'<br>');
?>
<form method='get'>
    <input type='hidden' name='type' value='article'>
    <p>
        введите артикул: <input type='text' name='q' value='<?=$sQValue?>' placeholder='48510-29176' pattern='.{0,15}' required>
        <input type='submit' value='Найти'>
    </p>
    <hr>
    <p>
        <input type='checkbox' name='strict'<?=$sStrictStatus?>> строгое соответствие В*
        <input type='checkbox' name='cross'<?=$sCrossStatus?>> с кроссами В*
        <input type='checkbox' name='replace'<?=$sReplaceStatus?>> с заменами
    </p>
</form>

@Gcom · 28.02.2016, 07:02 **[ТС]**

Сообщение от pav1uxa

Но основное - зачем-то меняете value у checkbox'ов. Это бессмыслица.

Хотел привязать к переменным и в дальнейшем использовать эту связку + нужно было "сохранять" выбранное значение флажков.

PHP

1
2
3

$sStrict  = '0'; // точный поиск (1-да/0-нет) "0"
$sCross   = '0'; // поиск с кросами (1-да/0-нет) "0"
$sReplace = '0'; // поиск с заменами (1-да/0-нет) "0"

Добавлено через 4 минуты
pav1uxa, код конечно шикарен, в плане минимизации, но меня всегда смущали конструкции вида:

PHP

1	<переменная> = <условие> ? <действие если да> : <действие если нет> ;

А что если после сработки условия нужно сделать несколько действий, тогда как быть?

@pav1uxa · 28.02.2016, 07:38

Сообщение от Gcom

Хотел привязать к переменным и в дальнейшем использовать эту связку + нужно было "сохранять" выбранное значение флажков.

Ну у меня переменные 0\1 тоже сохраняются. А выбранные значения флажков устанавливаются через атрибут checked, а не через value="0/1".

Сообщение от Gcom

код конечно шикарен, в плане минимизации

Он шикарен еще и потому что работает

Сообщение от Gcom

А что если после сработки условия нужно сделать несколько действий, тогда как быть?

Если да кабы... Если б нужно тогда бы я посмотрел. А то что у Вас это вообще абракадарбра. У Вас вот эти 6 строчек кода если удалить из программы, не изменится совершенно ничего

PHP

$sStrict  = '0'; // точный поиск (1-да/0-нет) "0"
$sCross   = '0'; // поиск с кросами (1-да/0-нет) "0"
$sReplace = '0'; // поиск с заменами (1-да/0-нет) "0"
 
if ($sStrict == '0')  $sStrictStatus  = ''; else $sStrictStatus  = 'checked';
if ($sCross == '0')   $sCrossStatus  = ''; else $sCrossStatus  = 'checked';
if ($sReplace == '0') $sReplaceStatus  = ''; else $sReplaceStatus  = 'checked';

Можно конечно переменным поназадавать значения по умолчанию, но тогда условия поубирать половину. А у Вас что то с чем то...

Сообщение от Gcom

но меня всегда смущали конструкции вида:

Странно что Вас совершенно не смущают конструкции вида

PHP

1	if ($sStrict == '0') $sStrictStatus = ''; else $sStrictStatus = 'checked';

Поверьте - они выглядят гораздо ужаснее...

@Gcom · 29.02.2016, 05:42 **[ТС]**

Сообщение от pav1uxa

А то что у Вас это вообще абракадарбра. У Вас вот эти 6 строчек кода если удалить из программы, не изменится совершенно ничего

Почему не нужны, они как раз задают так сказать стартовое значение флажков, что бы если понадобилось например при "старте" поменять что допустим нужен включить флажок номер 2, я просто меняю в коде это:

PHP

1	$sCross = '0'; // поиск с кросами (1-да/0-нет) "0"

на это:

PHP

1	$sCross = '1'; // поиск с кросами (1-да/0-нет) "0"

И все по прежнему работает, мудрено конечно наверно все это, но на что хватило мозгов, то и придумал

Добавлено через 21 час 9 минут
pav1uxa, СПС Вам, все заработало как надо, сделал на основе Вашего кода (работает как часики)

PHP/HTML

<?php
header ("Content-Type:text/html; charset=UTF-8");
ini_set ('display_errors', 1); 
error_reporting (E_ALL);
 
$sStrict  = '0'; // точный поиск (1-да/0-нет) "0"
$sCross   = '1'; // поиск с кросами (1-да/0-нет) "0"
$sReplace = '1'; // поиск с заменами (1-да/0-нет) "0"
 
$sStrictStatus = ($sStrict == '0') ? '' : 'checked';
$sCrossStatus = ($sCross == '0') ?  '' : 'checked';
$sReplaceStatus = ($sReplace == '0') ? '' : 'checked';
 
if(isset($_REQUEST['q'])) { // Обработка кнопки "Найти"
    $sQValue = $_REQUEST['q'];
 
    $sStrict  = isset($_REQUEST['strict']) ? '1' : '0';
    $sCross   = isset($_REQUEST['cross']) ? '1' : '0'; 
    $sReplace = isset($_REQUEST['replace']) ? '1' : '0';
 
    $sStrictStatus  = isset($_REQUEST['strict']) ? 'checked' : '';
    $sCrossStatus   = isset($_REQUEST['cross']) ? 'checked' : ''; 
    $sReplaceStatus = isset($_REQUEST['replace']) ? 'checked' : '';
 
} else $sQValue = ''; // if(isset($_REQUEST['q']))
 
echo ('/strict '.$sStrict.'/cross '.$sCross.'/replace '.$sReplace.'<br>');
?>
<form>
    <input type='hidden' name='type' value='article'>
    <p>
        введите артикул: <input type='text' name='q' value='<?=$sQValue?>' placeholder='48510-29176' pattern='.{0,15}' required>
        <input type='submit' value='Найти'>
    </p>
    <hr>
    <p>
        <input type='checkbox' name='strict' value = "1" <?=$sStrictStatus?>> строгое соответствие *
        <input type='checkbox' name='cross' value = "1" <?=$sCrossStatus?>> с кроссами *
        <input type='checkbox' name='replace' value = "1" <?=$sReplaceStatus?>> с заменами
    </p>
</form>

Добавлено через 3 минуты

Сообщение от pav1uxa

Поверьте - они выглядят гораздо ужаснее...

Ну да так на много приятнее на ощупь

PHP

1	$sStrictStatus = ($sStrict == '0') ? '' : 'checked';

Добавлено через 20 минут
И еще такой момент чем хуже это:

PHP/HTML

1
2
3

if(isset($_REQUEST['q']))
...
<form>

Чем это:

PHP/HTML

1
2
3

if(isset($_GET['q']))
...
<form method='get'>

Почему все применяют GET и POST вместо REQUEST?

@pav1uxa · 29.02.2016, 09:15

Сообщение от Gcom

Почему все применяют GET и POST вместо REQUEST?

В суперглобальный массив $_REQUEST входят суперглобальные массивы $_GET, $_POST и $_COOKIE. Зачем мне брать $_GET, $_POST и $_COOKIE, если я знаю что данные приходят через GET запрос.

А если method='get' не использовать, они все равно приходят через GET. Так просто нагляднее.

@Kerry_Jr 3106 / 2591 / 1219 Регистрация: 14.05.2014 Сообщений: 7,236 Записей в блоге: 1
	27.02.2016, 15:04	2
	Gcom, у вас сначала все считается, а потом по результатам расчетов рисуется форма. Т.е. чтобы единичка появилась в строке запроса, нужно поставить галочку, нажать на кнопку, а потом ещё раз нажать на кнопку. Можете даже перед вторым нажатием снять эту галочку и/или поставить другие галочки. 0

@pav1uxa 1943 / 1768 / 825 Регистрация: 23.01.2014 Сообщений: 6,230
	29.02.2016, 09:15	7
	Сообщение от Gcom Почему все применяют GET и POST вместо REQUEST? В суперглобальный массив $_REQUEST входят суперглобальные массивы $_GET, $_POST и $_COOKIE. Зачем мне брать $_GET, $_POST и $_COOKIE, если я знаю что данные приходят через GET запрос. А если method='get' не использовать, они все равно приходят через GET. Так просто нагляднее. 0

Работа с формой

Решение