Не важно на каком языке и что Вы пишите — рано или поздно Вам придется любой свой код менять и улучшать. Конечно это при условии что Вы не пишите одноразовые салфетки — тогда да, можно использовать и выкинуть, но выкидывать код клиенту не нужно)
Это примеры нехорошего кода, который нужно искоренять у себя из подкорочки даже.
Переукорачивание кода
А теперь скажи читатель, бросив косой взгляд на это — с каким набором значений в переменной мы будем для $type_content==TYPE_DISEASE_ID в конце концов…
Использование нескольких псевдонимов для одного и того-же
Можно еще парочку запилить, только лучше все свести к одному все-таки… чтобы не угадывать
Использование одноразовых переменных
Очевидно «графоманская» штука. К ней я для сравнения даже улучшенную версию приведу
Плохо
Хорошо
Циклы пустышки
Была проверка — не проходим — ну значит закомментим. А цикл оставим. Правильно было бы разобраться какая проверка сбоит.