Про форматирование кода и нормальные имена все уже знают, копнём глубже.
Правильный код должен быть модульным. Т.е. программа должна собираться из самостоятельных отдельных моделей, которые могут работать независимо друг от друга. Так можно использовать старый код много раз и вообще писать легче ибо неразберихи будет меньше, достаточно будет просто отслуживать внутреннее пространство каждого конкретного модуля, выявлять ошибки так легче.
Код должен идти частями. Т.е. не надо создавать множество вложенных друг в друга условий. Надо по ходу программы проверять какое-либо условие и тут же делать действия с текущими переменными если нужно и дальше идти по коду будто этого самостоятельного куска и не было. Собственно, это так же относится к модульности, но тут просто облегчается логика, написание и отладка т.к. а повторное использование не используется. В общем, множественные вложенные друг в друга условия это зло, лучше тогда использовать методы/функции, которые часть логики будут скрывать в себе, чем городить миллион вложенных условий. В общем, это к первой части.
Не должно быть никакой избыточности и двухсмысленности, т.к. это затрудняет отладку и составление логики. Всё должно идти точно так как было задумано без вариантов.
Все одинаковые моменты должны пропускаться через один проход. Если это сайт, то обязательно общая точка входа. Одной и тоже действие должно делаться только в одном месте с одной общей реализацией, а не везде по-своему.
Всегда всё делать максимально независимо друг от друга, чтобы потом не пришлось исправлять коллизии. Если надо поработать с текстом в текстовом поле, то сначала его получаем оттуда в свою переменную, а потом, если требуется, вставляем обратно, а не работать со свойством объекта текстового поля.