Как лучше ветвить метод

@Fylhtq05 · Регистрация: 09.06.2021

Студворк — интернет-сервис помощи студентам

Есть ли какая-нибудь выгода:

C#
1
2
3
4
string s;
if(a)s="a";
else s="b";
AnyMethod(s);

перед:

C#
1
2
if(a)AnyMethod("a");
else AnyMethod("b");

Или без разница и писать можно как угодно?

@kolorotur · 20.10.2021, 01:00

Сообщение от Fylhtq05

Есть ли какая-нибудь выгода

Практической выгоды никакой.
Можете писать вообще так:

C#
1
AnyMethod(a ? "a" : "b");

@Fylhtq05 · 20.10.2021, 01:30 **[ТС]**

Сообщение от kolorotur

Практической выгоды никакой.

Хотел узнать о такой ситуации: есть участок кода, который производит несколько вычислений. Вычисления это две или три строки кода. Либо для каждой из двух ветвей писать этот код со своими параметрами (т.е. для второй ветви просто копирование и подставка нужных параметров), либо ветвить инициализацию параметров и передавать их в участок кода, в котором происходят вычисления. С точки зрения быстроты кодирования первый вариант предпочтительнее. Но может есть в нем недостатки? Первый способ похож на инлайн методов. Но почему-то не все методы компилятор инлайнит. Значит есть какие-то недостатки?

@kolorotur · 20.10.2021, 01:35

Fylhtq05, я бы в этом случае сначала вычислял аргументы, а потом после условия передавал их в метод — как у вас в первом варианте. Код обычно более читаемый с таким подходом.

Сообщение от Fylhtq05

Но может есть в нем недостатки?

Зависит от того, сколько переменных вычисляется и что за вычисления.
Если там чего-то много происходит, то можно вообще вынести в отдельный метод, чтобы не валить все в одну кучу.

@Fylhtq05 · 20.10.2021, 01:52 **[ТС]**

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

@kolorotur · 20.10.2021, 02:04

Сообщение от Fylhtq05

Есть у инлайна, кроме увеличения экзешника, другие недостатки?

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

@Fylhtq05 · 20.10.2021, 13:12 **[ТС]**

Сообщение от kolorotur

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

А что он там конкретно оптимизирует? Разве он не переводит просто IL в машинные коды? Если программист опытный, то сможет ли что-то джиттер оптимизировать?

@kolorotur · 20.10.2021, 14:38

Сообщение от Fylhtq05

А что он там конкретно оптимизирует?

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

Сообщение от Fylhtq05

Разве он не переводит просто IL в машинные коды?

Любой компилятор переводит инструкции с одного языка на другой.

Сообщение от Fylhtq05

Если программист опытный, то сможет ли что-то джиттер оптимизировать?

Современные компиляторы оптимизируют намного лучше, чем 99% программистов.
Остальные 1% пишут на ассемблере или сях с ассемблерными вставками.

@Fylhtq05 · 20.10.2021, 15:43 **[ТС]**

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

@kolorotur · 20.10.2021, 15:54

Сообщение от Fylhtq05

есть большущий метод на несколько 1000 строк

Дальше можно и не читать — метод срочно нуждается в рефакторинге

Сообщение от Fylhtq05

не нужно ли его все равно разбивать на несколько методов, чтобы джиттер смог делать оптимизацию?

Тут уже не вопрос оптимизации, а банального восприятия.
Метод на 1000 строк — это кошмар.

Сообщение от Fylhtq05

и потому никакого выигрыша не будет?

Сделайте сначала человеческий рефакторинг, потом прогоните оба варианта через бенчмарк и посмотрите на результаты.

@Usaga · 20.10.2021, 15:58

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

@Fylhtq05 · 20.10.2021, 16:18 **[ТС]**

Сообщение от kolorotur

метод срочно нуждается в рефакторинге

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

@Usaga · 20.10.2021, 16:33

Сообщение от Fylhtq05

А какой может быть рефакторинг, если нет повторяющихся участков кода?

Выносят не только повторяющийся код. Код может разъехаться не просто по разным методам, но и по разным классам, для разделения ответственностей или введения нужно уровня абстракций.

@kolorotur · 20.10.2021, 16:51

Сообщение от Fylhtq05

А какой может быть рефакторинг, если нет повторяющихся участков кода?

Ну если там 1000 строк, то наверняка различные процессы свалены в одну кучу.
Начать можно с разделения этих процессов на отдельные методы, чтобы глядя на код метода даже незнакомому с кодом человеку было понятно что там происходит.
Кому интересны детали — посмотрит реализацию методов.

Сообщение от Fylhtq05

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

Именно!

Сообщение от Fylhtq05

Вместо разбивки метода в редакторе сделаны отступы и комментарии

Это, кстати, хороший ориентир при рефакторинге: вынести то, что описано комментариями/отступами в отдельные методы.
Наверняка где-то в комментарии есть и название метода.

@Fylhtq05 · 20.10.2021, 17:27 **[ТС]**

Сообщение от kolorotur

Ну если там 1000 строк, то наверняка различные процессы свалены в одну кучу.

Совершенно верно. В этом есть что-то плохое? Я только для себя пытаюсь написать программу арбитража криптовалютной биржи. Потому, какие сложности испытывал бы другой человек, читая программу (мне его было бы жалко), меня в данном случае не интересует? Забота только о производительности. И вообще непонятно, чем вынесение кода для восприятия в метод лучше чем сворачивающийся region с комментариями?

@IamRain · 20.10.2021, 20:29

Сообщение от Fylhtq05

Забота только о производительности. И вообще непонятно, чем вынесение кода для восприятия в метод лучше чем сворачивающийся region с комментариями?

Ну можно и картоху жареную всей семьей из 6 человек есть прямо из большой сковородки, просто разделяя секторами нужные куски с помощью кетчупа. Так тоже едят.

Но все-таки чаще всего по-человечески (каждому отдельно) будет удобнее.
Так и тут - код читают люди.

@Fylhtq05 · 20.10.2021, 21:01 **[ТС]**

Сообщение от IamRain

Так и тут - код читают люди.

Про картошку ничего говорить не буду, а по поводу кода непонятно, чем region хуже метода? Хотелось чтобы не рассматривали мой вопрос, как позицию. Здесь было сказано, что короткие методы джиттер оптимизирует. Это плюс к тому, чтобы большой метод разбивать на маленькие. А вот с точки зрения читабельности пока преимуществ перед region с комментариями не нахожу. Мало того, регион можно развернуть и просмотреть быстрее, чем в меню перейти к методу, что приведет к строкам в другом файле, а стрелочка вернуться назад не факт, что вернет к строке кода, с которой начат переход к методу а не к другой строке в том файле, где интересующий метод. А если в меню нажать показать метод, то это тоже уступает региону: точку останова не поставить, рефакторинг не сделать. Придется закрывать окошко и нажимать перейти к методу. Т.е. хотелось бы услышать в чем читабельное преимущество разбиения на методы перед регионами с комментариями. И как-это связано с картошкой? Еще раз - это вопрос.

@IamRain · 20.10.2021, 21:15

Сообщение от Fylhtq05

Т.е. хотелось бы услышать в чем читабельное преимущество разбиения на методы перед регионами с комментариями.

В здравом смысле.
Метод - логически завершенный повторно используемый кусок кода, имеющий имя.
Есть один логический бизнес-процесс, состоящий из двух подпроцессов - соответственно, логически создавайте два метода, с осмыленными имена. Самодокументируемый код здесь - наше все.
А то, что там глазками навигацию сложно выполнять - так это следствие сложности разрабатываемой ИС. Вы и в регионах запутаетесь.
10 логически разных "проектов" в 10 файлах? А 20? А 50? Для такого кодописаки надо мачету...

И размер exe в случае inline-инга методов - особо не проблема. Вы размер exe основного сервиса СУБД Postgresql видели? - 9-10 MB.
В 21 веке то, где рулят микросервисы...да пофиг.

@Fylhtq05 · 20.10.2021, 21:25 **[ТС]**

Сообщение от IamRain

В здравом смысле.

Угу, по определению, значит.

Сообщение от IamRain

Метод - логически завершенный повторно используемый кусок кода, имеющий имя.

Вот именно, повторно. А если нет повтора?

Сообщение от IamRain

Вы и в регионах запутаетесь.

Что уж говорить о методах, когда даже в регионах запутаюсь.

Сообщение от IamRain

Для такого кодописаки надо мачету...

Спасибо за совет.

@IamRain · 20.10.2021, 21:31

Сообщение от Fylhtq05

Вот именно, повторно. А если нет повтора?

Ну так никто не заставляет ведь. Ешьте из сковороды, даже если живет один.

Полезно для ознакомления: nice to have

Добавлено через 1 минуту
ps: за 49 баксов, думаю, подойдет.

Новые блоги и статьи Все статьи Все блоги /
Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем	10 пpимет, которые всегда сбываются Maks 31.03.2026 1. Чтобы, наконец, пришла маршрутка, надо закурить. Если сигарета последняя, маршрутка придет еще до второй затяжки даже вопреки расписанию. 2. Нaдоели зима и снег? Не надо переезжать. Достаточно. . .	Перемещение выделенных строк ТЧ из одного документа в другой Maks 31.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .
Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .	Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .

@Fylhtq05 109 / 52 / 16 Регистрация: 09.06.2021 Сообщений: 480
	20.10.2021, 01:52 [ТС]
	А если все-таки не выносить код в метод и не инициализировать параметры, а просто делать копирование с подставкой параметров, т.е. аналог инлайна. Есть у инлайна, кроме увеличения экзешника, другие недостатки? Кажется, где-то читал, что в будущем можно будет инлайнить методы по своему усмотрению. Если недостатков нет, кроме размера исполняемого файла, имеет смысл инлайнить почти все методы, т.к. диски сейчас большие, а при инлайне не происходит задержка, связанная с вызовом метода и передачи в него параметров? 0

@Fylhtq05 109 / 52 / 16 Регистрация: 09.06.2021 Сообщений: 480
	20.10.2021, 15:43 [ТС]
	В таком случае если есть большущий метод на несколько 1000 строк, в котором нет фрагментов, которые могли бы использоваться в других методах, не нужно ли его все равно разбивать на несколько методов, чтобы джиттер смог делать оптимизацию? Или все равно в таком случае сначала компилятор сделает инлайн и потому никакого выигрыша не будет? 0

@Usaga 14314 / 9398 / 1355 Регистрация: 21.01.2016 Сообщений: 35,431
	20.10.2021, 15:58
	Fylhtq05, если человек пишет методы на тысячи строк, то оптимизировать он в них ничего не сможет по определению и думать ему нужно в первую очередь о порядке в коде. 1

Как лучше ветвить метод

Решение

Решение