Как исправить эту ошибку?

Question

[MC Додик]

Шалом! Как исправить эту ошибку? Fatal error: Call to a member function query() on a non-object in Z:\home\tronmoney.online\www\engine\functions.php on line 12
Вот код

function fakePayout ($user, $amount, $time) {
   $referer = 'P'.rand(4, 6).rand(1, 0).rand(000000, 999999);
   $refererIncome = $amount/100*150;
   $mysqli->query("INSERT INTO `deposits` (`user`, `amount`, `time`, `type`, `fake`) VALUES ('$user', '$amount', '$time', '1', '1')");
   $mysqli->query("INSERT INTO `deposits` (`user`, `amount`, `time`, `type`, `fake`) VALUES ('$referer', '$refererIncome', '$time', '2', '1')");
   return true;
}

вот подключение к базе

class DataBase
{
   public $template = 'default';
   public $db_host;
   public $db_user;
   public $db_passord;
   public $db_name;

   public $mysql_port;
   public $mysql_socket;

   public function __construct($db_host, $db_user, $db_passord, $db_name, $mysql_port, $mysql_socket)
   {
      $this->db_host = $db_host;
      $this->db_user = $db_user;
      $this->db_passord = $db_passord;
      $this->db_name = $db_name;

      $this->mysql_socket = $mysql_socket;
      $this->mysql_port = $mysql_port;

      $mysqli = @new mysqli($this->db_host, $this->db_user, $this->db_passord, $this->db_name, $this->mysql_port, $this->mysql_socket);
      if (mysqli_connect_error()) {
         echo 'Подключение к серверу Mysql невозможно. Код ошибки: '.mysqli_connect_error();
         exit;
      }
   }
}

подключаю так

include_once(dirname(__FILE__).'/engine/classes/Config.class.php');
$mysqli = new DataBase("localhost", "root", "", "test1488", NULL, NULL);

Comments

[Arman]

function fakePayout ($mysqli, $user, $amount, $time) {

[Евгений Самсонов]

1. Где находится метод fakePayout? Есть в области видимости метода переменная $mysqli?
2. В DataBase нет метода query

Answer 1

[FanatPHP]

1. Выкинуть класс DataBase на помойку, он здесь явно лишний.
2. создать файл engine/что-нибудь/mysqli.php и записать в него код отсюда
3. Добавить а параметрам функции $mysqli

function fakePayout ($mysqli, $user, $amount, $time) {

4. Переписать код запросов на подготовленные выражения

function fakePayout ($mysqli, $user, $amount, $time) {
   $referer = 'P'.rand(4, 6).rand(1, 0).rand(000000, 999999);
   $refererIncome = $amount/100*150;
   $stmt = $mysqli->prepare("INSERT INTO `deposits` (`user`, `amount`, `time`, `type`, `fake`) VALUES (?,?,?, '1', '1')");
    $stmt->bind_param("sss", $user, $amount, $time);
    $stmt->execute();
    $stmt = $mysqli->prepare("INSERT INTO `deposits` (`user`, `amount`, `time`, `type`, `fake`) VALUES (?,?,?, '2', '1')");
    $stmt->bind_param("sss", $referer, $refererIncome, $time);
    $stmt->execute();
}

5. Вызывать как обычно

include_once(dirname(__FILE__).'/engine/что-нибудь/mysqli.php');
fakePayout ($mysqli,$user, $amount, $time);

Answer 2

[Игорь Воротнёв]

Проблема в области видимости - переменная $mysqli недоступна в функции fakePayout(). Решается одним из способов:

1. Явной передачей переменной внутрь функции function fakePayout($mysqli, ...) {
2. Корректным доступом к данной переменной, поместив ее в свойство класса, содержащего функцию fakePayout():

class SomeClass 
{
    private $mysqli;

    public function __construct(DataBase $mysqli)
    {
        $this->mysqli = $mysqli;
    }

    public function fakePayout($user, $amount, $time)
    {
        // Теперь можно использовать
        $this->mysqli->query()
    }
}

3. Вариации #2 с dependency injection контейнерами, фасадами и прочее.

ЗЫ: Можно еще внутри самой функции создавать соединение, или хранить его в глобальной переменной / синглтоне - но это уже совсем дичь.

Comments

[FanatPHP]

И в тоге аффтар получит тыкву

[Игорь Воротнёв]

FanatPHP, Почему? (ответ "потому что там говнокод и все надо переписать" - не катит, мы это как бы понимаем, задачу надо решить минимальной кровью)

[Игорь Воротнёв]

FanatPHP, впрочем, почитал код класса DataBase и пожалуй соглашусь :) Тут после решения вопроса с областью видимости другая ошибка вылезет, потом еще, и так пока в итоге все не отрефакторить.

[MC Додик]

Игорь Воротнёв, вот какая теперь ошибка
Fatal error: Call to undefined method DataBase::query() in Z:\home\tronmoney.online\www\engine\functions.php on line 18

//file functions.php
function showDeposits ($mysqli) {
   $sql = $mysqli->query("SELECT * FROM `deposits`");
   $result = $sql->fetch;
   return $result;
}

//file index.php
include_once(dirname(__FILE__).'/engine/classes/Config.class.php');
$mysqli = new DataBase("localhost", "root", "", "test1488");

//file main.php
$depositsArray = showDeposits($mysqli);

[Игорь Воротнёв]

MC Додик,

Call to undefined method DataBase::query()

В классе DataBase метода query() НЕТУ.

FanatPHP ты был прав :)

[Arris]

или хранить его в глобальной переменной / синглтоне - но это уже совсем дичь.

Чё это дичь то? Популярные фреймворки этим без зазрения совести пользуются, а как вне фреймворка - сразу "дичь".

[FanatPHP]

Arris, у них не синглтон, у них - фааабрика

[Игорь Воротнёв]

Arris, см "3. Вариации #2 с dependency injection контейнерами, фасадами и прочее." Централизовано - не дичь, даже если сам класс БД синглтон. Делать как тот же WordPress в виде глобальной переменной $wpdb - дичь.

[Arris]

Игорь Воротнёв, короче говоря, если я в рамках своего приложения заведу синглтон DB - меня никто анафеме не предаст, лишь бы не писал
`global $DBCONNECTION;` ? :)

P.S. К слову, в исходниках форума по ларавелю глобальными переменными очень даже балуются, хотя казалось бы...

[FanatPHP]

Arris, предаст.
По сути это одно и то же

[Игорь Воротнёв]

Arris, пуристы предадут, обязательно. Синглтон в чистом виде это та же глобальная переменная. Используйте DI.

[Arris]

Игорь Воротнёв, а как "кошерно" хранить конфиги проекта? :)

[Игорь Воротнёв]

Arris, .env, .yaml, .json, .ini и/или банальные массивы:

<?php
return [
    'key' => 'value',
];

[Arris]

Игорь Воротнёв, да я не про это. Я про то, как к ним доступ из кода получать. Как хранить - понятно, мильон решений.

[Игорь Воротнёв]

Arris, так писали ж выше, для этих целей используется container

[Arris]

Игорь Воротнёв, что-то пока то что я вижу за этой идеей - это ничем не отличается от жутчайшего легаси, которое я сейчас переписываю. В этом легаси тоже используется некая эмуляция DI, только в каждый конструктор передается $this

но, возможно, я чего-то не понимаю в этих ваших "современных" технологиях?

[FanatPHP]

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

[Игорь Воротнёв]

Arris, вы задаете какие-то пространные вопросы из разряда абстрактной научной теории, сложно сформулировать вам более-менее четкий ответ, потому что четкого вопроса нету. В целом FanatPHP дело говорит.

[Arris]

FanatPHP, поясню, что меня напрягает в этой идеологии.

Моделька вывода статьи берет соединение с БД (по идее оно должно быть установлено прямо сейчас, в модели, так?), дергает статью и суёт её куда-то на обработку. Потом к статье нужно выдернуть комментарии. Этим занимается другая моделька. Мы её инстанциируем, передаем ей параметры и... она дёргает из базы комментарии к статье.

а) она снова создает соединение с БД - но это неэкономично
б) она получает это соединение (инстанс PDO скажем) от предка. Но значит и предок получает этот инстанс откуда-то. Откуда?

А если он получает его от модели самого верхнего уровня - то какая разница, передаем мы DBConnection через параметр конструктора или каким-то другим способом (через синглтон, например или глабальную переменную)

Или (в нормальных фреймворках) всё работает совсем иначе?

P.S. Вообще конечно это тема отдельной дискуссии, но у меня нет желания создавать вопрос на тостере с этой темой)

[Игорь Воротнёв]

Arris,

она снова создает соединение с БД - но это неэкономично

1. Соединение с БД создается один раз на весь lifecycle, если оно нужно и не было создано ранее. Соединение мы спрашиваем у контейнера, а он уже дирижирует дальше.
2. Комментарии к статье лучше получать сразу вместе со статьей, если они нам будут нужны и спрашиваем мы всего 1 статью.

Или (в нормальных фреймворках) всё работает совсем иначе?

Возьмите Symfony, Zend и Laravel и посмотрите как у них это реализовано. Можете пошагово xdebug'ом пройтись от начала загрузки фреймворка до запроса в БД.

[FanatPHP]

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

А где происходит соединение - нам, по большому счету, неважно. Главное, чтобы это не происходило "прямо сейчас, в модели" - это будет действительно треш.

[Arris]

FanatPHP, я примерно понял логику.

Возложите инициализацию нужных вам инстансов на механизм зависимостей - он справится без вас так, как нужно тогда когда нужно, так?

[Arris]

Игорь Воротнёв, ну, пример с комментариями к статье и статьями возможно не самый удачный.

Но все равно, спасибо за ответы.

[FanatPHP]

Arris, да.
если не считать того что зависимости все равно должны быть описаны программистом, а вот их создание - это дело фреймворка

Answer 3

[Алексей]

Внутри функции fakePayout переменная $mysqli не определена. Передавайте ее как еще один параметр к функции.

function fakePayout ($user, $amount, $time, $mysqli) {
.....
}

Comments

[MC Додик]

Вот какая ошибка теперь Warning: Missing argument 4 for fakePayout(), called in Z:\home\tronmoney.online\www\pattern\default\main.php on line 110 and defined in Z:\home\tronmoney.online\www\index.php on line 15

Fatal error: Call to a member function query() on a non-object in Z:\home\tronmoney.online\www\index.php on line 18
код

include_once(dirname(__FILE__).'/engine/classes/Config.class.php');
$mysqli = new DataBase("localhost", "root", "", "test1488", NULL, NULL);

function fakePayout ($mysqli, $user, $amount, $time) {
   $referer = 'P'.rand(4, 6).rand(1, 0).rand(000000, 999999);
   $refererIncome = $amount/100*150;
   $mysqli->query("INSERT INTO `deposits` (`user`, `amount`, `time`, `type`, `fake`) VALUES ('$user', '$amount', '$time', '1', '1')");
   $mysqli->query("INSERT INTO `deposits` (`user`, `amount`, `time`, `type`, `fake`) VALUES ('$referer', '$refererIncome', '$time', '2', '1')");
   return true;
}

[Игорь Воротнёв]

MC Додик, там где вы вызываете fakePayout() надо тоже $mysql добавить

[MC Додик]

Игорь Воротнёв, так я её не где не вызываю, какого хера функция срабатывает?

[Игорь Воротнёв]

MC Додик, Серьезно? PHP с вами не согласен и говорит вам прямым текстом - fakePayout(), called in Z:\home\tronmoney.online\www\pattern\default\main.php on line 110. Вы вызываете эту функцию в файле main.php в строке 110.