Поправьте код на C++

[dvano]

Добрый день. Практика работы с C++ отсутствует, поэтому я попрошу дать мне любой совет, который бы помог дальше правильно писать программы на этом языке. Ниже имеются исходники обычной "обертки" над библиотекой SDL2, которые написаны просто для оценки "правильности".

(Нажмите, чтобы показать/скрыть)

Код: (c) [Выделить]

#include <iostream>

#include "SDL2Helper.h"

using namespace std;

int main(int argc, char** argv) {
    SDL2initialize(SDL_INIT_VIDEO);
   
    SDL_Window* window = SDL2createWindow("Test", 100, 100, 640, 480, SDL_WINDOW_SHOWN);
   
    SDL_Delay(2000);
   
    SDL2destroyWindow(window);
    SDL2destroy();
   
    return 0;
}



Код: (c) [Выделить]

#ifndef SDL2EXCEPTION_H
#define SDL2EXCEPTION_H

#include <exception>
#include <string>

class SDL2Exception : std::exception
{
    const std::string info;
   
    public:
        SDL2Exception(const char* text) throw();
        virtual ~SDL2Exception() throw();
       
        const char* what() const throw();
};

#endif /* SDL2EXCEPTION_H */



Код: (c) [Выделить]

#include "SDL2Exception.h"

SDL2Exception::SDL2Exception(const char* text) throw() : info(text) {}

SDL2Exception::~SDL2Exception() throw() {}

const char* SDL2Exception::what() const throw()
{
    return this->info.c_str();
}



Код: (c) [Выделить]

#ifndef SDL2HELPER_H
#define SDL2HELPER_H

#include <SDL2/SDL.h>

#include "SDL2Exception.h"

void SDL2initialize(const Uint32& flags) throw(SDL2Exception);

void SDL2destroy() throw(SDL2Exception);

void SDL2destroyWindow(const SDL_Window* window) throw(SDL2Exception);

void checkSDL2Initialized() throw(SDL2Exception);

SDL_Window* SDL2createWindow(const char* title, const unsigned& x,
        const unsigned& y,
        const unsigned& width, const unsigned& height, const Uint32& flags)
        throw(SDL2Exception);

const bool& SDL2isInitialized();

#endif /* SDL2HELPER_H */



Код: (c) [Выделить]

#include "SDL2Helper.h"

namespace
{
    bool initialized = false;
}

void SDL2initialize(const Uint32& flags) throw(SDL2Exception)
{
    if (SDL_Init(flags)) {
        throw SDL2Exception("Unable to initialize SDL2.");
    }
   
    initialized = true;
}

void SDL2destroy() throw(SDL2Exception)
{
    checkSDL2Initialized();
   
    SDL_Quit();
   
    initialized = false;
}

void SDL2destroyWindow(const SDL_Window* window) throw(SDL2Exception)
{
    if (!window) {
        throw SDL2Exception("Window pointer == NULL");
    }
   
    SDL_DestroyWindow(const_cast<SDL_Window *>(window));
}

void checkSDL2Initialized() throw(SDL2Exception)
{
    if (!initialized) {
        throw SDL2Exception("SDL2 wasn't initialize");
    }
}

SDL_Window* SDL2createWindow(const char* title, const unsigned& x,
        const unsigned& y,
        const unsigned& width, const unsigned& height, const Uint32& flags)
        throw(SDL2Exception)
{
    checkSDL2Initialized();
   
    SDL_Window* window = SDL_CreateWindow(title, x, y, width, height, flags);
   
    if (!window) {
        throw SDL2Exception("Failed to create the SDL2 window.");
    }
   
    return window;
}

const bool& SDL2isInitialized()
{
    return initialized;
}



Как бы вы реализовали выше приведенную "обертку" над библиотекой. Правильно ли я использую исключения? Что нужно исправить? Правильно ли то, что я использую namespace для сокрытия переменной от сторонних факторов, и правильно ли то, что функция const bool& isInitialized() возвращает константную ссылку, в которой можно потом убрать const и изменить ее? Или лучше возвращать обычную копию?

[aSmile]

Я не знаю ничего про SDL, но раз уж ты пишешь на c++, то почему делаешь глобальные функции и переменные вместо классов? Например, переменную initialized можно было бы сделать private членом. По поводу исключений - если ты их используешь, то почему не пытаешься перехватывать?

[dvano]

почему делаешь глобальные функции и переменные вместо классов?

Потому что я не понимаю, что делаю. Зачем в данной ситуации классы, если можно просто подключить заголовок, и работать с ним? Как я понимаю, экземпляры класса можно создавать сколько угодно, но мне не нужно столько.
Пользователь решил продолжить мысль 05 Февраля 2015, 20:57:36:

По поводу исключений - если ты их используешь, то почему не пытаешься перехватывать?

Это просто пример. Мне интересно то, правильно ли я их реализовываю?

[Protopopulus]

Автор, почитай полезную книгу.