Почему ошибка Warning: Undefined variable $db in?

Question

[Александр Андропов]

Добрый день. Есть код:

<?php
require $_SERVER['DOCUMENT_ROOT'] . '/inc/connect.php';

function CheckDestiny ($variable,$type) {
    if ($type == "ONE") {
        switch ($variable) {
        case $variable > 100000 and $variable < 249999:
            echo "QUERY 1";
            $add_destiny = mysqli_query($db, "UPDATE `data` SET `column` = '$variable; WHERE `symbol` = '$db_symbol[$t]'");            
            break;
        case $variable > 250000 and $variable < 499999:   
            echo "QUERY 2";
            break;                     
        }
    }  
}

    $search_destiny = mysqli_query($db, "SELECT * FROM `items` ORDER BY `id` LIMIT 0,35");
	
    while ($select_destiny = mysqli_fetch_assoc($search_destiny)) 
	{ $db_ticker[] = $select_destiny['name']; 
	  $db_symbol[] = $select_destiny['sym'];
	  $db_price[] = $select_destiny['price'];
	}

    for ($t = 0; $t < count($db_symbol); $t++) {
        echo $t . ". - " . $db_symbol[$t] . " == Цена <b>" . $db_price[$t] . "</b> <br>";
		
		// ....... etc
		
        CheckDestiny($sum_usdt,"ONE");
?>

В итоге я получаю Warning: Undefined variable $db in /home/users/j/j82769333/domains/site.ru/main.php on line 9

Не пойму почему ошибка. Если вне функции выполнять запрос в БД - то всё нормально.

connect.php выглядит так:

<?php
$driver = "mysql";
$host = "localhost";
$db_name = "j82769333";
$db_user = "j82769333";
$db_pass = "passwd";
$charset = "utf8";
$options = [PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION];

$db = mysqli_connect($host, $db_name, $db_pass, $db_name);
?>

Подскажите пожалуйста, почему выскакивает ошибка о том, что нет переменной $db когда пытаюсь выполнить запрос в БД в теле функции ?

Comments

[Алексей Уколов]

почему выскакивает ошибка о том, что нет переменной $db когда пытаюсь выполнить запрос в БД в теле функции ?

Ошибка выскакивает потому, что в теле функции нет переменной $db.

[Сергей delphinpro]

https://yandex.ru/search/?clid=1882628&text=php+об...

Answer 1

[FanatPHP]

Самый простой и правильный вариант

function CheckDestiny ($variable, $type, $db) {

Answer 2

[Сергей delphinpro]

Учитывая, что ваш код и так не ахти, можете сделать так

connect.php

$db = mysqli_connect($host, $db_name, $db_pass, $db_name);

if (!function_exists('db')) {
  function db () { global $db; return $db; }
}

или так

if (!function_exists('db')) {
  function db () {
    static $db;
    if (!$db) $db = mysqli_connect($host, $db_name, $db_pass, $db_name);
    return $db;
  }
}

И потом использовать функцию, она всегда будет доступна.

$add_destiny = mysqli_query(db(), "UPDATE `data` SET `column` = '$variable; WHERE `symbol` = '$db_symbol[$t]'");

Еще вариант - пробросить переменную в функцию

function CheckDestiny ($variable,$type) use ($db) {
 // ...
}

PS. Знатоков прошу не умничать по поводу предложенных вариантов.
Главное, что они заработают у автора =))

Comments

[Александр Андропов]

Спасибо. Какие свежие курсы порекомендуйте для изучения PHP ?

[FanatPHP]

я думаю, проблема с этим ответом не в том что он заработает у автора, а в том что это говнокод

[Александр Андропов]

FanatPHP, Покажите пожалуйста как правильно и почему.

[FanatPHP]

Александр Андропов, вы это кому?

[Сергей delphinpro]

FanatPHP,

это говнокод

Спорить не буду =)
но я бы назвал это костылём, чтобы сделать переменную доступной везде.

[Александр Андропов]

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

В итоге, что я вижу в каждом 2-ом вопросе на этом "профессиональном сайте": Вы все говнокодеры, мы профессионалы, вы просто немощные людишки, которые недостойны нажимать на кнопки, - а мы боги программирования. Ну и что толку-то? Я понял, что здесь часть определенных личностей с зашкаливающим ЧСВ, но какую пользу Вы несёте просиживая здесь своё время ?

Если для Вас этот вопрос кажется глупым написанным говнокодером то и не надо отвечать ничего. Более адекватный человек ответит без данных выпадов.
Однажды, когда вы придете в неизвестную Вам тему, вы получите точно такое же к себе отношение и скорее всего, будете максимально удивлены, почему так несправедливо с вами поступают.

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

[FanatPHP]

Непонятно, почему вы принимаете это так близко к сердцу.
Я комментировал не ваш вопрос, а конкретный ответ. Причем его последнюю строчку.

Выше вы спросили, как правильно, при том что я ещё вчера написал ответ.
Я не понимаю, ни ваших претензий, ни ваших вопросов.

Если у вас есть конкретные вопросы, то можете задать их, только желательно без ненужной беллетристики.

[Александр Андропов]

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

[FanatPHP]

Александр Андропов, да, если добавить переменную $db, как аргумент функции, то конкретно это место перестанет быть говнокодом. В работе функции не будет магии и берущихся из неоткуда переменных, всё наглядно.

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

В целом же, чтобы исправить код целиком, надо потратить довольно много усилий.
Начать можно отсюда
https://phpdelusions.net/mysqli/mysqli_connect
https://phpdelusions.net/mysqli_examples

[Александр Андропов]

Спасибо большое! Буду изучать.