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