c/c++ SDL podejście obiektowe w przypadku biblioteki, która nie wspiera obiektowości...

Question

Witam wszystkich.

Mam napisaną prostą gierkę w cpp z użyciem biblioteki SDL (multiplatformowa gierka)

Gra napisana strukturalnie dzięki czemu zaczyna być spory bałagan w kodzie. Postanowiłem przepisać kod tak aby był w pełni obiektowy, problem w tym, że biblioteka SDL nie wspiera obiektowości jak jest to w przypadku SFML. Stworzyłem w tym celu prosty szkielet, który w mojej ocenie pozwoli na wykorzystanie SDL w sposób obiektowy. Pytanie: czy moje podejście będzie poprawne ? może coś pozmieniać ?

Zaznaczę, że jest to jedynie szkielet, który wyświetla tylko okno aplikacji, chcę się upewnić, że moje podejście jest dobre zanim przepiszę cały kod.

main.cpp

#include <iostream>
#include <string>

#include "main.h"
#include "win.h"

AppInit::AppInit()
{
    SDL_Init(SDL_INIT_VIDEO|SDL_INIT_AUDIO);
}

int main(int argc, char **argv)
{
    AppInit app;
    Win win;
    //------------------------------------------------------------





    SDL_Delay(10000);

}

main.h

#ifndef MAIN_H
#define MAIN_H

#include <SDL.h>

class AppInit
{
    public:
    AppInit();
};

#endif

win.cpp

#include "win.h"

Win::Win()
{
    win = SDL_CreateWindow("SDL Game joszi", 100, 100, 800, 600, SDL_WINDOW_SHOWN);
    src = SDL_GetWindowSurface(win);
}

win.h

#ifndef WIN_H
#define WIN_H

#include <SDL.h>

class Win
{
    public:
    Win();

    SDL_Window *win;
    SDL_Surface *src;
};

#endif

makefile

CXX = g++

SDLFLAGS = /home/joszi/sdl_lin/bin/sdl2-config --cflags
SDLLIBS = /home/joszi/sdl_lin/bin/sdl2-config --libs

OPT = -O2 -Wall

APPNAME = game

OBJDIR = objlin

all: $(OBJDIR)/main.o $(OBJDIR)/win.o
	rm -f bin/$(APPNAME)
	$(CXX) $(OBJDIR)/*.o `$(SDLLIBS)` -o bin/$(APPNAME)

clean:
	rm -f $(OBJDIR)/*.o

$(OBJDIR)/main.o: src/main.cpp src/main.h
	$(CXX) src/main.cpp $(OPT) -c `$(SDLFLAGS)` -o $(OBJDIR)/main.o

$(OBJDIR)/win.o: src/win.cpp src/win.h
	$(CXX) src/win.cpp $(OPT) -c `$(SDLFLAGS)` -o $(OBJDIR)/win.o

 

Comments

A na boku:

SDL_GetWindowSurface

nie chcesz renderować kartą graficzną?

---

adrian prawdopodobnie tak, jednak najpierw muszę zając się przepisaniem tego na obiekty bo gubię się w swoim kodzie :)

---

zając się przepisaniem tego na obiekty bo gubię się w swoim kodzie

OOP to nie automatyczne panaceum na słaby kod, to tylko inne podejście - na oba sposoby można pisać kod ładny lub nieczytelny.

---

no tak ale w przypadku SDLA trzeba używać zmiennych globalnych, Jeśli zastosuję OOP nie muszę

Tak mi wyszedł ten kod... stałych globalnych nie udało mi się zastosować (muszę zmieniać rozmiar okna w trakcie działania programu, problem jest tylko z oknem)

---

no tak ale w przypadku SDLA trzeba używać zmiennych globalnych, Jeśli zastosuję OOP nie muszę

...nie, przepraszam ale to zdanie nie ma sensu. OOP nie "zmniejsza użycia zmiennych globalnych". Co dokładnie masz na myśli?

---

OOP pozwala na uporządkowanie kodu, wiem, że gdybym posiedział nad wskaźnikami i referencjami to prawdopodobnie nie musiałbym tworzyć zmiennej globalnej...

Nie znam zbyt dobrze cpp, uczę się (dlatego zrobiłem bałagan w swoim kodzie)

OOP jest dla mnie czytelniejsze (dlatego chcę to przepisać)

Answer 1

Po pierwsze, istota "podejścia obiektowego" oznacza "obiekty komunikujące się ze sobą". Nie oznacza "zamieniam wszystko na klasę jak popadnie".

Więc to:

AppInit::AppInit()
{
    SDL_Init(SDL_INIT_VIDEO|SDL_INIT_AUDIO);
}

int main(int argc, char **argv)
{
    AppInit app;

na chwilę obecną nie ma żadnej wartości ani przewagi (a tylko konfunduje) nad:

void init_sdl()
{
    SDL_Init(SDL_INIT_VIDEO|SDL_INIT_AUDIO);
}

int main(int argc, char **argv)
{
    init_sdl();

Po drugie, w C++ie, jedną z największych sił C++a i jego systemu klas są destruktory pozwalające na automatyczne zarządzanie zasobami. Tutaj:

Win::Win()
{
    win = SDL_CreateWindow("SDL Game joszi", 100, 100, 800, 600, SDL_WINDOW_SHOWN);
    src = SDL_GetWindowSurface(win);
}

Tworzysz zasoby w konstruktorze, ale zgubiłeś destruktor zwalniający te zasoby (SDL_DestroyWindow etc).

Comments

Słuszna uwaga, zrobiłem tak:

#include <iostream>
#include <string>

#include "main.h"
#include "win.h"

AppInit::AppInit()
{
    SDL_Init(SDL_INIT_VIDEO|SDL_INIT_AUDIO);
}

AppInit::~AppInit()
{
    SDL_Quit();
}

int main(int argc, char **argv)
{
    AppInit app;
    Win win;
    //------------------------------------------------------------





    SDL_Delay(10000);

}

#ifndef MAIN_H
#define MAIN_H

#include <SDL.h>

class AppInit
{
    public:
    AppInit();
    ~AppInit();
};

#endif

#include "win.h"

Win::Win()
{
    win = SDL_CreateWindow("SDL Game joszi", 100, 100, 800, 600, SDL_WINDOW_SHOWN);
    src = SDL_GetWindowSurface(win);
}

Win::~Win()
{
    SDL_DestroyWindow(win);
    SDL_FreeSurface(src);
}

#ifndef WIN_H
#define WIN_H

#include <SDL.h>

class Win
{
    public:
    Win();
    ~Win();

    SDL_Window *win;
    SDL_Surface *src;
};

#endif

 

---

Jeszcze pytanie odnośnie sprzątania...

Nie jest tak, że tworzony jest domyślny destruktor, który to posprząta gdy obiekt jest usuwany ?

---

A skąd C++ ma wiedzieć, że wskaźnik stworzony przez `SDL_CreateWindow` należy zwolnić przez `SDL_DestroyWindow`? ;)

---

No fakt, dzieki za pomoc :D