Логическая ошибка в коде фильтра усреднения

@Andy24 · Регистрация: 03.11.2015

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

На парах попросили написать код фильтра усреднения.

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
38
private void button5_Click(object sender, EventArgs e)
        {
            
            Bitmap b2 = new Bitmap(b1);
 
            for (int i = 1; i < b1.Width-1; i++)
            {
                for (int j = 1; j < b1.Height-1; j++)
                {
                    int sumR = 0, sumG = 0, sumB = 0;
 
                    for (int n = -1; n < 2; n++)
                    {
                        for (int k = -1; k < 2; k++)
                        {
 
                            c1 = b1.GetPixel(i + n, j + k);
                            sumR += c1.R / 9;
                            sumG += c1.G / 9;
                            sumB += c1.B / 9;
                        }
                        if (sumR > 255) sumR = 255;
                        if (sumR < 0) sumR = 0;
                        if (sumG > 255) sumG = 255;
                        if (sumG < 0) sumG = 0;
                        if (sumB > 255) sumB = 255;
                        if (sumB < 0) sumB = 0;
 
 
                        c2 = Color.FromArgb(sumR, sumG, sumB);
                        b2.SetPixel(i, j, c2);
                    }
                    
                }
            }
            pictureBox2.Image = b2;
            pictureBox2.Refresh();
        }

Переменный с2 и с1 созданы заранее,выше кода идёт подгрузка изображения через OpenFileDialog (отдельный метод, он не указан, т.к. не важен). В общем вопрос такой этот код работает изображение изменяется, но преподаватель сказал что в нём есть логическая ошибка, которая потом приводит к большой ошибке, но как я ни старался я так её и не нашёл. Надеюсь на вашу помощь дорогие форумчане.

@nedel · 03.11.2015, 18:35

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
for (int i = 1; i < b1.Width - 1; i++)
            {
                for (int j = 1; j < b1.Height - 1; j++)
                {
                    int sumR = 0, sumG = 0, sumB = 0;
 
                    for (int n = -1; n < 2; n++)
                    {
                        for (int k = -1; k < 2; k++)
                        {
 
                            c1 = b1.GetPixel(i + n, j + k);
                            sumR += c1.R;
                            sumG += c1.G;
                            sumB += c1.B;
                        }
                    }
                    sumR /= 9;
                    sumG /= 9;
                    sumB /= 9;
                    /* это мне кажется лишним
                    if (sumR > 255) sumR = 255;
                    if (sumR < 0) sumR = 0;
                    if (sumG > 255) sumG = 255;
                    if (sumG < 0) sumG = 0;
                    if (sumB > 255) sumB = 255;
                    if (sumB < 0) sumB = 0;
                    */
 
                    c2 = Color.FromArgb(sumR, sumG, sumB);
                    b2.SetPixel(i, j, c2);
 
 
                }
            }

@Andy24 · 03.11.2015, 18:47 **[ТС]**

C#
1
2
3
4
5
6
7
8
 /* это мне кажется лишним
                    if (sumR > 255) sumR = 255;
                    if (sumR < 0) sumR = 0;
                    if (sumG > 255) sumG = 255;
                    if (sumG < 0) sumG = 0;
                    if (sumB > 255) sumB = 255;
                    if (sumB < 0) sumB = 0;
                    */

А разве это не условие того чтобы не выходило за границы изображения??

@nedel · 03.11.2015, 19:37

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

А разве это не условие того чтобы не выходило за границы изображения??

это условие, чтобы компоненты цвета были в границах от 0 до 255, но они и так там будут

Добавлено через 10 минут
а так будет еще лучше, а то у вас крайние пиксели не подменялись

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
for (int i = 0; i < b1.Width; i++)
            {
                for (int j = 0; j < b1.Height; j++)
                {
                    int sumR = 0, sumG = 0, sumB = 0, cnt = 0;
 
                    for (int n = -1; n < 2; n++)
                    {
                        for (int k = -1; k < 2; k++)
                        {
                            int x = i + n;
                            int y = j + k;
                            if (x >= 0 && x < b1.Width && y >= 0 && y < b1.Height) //условие того чтобы не выходило за границы изображения
                            {
                                c1 = b1.GetPixel(x, y);
                                sumR += c1.R;
                                sumG += c1.G;
                                sumB += c1.B;
                                cnt++;
                            }
                        }
                    }
                    sumR /= cnt;
                    sumG /= cnt;
                    sumB /= cnt;
                    c2 = Color.FromArgb(sumR, sumG, sumB);
                    b2.SetPixel(i, j, c2);
                }
 
            }

@Andy24 · 04.11.2015, 07:53 **[ТС]**

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

C#
1
2
3
4
5
6
7
 cnt++;
                            }
                        }
                    }
                    sumR /= cnt;
                    sumG /= cnt;
                    sumB /= cnt;

Новые блоги и статьи Все статьи Все блоги /
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. На борту пять. . .

Логическая ошибка в коде фильтра усреднения

Решение