CodeReview - Githubissues

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Console/Commands/ReindexCommand.php#L11-L40

не хватает пробелов между phpDoc, свойствами и методами
старайся делать однотипные phpDoc у свойств, т.е. либо многострострочные, либо однострочные

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Jobs/RabbitMQJob.php#L29

старайся писать комментарии на английском языке

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Jobs/RabbitMQJob.php#L35

не хватает пробелов до и после =

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Mail/OrderShipped.php#L38

typeHinting return value

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Mail/OrderShipped.php#L43

такие вещи необходимо в константы, либо в конфиг. Если подразумевается локализация, то тексты необходимо вынести в лэнг-файлы

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Products/ElasticsearchRepository.php#L2-L3 https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Products/ElasticsearchRepository.php#L11-L12

лишние пробелы

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Products/ElasticsearchRepository.php#L16-L26

здесь наоборот, пробелы отсутствуют, методы необходимо отделять пробелом

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Products/EloquentRepository.php#L10

наверное стоит изменить наименование класса и интерфейса наоборот, выделить общий интерфейс - EloquentRepository, и класс для работы с продуктами ProductsRepository
так же как вариант вынести контракты в отдельную папку, можно добавлять к имени интерфейса i, например IEloquentRepository (как минимум удобнее ориентироваться по файлам)

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Providers/AppServiceProvider.php#L19-L29

получается bind репозитория зависит от конфига, а подключение клиента нет?

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Filters/Models/Product/RateMaxFilter.php#L13

typeHinting (phpDoc)

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Http/Controllers/Admin/AdminController.php#L14

съехала табуляция

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/resources/views/admin/insurance-company/create.blade.php#L24

при возникновении ошибки, поля потеряют введенные значения, можно добавить хелпер в поле value="{{ old("name") }}"

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Http/Controllers/Admin/InsuranceCompanyController.php#L67-L74

можно упростить до:

    if ($insuranceCompany->products()->count() === 0) {
        $insuranceCompany->deleteOrFail();
        return redirect()->route('admin.insurance-company.index');
    }

    return redirect()->route('admin.insurance-company.index')
        ->with('error', 'Компания имеет продукты!');

https://github.com/nemirova-e/web-practikum/blob/5adf74530eb0cabd763b5fe31b9665ec89d91c89/app/Http/Controllers/Agent/AgentController.php#L17-L18

не хватает пробелов

Общее

используй declare(strict_types=1)
используй более релевантные названия: например репозитории почему-то лежат в папке Products
тексты ошибок, заголовки, название полей таблиц, кнопок и пр., особенно если они не на английском, необходимо вынести в лэнг-файлы
следи за пробелами, переносами строк, табами и пр., обрати внимание на PSR-1, PSR-12 - https://www.php-fig.org/psr/

nemirova-e / web-practikum

CodeReview #5