Как передать параметры в функцию PHP?

Question

[Epic18]

Имеется функция

function sel($pole, $table){
	$result = mysql_query ("SELECT $pole FROM $table", $db);
		$myrow = mysql_fetch_array ($result);
			do{
				$number = $myrow[$pole];
				printf("<option>%s</option>",$number);
				}
			while($myrow = mysql_fetch_array ($result));
	}

Если попробовать её вызвать

$p = "pole";
$t = "table";
/* хоть так */
sel($p, $t);
/* хоть так */
sel("pole", "table");

То функция не работает

Но если сделать его не функцией

$pole = "pole";
$table = "table";

	$result = mysql_query ("SELECT $pole FROM $table", $db);
		$myrow = mysql_fetch_array ($result);
			do{
				$number = $myrow[$pole];
				printf("<option>%s</option>",$number);
				}
			while($myrow = mysql_fetch_array ($result));

То все работает. Но мне нужно сделать функцией, тут какая то ошибка возникает при передаче параметров в функцию, кто подскажет в чем дело?

PS Сама функция вызывается потому как появляется пустая строка, но вот параметры функция не видит

Comments

[Максим Федоров]

function sel($pole, $table){
  $result = mysql_query ("SELECT $pole FROM $table", $db);
...

Вам нужно обратить внимание, что у функций своя видимость, соответственно переменные снаружи не видны, но в коде у вас используется переменная $db
Читать: Область видимости переменной ¶

Нужно:
- Либо подойти архитектруно с другой стороны — через ООП парадигму, когда все функции класса (методы) замкнуты на состояние обьекта (его переменные), описанного классом и могут с ними работать
- Либо написать функционально — передать в функцию запроса к БД (принимающую переменную с БД и функцию запроса) и передать в нее другую функцию с подгтовлением запросом

[Epic18]

Максим Федоров, кажется вы все усложняете

..
function sel($pole, $table, $db){
		$result = mysql_query ("SELECT ".mysql_real_escape_string($pole)." FROM ".mysql_real_escape_string($table), $db);
..

[Максим Федоров]

Epic18, как вам угодно

[Epic18]

Максим Федоров, наконец то можете прямо ответить "что не так" с кодом?

Answer 1

[Artem Ivanov]

$db
коннекта к базе нет
объявляйте переменную глобальной

Comments

[Adamos]

Будьте любезны, приподнимите клавиатуру над головой и несколько раз резко опустите, приговаривая "и так будет с каждым, кто советует использовать в пыхе глобальные переменные!!!". Спасибо.

[Антон Р.]

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

[Artem Ivanov]

Adamos, кыш

[Epic18]

Спасибо, решил проблему

..
$result = mysql_query ("SELECT $pole FROM $table", $GLOBALS['db']);
..

[FanatPHP]

Смотри ты как папуасы раздухарились

[Rsa97]

Adamos, Учитывая, что в этом коде надо менять всё, начиная от использования библиотеки mysql_ и заканчивая do...while, вместо нормального while, глобальная переменная будет наименьшей из проблем.

[AUser0]

Epic18, во-первых защищайтесь от SQL injection, иначе это станет вашей боооо-о-ольшой проблемой:

$result = mysql_query ("SELECT ".mysql_real_escape_string($pole, $GLOBAL['db'])." FROM ".mysql_real_escape_string($table, $GLOBAL['db']), $GLOBAL['db']);

Во-вторых, если запрос к базе данных не вернёт каких-либо записей, то обращение к переменной $myrow[$pole] вызовет ошибку отсутствия данных. Использование

while ($myrow = mysql_fetch_array ($result)) { printf("<option>%s</option>",$myrow[$pole]); }

избавит от такой проблемы.

[FanatPHP]

AUser0, с каких это пор mysql_real_escape_string стала защитой от SQL инъекций?

[Koksmos]

ответ уже дали, но лучше изучи и используй PDO

[Epic18]

AUser0,
Я передал переменную отвечающее за подключение к БД как параметр это раз,
потом убрал его в функции mysql_real_escape_string, итого вот что получилось:

..
function sel($pole, $table, $db){
		$result = mysql_query ("SELECT ".mysql_real_escape_string($pole)." FROM ".mysql_real_escape_string($table), $db);
...

А зачем вы его добавили в функцию mysql_real_escape_string не понятно
А еще я прочитал тут (https://www.php.net/manual/ru/function.mysql-real-...
что это функция устарела