Как организовать инстанс класса вызванного через call_user_func ?

Question

[nepster-web]

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

$rangs - массив всех рангов

foreach($rangs as $rang) {
            
            $alias = ucwords($rang['alias']);
        
            // проверяем наличие класса для определения ранга 
            if(!class_exists($pathToRangsClass.$alias)) {
                throw new \Exception('Не найден класс '.$pathToRangsClass.$alias);
            }
            
            $verify = call_user_func("\\modules\\rangs\\libraries\\{$alias}::verify", $userId);
            
       }

Вызывается метод класса, все хорошо. Теперь в самом классе мне нужно организовать, что то вроде:

public static function getInstance($className=__CLASS__)
    {
        return new $className;
    }

что бы был доступ к $this. Но вот никак не получается это организовать.

Вот такие вот приколы не выходят:

$verify = call_user_func("\\modules\\rangs\\libraries\\{$alias}::getInstance", $userId);
$verify = call_user_func("\\modules\\rangs\\libraries\\{$alias}::getInstance()->verify", $userId);

Answer 1

[Сергей Протько]

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

Comments

[nepster-web]

а какие есть еще идеи реализации ? Возможно я еще не открыт что-то другое.

[Igor Makarov]

@nepster09 не понятно, какой в итоге нужен результат, и код читать непросто:

Для чего делать конкатенацию десять раз?

$alias = ucwords($rang['alias']);
$className = DESIRED_NAMESPACE . $alias;

$rang = $className::getInstance();
// Или лучше
$rang = $application->getUserRang($alias);

// Проверка юзера
$rang->isUserMeetRequirements($user);

// $pathToRangsClass - это пространство имен или путь к файлу?
if(!class_exists($pathToRangsClass.$alias))

Какой то странный синглтон

public static function getInstance($className=__CLASS__)
{
    return new $className;
}

[Сергей Протько]

@nepster09 просто не просто предлагать решение, когда в исходных данных каша. У вас в коде мягко скажем треш. И почему у вас нету автозагрузчика? Почему такие кастыли?... Вот честно, лет 7 назад я не особо удивился бы такому коду, но сейчас подобное повергает меня в уныние... Хоть запускается это нечто через гермен а не по крону, уже радует.

[nepster-web]

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

Классы рангов находятся в отдельной категории с неймспейсом business\modules\rangs\libraries

ну и собственно в коде, я пытался по имени класса подключить нужный неймспейс.

Но как я понял лучше сделать с помощью фабрики ?

[Сергей Протько]

Как минимум да, хотя делать для рангов отдельный класс... хотя я не вкурсе особенностей логики вашего приложения. Мне кажется вы излишне усложняете и утяжеляете логику.

[nepster-web]

@Fesor я выбрал путь каждый класс для каждого ранга, так как там логика определения ранга неплохо отличается. Еще момент в том, что я не знаю будут ли изменятся ранги, добавлять или удалятся. Поэтому хочу сделать за ранее гибкую настройку. Тоесть чтобы добавить новый ранг, все что нужно это создать новый класс. Тоесть сейчас я пишу единый интерфейс для управления рангами.

[Сергей Протько]

Вам нужен класс RankDetector в таком случае. Старайтесь выделять компоненты по их функциям а не придумывать абстрактных коней в вакууме. Он же будет единым интерфейсом, а что будет происходить внутри - не столь важно. По сути интерфейс этого класса должен иметь один метод - getRank(User $user), который будет либо возвращать нужную сущность либо алиас ранка либо что там вам нужно по бизнес логике.

Реализовать внутри него можно уже при помощи отдельных драйверов-детекторов, каждый под вой ранг. Тогда и тестами покрывать удобно, и архитектура упрощается для понимания.

Answer 2

[Дмитрий Энтелис]

Не используйте call_user_func никогда. Он очень медленно работает.

Comments

[nepster-web]

у меня это жирный скриптяра, он улетает в очередь на сервер германа. Так что не принципиально

[Igor Makarov]

Не используйте PHP никогда, он очень медленно работает! Нет, ну правда! Вы мне не верите? :)

Извините, не удержался.

Answer 3

[Назар Мокринский]

Попробуйте так:

$verify = call_user_func(
    [
        "\\modules\\rangs\\libraries\\{$alias}",
        "getInstance"
    ],
    $userId
);

В массив первым элементом передаете имя класса или инстанс объекта, вторым - название метода.
Почитайте подробнее в официальном источнике: php.net/manual/en/language.types.callable.php

Но исходя из того, что я понял из вашей формулировки нужно что-то типа:

$verify = call_user_func([
    "\\modules\\rangs\\libraries\\{$alias}",
    "getInstance"
])->verify($userId);

Comments

[nepster-web]

да, то то такое. Но походу тут идея родилась как сделать по проще, через фабричный метод. Что скажете ?

[Назар Мокринский]

@nepster09 Я не в курсе деталей вашего приложения, просто дал решение того, о чём вы попросили. На самом деле поддерживаю Fesor, кажется, вы излишне намудрили, очень странно что вам для этого нужны разные классы.

Answer 4

[AxisPod]

ООП не ваш путь?

Comments

[nepster-web]

а ?