Что означает «код не как у сениора»; и как его подтянуть?
Если опустить паттерны / SOLID и прочую архитектуру и открыть любой рандомный php-файл, то сразу можно увидеть хороший код или шляпу. Обычно у меня такие критерии последнего:

Отсутствие кодсайла PSR или другого. Убогое форматирование. Двойные-тройные пустые строки; Лишние пробелы; Или, наоборот, код слеплен без единой пустой строки, которая логически бы отделяла блоки.
Отсутствие scrict-сравнения; Всегда стараюсь писать через ===, declare(strict_types=1), добавлять true в конец in_array и подобных;
Отсутствие type-hint'ов; хотя бы в phpdoc; Отсутствие указания области видимости метода / поля / константы;
Неадекватные названия переменных и методов, по которым непонятно, что это. Лишние переменные. Или отсутствие пемененной, где она была бы полезной, для улучшения понимания кода;
Тупые и очевидные комментарии ради комментариев;
Длинные портянки. Если метод, условно, больше 80 строк, то, мб, надо задуматься о декомпозиции;
Куча параметров в сигнатуре функции
Неиспользуемые переменные / константы / методы "на будущее".
Закомментированный код;
Хард код. Какие-нибудь api url прям в коде в переменной, вместо конфига или хотя бы константы.
Забивание на работу с ошибками. unserialize вернул false? Да неее, такое точно не произойдёт


Ну, это что на первый взгляд бросается. Потом видишь неоптимальный код. Какой-нибудь INSERT в бд по одной строке в цикле foreach.
гляньте https://qna.habr.com/q/276441