Slightly improve code quality of basic display

Author: stevenewald

CMakeLists.txt

@@ -18,6 +18,7 @@ include(cmake/variables.cmake)
 add_library(
     fractal-generator_lib OBJECT
     source/lib.cpp
+    source/graphics/basic_display.cpp
 )
 
 target_include_directories(

source/graphics/basic_display.cpp

@@ -0,0 +1,49 @@
+#include "basic_display.hpp"
+
+#include <SFML/Graphics.hpp>
+
+namespace fractal {
+void BasicDisplay::set_pixel(std::size_t x_pos, std::size_t y_pos, uint8_t value)
+{
+    pixels_.at(x_pos).at(y_pos) = value;
+}
+
+void BasicDisplay::display_window()
+{
+    sf::RenderWindow window(sf::VideoMode(WINDOW_WIDTH, WINDOW_HEIGHT), "SFML Window");
+
+    window.setFramerateLimit(FRAME_RATE);
+
+    sf::Image image;
+    image.create(WINDOW_WIDTH, WINDOW_HEIGHT);
+    for (std::size_t x_pos = 0; x_pos < WINDOW_WIDTH; ++x_pos) {
+        for (std::size_t y_pos = 0; y_pos < WINDOW_HEIGHT; ++y_pos) {
+            std::uint8_t pixel_value = pixels_.at(x_pos).at(y_pos);
+            image.setPixel(
+                static_cast<unsigned int>(x_pos), static_cast<unsigned int>(y_pos),
+                sf::Color(pixel_value, pixel_value, pixel_value)
+            );
+        }
+    }
+
+    sf::Texture texture;
+    texture.loadFromImage(image);
+
+    sf::Sprite sprite(texture);
+
+    while (window.isOpen()) {
+        sf::Event event;
+        while (window.pollEvent(event)) {
+            if (event.type == sf::Event::Closed) {
+                window.close();
+            }
+        }
+
+        window.clear(sf::Color::Black);
+
+        window.draw(sprite);
+
+        window.display();
+    }
+}
+} // namespace fractal

source/graphics/basic_display.hpp

@@ -2,9 +2,6 @@
 
 #include "config.hpp"
 
-#include <SFML/Graphics.hpp>
-
-#include <cstddef>
 #include <cstdint>
 
 #include <array>
@@ -14,47 +11,7 @@ class BasicDisplay {
     std::array<std::array<uint8_t, WINDOW_HEIGHT>, WINDOW_WIDTH> pixels_{};
 
 public:
-    void set_pixel(std::size_t x_pos, std::size_t y_pos, uint8_t value)
-    {
-        pixels_.at(x_pos).at(y_pos) = value;
-    }
-
-    void display_window()
-    {
-        sf::RenderWindow window(
-            sf::VideoMode(WINDOW_WIDTH, WINDOW_HEIGHT), "SFML Window"
-        );
-
-        window.setFramerateLimit(FRAME_RATE);
-
-        sf::Image image;
-        image.create(WINDOW_WIDTH, WINDOW_HEIGHT);
-        for (std::size_t x = 0; x < WINDOW_WIDTH; ++x) {
-            for (std::size_t y = 0; y < WINDOW_HEIGHT; ++y) {
-                uint8_t pixel_value = pixels_.at(x).at(y);
-                image.setPixel(x, y, sf::Color(pixel_value, pixel_value, pixel_value));
-            }
-        }
-
-        sf::Texture texture;
-        texture.loadFromImage(image);
-
-        sf::Sprite sprite(texture);
-
-        while (window.isOpen()) {
-            sf::Event event;
-            while (window.pollEvent(event)) {
-                if (event.type == sf::Event::Closed) {
-                    window.close();
-                }
-            }
-
-            window.clear(sf::Color::Black);
-
-            window.draw(sprite);
-
-            window.display();
-        }
-    }
+    void set_pixel(std::size_t x_pos, std::size_t y_pos, std::uint8_t value);
+    void display_window();
 };
 } // namespace fractal

source/main.cpp

@@ -3,7 +3,7 @@
 int main()
 {
     fractal::BasicDisplay display;
-    for (size_t i = 0; i < 100; i++) {
+    for (std::size_t i = 0; i < 100; i++) {
         display.set_pixel(100, 100 + i, 255);
     }
     display.display_window();