parser


 

 
   команды управления поиском

Методы и должны быть мелкими...

Sumo 18.12.2006 17:37

... почитайте Фаулера, Бека и пр. Теперь мне более-менее понятно откуда Ваши "претензии" возникли. Тут, уж извинити,проблемы не языка, а программиста.

Теперь по поводу локальных переменных в методах. Надо учитывать, что у Парсера кроме явно описанных, локальными являются все аргументы метода, а также переменная result. Т.е. мы уже имеем некоторое количество локальных хранилищ - как правило этого хватает (у меня больше половины методов не используют локальные переменные вобще). Если какой-либо метод содержит дополительно больше 3 локальных переменных, то это первый сигнал для выделения кусков кода в отдельные методы. Аргументация такая: если часть Вашего метода формирует некий результат, который используется дальше в коде, то такой кусок спокойно может быть вынесен в отдельный метод, причем зачастую хранить результат его работы в промежуточной переменной просто не потребуется - просто подставбте вызов метода вместо промежуточной переменной.