Как можно изменить значения в БД при помощи PHP?

Question

[miniven]

Есть таблица со значениями. Я хочу их изменять из админки. Для этого мне нужно, чтобы в ней появлялись столько input'ов, сколько значений в таблице. ЭТо я могу сделать с помощью foreach, но как потом отправлять введенные значения в БД?

Сейчас я изменяю значения таким образом, например:

if (!empty($_POST)) {
		$id = $_POST['id'];
		$name = $_POST['reviewName'];
		$body = $_POST['reviewBody'];
		$userpic = $_POST['userPic'];

		$update_review = $db->prepare(
		'UPDATE reviews SET 
			reviews_name = :name, 
			reviews_body = :body,
			reviews_logo = :userpic
		WHERE id = :id'
	    );

		$update_review->execute([
			'name' => $name,
			'body' => $body,
			'userpic' => $userpic,
			'id' => $id
		]);

Но с использованием foreach для данной задачи еще не сталкивался

Или есть какой-то более грамотный подход?

Answer 1

[ThunderCat]

Хорошо что прикручены препарестатменты, но все равно, неужели все так и кидается в базу без проверок?
Неплохо было бы какой-никакой фильтр все же прикрутить.
А так, если по коду:

$querystring = array();
foreach($_POST as $key=>$val){
if($key!="id")$querystring[] = "`$key`=:$key";
}
if(!empty($querystring))$querystring = implode(",",$querystring);
$querystring = "UPDATE reviews SET ".$querystring." where `id`=:id";
$update_review->execute($_POST);

Сори за кривоватый код, пишу с планшетки

Comments

[miniven]

А о каком фильтре вы говорите?
Просто я в php полный 0. Пытаюсь разбираться с сжатые сроки

[ThunderCat]

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

[miniven]

ThunderCat: А, ну да, это я понимаю. Просто сейчас это не первостепенная задача

[Николай]

ThunderCat: Если чисто для защиты - то не нужен ни какой фильтр. Судя по всему, используется PDO - а он отлично экранирует все инъекции.
Не надо делать двойную работу и увеличивать код ^^

[ThunderCat]

Николай: вопрос не в инъекциях, а в неправильных данных, например userpic - я могу предать туда адрес чего угодно, вплоть до екзешника на левом хосте.

[miniven]

ThunderCat: А вообще такой подход, как я пытаюсь реализовать, используется?

[ThunderCat]

да, это довольно стандартный подход, но не в такой форме, то есть смысл такой, но вся фишка в отдельных объектах для работы с данными и для записи этих данных, смотрите как реализовано в более взрослых проектах, хотите писать правильно и красиво - учите принципы ооп и паттерны использования объектов. должно быть проще и изящней в итоге. Что-то типа
$object = new Object($_POST);
$object->save();

Answer 2

[Константин]

Если предположить, что у вас таблица вида

|   id   | reviews_name | reviews_body | reviews_logo |
-------------------------------------------------------
|    1   | Вася         | тут был вася | /pic.jpg     |
-------------------------------------------------------
|    2   | Петя         | Петя тут был | /pic2.jpg    |
-------------------------------------------------------

И вам надо иметь возможность сделать редактирование нескольких строк сразу, то можно вот так:

$model = [ 'id' => 'Идентификатор', 'reviews_name' => 'Имя', 'reviews_body'=> 'Текст', 'reviews_logo'=>'Аватар' ];

if (!empty($_POST)) {
   $update_review = $db->prepare(
    'UPDATE reviews SET 
      reviews_name = :name, 
      reviews_body = :body,
      reviews_logo = :userpic
    WHERE id = :id'
      );

    $id = $_POST['id'];
    $name = $_POST['reviewName'];
    $body = $_POST['reviewBody'];
    $userpic = $_POST['userPic'];

    foreach ($_POST['reviews']  as $pr) {
        $data = [];       
        foreach ($model as $field=>$field_name) {
            $data[$field] = !empty($pr[$field]) ? $pr[$field] : '';
        }
        $update_review->execute($data);
   }
    $update_review->execute([
      'name' => $name,
      'body' => $body,
      'userpic' => $userpic,
      'id' => $id
    ]);

$reviews = $db->select ... // получаем отзывы, которые нам нужны

foreach ($reviews as $r) {
   foreach ($model as $field=>$field_name) {
       ?>
       <?=$field_name?>: <input type="text" name="reviews[<?=$r['id']?>][<?=$r[$field]?>]" value="<?=$r[$field]?>" />
       <?php
   }
   ?>
   <br />
   <?php
}
?>
<input type="submit" />

Чтобы добавить новое поле - понадобится по сути, только обновить таблицу в БД и массив $data.