CDraw *mDraw = new CDraw (); // Pieces CPieces *mPieces = new CPieces; // Board CBoard *mBoard = new CBoard (mPieces, mScreenHeight); // Game CGame *mGame = new CGame (mBoard, mPieces, mDraw, mScreenHeight); // game loop // ----- Free ----- delete mBoard; delete mPieces; delete mDraw; delete mGame;
This makes me raise an eyebrow. You dont really need to dynamically allocate these do you? The goal should be to use the stack unless you cant.
while (!mDraw->IsKeyDown (SDLK_ESCAPE))
I wanted to highlight that your CDraw object handles input. This is not an ideal division of responsibilities.
Quote:I'm asking myself... do I have lot of english mistakes?
I only skimmed it and I did notice a few.
PS: Don’t forget to play with the “defines”. Crazy example:
#define BLOCK_SIZE 5 // Width and Height of each block of a
#define BOARD_WIDTH 90 // Board width in blocks
#define BOARD_HEIGHT 90 // Board height in blocks
[Edited by - Loover on December 14, 2008 8:38:34 PM]
