Большинство стандартов программирования выступают за разделение длинных функций на более короткие и использование пустых строк для разделения функции на меньшие единицы.
Две основные причины этого - читаемость и проверяемость.
Если вы разделите длинную функцию на более мелкие, каждая меньшая функция может быть проверена отдельно.
Что вы делаете, лучше, чем просто иметь большую функцию, но вы упускаете часть проверяемости.
В зависимости от того, как выглядит ваш код, вы также можете упускать возможность чтения. Помните, что функция имеет имя, которое описывает это. У блоков нет имен, поэтому для его описания вам понадобится комментарий.
При правильном использовании блоки могут использоваться для повышения удобочитаемости. Однако то же самое верно для простой пустой строки. Преимущество блока над пустой строкой заключается в том, что он ограничивает область локальных переменных, что повышает читаемость.
Итак, для каждого создаваемого вами блока вы должны учитывать, соответствует ли он пустой строке или вызову функции. В последнем случае преобразуйте блок в отдельную функцию.