Как объединить строки таблицы?

Question

[kinish_kinish]

Здравствуйте.Создал таблицу, для записи данных из трех форм.При отправке данных в таблицу, данные записываются, как три отдельных стройки.Пытался разобраться с функцией SELECT для объединения этих данных в одну строку, но что то не вышло.Хотелось бы увидеть, как с этой проблемой разбираются профессионалы, что бы понять, как это должно работать.

<?php
$Fam = filter_var(trim($_POST['Fam']), FILTER_SANITIZE_STRING);
$Nam = filter_var(trim($_POST['Nam']), FILTER_SANITIZE_STRING);
$Otch = filter_var(trim($_POST['Otch']), FILTER_SANITIZE_STRING);
$Email = filter_var(trim($_POST['Email']), FILTER_SANITIZE_STRING);
$Data = filter_var(trim($_POST['Data']), FILTER_SANITIZE_STRING);
$Tel = filter_var(trim($_POST['Tel']), FILTER_SANITIZE_STRING);



$servername = "localhost";
$username = "root";
$password = "";
$dbname = "base";


$mysql = new mysqli('localhost','root','','base');
$mysql ->query("INSERT INTO `table` (`Fam`, `Nam`, `Otch`, `Email`, `Data`, `Tel`) VALUES('$Fam', '$Nam', '$Otch', '$Email', '$Data', '$Tel')");
$mysql -> close();

header('location: vopros.html');

?>

<?php

$family = filter_var(trim($_POST['family']), FILTER_SANITIZE_STRING);
$avto = filter_var(trim($_POST['avto']), FILTER_SANITIZE_STRING);
$programm = filter_var(trim($_POST['programm']), FILTER_SANITIZE_STRING);
$game = filter_var(trim($_POST['game']), FILTER_SANITIZE_STRING);
$music = filter_var(trim($_POST['music']), FILTER_SANITIZE_STRING);
$social = filter_var(trim($_POST['social']), FILTER_SANITIZE_STRING);
$alcogol = filter_var(trim($_POST['alcogol']), FILTER_SANITIZE_STRING);
$airplane = filter_var(trim($_POST['airplane']), FILTER_SANITIZE_STRING);



$servername = "localhost";
$username = "root";
$password = "";
$dbname = "base";



$mysql = new mysqli('localhost','root','','base');
$mysql ->query("INSERT INTO `table` (`family`, `avto`, `programm`, `game`, `music`, `social`, `alcogol`, `airplane`) VALUES('$family', '$avto', '$programm', '$game', '$music', '$social', '$alcogol', '$airplane')");
$mysql -> close();

header('location: feedback.html');

?>

<?php


$a1 = filter_var(trim($_POST['a1']), FILTER_SANITIZE_STRING);
$a2 = filter_var(trim($_POST['a2']), FILTER_SANITIZE_STRING);



$servername = "localhost";
$username = "root";
$password = "";
$dbname = "base";


$mysql = new mysqli('localhost','root','','base');
$mysql ->query("INSERT INTO `table` (`a1`, `a2`) VALUES('$a1', '$a2')");
$mysql -> close();


?>

Answer 1

[Vladimir Medvedev]

Прозвучит глупо, но лучший способ создать одну форму, но если никак, то вариантов много, и если не пользоваться сторонними библиотеками, фреймворками и тд, то...
К сожалению, не очень понятно, сразу ли приходят все данные или нет и есть ли регистрация. Если нет, и например $Fam, $Nam, $Otch, $Email, $Data, $Tel приходят сначала, отправляются в базу, в базе им присваивается id (судя по скрину, он автогенерируется у вас), берем id (так как это последняя запись, то Select max(`id`) as `maxid` from `table`), после этот id присваивается пользователю и сохраняется в сессию (если без регистрации), дальше получаете $family, $auto и тд, и добавляете в таблицу где id = id. (например id 17)
UPDATE table SET family = $family, auto = $auto WHERE id = 17;
И так далее.

Еще можно данные всех форм объединить на стороне пользователя (если нет перезагрузки страницы), то есть, собрать данные с разных форм в переменные и отправить их одним постом (не одной формой, так как этот вариант видимо не доступен для вас). Если используете во фронте vue или react, то там кучу возможность сохранить данные. Если нет, то главное чтобы не перегружалась страница.

Вытаскивать данные в таком виде, как у вас, не получиться, так как если одновременно будет отправлено больше одной формы, то могут перемешаться очередность данных. Это если вдруг вы придете к все такой же, как у вас схеме, а потом будете вытаскивать данные с id++, и их объединять.

Так же чтобы не мучиться, вы можете создать отдельный конфиг для дб, потом модели для ваших сущьностей (пользователь или что там у вас), и отдельно функции для работы с глобальными. Потом ловить post и распределять данные.
Например

Comments

[Vladimir Medvedev]

<?php
class DB 
{
    private static $item; //нужен только в этом класе

    public static function getInstance() //функция подключения к базе
    {
        if (empty(static::$item)) { //если нет подключения, то создаем новое, если есть, то просто его возвращаем
            static::$item = new static();
        }
        return static::$item;
    }


    private $config = [  //массив данных для подключения, удобно потом менять, когда будете загружать на боевой 
                                     //сервер, где могут отличаться настройки 
        'driver' => 'mysql', //тип бд
        'host' => 'localhost', //хост
        'dbName' => 'sto', //имя вашей бд
        'charset' => 'UTF8', //кодировка
        'login' => '      ', //логин для входа
        'password' => '' //пароль
    ];
    private $connect; //переменная для определения, есть ли соединение 

    public function getConnect() //конект через PDO, можете почитать в оф документации 
                                                             //https://www.php.net/manual/ru/book.pdo.php
    {
        if (empty($this->connect)) { //если нет конекта, то создаем
            $this->connect = new \PDO(
                $this->getSdnString(),
                $this->config['login'],
                $this->config['password']
            );
            $this->connect->setAttribute(\PDO::ATTR_DEFAULT_FETCH_MODE, \PDO::FETCH_ASSOC);
        }
        return $this->connect;
    }

    private function getSdnString()
    {
        return sprintf(
            '%s:host=%s;dbname=%s;charset=%s',
            $this->config['driver'],
            $this->config['host'],
            $this->config['dbName'],
            $this->config['charset']
        );
    }
/* тут "ПЛОХАЯ" штука дальше, но так можно "защитить" этот класс от лишнего, в дальнейшем, когда будете лучше разбираться, или использовать laravel оно вам не пригодится 
    protected function __construct()
    {

    }

    protected function __clone()
    {

    }

    protected function __wakeup()
    {

    }
    */

    private function query(string $sql, $params = [])  //функция отбора
    {
        $pdoStatement = $this->getConnect()->prepare($sql);
        $pdoStatement->execute($params);    
        return $pdoStatement;
    }

    public function find($sql, $params) //что-то найти 
    {
        return $this->query($sql, $params)->fetch();
    }

    public function findAll($sql, $params = []) //вытащить все
    {
        return $this->query($sql, $params)->fetchAll();
    }

    public function createObj(string $sql, $class, $params = []) //создать нужный нам объект, ниже будет об этом 
    {
        $pdoStatement = $this->query($sql, $params);
        $pdoStatement->setFetchMode(\PDO::FETCH_CLASS, $class);
        return $pdoStatement->fetch();
    }

    public function createObjAll(string $sql, $class, $params = [])
    {
        $pdoStatement = $this->query($sql, $params);
        $pdoStatement->setFetchMode(\PDO::FETCH_CLASS, $class);
        return $pdoStatement->fetchAll();
    }

    public function exe(string $sql, array $params = []) {
        return $this->query($sql, $params)->rowCount();
    }

}

после создаем нужный нам объект

<?php
//подключаем сюда файл с ДБ includ или через use (лучше use и через namespace)
//в теории использовать бы тут наследие от модели, но наверное будет очень много и так 
class Client
{
//тут нужные вам переменные (все столбцы БД!), все писать не буду 
  public $id;
  public $fam;
  public  $auto;
  public $tableName;

       public function construct($id = '', $fam = '', $auto = '') //это конструктор класса, он вызывается, //когда мы его создаем
    {   
        $this->id = $id;
        $this->fam = $fam;
        $this->auto = $auto;
    }

    protected function getDB() //таким образом устанавливаем соединение с БД
    {
        return DB::getInstance();
    }

   public function getMaxId() 
     {
         $tableName = $this->tableName; 
         $sql = "Select max (`id`) as `maxid` from {tableName}"
         return $this->getDB()->query($sql); 
     }

    public function getLast($id) //так будем искать по id, для объединения 
    {
        $tableName = $this->tableName; // -> такой стрелкой идем обращение к элементам объекта, что-то типа //this<b>.</b>в 
                                                             // js
        $id = $this->getMaxId();
        $sql = "SELECT * FROM {$tableName} WHERE id = :id"; //создаем шаблон запроса " :id", значит, что сюда 
                                                                                                //подставиться значение, которое мы передадим 
        $params = [':id' => $id];//как раз значение
        return $this->getDB()->createObj($sql, 'Client' . ucfirst($tableName), $params); //и создаем наш объект
    }//такие сложно для защиты от инъекций https://habr.com/ru/post/130826/#:~:text=SQL%20%D0%B8%D0%BD%D1%8A%D0%B5%D0%BA%D1%86%D0%B8%D1%8F%20%E2%80%94%20%D1%8D%D1%82%D0%BE%20%D0%BE%D0%B4%D0%B8%D0%BD%20%D0%B8%D0%B7,%D0%B2%D1%81%D0%B5%D0%B3%D0%BE%20%D1%8D%D1%82%D0%BE%20MySQL)%20%D1%87%D1%82%D0%BE%20%D1%83%D0%B3%D0%BE%D0%B4%D0%BD%D0%BE.

    public function getAll() //берем все
    {
        $tableName = $this->tableName;
        $sql = "SELECT * FROM {$tableName}";
        return $this->getDB()->createObjAll($sql, 'Client' . ucfirst($tableName));
    }

   public function addSecondForm( $auto) //и остальные переменные из второй формы
   {
       return $this->auto = $auto;
   }

    private function getColumns() //берем все колонны, для дальнейшего их заполнения
    {
        $element = $this->getAll()[0];
        $columnsNames = [];
        foreach ($element as $columns => $value) {
            if ($columns == 'id') {
                continue;
            } else {
                array_push($columnsNames, $columns);
            }
        }
        return $columnsNames;
    }

    private function insert($params = []) //тут мы описываем логику загрузки в базу на подобии кода //выше
    {
        $columns = $this->getColumns();
        $params = (array) $params;
        unset($params['id']);
        $columnsNames = implode(', ', $columns);
        $params = implode("', '", $params);
        $sql = "INSERT INTO {$this->tableName} ({$columnsNames})
                VALUES ('{$params}')";
        if(!$res = $this->getDB()->exe($sql)) {
            return false;
        }else return true;
    }
     
      private function update($element = []) //описываем обновление данных
    {
        $res = false;
        foreach ($this as $key => $value) {
            if ($element["$key"] != $value) {
                $params[":{$key}"] = $value;
                $placeholders = implode(", ", array_keys($params));
                $sql = "UPDATE {$this->tableName()} SET {$key} = {$placeholders} WHERE id = {$element['id']}";
                if (!$ok = $this->getDB()->exe($sql, $params)) {
                    $res = false;
                }else {
                    $res = true;
                };
            }
        }
        return $res;
    }//мы получали все данные из созданного объекта и сверяли с базой, после добавляли, что отличалось 

    public function save()//а дальше одну функцию,  чтобы меньше дублировать /*
тут мы берем все объекты, сравниваем если ли такой же, если да, но обновляем, если нет, то создаем новый
*/
    {
        $allFromTable = $this->getALl();
        foreach ($allFromTable as $element) {
            $element = (array) $element;
            if ($this->id == $element['id']) {
                if (!$res = $this->update($element)) {
                    echo '<h1>error update</h1> <hr> <br>';
                    return false;
                }else {
                    echo "<h1>OK Update</h1> <hr> <br>";
                    return true;
                }
            }
        }
        if (!$res = $this->insert($this)) {
            echo '<h1>error insert</h1> <hr> <br>';
            return false;
        }else {
            echo "<h1>OK Insert</h1> <hr> <br>";
            return true;
        }
    }

}

и теперь ipdex.php

<?php
   session_start(); //начинаем сессию
   require //все до этого созданные файлы (не onec, так как они не супер огромные, а проверять каждый раз их 
//подключение затратно, по хорошему написать загрузчик или composer или laravel
/*
        $this->id = $id;
        $this->fam = $fam;
        $this->auto = $auto;
*/
   if ($_SERVER['REQUEST_METHOD'] === 'POST') { //к глобальным переменным лучше делать отдельные функции,
 //но и так уже расписал в 100 раз больше, чем думал сначала :)
            if($id = $_POST['id']) { //есть уже отправляется id из сессии 
                        $avto = filter_var(trim($_POST['avto']), FILTER_SANITIZE_STRING);
                        $model = new Client($id);
                        $client = $model->getLast();
                        $client->addSecondFrom($avto);
                        $client->save();
                 }else {
                        $fam = filter_var(trim($_POST['Fam']), FILTER_SANITIZE_STRING);
                        $client = new Client('', $fam); //обязательно должно совпадать с очередностью в конструкторе класса
                        $id = $client->getMaxId();
                        $_SESSION['id'] = $id;
                   }
        }else {
            return false;
        }

[Vladimir Medvedev]

Я далеко не профи, и наверняка будут более корректные ответы, ведь данный способ точно кривой, и такого не встретите, но, на мой взгляд, это такой минимум если вы без опыта и делаете без сторонних фреймворков, библиотек, апи и тд. Сможете посмотреть на разные возможности базы, на классы, на условия и тд.
ЗЫ. хотел написать лишь образно, что примерно надо делать, но немного понесло, часть скопировал со своих старых реп, часть писал прям тут, так что не советую прям ctrl+c ctrl+v, но если что-то будет не понятно или не работать, пишите комментарии, постараюсь ответить .

[kinish_kinish]

Vladimir Medvedev, Спасибо за ваш код, но мне дали это как задание второго курса колледжа, и уже который день с одногруппниками мучаемся над этой задачей.Преподаватель, что то говорил про функции SELECT, с дальнейшим отбором по id.Я очень удивлён таким большим ответом, попробую разобраться, но это будет сложно)

[kinish_kinish]

Vladimir Medvedev, то есть он имел ввиду, что решение буквально в пару строк.

[Vladimir Medvedev]

kinish_kinish, если это учебное задание, то тогда и правда можно просто создать переменные и данными из 3х форм и их объеденить в третей. И в таком случае не разбирайтесь в коже выше, если пока сосредоточены на учёбе) Можно создать функцию, которая будет принимать параметры, и их дальше вставлять в sql запрос. Но не знаю, какими инструментами можно вам пользоваться, так что самый топорный.

$firstForm = mysql ->query(" SELECT fam, nam, …
FROM 'tsble' WHERE id = число; ");
$secondForm = mysql ->query(" SELECT family , auto, …
FROM 'table' ;");
$thirdForm = 'вы поняли идею';

$mysql -> close();

$fullData = $firstForm. $secondForm. $thirdForm;

Вот и все вместе. Можно для этого обойтись одной функцией, но опять же не знаю что можно использовать)

Писал с телефона в дороге, так что тут скорее логика, чем точный синтаксис)