neol

Все решения будут дико тормозить. Сайт по ссылке тормозит на macbook pro 2018 с i7, во время скролла лишь появляется желание побыстрее закрыть сайт.
Лучше нативного скролла нет ничего, убеждался в этом множество раз. Если нужна плавная прокрутка к нужной позиции, то тоже есть нативные решения: css scroll-behavior или на js параметр behavior у scrollTo

Вам нужно использовать LLD - low-level discovery

Да никто не лучше. Этот холивар длится примерно вечность и никогда не закончится. Каждая из этих технологий решает примерно одни и те же задачи, но немного по-разному. Вот и все.

А остальное "лучше-хуже" уже особенности проекта и радиус кривизны рук (недостаток опыта). Есть еще 100500 баз данных и что?

$a = rand(0, 100);
$b = rand(0, 50);

if ($a > $b) {
    $b += $a;
    $a = $b - $a;
    $b -= $a;
}

echo $a, ' ', $b;

Если нужно поменять значение 2-х переменных, то надо использовать третью переменную или изощрятся используя промежуточную функцию как буффер.

$e = rand(0, 100);
$l = rand(0, 50);
if($e > $l)
{
    list($e, $l) = array($l, $e);
}

echo $e;
echo $l;

Первое самое глобальное - почему laravel 5.8?
Вы документацию используете же наверняка, видели же по любому что там уже больше полугода зарелизен 6.0. А 5.8 даже уже не поддерживается критикал патчами

Шаблоны слеплены по старинке кусками без структуры.
При том что и @extends и @section вполне себе используются, но код выглядит как будто в вордпресс заглянул где пачка файлов и открываются теги в одном, а закрываются в другом. Не надо так.

Код стайла никакого, всё то разброс, то наоборот слеплено. Везде кучи пустых строк в разных местах ну и всё такое прочее.

Нейминг тоже пляшет

Трейты это хорошо, но довольно странное использование. Как будто пытались изобрести репозитории или хотя бы вынести повторяющийся код, но почему тогда хотя бы не в моделях методами?

Что это такое и зачем я вообще не понял. Когда есть response()->json() например (и опять же местами оно есть тоже). Про ->data я уж вообще молчу.

JSON роуты намешаны в кучу с обычными. Хотя есть даже отдельный файлик api.php в routes. Это в целом-то как угодно, но опять же про кодсайтл и практики устоявшиеся внутри Laravel.

Методы которые не являются собственно обработчиками запросов в контроллерах тоже такое себе. Не сказать что криминал, но просто опять же создание мешанины.

Requests это хорошо, но почему там только required везде, а если туда массивы какие-нибудь слать начать, всё поляжет.

Используете мидлвар с параметром role:admin, а на самом деле admin захардкожен

И снова про кодстайл array() уже не очень актуально, гораздо более лаконично использовать короткий синтаксис [].
Хотя опять же строгих рекомендаций на этот счет нет.

Указание классов $this->belongsTo('App\City'); не очень красиво, да и в IDE поддержка какая-то непонятная, как будто от этого уже давно ушли. Проще и удобнее писать $this->belongsTo(City::class);. Бонусом подсветка в IDE, автодополнение и класс импортировать через use можно в таком случае.

Зачем такое выносить? Это переиспользуемый код? Он помог сделать код чище? Вместо одной строчки кода сделали 4 на ровном месте.

Ну в общем еще много такого по мелочи можно найти. Глобально ну круд и круд, работает значит хорошо. Круды умеете писать и ларавел не первый раз увидели уже.

По сути всё про:
1. Кодстайл php, автоформатирование и вот это всё. Поставьте IDE или что-нибудь что может форматировать по PSR.
2. Кодстайл/структура Laravel тоже стоит придерживаться того как всё устроено конкретно внутри фреймворка. Чтобы код был однородным.

этих знаний хватит для поиска работы?

Пробуйте, каким-нибудь трейни-джуном-то должны взять, но по этому коду трудно сказать что вы умеете, тут и php функций как таковых не используется и логика вся "прямая" как не знаю что :)
По коду понятно только что на Laravelе что-то смастерить наверно сможете хоть и из говна и палок. На php например сможете сделать тоже самое? Даже пусть с копозером и пакетами какими-нибудь, но собрать из разных частей и склеить вместе.

$fields = [
  'clientName' => 'name',
  'clientPhone' => 'phone',
  'clientEmail' => 'email'
];
$where = [];
$resultArr = [];
foreach($fields as $key => $field) {
  if($arr[$key]) {
    $where[] = "`$field` LIKE ?";
    $resultArr[] = '%' . $arr['clientName'] . '%';
  }
}
if(count($where) > 0) {
  $sql .= ' WHERE ' . implode(' AND ', $where);
}

подключить систему мониторинга. она все покажет.
от элеметарного atop до монстров типа munin
https://en.wikipedia.org/wiki/Comparison_of_networ...

Обязательно ли в ларавеле использовать Eloquent ORM?

Нет, не обязательно.

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

Делайте сырые запросы к базе. Не нужно натягивать сову на глобус выборки на ORM.

ORM помогает когда вам нужны модели с логикой для какой-то операции. Если цель - просто получить данные от базы - выборки через SQL, и маппинг на DTO по прежнему остаются лучшим выбором.

VPN - это тоже трафик...
Поэтому - никак.