Доработка формы, правильно ли я сделал?

Question

[lere]

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

/* Задаем переменные */
$lastname = htmlspecialchars($_POST["lastname"]); //Фамилия
$name = htmlspecialchars($_POST["name"]); //Имя
$middlename = htmlspecialchars($_POST["middlename"]); //Отчество
$birthday = htmlspecialchars($_POST["birthday"]); //День рождения


$drivers = htmlspecialchars($_POST["driver-serie"]); //Вод. серия
$drivern = htmlspecialchars($_POST["driver-number"]); //Вод. номер

$passports = htmlspecialchars($_POST["passport-serie"]);
$passportn = htmlspecialchars($_POST["passport-number"]);

$phone = htmlspecialchars($_POST["phone-number"]);
$email = htmlspecialchars($_POST["email"]);
/* Предыдущие данные */
$lastname = htmlspecialchars($_POST["lastname1"]); //Предыдущая фамилия
$name = htmlspecialchars($_POST["name1"]); //Имя
$middlename = htmlspecialchars($_POST["middlename1"]); //Предыдущее отчество
$drivers = htmlspecialchars($_POST["driver-serie1"]); //Предыдущее Вод. серия
$drivern = htmlspecialchars($_POST["driver-number1"]); //Предыдущее Вод. номер

$passports = htmlspecialchars($_POST["passport-serie1"]);
$passportn = htmlspecialchars($_POST["passport-number1"]);


 
/* Ваш адрес и тема сообщения */
$address = "@yandex.ru";
$sub = "Сообщение с сайта ";
 
/* Формат письма */
$mes = "Сообщение с сайта.\n
Имя отправителя: $lastname, $name, $middlename
Дата рождения: $birthday
Водительское удостоверение: Серия $drivers  Номер $drivern
Паспорт: Серия $passports Номер $passportn
Телефон: $phone
E-mail: $emailw"; 
Предыдущие данные:
Имя отправителя: $lastname1, $name1, $middlename1
Водительское удостоверение: Серия $drivers1  Номер $drivern1
Паспорт: Серия $passports1 Номер $passpor4tn1

Answer 1

[Сергей delphinpro]

$fieldNames = [
    "lastname",
    "name",
    "middlename",
    "birthday",
    "driver-serie",
    "driver-number",
    "passport-serie",
    "passport-number",
    "phone-number",
    "email",
    "lastname1",
    "name1",
    "middlename1",
    "driver-serie1",
    "driver-number1",
    "passport-serie1",
    "passport-number1"
    ];

$fileds = [];

// всё, добавляйте полей, сколько хотите...
foreach ($fieldNames as $fName) {
    $fields[$fname] = htmlspecialchars($_POST[$fname]);
}

$mes = "Сообщение с сайта.\n
Имя отправителя: {$field['lastname']}, {$field['name']}, {$field['middlename']}
Дата рождения: {$field['birthday']}
...
";

Comments

[Дмитрий Гаджиев]

а если какое поле не так сработает, искать ошибку в массиве он будет полдня... на переменных хоть понятней и проще))

[Сергей delphinpro]

Дмитрий Гаджиев: а если нужно будет поправить валидацию/фильтрацию? Все 20 будет править. Нафиг этот геморр.

[Дмитрий Гаджиев]

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

[Дмитрий Гаджиев]

да и валидация для 20 полей одинаковой у него не будет, однозначно. так что все равно делать все ручками)))

[Сергей delphinpro]

Дмитрий Гаджиев: да вот нифига )
Я могу грубо говоря сделать как-то так

$fields = [
  [
    "name" => "firstname",
    "validation" => "string"
  ],
  [
    "name" => "age",
    "validation" => "integer"
  ]
]

Ну а дальше, надеюсь понятен ход мысли

[Дмитрий Гаджиев]

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

в данном вопросе я бы поступил совсем по другому, но если есть готовая структура - зачем её усложнять? надо быстро найти ошибку и исправить, а красивости можно навести и в процессе, как будет свободное время.

[Сергей delphinpro]

Дмитрий Гаджиев: ну ладно, если "по-быстрому" надо, тогда не спорю =)