Привет, народ. В методе update есть несколько блоков условных операторов,

вроде:

Update()
{
{
if (…)
{ }
else if (…)
{ }
else
{ }
}

{
if (…)
{ }
else
{ }
}
}

Нужно ли мне эти два блока как-то разграничивать между собой?

10 ответов

17 просмотров
Михаил-Попов Автор вопроса

То есть, пихать все в отдельные методы, а потом вызывать

1 совет ужасен

invariance
1 совет ужасен

нет. Какой максимально длинный метод и как долго ты поддерживал? Моя личная практика: 1. методы длиннее 20-30 строк - для лохов. 2. методы с тремя и более условиЯми/циклами - для длхов В целом, если ты, открыв метод через полгода, не можешь за 5-10 секунд ТОЧНО понять, как он работает - мтеод говно

Moon Walker
нет. Какой максимально длинный метод и как долго т...

> методы длиннее 20-30 строк - для лохов. Ну ок ☹️

Михаил-Попов Автор вопроса
invariance
не делай так

А как лучше? В целом, то, до чего я додумался, на жизнь имеет право? Если внутри Update создавать отдельные тела Update() { { … } { … } }

Михаил Попов
То есть, пихать все в отдельные методы, а потом вы...

в отдельные логические методы, да. Просто представь. У тебя в апдейте уже два НЕЗАВИСИМЫХ блока. А потом появится третий. Четвертый. А потом один из блоков потребуется использовать в другом месте. И вот через неделю или месяц ты обнаружишь гигантский апдейт на 200 строк, в котором делается ВСЁ. Не хочешь методы - хотя бы разбивай комментами на блоки. Твоя задача: сделать код таким, что если ты его бросишь прямо сейчас, а через полгода похмельным утром 1 января ты его откроешь - тебе понадобится 10 секунд чтобы понять, что и как делает ЛЮБОЙ метод. Любой метод должен быть понятен от и до за 10 секунд. Не понятен - бей на методы или добавляй комменты

Moon Walker
нет. Какой максимально длинный метод и как долго т...

я работаю в фирме и у меня может быть sql запрос больше 100 строк, а потом linq еще на столько же и каким образом у меня метод будет 20 строк?

Похожие вопросы

Обсуждают сегодня

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта