Как вы код разбиваете на функции? В книжке читал что тело функции должно умещаться в один экран и стараюсь придерживаться этого правила. Правильно ли это?
-
3да, это правильно. А какие у вас основания в этом сомневаться?– DreamChildCommented 10 мар. 2013 в 9:37
-
1я код на функции логически разбиваю(если вообще можно так выразиться). а критерий сомнительный, вот в книжке наверняка имелся ввиду экран редактора в консоли 80x25, т.е. тело функции не больше 25ти строк, реально, но придерживаетесь ли вы этого ? И как избегаете спагетти ?– zb'Commented 10 мар. 2013 в 9:40
-
2ну есть у меня такое мнение что функция должна завершать код логически. то есть расматриватся как логически завершонная задача и поэтому бывает зачастую выхожу за рамки одного экрана.– perfectCommented 10 мар. 2013 в 9:40
-
1@DreamChild Один экран -- это сколько (один знакомый устанавливал очень мелкий шрифт на 1280х1024)? Язык? Одно дело -- ассемблер, другое -- C++/Java (про Haskell и вспоминать не будем) Главное -- чтобы читать было удобно. Жёстких стандартов нет и быть не может.– alexlzCommented 10 мар. 2013 в 10:04
-
4@Flammable догматика из серии "заставь дурака Богу молиться"– alexlzCommented 10 мар. 2013 в 15:43
1 ответ
Отвечу несколько более развернуто. Компилятору в принципе, совершенно плевать на размеры ваших функций/методов/всего остального - ему важно лишь отсутствие в нем синтаксических ошибок. То есть размер функции "в один экран" - это лишь удобство для самого разработчика и для тех, кто в последствии будет иметь счастье (или несчастье) это читать и сопровождать. Есть пример из собственного опыта - та система, что я пишу/сопровождаю на работе, писалась несколько лет тому назад не очень опытным программистом, отчего сейчас она изобилует огромными километровыми портянками кода, который на протяжении всех этих километров усердно сам себя повторяет и содержит всяческое непотребство наподобие таких вот идентификаторов: db_some_important_class_from_db_for_updating_this_table. Более того, методы классов, которыми заполнен этот всесторонне замечательный код, имеют дикие размеры в 1-2 тысячи строк, и выполняют все на свете - получение данных из GUI, выполнение какой-то бизнес-логики, запись в базу данных и много-премного всяческих других действий. Так вот работать с этим очень тяжело ии чревато множеством ошибок. Код словно по волшебству становится связанным миллионом отношений, которые находятся в очень хрупком взаимодействии, и элементарные изменения могут порушить все эти связи к чертовой матери, что неоднократно и случалось - после, казалось бы, совершенно безобидного изменения логики рушилась половина функциональности приложения. Так или иначе, из вышесказанного вытекает нижеследующее (мораль, так сказать):
В подавляющем большинстве случаев для выполнения какой-либо задачи достаточно функций указанного размера. В противном случае нужно либо пересмотреть свой стиль кодирования и сделать его более лаконичным, либо разбить задачу на ряд меньших - еще раз повторюсь, как из собственного опыта, так и из опыта других, куда более компетентных людей, могу сказать, что если вам кажется, что соблюсти это правило невозможно, то, скорее всего, вам это лишь кажется. Ну и напоследок почитайте вот это
-
-
а вот в этом вам поможет вот эта ozon.ru/context/detail/id/1308678 без преувеличения прекрасная книга Commented 10 мар. 2013 в 10:06
-
Я уже приводил цитату из эссе пионера веб-программирования П. Грэма, что ООП -- это крэк для тех, кто в старые времена писал функции длиной в страницу распечатки: 5-10 строк кода, остальное -- комментарии.– alexlzCommented 10 мар. 2013 в 10:09
-
-
1@good, если уж на то пошло, то в ООП "это" называется не функциями, а методами класса. А уже между этими двумя понятиями существует несколько серьезных отличий.– AseNCommented 10 мар. 2013 в 13:21