Форум программистов, компьютерный форум, киберфорум
C# для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.89/18: Рейтинг темы: голосов - 18, средняя оценка - 4.89
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236

Убрать однотипные и повторяющиеся участки кода

06.07.2015, 23:40. Показов 4002. Ответов 22
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
static void fns(int val)
{
    int a = 0, b = 0, c = 0;
    
    if (val > 0)
    {
        a++;
        b++;
    }
    else if (val < 0)
    {
        a++;
        b++;
        c++;
    }
    else
    {
        c++;
    }
}
это только пример, что пришло в голову да бы продемонстрировать суть вопроса, как видите в if есть повторяющийся код, в частности:
C#
1
2
a++;
b++;
можно как-нибудь сделать, что бы не дублировать каждый раз?

PS создание именно отдельного метода, делающего эту грязную работу не подойдет, уж проще дублировать.
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
06.07.2015, 23:40
Ответы с готовыми решениями:

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

Участки кода, выполняемые только в режиме Debug
Есть у меня 2 вопроса: 1. Можно ли сделать так, чтобы определенные строки кода выполнялись только тогда, когда в студии выбрана сборка...

Объяснить некоторые участки кода (создание класса: методы, перегрузки, .)
Мне помогли написать код using System; internal class Complex : IEquatable&lt;Complex&gt; { // int a действительная часть - int...

22
 Аватар для diadiavova
7259 / 2606 / 744
Регистрация: 11.04.2015
Сообщений: 4,150
Записей в блоге: 43
06.07.2015, 23:50
VLK, а в чем смысл? В реальном коде много таких конструкций или просто хочется одну-две строчки сэкономить? Ну вот так вроде тот же результат будет
C#
1
2
3
4
5
6
            if (val != 0)
            {
                a++;
                b++;
            }
            if (val <= 0) c++;
1
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
06.07.2015, 23:55  [ТС]
diadiavova, Даценд, нет это не то, то что я написал это только пример, в реальности там другие условия и побольше кода, а не а++..
тут надо именно как то через делегаты или лямбда выражения..

ну если часто встречается, наверное кто то и призадумывался как упростить такое.
0
 Аватар для diadiavova
7259 / 2606 / 744
Регистрация: 11.04.2015
Сообщений: 4,150
Записей в блоге: 43
06.07.2015, 23:57
VLK, я думаю, что если ты не дашь более-менее близкий к реальности пример, то вряд ли можно будет понять, что именно тебе надо.
1
52 / 13 / 0
Регистрация: 05.09.2014
Сообщений: 92
07.07.2015, 00:05
Не совсем уверен правильно ли я вас понял ...
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
static void fns(int val)
{
int a=0, b=0, c=0;
if(val!=0)
{
a++;
b++;
}
if(val<=0)
{
c++;
}
}
Это не то ?

Добавлено через 26 секунд
Пока писал, уже столько накатали.

Добавлено через 4 минуты
VLK, Я тоже голову ломал, пока умные люди на сайте меня не просветили. Оказывается в компиляторы обычно встраиваются оптимизаторы кода, то есть программы анализирующие код и делающие его более эффективным. Так что то, о чем вы думаете, уже сто раз обдумано и сделано.
1
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
07.07.2015, 00:08  [ТС]
diadiavova, а зачем тонны непонятного кода? главное тут, что данную проблему нельзя решить путем изменения условий в if - else.


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

Добавлено через 2 минуты
Tini, что там под капотом будет мне не суть важно, сам код написан нормально, мне бы сделать как то так, что бы мне меньше писать.
0
52 / 13 / 0
Регистрация: 05.09.2014
Сообщений: 92
07.07.2015, 00:09
А почему нельзя дублируемый код вывести в отдельную функцию ?
1
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
07.07.2015, 00:10  [ТС]
еще раз повторюсь, в реальном коде невозможно поменять условия if else, тут надо как то по другому решать

Добавлено через 29 секунд
Tini, ну мне кажется оно того не заслуживает
0
52 / 13 / 0
Регистрация: 05.09.2014
Сообщений: 92
07.07.2015, 00:16
VLK, Я как-то плохо Вас понимаю. Нельзя поменять условия if else ? То есть Вы не можете поменять код ? Уверен, если бы вы привели здесь хотя бы упрощенный код вашей программы, передающий основную суть проблемы, вам бы помогли, а пока что это топтание на месте без конкретики.
1
 Аватар для diadiavova
7259 / 2606 / 744
Регистрация: 11.04.2015
Сообщений: 4,150
Записей в блоге: 43
07.07.2015, 00:23
Цитата Сообщение от VLK Посмотреть сообщение
а зачем тонны непонятного кода?
Не знаю. А почему этот вопрос адресован мне? Я разве выступал за тонны непонятного кода? Я только сказал, что на данный момент непонятна сама задача.
То, что ты описываешь дальше не особенно проливает свет на задачу. В принципе выделить повторяющуюся часть кода и скомбинировать ее с изменяющейся - это задача не просто разрешима, а очень многое во всех языках программирования направлено на решение именно этой задачи. Но более предметно об этом можно говорить только на более конкретных примерах.
1
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
07.07.2015, 00:27  [ТС]
diadiavova, Tini,
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
WebClient wc = new WebClient();
string html = null;
 
do {
    
    try
    {
        html = DownloadString(uri);
        if(/* тут проверяется html и если... */) // допустим тот или не тот код
        {
            // тут выполняется несколько действий 
            // из которых 2 будут выполнятся во всех условиях дальше, в том числе и в catch
            break; 
        }
        else
        {
            // тут выполняется несколько действий включая те 2, что описаны выше
            continue; 
        }
    }
    catch (WebException ex)
    {
        if(/* если ex что то там, тогда.. */) // допустим был сайт недоступен или же мы получили 404
        {
            // тут выполняется несколько действий включая те 2, что описаны выше
            continue; 
        }
        else
        {
            // тут выполняется несколько действий включая те 2, что описаны выше
            break; 
        }
    }
 
} while(/* какое то условие */);
я надеюсь этот код внесет понимание происходящего
0
Эксперт .NET
 Аватар для Даценд
5878 / 4755 / 2939
Регистрация: 20.04.2015
Сообщений: 8,361
07.07.2015, 00:31
VLK,
А если те 2 выполнить в блоке finally. Или важна последовательность?
1
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
07.07.2015, 00:32  [ТС]
Даценд, а continue и break разве не обломят этот finally? вроде как будет недостижимый код.
0
Эксперт .NET
 Аватар для Даценд
5878 / 4755 / 2939
Регистрация: 20.04.2015
Сообщений: 8,361
07.07.2015, 00:37
Цитата Сообщение от VLK Посмотреть сообщение
continue и break разве не обломят этот finally?
Да его даже return не обломит. Если только Application.Exit справится.
1
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
07.07.2015, 00:46  [ТС]
Даценд, тогда в данном случае прокатит, а в целом, еще как-нибудь можно это решить? я вот сейчас подумал можно через Action решить.
Я просто тут увлекся РНР, так там аналог делегата это анонимная функция, это именно функция, она не имеет доступа к свойствам класса, а тут делегат выступает как метод класса и имеет доступ к свойствам, оказывается.

Добавлено через 6 минут
если допустим в этом коде в одном из мест не надо будет использовать дублирование:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
WebClient wc = new WebClient();
string html = null;
 
do {
    
    try
    {
        html = DownloadString(uri);
        if(/* тут проверяется html и если... */) // допустим тот или не тот код
        {
            // тут выполняется несколько действий 
            // из которых 2 будут выполнятся во всех условиях дальше КРОМЕ ОДНОГО, в том числе и в catch
            break; 
        }
        else
        {
            continue; 
        }
    }
    catch (WebException ex)
    {
        if(/* если ex что то там, тогда.. */) // допустим был сайт недоступен или же мы получили 404
        {
            // тут выполняется несколько действий включая те 2, что описаны выше
            continue; 
        }
        else
        {
            // тут выполняется несколько действий включая те 2, что описаны выше
            break; 
        }
    }
 
} while(/* какое то условие */);
выходит тут уже finally не прокатит, как быть?
0
Заблокирован
07.07.2015, 01:07
Цитата Сообщение от VLK Посмотреть сообщение
если допустим в этом коде в одном из мест не надо будет использовать дублирование:
а вы уверены что правильно обернули конструкцию в try - catch? На каком этапе может возникнуть WebException?

Не по теме:

Цитата Сообщение от VLK Посмотреть сообщение
Я просто тут увлекся РНР, так там аналог делегата это анонимная функция, это именно функция, она не имеет доступа к свойствам класса, а тут делегат выступает как метод класса и имеет доступ к свойствам, оказывается.
VLK, не путайте делегаты и лямбда-выражения.

1
Эксперт .NET
 Аватар для Wolfdp
3790 / 1767 / 371
Регистрация: 15.06.2012
Сообщений: 6,543
Записей в блоге: 3
07.07.2015, 05:35
Тоже когда-то страдал похожим. Главное в коде, как правило, читаемость. Поєтому я бьі оставил if/else и не трогал. Плюс если оборачивать в методьі/лямда, єто все кушает время вьізова, на десятке итераций не заметно, а вот на 1-2кк уже начинаешь замечать разницу даже на глаз.

Цитата Сообщение от VLK Посмотреть сообщение
я вот сейчас подумал можно через Action решить
делегат, єто по сути тот же метод, только обернутьій в цельій класс. С ним есть пару ньюансов, по поводу захвата переменьіх, а так ничего отличного.

В общем, если ваши 2 действия вмещаються в две строчки по два-три десятка символа в каждой, лучше вообще не химичить.
1
 Аватар для diadiavova
7259 / 2606 / 744
Регистрация: 11.04.2015
Сообщений: 4,150
Записей в блоге: 43
07.07.2015, 10:26
Цитата Сообщение от VLK Посмотреть сообщение
а тут делегат выступает как метод класса и имеет доступ к свойствам, оказывается.
А это принципиальный вопрос? Все, к чему метод не имеет непосредственного доступа, можно передать с аргументами.
0
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
07.07.2015, 13:45  [ТС]
Ev_Hyper, метод используя WebClient через прокси (WebProxy) получает html страницы, но выяснилось что прокси довольно коварная штука:
некоторые просто не отвечают (скорее всего не работаю) тут возникает WebException,
некоторые возвращают код ошибки прокси, ну т.е. понятно что виновен прокси, тут возникает WebException,
а некоторые при попытке получить существующею страницу возвращают ошибку 404 (страница не существует), хотя она существует, тут тоже возникает WebException.

я еще пока не сталкивался, но так же возможно я могу получать от прокси html но в итоге это будет совсем не тот html что мне нужно (ну например прокси сгенерирует свой html, типа "куда лезешь").

по этому там нужно много проверок, как в try так и catch и как минимум в 3-х случаях у меня повторяется код с подсчетом количества ошибок (сколько сделано запросов, сколько из них успешный и т.д.), по этому хотелось бы его не каждый раз писать, а вот как то, я думал можно есть какой то способ но я просто о нем не знаю, но как я понимаю нет.

Делегат (Action) не проканает, т.к. мне получается ему надо будет передавать по ссылке данные, а он не принимает ссылки.

PS еще немного об устройстве метода:
у меня есть список очередь (Queue) прокси
я в цикле do - while выдергиваю прокси, делаю запрос
если потерпел неудачу, я этот прокси засовываю обратно в конец очереди, беру новый и по новой выполняю запрос
пока не получу html или пока та или иная ошибка не повторится допустим 10 раз

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

Добавлено через 40 секунд
diadiavova, по ссылке он данные не принимает, это обламывает.
0
 Аватар для VLK
198 / 170 / 19
Регистрация: 05.05.2013
Сообщений: 1,236
13.08.2015, 21:21  [ТС]
Решение нашлось
Выяснилось что в делегат Action и Func не надо ни чего передавать, они видят содержимое метода и без передачи, что приятно удивило.

C#
1
2
3
4
5
6
7
8
9
10
11
static void test()
{
    int i = 0;
    Action act = delegate { i++; };
 
    while (i < 10) 
    {
        Console.WriteLine(i);
        act();
    }
}
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
13.08.2015, 21:21
Помогаю со студенческими работами здесь

Почему участки кода со "switch-case" - это плохо и реплейсятся при рефакторинге?
Попалась мне статейка одна на Хабре, где давались рекомендации по грамотному написанию кода. Одним из пунктов в ней был отказ от...

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

Убрать повторяющиеся пути через .Distinct();
Как убрать повторяющиеся файлы элементов листа, состоящего из путей к этим файлам(string)? например List&lt;string&gt; paths = new...

Оптимизировать повторяющиеся участки кода
:wall:Как зациклить уровнение p, чтобы не писать уравнение три часа? вот код с прогой: program rgr2n3; const s=20;t=10;n=12; ...

Samsung SCX-4220 при печати получаю повторяющиеся участки с размытием
Приветствую. МФУ Samsung SCX-4220 при печати получаю повторяющиеся участки с размытием в центре листа(скрин прилагаю). Картридж родной,...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru