Нашел классный комент:
Правило 30 секунд заключается в том, что если посторонний программист за 30 секунд не может разобраться в том, что делает метод, то его нужно переписывать. Если метод длинный, то разбить на несколько более мелких с говорящими именами. Если метод короткий, то значит что-то не так с naming’ом.
Прыгать по методам интересно будет, когда их нужно будет править, или когда интересны будут детали реализации. А для того, чтобы понять, что делает код в общем, достаточно хороших имен и короткого метода.
Кстати, у тебя этот прием использован во втором примере: WritePageNumberOnBitmap ты вынес в отдельную функцию, хотя вызывается она всего в одном месте. Точно так же стоит поступать и с остальными, т.е. получится что-то вроде (за синтаксис не пинать, ибо на Delphi уже больше 10 лет не писал).
Комментариев нет:
Отправить комментарий