Народ посмотрите на код и скажите как его можно улучшить? PHP: form action="" method="POST" > <table> <tr><td>Порядковый номер:</td><td> <input type="text" name="id"></input></td></tr> <tr><td>Имя сотрудника:</td><td> <input type="text" name="name"></input></tr></td> <tr><td><input type="submit" value="Внести данные"></td><tr> </table> </form> <a href="index.php">Посмотреть список лиц</a> <?php require_once "connectionDB.php"; if(isset($_POST[name])) { echo "Принял данные: " . $_POST[name] . "</br>"; } else { echo "Данные отсутствуют</br>"; } //Скрипт на добавление данных в бд $id = $_POST[id]; $name = $_POST[name]; $query = "INSERT INTO Users (id, name) VALUES (:id, :name)"; $params = [ ':id' => $id, ':name' => $name ]; $stmtin = $db->prepare($query); $stmtin->execute($params); ?>
Защиту от обновления страницы, защиту от дублей, проверку того что водят в поле name. Обычно id это AUTO_INCREMENT зачем вы его вручную вводите?
Когда создавал таблицу не подумал об этом. Можешь подробнее расписать про защиту от обновления страницы? В чем суть?
Если запрос INSERT выполнился успешно, нужно делать «серверный» редирект методом GET. Но вам до этой логики еще далеко, судя по этому: PHP: } else { echo "Данные отсутствуют</br>"; } //Скрипт на добавление данных в бд $id = $_POST[id]; $name = $_POST[name]; --- Добавлено --- P.S. И сим. ключи ассоциативного массива нужно заключать в кавычки