Можно ли один раз использовать глобальную переменную?

Question

[Max Ba]

Всем доброго дня. С недавнего времени перевел сайты на PDO. И всёб ничего, но внутри некоторых функций прежний объект PDO не виден. Вопрос: сильно ли я накосячу, добавив перед $pdo переменную global? На всём проекте это будет одна всего лишь глобальная переменная. Программируем втроем. Договориться сможем, чтоб не переназначили случайно. Как вариант можно и синглтон использовать, но по моему это те же яица. Вместо куска кода, одно слово.

Это был первый вопрос. И заодно на второй ответьте, если знаете.
В данную конструкцию не пролезет никак инъекция?

$res = DB::prepare("UPDATE `users` SET `email` = :email WHERE `id` = :id");

$data = array(
  'email' => $_POST['email'],
  'id' => $_POST['id']
);

$res->execute($data);

Comments

[Максим Тимофеев]

Насколько prepare и execute обеспечивают безопасность sql-запроса от инъекции?

[Max Ba]

Спасибо за ссылку.

Answer 1

[FanatPHP]

Странно, в тексте упоминается объект $pdo, а в коде статический вызов, у которого никаких проблем с видимостью в принципе быть не может.

Но есть отвечать буквально, то

1. Если код процедурный, то global для $pdo или статический вызов через DB:: - это нормально. Если код объектный, то передавать $pdo как параметр в конструктор, присваивать локальной переменной класса и потом уже использовать только ее.
2. В данную конструкцию никакая инъекция не пролезет

Писать, кстати, можно короче, чтобы не канителиться.

$data = array(
  'email' => $_POST['email'],
  'id' => $_POST['id']
);
$res = DB::prepare("UPDATE `users` SET `email` = :email WHERE `id` = :id")->execute($data);

Comments

[Max Ba]

Статический вызов это у меня Синглтон стоит. Я решал его заменить на global, тк не вижу разницы. По этому вопрос то и создал. Спасибо за коммент. Все понял!

Answer 2

[Axian Ltd.]

1. Глобальными переменными не гнушаются и гранды. Я бы только подумал чтобы вынести все в отдельный файл.
   
2. А то как же ;-) Кто фильтровать ввод будет?
   

Comments

[FanatPHP]

от инъкций ничего "фильтровать" не надо

[Axian Ltd.]

угу, а теперь рассмотрим что я id сделаю таким '104 or 1=1'?

[FanatPHP]

А теперь сядем за парту в первый класс и откроем для себя PDO

Answer 3

[Zubastik_1]

Я бы искал причину если проект планируете развивать и улучшать программу а не просто поддерживать на плаву. Где то по видимому косяк.

Comments

[Max Ba]

Да смысл его разворачивать? есть рабочий проект. Дали задание заменить mysql на PDO, что я и сделал. Проект переписывать не хочу, тк нет нужды. Процедурный подход :)