Set variable in __construct in controller is correct?!

[MohammadRezaKiani]

this is my code is correct or not?

class UserController extends Controller
{

    protected $user;

    public function __construct(Request $request)
    {
            $this->user = User::find($request->user_id);
    }

    public function changeStatus(Request $request)
    {
        if ($request->user_id){
            if($this->user->isBlock == 0){
                $this->user->isBlock = 1;
                $this->user->save();
                alert()->success('Ban.');
                return redirect()->route('admin.user.list');
            }elseif ($this->user->isBlock == 1){
                $this->user->isBlock = 0;
                $this->user->save();
                alert()->success('unBAN.');
                return redirect()->route('admin.user.list');
            }
        }else{
            return redirect()->route('admin.user.list');
        }
    }

}

[henzeb]

I always say: "If it works, it works" so if if it works, why the **** not?