Thread: Vector Construction problem..

Ok so here's my problem:

I'm making a little game project that's like a clone of Breakout, which I'm sure everyone has played before, and I encapsulated the game variables in its own class called "Game" that looks something like this...

Code:

//game.h
//Game class header file
#ifndef GAME_H
#define GAME_H

#include <vector>

#include "player.h"
#include "brick.h"
#include "ball.h"

using namespace std;

class Game
{
public:
          Game(int player_x, int player_y, unsigned int player_w, unsigned int player_h, int player_lives,
                   int ball_x, int ball_y, unsigned int ball_w, unsigned int ball_h, double ball_XSpeed, double ball_YSpeed,
                   int brick_x, int brick_y, unsigned int brick_w, unsigned int brick_h, int brick_hp,
                   int level_width, int level_height);
          
          void Reset();
          void Play();
          
          Player player1;
          Ball ball;
          vector<vector<Brick> > bricks;
};

#endif

Not the most efficient code on the planet, I KNOW. . -.-

But! I'm having trouble creating the constructor of this Game class. As you can see, the "bricks" variable is a 2-dimensional vector. What I'm having trouble doing is creating a constructor that assigns two of the parameters "int level_width, int level_height" as the width and height dimensions of the vector.

I hope I explained that fully. O_o

Thanks for the help, and sorry in advance about all my constructor parameters - I want to keep it that way for certain reasons though. >.<

Bah, this board is so slow with replys I had to reply to my own problem..

I got a constructor working and I improved the "clenliness" of my Game class. It looks like this:

Code:

Game::Game(unsigned int player_w, unsigned int player_h, int player_lives,
               unsigned int ball_w, unsigned int ball_h,
               int brick_x, int brick_y, unsigned int brick_w, unsigned int brick_h, int brick_hp,
               int rows, int columns):
      player1(0,0,player_w, player_h, player_lives),
      ball(0,0,ball_w, ball_h,0,0)
{
      bricks.reserve(rows);
      for(int y = 0; y < rows; y++)
      {
              bricks[y].reserve(columns);
              for(int x = 0; x < columns; x++)
              {
                      bricks[y][x].SetWidth(brick_w);
                      bricks[y][x].SetHeight(brick_h);
                      bricks[y][x].SetHP(brick_hp);
                      bricks[y][x].SetX( brick_x + (brick_w * x) );
                      bricks[y][x].SetY( brick_y + (brick_h * y) );
              }
      }
      
      Reset();
}

I compiled it and everything is running great. ^^

Although it would still help if other people showed me an example of theirs.

Edit: notice the time of my posts, it took me 4 freaking hours to do that. T_T

Code:

                      bricks[y][x].SetWidth(brick_w);
                      bricks[y][x].SetHeight(brick_h);
                      bricks[y][x].SetHP(brick_hp);
                      bricks[y][x].SetX( brick_x + (brick_w * x) );
                      bricks[y][x].SetY( brick_y + (brick_h * y) );

Does every brick really need to know it's own width, height, and position in the grid?

I agree with you, I really should have posted on the regular C++ board. heheh..

But yeah, I'll just stick with reserve(). It seems better and it works in my constructor, so I'm lazy and I don't want to change much code.

Thanks for your input, here's all the Brick class is now...

Code:

//brick.h
//Brick class header file
#ifndef BRICK_H
#define BRICK_H

class Brick
{
public:
            Brick(int new_hp);
            void SetHP(int num);
            int GetHP();
protected:
            
private:
            int hp;
};

#endif

Thanks much.

If this is a breakout clone you should already know how many bricks are on the board. If this is true then a simple dynamic array would work just fine. To remove a brick from the array you would need to take the ball x coord and ball y coord and divide it by the brick width and brick height. Then you find the offset by doing row*gridwidth+column.

Code:

void CGame::FindBrickOffset(int x,int y)
{
  int row=y/m_iBrickHeight;
  int col=x/m_iBrickWidth;

  return row*m_iBoardWidth+col;
}

Set the brick offset to 0 and then in your render loop if the current offset's value is 0...you don't draw anything. Since you are drawing from memory when you change the memory value it is immediately reflected on the screen.
To do borders you don't draw anything on the edges of the bricks.

Hey Bubba, thanks for the reply!

I'll start by saying: I never even thought of using a dynamic array.. mostly because I've never used them before. I'll have to look more into that juicy goodness, hahah. It's these kinds of things that self-taught C++ noobies like me seem to miss..

If it's more efficient, then I should definitly use it, but I was making this project so I could get used to vectors specifically. XD I do appreciate the input though, thanks!

Perhaps I'll do something easy after I finish this to practice dynamic arrays..

You can do the same thing with a vector, just make it a one-dimensional vector and use the same technique.