https://en.cppreference.com/w/cpp/language/eval_order

2) If a side effect on a memory location is unsequenced relative to a value computation using the value of any object in the same memory location, the behavior is undefined.

cout << i << i++; // undefined behavior until C++17
a[i] = i++;       // undefined behavior until C++17
n = ++i + i;      // undefined behavior

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

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

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

К перечисленному советую почитать книжку по DDD. Перестраивает мозги.