Рабочее
Иногда смотришь в код, смотришь.. и понимаешь, что надо переписать вообще.
Когда для переменной заводят копию (копию ссылки естественно), и в дальнейшем работают как с копией данных... когда код пестрит кучей приватных методов из двух-трех строк, в результате чего одни и те же проверки выполняются не один раз, а как минимум два, а то и три... когда самые явные условия (не требующие дополнительных ресурсов), которые могут прекратить дальнейшее выполнение сценария, выводят в конец метода, а запросы к базе данных (которые могут и не потребоваться) - наоборот всем скопом вынесены в начало... хочется вырвать себе глаза. Ну и заодно автору кода - руки.
Когда для переменной заводят копию (копию ссылки естественно), и в дальнейшем работают как с копией данных... когда код пестрит кучей приватных методов из двух-трех строк, в результате чего одни и те же проверки выполняются не один раз, а как минимум два, а то и три... когда самые явные условия (не требующие дополнительных ресурсов), которые могут прекратить дальнейшее выполнение сценария, выводят в конец метода, а запросы к базе данных (которые могут и не потребоваться) - наоборот всем скопом вынесены в начало... хочется вырвать себе глаза. Ну и заодно автору кода - руки.