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

Question

[Виталий Слюсар]

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

так выдает ошибку

<?php
	$con = mysqli_connect("localhost","root","","mybd");
	mysqli_set_charset($con,"utf8");

	if (!$con) {
	    die("Connection failed: " . mysqli_connect_error());
	}

function selected($db_name){
	$query = "select * from $db_name";
	$result = mysqli_query($con, $query);
	$row = mysqli_fetch_array($result);
	do
	{
		echo $row['name']."<br>";
	}while($row = mysqli_fetch_array($result));
}

selected('users');

а так все работает

$query = "select * from users";
	$result = mysqli_query($con, $query);
	$row = mysqli_fetch_array($result);
	do
	{
		echo $row['name']."<br>";
	}while($row = mysqli_fetch_array($result));

ошибка

Warning: mysqli_query() expects parameter 1 to be mysqli, null given

Можно ли вообще помещать запросы в функцию и как это делать правильно?
Заранее благодарю.

Answer 1

[Alex Safonov]

В вашей функции
$result = mysqli_query($con, $query);

$con - переменная которая не попадает в scope. Функции она не известна.

Попробуйте

function selected($db_name){
  global $con;
...
}

А в целом, должен заметить что так уже давно никто не пишет. Оберните $con хотя бы в синглтон. Чтобы быть уверенным что никто эту переменную не перепишет в коде рантайм.

Comments

[Виталий Слюсар]

Спасибо! Действительно простое решение. Аж стыдно, что на области видимости затупил))

[Виталий Слюсар]

А как сейчас принято писать?

[Alex Safonov]

По-разному.
Используют ORM, готовые обертки.
Я думаю, что для самостоятельного написания наиболее понятно решение с инкапсуляцией в синглтон.
https://gist.github.com/jonashansen229/4534794

Есть смысл вынести настройки подключения в константы, в конфигурационный файл.

[Алексей С.]

Виталий Слюсар: Решение то простое, но так писать программы нельзя )))

[Alex Safonov]

Алексей С.: Можно. Чтобы потом понять, зачем нужны Best Practices. Не поняв что плохо, нельзя понять что хорошо.

Answer 2

[Макс]

$result = mysqli_query($con, $query); - в функции используются локальные переменные. $con у вас появляется ДО вызова функции.
а так - без проблем

Answer 3

[Алексей С.]

ваши переменные внутри функции не видны
function selected($con,$db_name){
Ну и соответственно: selected($con,'users');