Можно ли передать конструктор модели в метод контролера параметрам?

Question

[Andrey Suha]

Есть модель Visitor
вот так она подключатся в контролере VisitorCotroller
use App\Models\Visitor;
Далее я ее могу вызвать так Visitor::create($data)
VisitorCotroller наследуется от AppUserController
Внутри VisitorCotroller
я хочу сделать что то типа этого
$this->createUser(Visitor) где Visitor конструктор App\Models\Visitor, а метод createUser наследуется от AppUserController
и выглядит так

protected function createUser($userModel) {
       $user = $userModel::create($data);
}

Но получаю ошибку Undefined constant Visitor, пробовал передавать так $this->createUser(App\Models\Visitor)

Answer 1

[ThunderCat]

protected function createUser($userModel) {
       $user = $userModel::create($data);
}

откуда там $data берется??? И зачем там $user который потом никуда не отдается???

Чет не понятно зачем все эти танцы?
$visitor = Visitor::create($data);все.

PS:

Но получаю ошибку Undefined constant Visitor, пробовал передавать так $this->createUser(App\Models\Visitor)

$this->createUser(new Visitor());если так хочется извращений.

Comments

[Andrey Suha]

protected function createUser($userModel) {
       $user = $userModel::create($data);
}

это упрощенный пример

protected function createUser(Request $request, $userModel) {
       $data = $request->toArray()
       // тут какой-то код
       $user = $userModel::create($data);
       // тут какой-то код
}

Чет не понятно зачем все эти танцы?

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

[Andrey Suha]

$this->createUser(new Visitor());если так хочется извращений.

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

[ThunderCat]

Andrey Suha, вот валидации и вынесите в отдельный метод, а объект нормально инициализируйте по месту. Или если так хочется все впихнуть - $this->createUser($request, 'Visitor');

[Andrey Suha]

спасибо

[ThunderCat]

Andrey Suha, если совсем никак, но очень хочется

$this->createUser($request, 'Visitor');
...
protected function createUser(Request $request, $userModelName) {
....
$user = $userModelName::create($data);
...
}

[Andrey Suha]

ThunderCat, сделал так как ниже написано. Я бы вынес только валидацию, но после инциализации объекта создается еще связь с одной моделью.

в общем такой метод получился

protected function createUser(Request $request, $userModel) {
        $data = $request->toArray();
        $validate = Validator::make($data, $this->validateRulesCreate);
        if($validate->fails()) return [ "success" => false, "validator" => $validate ];
        $data["password"] = Hash::make($data["password"]);
        $appUser = $userModel::create($data);
        $appUser->user()->create($data);
        event(new Registered($appUser->user));
        return [ "success" => true, "user" => $appUser ];
    }

Вы бы реализовывали по другому?

[ThunderCat]

Andrey Suha, хз, я вообще ларавель не очень люблю, все эти статик методы для инициализации, возврат коллекций из объекта... В вашем случае имя класса можно передать как строковый параметр, или как Sergei Iamskoi посоветовал, более красиво строку передать через метод класса.

[Andrey Suha]

ThunderCat, понял, спасибо))

Answer 2

[Sergei Iamskoi]

$this->createUser(Visitor::class)

или

$this->createUser(App\Models\Visitor::class)