Hmmm, this is going very wrong already... I don't understand the idea behind proper splitting up of files. I started programming Pong to teach myself this. Could you please look at my code and tell me how I should split things up and what needs to include what with a little explanation. Thanks in forward
main.cpp
/** * PongSDL * main.cpp ***/#include "game.h"int main ( int argc, char* argv[] ){ Game pong; if( pong.mainMenu() ) pong.play(); pong.~Game(); return 0;}/** * End of main.cpp ***/
game.h
/** * game.h ***/#ifndef GAME_H#define GAME_H#include <SDL/SDL.h> // For SDL_Event#include "graphics.h" // We need some graphics to play#include "ball.h" // The game has a ballstruct Gamestate{public: Ball ball;};class Game{public: Game(); ~Game(); bool mainMenu(); int play();private: Gfx gfx; SDL_Event event; Gamestate* current_state; Gamestate* previous_state;};extern Ball newBall( int x, int y, float velocity, float angle, float rad );extern void updateBall( Ball* b, double dt );#endif // GAME_H/** * game.h ***/
game.cpp
/** * game.cpp ***/#include "game.h"#include <SDL/SDL.h> // For SDL_Event#include "timer.h"#include "ball.h"void process( Gamestate* state, double t, double dt ){ // Move ball updateBall( &state->ball, dt ); // Todo: Rest of processing, eg. collision detection}Game::Game(){ gfx.init();}Game::~Game(){ gfx.~Gfx();}bool Game::mainMenu(){ return true; // Placeholder for a future main menu}int Game::play(){ Timer timer; const float DELTA = 0.01; previous_state = new Gamestate; current_state = new Gamestate; current_state->ball = newBall( 100, 100, 0.1, 0.0, 5.0 ); bool loop = true; while( loop ) { timer.fillAccu(); while( timer.pinchAccu( DELTA ) ) { // Process here *previous_state = *current_state; process( current_state, timer.getTimePassed(), DELTA ); } // End of processing while( SDL_PollEvent( &event ) ) { if( event.type == SDL_QUIT ) loop = false; } // End of event polling gfx.draw( current_state ); } // End of main loop return 0;}/** * End of game.cpp ***/
graphics.h
/** * graphics.h ***/#ifndef GFX_H#define GFX_H#include <SDL/SDL.h> // For SDL_Videostruct Gamestate; // Forward declarationclass Gfx{public: Gfx(); ~Gfx(); int init(); void draw( Gamestate* state );private: SDL_Surface* screen; void drawPixel( int x, int y );};const int GFX_SCREEN_W = 600;const int GFX_SCREEN_H = 400;#endif // GFX_H/** * End of graphics.h ***/
graphics.cpp
/** * graphics.cpp ***/#include "graphics.h"#include <SDL/SDL.h> // For SDL_Video#include "game.h" // To interact with game elements#include <math.h> // For sqrt()Gfx::Gfx(){ screen = 0;}Gfx::~Gfx(){ SDL_FreeSurface( screen );}int Gfx::init(){ if( SDL_Init( SDL_INIT_VIDEO ) < 0 ) { printf( "Unable to init SDL: %s\n", SDL_GetError() ); return -1; } SDL_WM_SetCaption( "PongSDL", NULL ); screen = SDL_SetVideoMode( GFX_SCREEN_W, GFX_SCREEN_H, 32, SDL_SWSURFACE ); if( !screen ) { printf( "Unable to set video: %s\n", SDL_GetError() ); return -1; } return 1;}void Gfx::draw( Gamestate* state ){ // WIP}void Gfx::drawPixel( int x, int y ){ SDL_Rect rect; rect.x = x; rect.y = y; rect.w = 1; rect.h = 1; SDL_FillRect( screen, &rect, SDL_MapRGB( screen->format, 0xFF, 0xFF, 0xFF ) );}/** * End of graphics.cpp ***/
timer.h
/** * timer.h ***/#ifndef TIME_H#define TIME_H// Time is measured in secondsclass Timer{public: Timer(); //~Timer(); void fillAccu(); bool pinchAccu( float dt ); float getTimePassed();private: float time_passed; float current_time; float accumulator; float new_time; float delta_time;};#endif // TIME_H/** * End of timer.h ***/
timer.cpp
/** * timer.cpp ***/#include "timer.h"#include <SDL/SDL.h> // For SDL_GetTicks()Timer::Timer(): time_passed(0.00), accumulator(0.00), new_time(0.00), delta_time(0.00){ current_time = (float)SDL_GetTicks() / 100;}void Timer::fillAccu(){ new_time = (float)SDL_GetTicks() / 100; delta_time = new_time - current_time; current_time = new_time; accumulator+= delta_time;}bool Timer::pinchAccu( float dt ){ if( accumulator >= dt ) { time_passed+= dt; accumulator-=dt; return true; } return false;}float Timer::getTimePassed(){ return time_passed;}/** * End of timer.cpp ***/
ball.h
/** * ball.h ***/#ifndef BALL_H#define BALL_Hstruct Ball{public: float x; float y; float velx; float vely; float r;};#endif // BALL_H/** * End of ball.h ***/
ball.cpp
/** * ball.cpp ***/#include "ball.h"#include "math.h"Ball newBall( int x, int y, float velocity, float angle, float radius ){ Ball b; b.x = x; b.y = y; // Set velocities in px/s. // 180.0*3.14 converts the angle from ° into rad b.velx = velocity * cos( angle / 180.0 * 3.14 ); b.vely = velocity * sin( angle / 180.0 * 3.14 ); b.r = radius; return b;}void updateBall( Ball* b, double dt ){ b->x+= dt * b->velx; b->y+= dt * b->vely;}/** * End of ball.cpp ***/
