Как в статических функциях абстрактного класса получить доступ к переменным дочернего класса?

Question

[Даниил Сидоров]

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

Есть класс Article, который наследуется от абстрактного класса Searchable

class Article extends Searchable
{
    public static $indexName = 'article';
    public static $mapping = [
        'properties' => [
            'url' => [
                'type' => 'keyword'
            ],
            'title' => [
                'type' => 'text',
                'boost' => 2
            ],
            'full_text' => [
                'type' => 'text'
            ],
        ]
    ];
}

abstract class Searchable
{
    public static function createIndex()
    {
        $client = ClientBuilder::create()->build();
        $indexExist = $client->indices()->exists(['index' => self::$indexName]);

        if (!$indexExist)
        {
            $params = [
                'index' => self::$indexName,
                'body' => [
                    'mappings' => self::$mapping
                ]
            ];

            $response = $client->indices()->create($params);
        }
    }
}

То есть в будущем я хочу просто вызывать методы следующим образом:
Article::createIndex();

Как мне в классе Searchable получить значения класса Article?

Comments

[Mofsy]

Вам нужно пойти поспать.

Answer 1

[grinat]

Статичные методы это хреновый подход. Наследование тоже.

Comments

[Сергей]

И давно это наследование стало хреновым подходом?)

[Даниил Сидоров]

Можете в двух словах объяснить почему?

[Дмитрий]

DaniLaFokc, а можно узнать кто вам посоветовал делать статику? Статика это даже не ооп

[Даниил Сидоров]

Дмитрий, Задача тех методов - сделать простое действие, например создать индекс. Делаю их статичными, чтобы лишний раз не создавать экземпляр класса. Или это не особо правильно?

[grinat]

DaniLaFokc, ну например твой код нельзя протестировать без запуска эластика, т.к. метод статичный и все данные лежат внутри замыкания, и дальше больше, раз он много везде будет использоваться, то это фактически означает невозможность unit тестирования по всему проекту.

[Дмитрий]

DaniLaFokc, проблема создать экземпляр?

Answer 2

[chtulhu]

self:: заменяем на static::

https://www.php.net/manual/ru/language.oop5.late-s...

Comments

[Даниил Сидоров]

Да, действительно работает. Я про это читал, даже написал, но phpstorm почему то подчеркивал этот код и я даже не стал проверять его работоспособность.

Пишет "field not found in Searcheable"

[Дмитрий Дерепко]

DaniLaFokc, создайте это свойство в модели Searcheble, и тогда шторм перестанет его подсвечивать.
А вообще, ребята в другом ответе правильно говорят - не использовать статику.
Кстати, Searcheble - имя для интерфейса, подберите для абстрактного класса другое имя, дабы потом не вызывать дискомфорт у людей.