Как уловить ошибку о которой умалчивает PHP интерпритатор?

Question

[Megas]

Работая над чужим кодом обнаружил баг примерно такого содержания:

function foo()
{
	$bar->test['a']['b'] = 5;
}

Очевидно, что переменная $bar тут не определена и соответственно обращение к ее свойству тут некорректно... но "гибкий" PHP просто создает пустой класс с нужной переменной и нужной вложенностью массива. Даже нотификации не выдает!

object(stdClass)[67]
  public 'test' => 
    array (size=1)
      'a' => 
        array (size=1)
          'b' => int 5

А сколько еще таких багов в коде? Что нужно сделать, чтоб интерпритатор ругался на такое? Директиву какую то прописать? Или в php.ini чтото изменить? На сервере стоит php 7.3

Comments

[Дмитрий]

ну как минимум error_reporting -1 и display_errors 1

[DevMan]

Дмитрий, не поможет.

Answer 1

[xmoonlight]

1. В начале каждого PHP-скрипта (глобально - не работает!): declare(strict_types=1);
2. Читаем тут и ставим свою перегрузку:

public function __get(string $var) : ?string {
    if (property_exists($this, $var)) {
        return $this->$var;
    }
    
    throw new Exception('property doesn’t exist');
  }

  public function __set(string $var, string $value) {
    if (property_exists($this, $var)) {
        $this->$var = $value;
        return;
    }
    
    throw new Exception('property ' . $var . ' doesn’t exist');
  }

3. В php.ini добавить (дополнить):

display_errors = On
error_reporting = E_ALL | E_STRICT

4. В начало PHP-скрипта добавить:

error_reporting(E_ALL | E_STRICT);
error_reporting(-1);
ini_set('error_reporting', E_ALL | E_STRICT);

5.В .htaccess:

php_flag display_startup_errors on
php_flag display_errors on

6. "Контрольный в голову": https://xdebug.org/docs/basic

7. PHPStan - проверка PHP-кода на ошибки без запуска.

PS: ещё есть также prepend/append скриптов в php.ini для установки любых значений (исполняется php-код внутри них) до/после загрузки любых скриптов, соответственно.

Comments

[Megas]

Зачем мне __set (и куда мне его засунуть), если суть именно в том, что переменная $bar не определена?

[xmoonlight]

Megas, выше поставьте (глобально)

__set() is run when writing data to inaccessible (protected or private) or non-existing properties.

[Megas]

xmoonlight, Да, только это работает для инстанса класса. А тут у нас просто неопределенная переменная.

[xmoonlight]

Megas, дополнил.

[Megas]

xmoonlight, Неа, все еще ничего. Напоминаю, php 7.3
Как уже написали в другом комментарии в php 7.4 да появляется ворнинг.

[xmoonlight]

Megas, если все пункты не дают нужного эффекта - проблема не в PHP.

[Megas]

xmoonlight, Вы сами то попробуйте, что рекомендуете...

[xmoonlight]

Megas, мне не нужно пробовать, т.к. если не работают первые 5 пунктов, то 6-й: работает 100%.
PS: ещё есть также prepend/append скриптов в php.ini для установки любых значений (исполняется php-код внутри них) до/после загрузки любых скриптов, соответственно.

[Кирилл Несмеянов]

2. Читаем тут и ставим свою перегрузку:

Ээээ... Если его проставить, то не будут вообще работать типизированные поля без дефолтных значений. Т.е. каждая установка значения будет вызывать фатал. Что-то попахивает этот совет.

[xmoonlight]

Кирилл Несмеянов, Да уж, поправил. Спасибо!

Answer 2

[Олег]

PhpShtorm для foo1 даст подсветку, что переменная не определена, во втором случае будет считать, что такое присвоение допустимо

function foo1()
{
    $bar->test['a']['b'] = 5;
}

/**
*@param  stdClass|object  $bar
*/
function foo2($bar)
{
    $bar->test['a']['b'] = 5;
}

здесь stdClass | object как иллюстрация, что он от них унаследован