Как вызвать метод класса внутри другого метода?

Question

[dch3]

На сервер приходит запрос от клиента с данными, которые нужно вначале провалидировать.
Этот запрос обрабатывает скрипт, а точнее метод createNewTitles класса WebController.
Этот класс содержит еще один метод validateItems, который вызывается внутри первого метода.

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

Ошибка наверное кроется вот в этом вызове $this->validateItems($request, $rules);, но что с ней не так не пойму. Пробовал делать метод validateItems статическим и вызывать его self::validateItems($request, $rules);, что тоже не принесло результата.

class WebController extends Controller {

    public function validateItems($request, $rules) {

        $validator = Validator::make($request->all(), $rules);
            if ($validator->fails()) {
                return response()->json($validator->errors(), 422);
            }

    }

public function createNewTitles(Request $request) {
  
        if ($targetButton == 'add-category') {

            $rules = ['newItem' => 'required|unique:categories,category'];
            $this->validateItems($request, $rules);

Comments

[Дмитрий]

Где ретурн?

Answer 1

[ThunderCat]

Вы НЕ возвращаете ответ ИЗ контроллера, вы возвращаете ответ в функцию. Так работать не будет, смотрите мой комментарий с выдержкой из доки.

-$this->validateItems($request, $rules);
+return $this->validateItems($request, $rules);

Answer 2

[Сергей delphinpro]

метод validateItems() может вернуть ответ с ошибками, но вы этот ответ в createNewTitles() не получаете и не пересылаете клиенту.

Comments

[dch3]

Сергей delphinpro
Спасибо!
Только я не совсем понял, почему методу createNewTitles() нужно получать ответ?
Ведь метод validateItems() содержит строку

return response()->json($validator->errors(), 422);

, которая в случае ошибки валидации возвращает клиенту сериализованный обьект ответа с данными ошибки и прекращает дальнейшее выполнение метода createNewTitles().

А в случае, когда ошибки нет, методу createNewTitles() тоже ответ не нужен, он просто спокойно себе выполняется дальше))

[Дмитрий]

dch3, метод возвращает, но вы же в методе где вызвали не возвращаете ничего

[Сергей delphinpro]

dch3,

Только я не совсем понял, почему методу createNewTitles() нужно получать ответ?

А как по другому вы вернете ответ с ошибками клиенту?

[Сергей delphinpro]

Этот метод в таком виде вообще не нужен. Он всё портит
Можно как-то так его переписать, к примеру

public function validateItems($request, $rules): bool|Validator {
    $validator = Validator::make($request->all(), $rules);
    if ($validator->fails()) {
        return $validator;
    }
    return true;
}

public function createNewTitles(Request $request) {
    if ($targetButton == 'add-category') {
        $rules = ['newItem' => 'required|unique:categories,category'];
        $resultValidation = $this->validateItems($request, $rules);
        if ($resultValidation instanceof Validator) {
            return response()->json($resultValidation->errors(), 422);
        }
        ...
}

Но это фигня какая-то получается.

Можно просто валидатор возвращать

public function validateItems($request, $rules): Validator {
    return Validator::make($request->all(), $rules);
}

public function createNewTitles(Request $request) {
    if ($targetButton == 'add-category') {
        $rules = ['newItem' => 'required|unique:categories,category'];
        $validator = $this->validateItems($request, $rules);
        if ($validator->fails()) {
            return response()->json($validator->errors(), 422);
        }
        ...
}

И тут вообще непонятно, зачем выносить одну строку в отдельный метод..

В общем это хрень какая-то, не надо так писать.

[Сергей delphinpro]

Если хотите контроллер поменьше сделать - вынесите валидацию в форму запроса https://laravel.com/docs/10.x/validation#form-requ...

[ThunderCat]

dch3,

All routes and controllers should return a response to be sent back to the user's browser. Laravel provides several different ways to return responses. The most basic response is returning a string from a route or controller. The framework will automatically convert the string into a full HTTP response

return response()->json($validator->errors(), 422);

абсолютно ничего не будет делать если вы не вызываете его из роутера или контроллера (т.е. это не команда типа header, как вы подумали).

[dch3]

@delphinpro
@Compolomus
я понял - я не полностью указал в тексте вопроса код метода createNewTitles()
Исправляюсь вот полный код (там есть ответ на клиента!):

public function createNewTitles(Request $request) {
    
        $transaction = $request->input('transaction');
        $newItem = $request->input('newItem');
        $targetButton = $request->input('targetBatton');

        if ($targetButton == 'add-category') {

            $rules = ['newItem' => 'required|unique:categories,category'];
            $this->validateItems($request, $rules);

            $categories = Category::where('transaction', $transaction)
                ->pluck('category')
                ->unique();

            return response()->json($categories);

        }
       elseif ($targetButton == 'add-subcategory') {
       $rules = ['subNewItem' => 'required|unique:categories,subcategory'];
            $this->validateItems($request, $rules);

            $subcategories =  Category::where([
                'transaction' => $transaction,
                'category' => $newItem
            ])->pluck('subcategory');

            return response()->json($subcategories);
}

[Сергей delphinpro]

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

[dch3]

Сергей delphinpro,
"И тут вообще непонятно, зачем выносить одну строку в отдельный метод.." - вынес из-за того, что мы имеем повторяющийся участок кода в методе createNewTitles:

$validator = Validator::make($request->all(), $rules);
            if ($validator->fails()) {
                return response()->json($validator->errors(), 422);
            }

который я вынес в отдельную функцию

[ThunderCat]

Исправляюсь вот полный код (там есть ответ на клиента!):

Я понял, вы считаете что метод response()->json() сам выполняет ответ клиенту и завершает работу скрипта. Во первых, это не так (иначе там не нужен был бы ретурн), во вторых - респонс ожидается только из контроллера и роутера, и уже там происходит вывод клиенту. Сам респонс является как бы инструкцией что и как отдать.

[Сергей delphinpro]

dch3, Можно переписать правила валидации и вынести проверку из условия, ничего повторяться не будет

public function createNewTitles(Request $request) {

    $validator = Validator::make($request->all(), [
        'transaction' => 'required',
        'targetBatton' => 'required',
        'newItem' => [
            // Исключить, если targetButton не равен add-category
            'exclude_unless:targetButton,add-category',
            'required',
            'unique:categories,category',
        ],
        'subNewItem' => [
            // Исключить, если targetButton не равен add-subcategory
            'exclude_unless:targetButton,add-subcategory',
            'required',
            'unique:categories,subcategory',
        ]
    ]);
    
    if ($validator->fails()) {
        return response()->json($validator->errors(), 422);
    }

    $transaction = $request->input('transaction');
    $newItem = $request->input('newItem');
    $targetButton = $request->input('targetBatton');

    if ($targetButton == 'add-category') {

        $categories = Category::where('transaction', $transaction)
            ->pluck('category')
            ->unique();

        return response()->json($categories);

    } elseif ($targetButton == 'add-subcategory') {

        $subcategories =  Category::where([
            'transaction' => $transaction,
            'category' => $newItem
        ])->pluck('subcategory');

        return response()->json($subcategories);
    }
}

[dch3]

@delphinpro
@ThunderCat
Вот только смог спокойно ознакомится с ответами и... все понял))
Спасибо большое за ответы. Сам не пойму, как я этого не увидел