Зачем нужен двойной return при использовании $this в контроллере Laravel?

А this тогда что делает? Получается не запускает новый метод, а запрашивает данные из нового метода в родительский (в отличие от this VUE где каждый метод также возвращает свое, но мне не нужен второй ретурн т.к. он есть в запускаемом методе)

$this это способ сослаться на текущий экземпляр объекта, для вызова его метода или атрибута.
В данном случае $this позволяет обратится к методу client() (включил режим гадалки) контроллера.

jazzus, в языке сперва разберитесь, потом во фреймворки лезьте.

Ниже избыточная информация - размышления над кодом.

Да и магическое число 10 лучше заменить на константу, которую прописать в модели User.

if ($user->type === User::ROLE_CLIENT) {
    // ...
}

Но не идеально.
В идеале (но не совсем) бы было красивее.

if ($user->isClient()) { // На самом деле customer :)
    // ...
}

Как вариант, если речь идёт о ролях, то возможно.

if ($user->isRole(new RoleClient)) {
    // ...
}

Это уже зависит от сложности проекта и возможности развития или изменения ролей.

И метод client имеет странное название, не отражающее его смысл. :)
Может проще redirectToClientsProfile() или что-то подобное или хотя бы goClients().
Впрочем всё относительно.
По мне метод вообще избыточен и следовательно не нужен.
Достаточно просто return redirect('/clients'); , ведь данный метод ничего другого не делает.

if ($user->isClient()) {
    return redirect('/clients');
}

Ну и $user = User::where('id', Auth::id())->firstOrFail(); то же избыточно
Достаточно $user = Auth::user();
Ведь речь идёт о Laravel и текущем авторизованном пользователе.

// В файле контроллера

    public function index()
    {
        $user = Auth::user();
        if ($user->isClient()) {
           return redirect('/clients');
        }
        // ...
    }

// В файле модели пользователя
class User extends Authenticatable
{
    const TYPE_CLIENT = 10;
    // ...
    public function isClient()
    {
        return (self::TYPE_CLIENT == $this->type);
    }
}

И если проверка делается во всех методах контроллера (контроллер не работает с клиентами),
то проще вынести проверку в миделвея и прикрутить к роутам с контроллером. :)

Евгений Педя, Спасибо за идеи по сокращению кода! По-поводу ролей - я уже сделал роли с пермишенами с несколькими связанными таблицами и т.д.. type - здесь тип аккаунта для редиректов, чтобы не лезть в отношения.

Но по-поводу this Laravel все равно есть вопросы :)

$this это способ сослаться на текущий экземпляр объекта, для вызова его метода или атрибута.
В данном случае $this позволяет обратится к методу client() (включил режим гадалки) контроллера.

В смысле "способ сослаться"? Можно человеческим языком?) Вызывает он метод на исполнение или запрашивает инфу из метода?

Евгений Педя, по поводу избыточности метода.. Я в итоге сделал отдельный трейт только для редиректов) Сегодня узнал, что в Laravel есть трейты. Пока не понял, чем они отличаются от контроллеров.. Но видимо придется переписать свои разработки тк. это очень удобно делать много методов типа 2+2=4 и собирать их в контроллерах

Евгений Педя, еще раз спасибо за идеи по поводу кода (не знал, что в моделях можно делать такие обращения). Единственное я не понял этот кусок

class User extends Authenticatable
{
    const TYPE_CLIENT = 10;
    // ...
    public function isClient()
    {
        return (self::TYPE_CLIENT == $this->type);
    }
}

я сделал просто проверку:

public function isClient()
    {
            if ($this->type==10) return true;
            return false;
    }

есть какое-то преимущество с константой? $user->isClient() проверил работает

В смысле "способ сослаться"? Можно человеческим языком?) Вызывает он метод на исполнение или запрашивает инфу из метода?

Откуда php по вашему знает как вызвать нечто с именем client()?
Это может быть функция. Это может быть метод другого класса.
Это может быть метод данного класса.

php.net/manual/ru/language.oop5.basic.php

узнал, что в Laravel есть трейты.

Они есть в языке php, а не в laravel.

php.net/manual/ru/language.oop5.traits.php

Трейты - это один из способов избежать дублирования кода. Может быть несколько классов, которые не имеют к друг другу отношения прямого, но в них могут использоваться схожие методы реализующие нечто. Иногда их проще вынести в трет и подключить в начале классов. Но это не идеальное решение. Оно подходит если нет смысла в создании отдельного объекта или наследования от общего класса.

По редиректам в laravel.
Смысл создания метода для редиректа есть если в redirect('/clients') может изменится '/clients' (правим в одном методе - работает везде). Но это легко решается именованными роутами. Имя в редиректах сохраняется, а правится url роута, если нужно.
По этому я считаю метод избыточным.
Да и при нормальной IDE найти и заменить redirect('/clients') скажем на redirect('/customers') довольно легко. А лишний метод заставит через некоторое время себя искать, что бы посмотреть, для чего он создан.

это очень удобно делать много методов

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

В общем я понял (наверное). задэдэшил этот this и увидел RedirectResponse, который в глубинах Ларавеля прописан)
получается, как и писал Евгений - this возвращает объект и редирект оказывается тоже может быть объектом с различными свойствами, атрибутами итд и который будет исполняться по запросу (в моем случае вторым ретурном). И можно сделать вывод, что this Laravel не запускает метод (как в Vue), а запрашивает объект метода (но это не точно))

И можно сделать вывод, что this Laravel не запускает метод (как в Vue), а запрашивает объект метода (но это не точно))

Нет! Я описал всё в комментарии к другому ответу подробно.

есть какое-то преимущество с константой? $user->isClient() проверил работает

По имени константы кто-то другой или вы сами (через пол года) можете понять смысл кроющийся в числе 10.
Или вы думаете, что будите помнить что скрывается за числом 10 и другими числами?
Кроме того в коде может быть куча чисел 10 с разными смыслами, а вам вдруг понадобилось сменить 10 на другое число для одного из контекстов. По имени константы можно сменить в любом нормальном IDE автоматом, а вот при замене 10ки можно зацепить и что-то не связанной в данном случае с типом пользователя.

Евгений Педя, по поводу константы. В модели 1 раз пишем

public function isClient()
    {
            if ($this->type==10) return true;
            return false;
    }

и уже во всем приложении потом исключительно $user->isClient() и все. Максимально просто. Мне кажется запутаться нереально т.к. "10" указывается один раз в одном месте c названием метода isClient() и более 10 нигде не появляется.

Нет! Я описал всё в комментарии к другому ответу подробно.

Почему нет-то? ( Я не увидел в вашем ответе противоречий

$this это способ сослаться на текущий экземпляр объекта, для вызова его метода или атрибута.

В моем случае this вызывает объект RedirectResponse из метода сlient(). Верно?
И этим можно объяснить почему this->сlient() не запускает сразу редирект т.к. мы только запросили объект редиректа, а чтобы его вернуть мы и используем второй return. Правильно??

Да. В начале в методе сlient() мы получаем объект редиректа из функции и тут же его возвращаем из метода (return). Далее в index() мы так же его возвращаем.

https://laravel.com/docs/5.7/responses

Евгений Педя, так я в соседней ветке и здесь тоже самое написал)

О боже.. выучите язык сначала, это же элементарные основы. Зачем тут люди вообще пытаются что-то говорить про качество кода, если человек вообще не понимает что делает?

Alex Wells, я и спрашиваю основы, чтобы разобраться. Если бы выучил язык не спрашивал. Учу) Написал же что новичок и занимаюсь этим недавно