Форумы портала PHP.SU :: Версия для печати :: не срабатывает проверка ошибок после нажатия кнопки

2. Строитель - 08 Января, 2017 - 14:26:44 - перейти к сообщению

alezander, строку №40 перенесите за пределы пхп кода вниз.

CODE (html):
скопировать код в буфер обмена

<form method="POST" action="/add.php"> 

3. alezander - 08 Января, 2017 - 14:50:55 - перейти к сообщению

Он и так находиться за приделами php если убрать не будет срабатывать.

CODE (htmlphp):
скопировать код в буфер обмена

<form method="POST" action="/add.php">   
    <?php
    if(isset($_POST['add']))
    {
        $errors = array();
        if( $_POST['username'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['password'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['class'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['type'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['cratedate'] == '0000-00-00')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if (empty($errors))
        {
        // include "add.php";
        }
        else 
        {
                echo $errors['0'];
        }
    }
    ?>
                        <tr>
                                <tr>
                                <th>#</th>
                                <th><input type="text" name="username" value = "<?php echo $_POST['username']?>"/><br/></th>
                                <th><input type="text" name="password" value = "<?php echo $_POST['password']?>"/><br/></th>
                                <th><input type="number" name="class" step="0.01" value = "<?php echo $_POST['class']?>"/><br/></th>
                                <th><input type="number" name="type" step="0.01" value = "<?php echo $_POST['type']?>"/><br/></th>
                                <th><input type="date" name="cratedate" value = "<?php echo $_POST['cratedate']?>"/><br/></th>
                                <th><input type="submit" name= "add" value="Добавить"/></th>
                        </tr>
        </form>

4. Строитель - 08 Января, 2017 - 14:55:17 - перейти к сообщению

alezander, вы не правильно меня поняли, или я не верно выразился. Вам нужно блок проверок на ошибки вынести за пределы html-формы

PHP:
скопировать код в буфер обмена

    <?PHP 
   if(isset($_POST['add']))
   {
        $errors = array();
        if( $_POST['username'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['password'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['class'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['type'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['cratedate'] == '0000-00-00')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if (empty($errors))
        {
       // include "add.php";
        }
        else
        {
                echo $errors['0'];
        }
   }
   ?>
 
<form action="">

(Добавление)
И ещё вместо этого echo $errors['0']; напишите так

PHP:
скопировать код в буфер обмена

echo join('<br />', $errors);

5. alezander - 08 Января, 2017 - 15:07:59 - перейти к сообщению

эффект тот-же добавляет строки не обращая внимания на проверки.

Спойлер (Отобразить)

CODE (htmlphp):
скопировать код в буфер обмена

    <?php
    if(isset($_POST['add']))
    {
        $errors = array();
        if( $_POST['username'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['password'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['class'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['type'] == '')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if( $_POST['cratedate'] == '0000-00-00')
        {
                $erorrs[] = 'Ведите Имя пользователя!';
        }
        if (empty($errors))
        {
        include "add.php";
        }
        else 
        {
                echo join('<br />', $errors);
        }
    }
    ?>
<form method="POST" action="/add.php">   
                        <tr>
                                <tr>
                                <th>#</th>
                                <th><input type="text" name="username" value = "<?php echo $_POST['username']?>"/><br/></th>
                                <th><input type="text" name="password" value = "<?php echo $_POST['password']?>"/><br/></th>
                                <th><input type="number" name="class" step="0.01" value = "<?php echo $_POST['class']?>"/><br/></th>
                                <th><input type="number" name="type" step="0.01" value = "<?php echo $_POST['type']?>"/><br/></th>
                                <th><input type="date" name="cratedate" value = "<?php echo $_POST['cratedate']?>"/><br/></th>
                                <th><input type="submit" name= "add" value="Добавить"/></th>
                        </tr>
        </form>
<tr>
        <td><?=$row['id']?></td>
        <td><?=$row['username']?></td>
        <td>Cуммма:</td>
        <td><? print $sum?></td>
        <td><? print $sum2?></td>
        <td><?=$row['cratedate']?></td>
    </tr>
    <?php
    print_r($POST);
    ?>
        </body>
</html>

6. Строитель - 08 Января, 2017 - 15:14:06 - перейти к сообщению

alezander, сейчас за свой комп перейду, посмотрю на локалке и отпишусь.

7. alezander - 08 Января, 2017 - 15:48:31 - перейти к сообщению

Спасибо. Если так будет быстрее вот все что написано.

8. Строитель - 08 Января, 2017 - 16:41:07 - перейти к сообщению

alezander, ошибка у вас была в названии переменной errors . Код я взял из вашего первого поста, немного его изменил. По хорошему надо всё из суперглобального массива POST фильтровать и сохранять в переменные, и далее работать только с переменными. Пример ниже это и демонстрирует

Спойлер (Отобразить)

PHP:
скопировать код в буфер обмена

<?PHP 
 
require 'includes/config.php';
 
$username  = !empty($_POST['username']) ? strip_tags($_POST['username']) : '';
$password  = !empty($_POST['password']) ? strip_tags($_POST['password']) : '';
$class     = !empty($_POST['class']) ? strip_tags($_POST['class']) : '';
$type      = !empty($_POST['type']) ? strip_tags($_POST['type']) : '';
$cratedate = !empty($_POST['cratedate']) ? strip_tags($_POST['cratedate']) : '';
 
$errors = array();
 
!empty($username)  ?: $errors[] = 'Ведите Имя!';
!empty($password)  ?: $errors[] = 'Ведите Пароль!';
!empty($class)     ?: $errors[] = 'Ведите Класс!';
!empty($type)      ?: $errors[] = 'Ведите Тип!';
!empty($cratedate) ?: $errors[] = 'Ведите Дату!';
    
if (isset($_POST['add'])) {
    if (!empty($errors)) {
        echo join('<br />', $errors);
    } else {
        include 'add.php';
    }
}
 
?>
            
<html>
    <head>
        <title><?=$config['title']?></title>
        <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
    </head>
    <body>
        <table border="2">
            <tr>
                <td>id</td>
                <td>Имя пользователя</td>
                <td>Пароль</td>
                <td>Класс</td>
                <td>Тип учетной записи</td>
                <td>дата создания</td>
            </tr>
            <?PHP 
 
            $result = mysqli_query($connection, 'SELECT * FROM users');
 
            while ($row = mysqli_fetch_array($result)):
                $sum += $row['class'];
                $sum2 += $row['type'];
                
                ?>
 
                <tr>
                    <td><?=$row['id']?></td>
                    <td><?=$row['username']?></td>
                    <td><?=$row['password']?> </td>
                    <td><?=$row['class']?></td>
                    <td><?=$row['type']?></td>
                    <td><?=$row['cratedate']?></td>
                    <td><input type="submit" value="Изменить"/></td>
                    <td><input type="submit" value="Удалить"/></td>
                </tr>
            
            <?PHP  endwhile; ?>
            
            <form method="POST" action="/add.php">
                <tr>
                    <th>#</th>
                    <th><input type="text" name="username" value="<?=$username?>"/><br/></th>
                    <th><input type="text" name="password" value="<?=$password?>"/><br/></th>
                    <th><input type="number" name="class" step="0.01" value="<?=$class?>"/><br/></th>
                    <th><input type="number" name="type" step="0.01" value="<?=$type?>"/><br/></th>
                    <th><input type="date" name="cratedate" value="<?=$cratedate?>"/><br/></th>
                    <th><input type="submit" name="add" value="Добавить"/></th>
                </tr>
            </form>
            <tr>
                <td><?=$row['id']?></td>
                <td><?=$row['username']?></td>
                <td>Cуммма:</td>
                <td><?=$sum?></td>
                <td><?=$sum2?></td>
                <td><?=$row['cratedate']?></td>
            </tr>
        </table>
    </body>
</html>

9. alezander - 08 Января, 2017 - 16:55:19 - перейти к сообщению

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

10. Строитель - 08 Января, 2017 - 17:08:44 - перейти к сообщению

alezander, тогда скопируйте мой код целиком, он рабочий.

11. alezander - 08 Января, 2017 - 18:37:25 - перейти к сообщению

все полностью скопировал эфект тот - же

12. Строитель - 08 Января, 2017 - 18:40:40 - перейти к сообщению

alezander пишет:

все полностью скопировал эфект тот - же

Больше не знаю, чем вам помочь, т.к. на локалке код тестировал, он работает без проблем.
(Добавление)
Включайте показ всех ошибок, только так вы что-то у себя сможете найти.

13. alezander - 08 Января, 2017 - 18:45:39 - перейти к сообщению

Строитель пишет:

alezander пишет:

все полностью скопировал эфект тот - же

Больше не знаю, чем вам помочь, т.к. на локалке код тестировал, он работает без проблем.
(Добавление)
Включайте показ всех ошибок, только так вы что-то у себя сможете найти.

Спасибо буду думать.

14. Quest - 09 Января, 2017 - 09:58:38 - перейти к сообщению

$erorrs

$errors