Сократить дублирующийся код

@Wolfdp · Регистрация: 15.06.2012

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

В общем, вот ^^" Это часть кода из масштабирования нарезки изображения. Как видите после условия if (this.countWidth > this.countHeight) идет зеркальная инверсия двух участков. Вроде бы и не критично, но не хотелось бы оставлять это безобразие.

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
36
int indent;
int size;
            if (this.countWidth > this.countHeight)
            {
                if (countHeight > 2)
                {
                    this.countHeight--;
                    do
                    {
                        this.countWidth--;
                        if (imageWidth / this.countWidth < imageHeight / this.countHeight)
                            size = Convert.ToInt32(imageWidth / this.countWidth);
                        else
                            size = Convert.ToInt32(imageHeight / this.countHeight);
                        indent = imageWidth - size * this.countWidth;
                    }
                    while (countWidth > 2 && indent >= 100);
                }
            }
            else
            {
                if (countWidth > 2)
                {
                    this.countWidth--;
                    do
                    {
                        this.countHeight--;
                        if (imageHeight / this.countHeight < imageWidth / this.countWidth)
                            size = Convert.ToInt32(imageHeight / this.countHeight);
                        else
                            size = Convert.ToInt32(imageWidth / this.countWidth);
                        indent = imageWidth - size * this.countWidth;
                    }
                    while (countWidth > 2 && indent >= 100);
                }
            }

Добавлено через 1 час 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
35
36
37
private void MinusButton_Click(object sender, RoutedEventArgs e)
{
    if (this.countWidth > this.countHeight)
    {
        if (countHeight > 2)
        {
            this.countHeight--;
            countWidth = this.Minus(this.countWidth, this.countHeight, this.imageWidth, this.imageHeight);
        }
    }
    else
    {
        if (countWidth > 2)
        {
            this.countWidth--;
            countHeight = this.Minus(this.countHeight, this.countWidth, this.imageHeight, this.imageWidth);
        }
    }
    this.countImage_TextBlock.Text = this.countWidth.ToString() + " x " + this.countHeight.ToString();
}
 
        private int Minus(int countA, int countB, int widthA, int widthB)
        {
            int indent;
            int size;
            do
            {
                countA--;
                if (widthA / countA < widthB / countB)
                    size = widthA / countA;
                else
                    size = widthB / countB;
                indent = widthB - size * countB;
            }
            while (countA > 2 && indent >= 100);
            return countA;
        }

Psilon · 25.02.2013, 00:02

C#
1
2
3
4
5
6
7
8
9
10
11
12
private void MinusButton_Click(object sender, RoutedEventArgs e)
{
    if (this.countWidth > this.countHeight)
    {
        if (countHeight > 2)        
            this.countHeight--;           
    }
    else if (countWidth > 2)
            this.countWidth--;
    countWidth = this.Minus(this.countWidth, this.countHeight, this.imageWidth, this.imageHeight);
    this.countImage_TextBlock.Text = this.countWidth.ToString() + " x " + this.countHeight.ToString();
}

лень думаать, это тут оставлю и все

@Wolfdp · 25.02.2013, 01:07 **[ТС]**

Psilon, ты проглядел, там в зависимости от главного условия в вызове this.Minus меняются местами переменные Width и Height. =^_^= Мне друг сказал, что можно использовать out и засунуть и вторую переменную в функцию.

private void MinusButton_Click(object sender, RoutedEventArgs e)
{
if (this.countWidth > this.countHeight)
{
countWidth = this.Minus(this.countWidth, out this.countHeight, this.imageWidth, this.imageHeight);
}
else
{
countHeight = this.Minus(this.countHeight, out this.countWidth, this.imageHeight, this.imageWidth);
}
this.countImage_TextBlock.Text = this.countWidth.ToString() + " x " + this.countHeight.ToString();
}

private int Minus(int countA, out int countB, int widthA, int widthB)
{
if (countB > 2)
{
countB--;
int indent;
int size;
do
{
countA--;
if (widthA / countA < widthB / countB)
size = widthA / countA;
else
size = widthB / countB;
indent = widthB - size * countB;
}
while (countA > 2 && indent >= 100);
}
return countA;
}

Psilon · 25.02.2013, 01:16

Wolfdp, немудрено проглядеть, когда везде понатыканно куча ненужных this. В методе в 10 строчек и так видно, какие переменные локальные, а какие поля класса, тем более, что тут ни 1 лок. переменной нет...

@Wolfdp · 25.02.2013, 19:38 **[ТС]**

Ну не знаю, сила привычки (в университете по правилам оформления лабы нужно было всегда прописывать this) и мне так легче читать код. Хотя соглашусь - читаемость ещё та...

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	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. На борту пять. . .

@Wolfdp 3790 / 1767 / 371 Регистрация: 15.06.2012 Сообщений: 6,543 Записей в блоге: 3
	25.02.2013, 01:07 [ТС]
	Psilon, ты проглядел, там в зависимости от главного условия в вызове this.Minus меняются местами переменные Width и Height. =^_^= Мне друг сказал, что можно использовать out и засунуть и вторую переменную в функцию. private void MinusButton_Click(object sender, RoutedEventArgs e) { if (this.countWidth > this.countHeight) { countWidth = this.Minus(this.countWidth, out this.countHeight, this.imageWidth, this.imageHeight); } else { countHeight = this.Minus(this.countHeight, out this.countWidth, this.imageHeight, this.imageWidth); } this.countImage_TextBlock.Text = this.countWidth.ToString() + " x " + this.countHeight.ToString(); } private int Minus(int countA, out int countB, int widthA, int widthB) { if (countB > 2) { countB--; int indent; int size; do { countA--; if (widthA / countA < widthB / countB) size = widthA / countA; else size = widthB / countB; indent = widthB - size * countB; } while (countA > 2 && indent >= 100); } return countA; } 0

Psilon Master of Orion 6102 / 4958 / 905 Регистрация: 10.07.2011 Сообщений: 14,522 Записей в блоге: 5
	25.02.2013, 01:16
	Wolfdp, немудрено проглядеть, когда везде понатыканно куча ненужных this. В методе в 10 строчек и так видно, какие переменные локальные, а какие поля класса, тем более, что тут ни 1 лок. переменной нет... 0

@Wolfdp 3790 / 1767 / 371 Регистрация: 15.06.2012 Сообщений: 6,543 Записей в блоге: 3
	25.02.2013, 19:38 [ТС]
	Ну не знаю, сила привычки (в университете по правилам оформления лабы нужно было всегда прописывать this) и мне так легче читать код. Хотя соглашусь - читаемость ещё та... 0