Проектирование (разработка логики приложения): как это делать правильно?

#include "Mouse.h"
#include "Color.h"
 
int main(int argc, char* argv[])
{
    Color prev, current;
    Mouse m(1);
 
    while (true)
    {
        if (GetAsyncKeyState(VK_F8) & 0x8000)
        {
            if (m.holded)
            {
                Sleep(100);
                continue;
            }
            current.GetRGB();
 
            if (!current.IsEqual(prev))
            {
                // "enemy detected"
                m.HoldButton();
            }
            prev = current;
        }
        else
        {
            if (m.holded)
            {
                m.ReleaseButton();
            }
            prev.Reset();
        }
        Sleep(100);
    }
    return 0;
}

#pragma once
#ifndef MOUSE_H
#define MOUSE_H
 
#include <windows.h>
 
class Mouse
{
public:
    bool holded;
 
    Mouse(short btn) : holded(false), button(btn), input({ 0 }) {}
    ~Mouse() {}
 
    void HoldButton();
    void ReleaseButton();
 
private:
    short button;
    INPUT input;
};
 
#endif

#include "Mouse.h"
 
void Mouse::HoldButton()
{
    holded = true;
    input.type = INPUT_MOUSE;
    input.mi.dwFlags = (button == 2 ? MOUSEEVENTF_RIGHTDOWN : MOUSEEVENTF_LEFTDOWN);
    ::SendInput(1, &input, sizeof(INPUT));
}
 
void Mouse::ReleaseButton()
{
    holded = false;
    ::ZeroMemory(&input, sizeof(INPUT));
    input.type = INPUT_MOUSE;
    input.mi.dwFlags = (button == 2 ? MOUSEEVENTF_RIGHTUP : MOUSEEVENTF_LEFTUP);
    ::SendInput(1, &input, sizeof(INPUT));
}

#pragma once
#ifndef COLOR_H
#define COLOR_H
 
class Color
{
public:
    Color();
    ~Color() {}
    bool IsEqual(Color &subject);
    void Reset();
    void GetRGB();
 
private:
    short r, g, b;
};
 
#endif

#include "Color.h"
#include <windows.h>
///
#include<iostream>
 
Color::Color()
{
    Reset();
}
 
void Color::Reset()
{
    r = -1;
    g = -1;
    b = -1;
}
 
bool Color::IsEqual(Color &subject)
{
    return (r == subject.r
        && g == subject.g
        && b == subject.b) || subject.r == -1;
}
 
void Color::GetRGB()
{
    static POINT pos;
    static HDC hdcScreen;
    static COLORREF pixel;
 
    GetCursorPos(&pos);
    hdcScreen = ::GetDC(NULL);
    pixel = ::GetPixel(hdcScreen, pos.x, pos.y);
    std::cout << pos.x << "," << pos.y << std::endl;
    ReleaseDC(NULL, hdcScreen);
 
    if (pixel != CLR_INVALID)
    {
        r = GetRValue(pixel);
        g = GetGValue(pixel);
        b = GetBValue(pixel);
        std::cout << r << "|" << g << "|" << b << std::endl;
    }
}