2

Иногда копаюсь в исходниках .NET ради интереса и очень часто встречаю длиннющие методы, например.

Во всяких книгах пишут, что метод не должен превышать размер одного экрана прокрутки.

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

Разве это правильно? Или Microsoft - это плохой пример для подражания?

2
  • 3
    Там же используется условная компиляция. Из-за нее сложно поделить на методы. К тому же, это mscorlib - ядро дотнета, самая часто испоьзуемая библиотека, она должна быть максимально эффективной и быстрой. В данном случае можно поступиться читаемостью методов, обеспечив производительность. Об этом говорит и использование unsafe. 21 дек 2018 в 10:25
  • Текущие ответы не полны. Почему-то никто не захотел в ответе указать то, о чём пишет @AlexanderPetrov
    – A K
    6 янв 2019 в 12:03

3 ответа 3

7

То что вы описали, скорее совет, нежели правило. Если логика метода такова, что ему нужно быть на 5 экранов, то почему нет. Другой вопрос, если можно логику разделить на отдельные методы, то для лучшей читаемости это стоит сделать. Но в то же время не стоит пораждать класс методами, которые будут вызываться лишь 1 раз и нигде больше не использоваться.

Во всем главное соблюдать баланс и придерживаться здравого смысла!

2
  • Ну не знаю. Мне кажется, что любой код можно разбить на маленькие методы, если конечно там не 5 экранов объявлений переменных... То что функция вызывается 1 раз фиксится созданием какого-нибудь внутренного статического класса хелпера или внутренних функций(C# 7.0)
    – iluxa1810
    21 дек 2018 в 5:50
  • я не программирую на C#, но подозреваю, что как и в любом языке статический класс-хелпер - это как раз тот кейс, когда есть "неудобный код", который нужен везде и в разных местах, но нельзя отнести к одному какому-либо случаю
    – Evgeny
    21 дек 2018 в 12:18
2

Если не забывать о том, что программирование это вид искусства, то правильного ответа на данный вопрос быть не может. У разных людей разные чувства стиля, вкуса, корректности, полноты, желания, темперамента и так далее. Есть только несколько рекомендаций, но люди понимают их по-своему. Пишите понятно, красиво, атомарно (с точки зрения цели функции, или метода). Если ваша атомарность это пять экранов, что же и в программировании есть место для Церетели ))

2

Во всяких книгах пишут...

В этих книгах пишут, в основном, про разработку корпоративного ПО.

У корпоративного и системного ПО (например, .NET core) разные требования, связанные со спецификой разработки и использования.

Ваш ответ

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

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