0

Как вы код разбиваете на функции? В книжке читал что тело функции должно умещаться в один экран и стараюсь придерживаться этого правила. Правильно ли это?

12
  • 3
    да, это правильно. А какие у вас основания в этом сомневаться?
    – DreamChild
    Commented 10 мар. 2013 в 9:37
  • 1
    я код на функции логически разбиваю(если вообще можно так выразиться). а критерий сомнительный, вот в книжке наверняка имелся ввиду экран редактора в консоли 80x25, т.е. тело функции не больше 25ти строк, реально, но придерживаетесь ли вы этого ? И как избегаете спагетти ?
    – zb'
    Commented 10 мар. 2013 в 9:40
  • 2
    ну есть у меня такое мнение что функция должна завершать код логически. то есть расматриватся как логически завершонная задача и поэтому бывает зачастую выхожу за рамки одного экрана.
    – perfect
    Commented 10 мар. 2013 в 9:40
  • 1
    @DreamChild Один экран -- это сколько (один знакомый устанавливал очень мелкий шрифт на 1280х1024)? Язык? Одно дело -- ассемблер, другое -- C++/Java (про Haskell и вспоминать не будем) Главное -- чтобы читать было удобно. Жёстких стандартов нет и быть не может.
    – alexlz
    Commented 10 мар. 2013 в 10:04
  • 4
    @Flammable догматика из серии "заставь дурака Богу молиться"
    – alexlz
    Commented 10 мар. 2013 в 15:43

1 ответ 1

8

Отвечу несколько более развернуто. Компилятору в принципе, совершенно плевать на размеры ваших функций/методов/всего остального - ему важно лишь отсутствие в нем синтаксических ошибок. То есть размер функции "в один экран" - это лишь удобство для самого разработчика и для тех, кто в последствии будет иметь счастье (или несчастье) это читать и сопровождать. Есть пример из собственного опыта - та система, что я пишу/сопровождаю на работе, писалась несколько лет тому назад не очень опытным программистом, отчего сейчас она изобилует огромными километровыми портянками кода, который на протяжении всех этих километров усердно сам себя повторяет и содержит всяческое непотребство наподобие таких вот идентификаторов: db_some_important_class_from_db_for_updating_this_table. Более того, методы классов, которыми заполнен этот всесторонне замечательный код, имеют дикие размеры в 1-2 тысячи строк, и выполняют все на свете - получение данных из GUI, выполнение какой-то бизнес-логики, запись в базу данных и много-премного всяческих других действий. Так вот работать с этим очень тяжело ии чревато множеством ошибок. Код словно по волшебству становится связанным миллионом отношений, которые находятся в очень хрупком взаимодействии, и элементарные изменения могут порушить все эти связи к чертовой матери, что неоднократно и случалось - после, казалось бы, совершенно безобидного изменения логики рушилась половина функциональности приложения. Так или иначе, из вышесказанного вытекает нижеследующее (мораль, так сказать):

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

12
  • внутри метода класса то же код разбивать надо
    – perfect
    Commented 10 мар. 2013 в 9:55
  • а вот в этом вам поможет вот эта ozon.ru/context/detail/id/1308678 без преувеличения прекрасная книга
    – DreamChild
    Commented 10 мар. 2013 в 10:06
  • Я уже приводил цитату из эссе пионера веб-программирования П. Грэма, что ООП -- это крэк для тех, кто в старые времена писал функции длиной в страницу распечатки: 5-10 строк кода, остальное -- комментарии.
    – alexlz
    Commented 10 мар. 2013 в 10:09
  • +1, хорошее объяснение.
    – VladD
    Commented 10 мар. 2013 в 13:01
  • 1
    @good, если уж на то пошло, то в ООП "это" называется не функциями, а методами класса. А уже между этими двумя понятиями существует несколько серьезных отличий.
    – AseN
    Commented 10 мар. 2013 в 13:21

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.