Как изменить данные из формы и вернуть их снова в массив?

Question

[0ldn0mad]

Есть функция, которая принимает в себя данные из $_POST и помещает их в базу

function addTask($addData){
		$db = new PDO("mysql:dbname=test; host=localhost", "root", "root");
		$sql = "INSERT INTO tasks (name, task) VALUES (:name, :task)";
		$statement = $db->prepare($sql);
		$statement->execute($addData);
	}

Хотелось бы прикрутить сюда жернова проверки и при необходимости изменения данных,

if(isset($_POST["name"])):
			$namePost = trim($_POST["name"]);
			$namePost = strip_tags($namePost);
			$namePost = htmlspecialchars($namePost,ENT_QUOTES);
			$namePost = stripslashes($namePost);
		endif;

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

Answer 1

[FanatPHP]

НЕ ХОТЕЛОСЬ БЫ.
Все эти идиотские "жернова" выкинуть на помойку.

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

function addTask($db, $name, $task){
    $sql = "INSERT INTO tasks (name, task) VALUES (?,?)";
    $statement = $db->prepare($sql);
    $statement->execute([$name, $task]);
  }

Comments

[0ldn0mad]

У нас нет идеальных людей, каждый наровит пощупать на прочность каким-то кодом из JS или весь шрифт разместить в теге Н1

[FanatPHP]

0ldn0mad, я писал не про людей, а про абсолютно бессмысленные телодвижения, которые ты собираешься совершать.

Но раз уж ты заговорил про пощупать на прочность, объясни-ка мне, от какого шрифта защищает функция stripslashes?
Ну и trim заодно

[0ldn0mad]

FanatPHP, на некоторых серверах автоматически добавляются обратные слеши к кавычкам вот мы и вырезаем их. Люди любят копипастить.

[0ldn0mad]

И да, идиотская "бессмысленная" обработка данных от идиотских людей - все относительно.

[FanatPHP]

0ldn0mad, на какаих серверах? Покажи хоть один.

А если я хочу запостить слеш? Ты его тоже вырезать будешь?
Вот допустим, я хочу написать вопрос, "мне сервер добавляет слеши вот так \'\'. Памагитя низнаю што делать". Во что превратился бы этот вопрос, если бы тостер использовал твою гениальную функцию?

[0ldn0mad]

FanatPHP, В рамках того ТЗ, который реализуется, слеши не предусмотрены.
Я всегда поддерживал правило - что не разрешено, то запрещено. На собственном опыте уже убедился что излишняя вольность приводит к печальным последствиям.

[FanatPHP]

Смотри ты какие мы слова знаем! "ТЗ", "опыт". А слово "мозги" и "думать своей головой" когда-нибудь слышал?
Не юли. Тебе сказали уже что все эти телодвижения - как мертвому припарки, папуасские пляски с бубном.

htmlspecialchars делай при выводе.
Для входящих данных по желанию можешь сделать trim.
Остальное на помойку.

[FanatPHP]

Не, ну какая красава.
Нашел где-то на помойке функцию, прилепил в свой код, и пытается еще про какое-то ТЗ рассказывать.
А что твое ТЗ говорит про остальные элементы ASCII таблицы? прямой слеш, знак доллара, процента, решетки? backtick? перевод строки? backspace? Что - все эти символы вдруг разрешены, а досталось одному слешу?

Хорошо, допустим у тебя именно такое ТЗ которое ты только что высосал из пальца. А ты уверен ,что это именно та функция, которая тебе нужна? что она действительно удаляет слеши? А если я ее легко обойду и символ \ спокойно пройдет твою функцию - куда отправится твое ТЗ?

Answer 2

[Станислав]

Ну так с исходным массивом и работать

$_POST['name'] = trim($_POST['name']; 
....
$_POST['name'] = stripslashes($_POST['name']);

Comments

[0ldn0mad]

Я тоже так думал сделать, но данные у себя обрабатываются, а функция забирает не обработанный $_POST:

function addTask($addData){
		if(isset($_POST["name"])):
			$_POST["name"] = trim($_POST["name"]);
			$_POST["name"] = strip_tags($_POST["name"]);
			$_POST["name"] = htmlspecialchars($_POST["name"],ENT_QUOTES);
			$_POST["name"] = stripslashes($_POST["name"]);
		endif;
		if(isset($_POST["task"])):
			$_POST["task"] = trim($_POST["task"]);
			$_POST["task"] = strip_tags($_POST["task"]);
			$_POST["task"] = htmlspecialchars($_POST["task"],ENT_QUOTES);
			$_POST["task"] = stripslashes($_POST["task"]);
		endif;
		$db = new PDO("mysql:dbname=test; host=localhost", "root", "root");
		$sql = "INSERT INTO tasks (name, task) VALUES (:name, :task)";
		$statement = $db->prepare($sql);
		$statement->execute($addData);
	}

[Станислав]

Не юзайте именованные параметры. Используйте знак вопроса ?
https://www.php.net/manual/ru/pdostatement.execute.php

<?php
function preparedData(array $data): array
{
   foreach($data as $key => $value) {
     $data[$key] = trim($value);
     $data[$key] = strip_tags($value);
     $data[$key] = htmlspecialchars($value, ENT_QUOTES);
     $data[$key] = stripslashes($value);
   }

  return $data;
}

function addTask(array $data)
{
 $data = preparedData($data);
 $db = new PDO("mysql:dbname=test; host=localhost", "root", "root");
 $sql = "INSERT INTO tasks (name, task) VALUES (?, ?)";
 $statement = $db->prepare($sql);
 $statement->execute($data);
}


addTask($_POST);

П.С: Набор проверок ваших бесполезен по сути)

[FanatPHP]

Ну, что использовать - вопрос или именованный параметр - это вопрос вкуса, никаких рекомендаций тут нет.
Тем более что с вопросами этот код не будет работать, а будет только с именованными как раз.

А вот что действительно не рекомендуется - это использовать массив вместо отдельных переменных в параметрах функции.