Выход за границы в методе изменения размера изображения

@Ifreqo · Регистрация: 08.12.2022

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

Мне необходимо реализовать метод изменения размера изображения. Написал его, но постоянно получаю выход за границы. Подскажите где не так написал. В предоставленном коде в функции main условное использование этого метода(в основной программе использоваться будет так же)

image.h

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
#pragma once
 
#include <array>
#include <cassert>
#include <cstddef>
#include <vector>
#include <string>
#include <stdexcept>
#include <filesystem>
#include <fstream>
 
namespace img_lib
{
    using namespace std::string_literals;
    using Path = std::filesystem::path;
 
    struct Color
    {
        Color() = default;
        Color(uint8_t r_, uint8_t g_, uint8_t b_) : r(r_), g(g_), b(b_) {}
        Color(uint8_t r_, uint8_t g_, uint8_t b_, uint8_t a_) : r(r_), g(g_), b(b_), a(a_) {}
 
        static Color Black()
        {
            return { 0, 0, 0, 255 };
        }
 
        static Color White()
        {
            return { 255, 255, 255, 255 };
        }
 
        uint8_t r;
        uint8_t g;
        uint8_t b;
        uint8_t a;
 
        Color operator*(float scalar_) const
        {
            return
            {
                static_cast<uint8_t>(r * scalar_),
                static_cast<uint8_t>(g * scalar_),
                static_cast<uint8_t>(b * scalar_),
                static_cast<uint8_t>(a * scalar_)
            };
        }
 
        Color operator+(const Color& other_) const
        {
            return
            {
                static_cast<uint8_t>(r + other_.r),
                static_cast<uint8_t>(g + other_.g),
                static_cast<uint8_t>(b + other_.b),
                static_cast<uint8_t>(a + other_.a)
            };
        }
 
        Color& operator+=(const Color& other_)
        {
            r += other_.r;
            g += other_.g;
            b += other_.b;
            a += other_.a;
            return *this;
        }
    };
 
    class Image
    {
    public:
 
        Image() = default;
        Image(int w_, int h_) : width(w_), height(h_) {}
        Image(int w_, int h_, Color fill_);
 
        Image(const Image& other_);
        Image& operator=(const Image& other_);
 
        Image(Image&& other_) noexcept;
        Image& operator=(Image&& other_) noexcept;
 
        explicit operator bool() const
        {
            return GetWidth() > 0 && GetHeight() > 0;
        }
 
        bool operator!() const
        {
            return !operator bool();
        }
 
        const Color& GetPixel(int x_, int y_) const;
        Color& GetPixel(int x_, int y_);
 
        std::vector<Color>& GetPixels();
        const std::vector<Color>& GetPixels() const;
 
        Color* GetLine(int y_);
        const Color* GetLine(int y_) const;
 
        int GetWidth() const;
        int GetHeight() const;
 
        int GetStep() const;
 
        const uint8_t* GetData() const;
 
        void SetPixel(int x, int y, const Color& pixel);
 
        Image ResizeImage(int newWidth_, int newHeight_) const;
 
    private:
 
        int width = 0;
        int height = 0;
        int step = 0;
 
        std::vector<Color> pixels;
 
        void CheckBounds(int x_, int y_) const;
        Color BilinearInterpolation(const Color& c00_, const Color& c01_, const Color& c10_, const Color& c11_, float dx_, float dy_) const;
    };
 
}//end namespace img_lib

image.cpp

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
#include "image.h"
 
namespace img_lib
{
    Image::Image(int w_, int h_, Color fill_) : width(w_), height(h_), step(w_), pixels(w_* h_, fill_) {}
 
    Image::Image(const Image& other_) : width(other_.width), height(other_.height), step(other_.step), pixels(other_.pixels) {}
 
    Image::Image(Image&& other_) noexcept : width(other_.width), height(other_.height), step(other_.step), pixels(std::move(other_.pixels))
    {
        other_.width = 0;
        other_.height = 0;
        other_.step = 0;
    }
 
    Image& Image::operator=(const Image& other_)
    {
        if (this != &other_)
        {
            width = other_.width;
            height = other_.height;
            step = other_.step;
            pixels = other_.pixels;
        }
        return *this;
    }
 
    Image& Image::operator=(Image&& other_) noexcept
    {
        if (this != &other_)
        {
            width = other_.width;
            height = other_.height;
            step = other_.step;
            pixels = std::move(other_.pixels);
 
            other_.width = 0;
            other_.height = 0;
            other_.step = 0;
        }
        return *this;
    }
 
    const Color& Image::GetPixel(int x_, int y_) const
    {
        CheckBounds(x_, y_);
        return pixels[y_ * step + x_];
    }
 
    Color& Image::GetPixel(int x_, int y_)
    {
        CheckBounds(x_, y_);
        return pixels[y_ * step + x_];
    }
 
    void Image::SetPixel(int x_, int y_, const Color& pixel_)
    {
        pixels[y_ * width + x_] = pixel_;
    }
 
    std::vector<Color>& Image::GetPixels()
    {
        return pixels;
    }
 
    const std::vector<Color>& Image::GetPixels() const
    {
        return pixels;
    }
 
    Color* Image::GetLine(int y_)
    {
        CheckBounds(0, y_);
        return &pixels[y_ * step];
    }
 
    const Color* Image::GetLine(int y_) const
    {
        CheckBounds(0, y_);
        return &pixels[y_ * step];
    }
 
    int Image::GetWidth() const
    {
        return width;
    }
 
    int Image::GetHeight() const
    {
        return height;
    }
 
    int Image::GetStep() const
    {
        return step;
    }
 
    const uint8_t* Image::GetData() const
    {
        if (pixels.empty())
        {
            return nullptr;
        }
        return reinterpret_cast<const uint8_t*>(pixels.data());
    }
 
    Image Image::ResizeImage(int newWidth_, int newHeight_) const
    {
        Image newImage(newWidth_, newHeight_);
 
        float scaleX = static_cast<float>(width) / newWidth_;
        float scaleY = static_cast<float>(height) / newHeight_;
 
        for (int y = 0; y < newHeight_; ++y)
        {
            for (int x = 0; x < newWidth_; ++x)
            {
                float srcX = (x + 0.5f) * scaleX - 0.5f;
                float srcY = (y + 0.5f) * scaleY - 0.5f;
 
                int x0 = static_cast<int>(srcX);
                int y0 = static_cast<int>(srcY);
                int x1 = x0 + 1;
                int y1 = y0 + 1;
 
                float dx = srcX - x0;
                float dy = srcY - y0;
 
                Color c00 = GetPixel(x0, y0);
                Color c01 = GetPixel(x0, y1);
                Color c10 = GetPixel(x1, y0);
                Color c11 = GetPixel(x1, y1);
 
                Color interpolatedColor = BilinearInterpolation(c00, c01, c10, c11, dx, dy);
                newImage.SetPixel(x, y, interpolatedColor);
            }
        }
 
        return newImage;
    }
 
    Color Image::BilinearInterpolation(const Color& c00_, const Color& c01_, const Color& c10_, const Color& c11_, float dx_, float dy_) const
    {
        Color result;
 
        result.r = static_cast<uint8_t>
            (
                (1 - dx_) * (1 - dy_) * c00_.r +
                (1 - dx_) * dy_ * c01_.r +
                dx_ * (1 - dy_) * c10_.r +
                dx_ * dy_ * c11_.r
            );
 
        result.g = static_cast<uint8_t>
            (
                (1 - dx_) * (1 - dy_) * c00_.g +
                (1 - dx_) * dy_ * c01_.g +
                dx_ * (1 - dy_) * c10_.g +
                dx_ * dy_ * c11_.g
            );
 
        result.b = static_cast<uint8_t>
            (
                (1 - dx_) * (1 - dy_) * c00_.b +
                (1 - dx_) * dy_ * c01_.b +
                dx_ * (1 - dy_) * c10_.b +
                dx_ * dy_ * c11_.b
            );
 
        result.a = static_cast<uint8_t>
            (
                (1 - dx_) * (1 - dy_) * c00_.a +
                (1 - dx_) * dy_ * c01_.a +
                dx_ * (1 - dy_) * c10_.a +
                dx_ * dy_ * c11_.a
            );
 
        return result;
    }
 
    void Image::CheckBounds(int x_, int y_) const
    {
        if (x_ < 0 || x_ >= width || y_ < 0 || y_ >= height)
        {
            throw std::out_of_range("Pixel coordinates out of range"s);
        }
    }
 
}//end namespace img_lib

main.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <iostream>
#include "image.h"
 
using namespace std;
using namespace img_lib;
 
int main()
{
    Image image(100, 100);
 
    cout << image.GetHeight() << " - " << image.GetWidth() << endl;
 
    image.ResizeImage(200, 200);
 
    cout << image.GetHeight() << " - " << image.GetWidth() << endl;
}

@TheCalligrapher · 29.09.2024, 09:21

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

Вылетало действительно каждый раз на GetPixel

Еще раз повторяю: если "вылетало действительно каждый раз на GetPixel", то вылетает и сейчас. Данное исправление никакого отношения к GetPixel не имеет.

Новые блоги и статьи Все статьи Все блоги /
Особенности gif сгенерированных в Си russiannick 21.10.2026 Рассмотрим генерацию gif файлов, которые в Си имеют свои особенности. Gif представляет собой графический формат популярный в интернете. Посмотрим на палитру. Gif файл сгенерированный в си отличается. . .	FPS контроллер с анимированными Y-Bot и 3D-звуком miniaudio (Box3D, Recast, BehaviorTree.CPP, ozz-animation, ImGui, OpenGL и SDL3) 8Observer8 18.10.2026 Содержание блога \| English Version Полное мультиплатформенное руководство по стартовому проекту с FPS-контроллером игрока, анимированными роботами Y-Bot с 4-слойным блендингом, 3D-физикой Box3D,. . .	Локальный http сервер Argus19 17.10.2026 Локальный http сервер На работе, на двух локальных компьютерах круглосуточно работают программы опроса приборов с записью в книги Excel,которые раз в сутки записываются на сервер компании. Т. к. . . .	Копировать Windows на новый диск на UEFI damix 17.10.2026 https:/ / uriesk. wordpress. com/ 2016/ 11/ 01/ move-window-10-to-ssd-on-a-uefi-dualboot-system-with-basic-linux-tools/ TL DR Вставляем диск, а старый подключаем через переходник, грузимся с лайва. . .
ИИ не может найти нужный язык в списке Supersumestria 05.10.2026 Я ему даю вот такое изображение и прошу найти и подчеркнуть немецкий язык. Возвращает он вот это: https:/ / i. **********/ vqBWLe2. png Нужную строчку в 3й колонке просто выдумал. . Это. . .	Новая последняя моя музыка в SUNO zorxor 05.10.2026 Здравствуйте, дорогие мои друзья! С большой радостью я хотел бы представить вам свою новую последнею музыку, которую сгенерировала мне по моей просьбе нейросеть SUNO. С уважением, zorxor. Это. . .	Программный домашний кинотеатр russiannick 27.09.2026 Сподобился на программный домашний кинотеатр. В качестве ЯВУ по традиции выбрал js. В помощники взял Яндекс-Алису. Было создано три зала на разные интересы. исторические и ретро сериал Хичкок. . .	Беседа с ИИ о программистах, недопускающих к созданию и правке кода генеративные ИИ и причины этого zorxor 21.09.2026 Раньше я радовался или получал некоторые эмоции, пусть небольшие, но всё же, от самого процесса написания кода, рекомпиляции и запуска, видя постепенное развитие программы и прочее. А теперь лень. . .