Следовать ли рекомендациям IDE?

Question

[HP_KDE_neon]

В файле index.php подключается файл settings.php, в котором используется переменная $container, созданная в файле index.php:

// index.php
$container = new Container();

require config_path('settings.php');

// settings.php
$container->set('settings', fn() => [
    # код
]);

Шторм подсвечивает мне, что нужно в settings.php прописать global $container;, я давно уже не встречал таких записей и ранее их не применял.




Почему разрабы шторма предлагают такое (работает ведь без всяких global), следовать ли рекомендациям IDE или запилить, как ниже и забыть?

// index.php
$container = new Container();

$settings = require config_path('settings.php');
$settings($container);

// settings.php
return function(Container $container) {
    $container->set('settings', fn() => [
        # код
    ]);
};

Comments

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

Следовать ли указаниям пожарной сигнализации? Стул подо мной не горит, зачем её включили?

[HP_KDE_neon]

Алексей Уколов, я вообще первый раз вижу, чтобы шторм такую ситуацию помечал ошибкой и предлагал прописать global, это всё новая версия по ходу, я раньше на 2023, 2024 сидел, поставил 2025.2

[Adamos]

Мне одному дико, что ТС использует ларовскую функцию, чтобы поперек фреймворка наколхозить подключение функции из папки конфига, а волнуют его при этом только рекомендации Шторма?..

[HP_KDE_neon]

Adamos, это не Лара, ента хелперы у меня такие есть :)

ранее использовался вариант в конце вопроса, перенято из Слима, хотел избавиться от анонимок и просто файлы подключить, где сборка идёт, а тут - global

будем возвращаться на исходный вариант, оно так и понятнее, что куда пошло

[Adamos]

HP_KDE_neon, я бы сказал - "переврано из Слима" ;) там-то как раз все грамотно и без "упрощений".

[HP_KDE_neon]

Adamos, строчек много, хотел сделать только 4 require :)

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

Adamos, просто авторы фреймворка, IDE и языка - дураки и не знают, как сделать более лучше.

Answer 1

[Rsa97]

Ну так в вашем файле переменная $container используется без предварительного определения. Вот вам и предлагается либо явно её определить и инициализировать, либо обозначить как global, как признак того, что она инициализируется в другом файле.
То, что вы этот файл гарантированно включаете где-то в другом месте, IDE не знает, и предполагает, что файл может использоваться сам по себе.

Comments

[HP_KDE_neon]

подсветка красным, я так понимаю определяется, как ошибка, если они хотели предупреждение, то хаки вроде они подсвечивают, предлагая обратить внимание и что-то подправить, напугали меня только, скорее удивили

то есть это не ошибка и исправлять не обязательно?

p.s. прикольно конечно: контейнер глобально по коду таскать, но это вроде не по феншую :)

[Rsa97]

HP_KDE_neon,

// MyContainer.php
class MyContainer extends Container
{
    public function __construct() {
        $this->set('settings', fn() => [...]);
    }
}

// index.php
$contaner = new MyContainer();

[HP_KDE_neon]

Rsa97, не ну так-то если контейнер собирать, то я могу просто файл container.php создать и подключить в index.php, определив $container, просто у меня $container используется типа как Слиме - ТЫК, хотел только убрать анонимки

ok, благодарю