Mysqli_query() expects parameter 1 to be mysqli, null given in

@Nemovok · Регистрация: 30.10.2015

Author24 — интернет-сервис помощи студентам

Здравствуйте. Такая проблема. Есть функция connect и из нее нужно передать переменную $connect в функцию welcome, в mysqli_query. Получаю ошибку "Undefined variable: connect in" и "mysqli_query() expects parameter 1 to be mysqli, null given in...". На другом форуме сказали что можно как-то через return. Но я не понимаю как(

PHP

function connect($host, $user, $password, $dbname)
    {
        $connect = mysqli_connect("$host", "$user", "$password", "$dbname");
        if (!$connect) 
        {
            $error = "Нет соединения с базой данных";
            echo "$error";
        }
    }
 
    function welcome()
    {
        if (!first_check($string = $_POST['login']) AND !second_check($string = $_POST['login'])) 
        {
            if (!connect($host = "localhost", $user = "root", $password = "root", $dbname = "admin")) 
            {
                $input_login = $_POST['login'];
                $sql = ("SELECT login, password FROM users WHERE login = '$input_login' ");
                $query = mysqli_query($connect, $sql);
            }
        }
    }

crautcher · 30.10.2015, 17:50

PHP

function connect($host, $user, $password, $dbname)
    {
        global $connect;
        $connect = mysqli_connect("$host", "$user", "$password", "$dbname");
        if (!$connect) 
        {
            $error = "Нет соединения с базой данных";
            echo "$error";
        }
    }
 
    function welcome()
    {
        global $connect;
        if (!first_check($string = $_POST['login']) AND !second_check($string = $_POST['login'])) 
        {
            if (!connect($host = "localhost", $user = "root", $password = "root", $dbname = "admin")) 
            {
                $input_login = $_POST['login'];
                $sql = ("SELECT login, password FROM users WHERE login = '$input_login' ");
                $query = mysqli_query($connect, $sql);
            }
        }
    }

@Vadim_Lasso · 30.10.2015, 17:57

Сообщение от crautcher

global $connect;

Следует по возможности избегать создание собственных глобальных переменных, т.к. засорение глобального пространства имён - это "дурной тон".

Nemovok, вот как вариант:

PHP

<?php
function connect($host, $user, $password, $dbname)
{
    if (!$connect = mysqli_connect("$host", "$user", "$password", "$dbname")) 
    {
        $error = 'Нет соединения с базой данных';
        echo $error;
        return false;
    } else
        return $connect;
}
 
function welcome()
{
    if (!first_check($string = $_POST['login']) AND !second_check($string = $_POST['login'])) 
    {
        if (!$connect = connect($host = "localhost", $user = "root", $password = "root", $dbname = "admin")) 
        {
            $input_login = $_POST['login'];
            $sql = ("SELECT login, password FROM users WHERE login = '$input_login' ");
            $query = mysqli_query($connect, $sql);
        }
    }
}
?>

@Nemovok · 30.10.2015, 17:58 **[ТС]**

Благодарю

crautcher · 30.10.2015, 17:59

welcome()
welcome()
welcome()
даст 3 коннекта

PHP

function connect($host, $user, $password, $dbname)
{
    static  $connect=null;
    if ( $connect !== null ) return  $connect;
    $connect = mysqli_connect("$host", "$user", "$password", "$dbname");
    if (!$connect) 
    {
        $error = "Нет соединения с базой данных";
        echo "$error";
        return false;
    } else
        return $connect;
}

@Raisin Zn · 30.10.2015, 18:07

Сообщение от Vadim_Lasso

Следует по возможности избегать создание собственных глобальных переменных, т.к. засорение глобального пространства имён - это "дурной тон".

А чем выражается этот "дурной тон"? Чем так плох global?

@Jewbacabra · 30.10.2015, 18:21

Сообщение от Raisin Zn

Чем так плох global?

Тяжело тестировать, искать ошибки и т.д.

@Vadim_Lasso · 30.10.2015, 18:22

Сообщение от Raisin Zn

Чем так плох global?

Есть немало причин, описанных на просторах интернета, по которым не стоит использовать этот оператор. Вот некоторые из них:
1. Сложно понять, где инициализируется переменная, доступ к которой осуществляется посредством global. Соответственно это ухудшает читабельность кода.
2. Глобальная переменная может быть переопределена в дебрях скрипта, и найти это место очень сложно. А значит велик риск сложно диагностируемых логических ошибок.
3. Код становится сильно связанным, выполнение одной части будет зависеть от другой, где глобальная переменная определена.
(с) гугл

@Raisin Zn · 30.10.2015, 18:57

Вас понял, спасибо!

@Nemovok · 30.10.2015, 19:51 **[ТС]**

Остановился на таком варианте. Кому интересно)))

PHP

       function connect ( $host = ' ' , $user = ' ' , $password = ' ' , $dbname = '' )
    {
        $connect = @mysqli_connect ( $host , $user , $password , $dbname );
        if ( !$connect ) 
        {
            $error = "Нет соединения с базой данных";
            echo "$error";
            return false;
        }
        return $connect;
    }
 
 
    function welcome (  )
    {
        if ( !first_check ( $s = $_POST ['login'] ) AND !second_check ( $s = $_POST ['login'] ) ) 
        {
            $connect = connect ( "localhost" , "root" , "root" , "admin" );
            if ( !$connect ) return;
            $i_login = $_POST ['login'];
            $i_password = $_POST ['password'];
            $sql = ( "SELECT login , password FROM users WHERE login = '$i_login' " );
            $query = mysqli_query ( $connect , $sql );
            $data = mysqli_fetch_assoc ( $query );
        }
        if ($data ['login'] != $i_login OR $data ['password'] != $i_password) 
        {
            $error = "Неверный логин или пароль";
            echo "$error";
            exit (  );
        }
    }

@Raisin Zn · 30.10.2015, 20:00

Сообщение от Nemovok

$connect = @mysqli_connect ( $host , $user , $password , $dbname );

Не думаю, что подавлять ошибку хорошая идея. Лучше так:

PHP

1	$connect = mysqli_connect ( $host , $user , $password , $dbname );

тем более, что у вас идет проверка:

PHP

1
2
3

        if ( !$connect ) 
        {
            $error = "Нет соединения с базой данных";

ИМХО, конечно...

@Nemovok 296 / 125 / 106 Регистрация: 30.10.2015 Сообщений: 690
	30.10.2015, 17:58 [ТС]	4
	Благодарю 0

@Raisin Zn 101 / 100 / 51 Регистрация: 19.04.2011 Сообщений: 958
	30.10.2015, 18:07	6
	Сообщение от Vadim_Lasso Следует по возможности избегать создание собственных глобальных переменных, т.к. засорение глобального пространства имён - это "дурной тон". А чем выражается этот "дурной тон"? Чем так плох global? 0

@Jewbacabra 4925 / 3920 / 1620 Регистрация: 24.04.2014 Сообщений: 11,441
	30.10.2015, 18:21	7
	Сообщение от Raisin Zn Чем так плох global? Тяжело тестировать, искать ошибки и т.д. 1

@Vadim_Lasso 433 / 352 / 259 Регистрация: 29.11.2011 Сообщений: 628
	30.10.2015, 18:22	8
	Сообщение от Raisin Zn Чем так плох global? Есть немало причин, описанных на просторах интернета, по которым не стоит использовать этот оператор. Вот некоторые из них: 1. Сложно понять, где инициализируется переменная, доступ к которой осуществляется посредством global. Соответственно это ухудшает читабельность кода. 2. Глобальная переменная может быть переопределена в дебрях скрипта, и найти это место очень сложно. А значит велик риск сложно диагностируемых логических ошибок. 3. Код становится сильно связанным, выполнение одной части будет зависеть от другой, где глобальная переменная определена. (с) гугл 1

@Raisin Zn 101 / 100 / 51 Регистрация: 19.04.2011 Сообщений: 958
	30.10.2015, 18:57	9
	Вас понял, спасибо! 0

	30.10.2015, 20:00