Help needed: Target an entire Array

[Dannehood]

So I am trying to make a canon that shoots bullets, the only problem that I'm having is that I don't know how to move/draw the objects sprite efficently. The commented parts are where the problem lies

Here is the code

Declarations.h

#ifndef DECLARATIONS_H
#define DECLARATIONS_H

class bullet
{
public:
	bullet(){shoot = true;}
	sf::Clock clock;
	sf::Sprite sprite;
	void setShoot(bool setShoot){shoot = setShoot;}
	bool getShoot(){return shoot;}
private:
	bool shoot;
};

#endif 

Main.cpp

#include <iostream>
#include <vector>
#include <SFML/System.hpp>
#include <SFML/Graphics.hpp>
#include <SFML/Audio.hpp>
#include <SFML/Window.hpp>
#include "Declarations.h"

using namespace std;

int main ()
{
	sf::RenderWindow App(sf::VideoMode(800, 600, 32), "Bullet");
	App.SetFramerateLimit(90);

	sf::Image Bullet_Bill;
	Bullet_Bill.LoadFromFile("Bullet_Bill.png");
	
	int i = 0;
	bullet bulletMember;
	bullet myBullet[10];

	vector <bullet> bulletlist;
	
	while(App.IsOpened())
    {
		sf::Event Event;

		while(App.GetEvent(Event))
		{
			if(Event.Type == sf::Event::Closed)
			App.Close();
			if((Event.Type == sf::Event::KeyPressed) && (Event.Key.Code == sf::Key::Escape))
			App.Close();
		}
		if (bulletMember.clock.GetElapsedTime() > 1.0 && bulletMember.getShoot() == true && i <= 8)
	    {
		    cout << "Bang! " << i << endl;
			i ++;
		    bulletlist.push_back(myBullet[i]);
			myBullet[i].sprite.SetImage(Bullet_Bill);
		    bulletMember.setShoot(false);
	    }
	    if (bulletMember.getShoot() == false)
	    {
		    bulletMember.clock.Reset();
		    bulletMember.setShoot(true);
	    }
	    myBullet[i].sprite.Move(150 * App.GetFrameTime(), 0); // only applies to 1 object at a time :(

	   	App.Clear(sf::Color(255,255,255,255));
		App.Draw(myBullet[i].sprite); // only applies to 1 object at a time :(
		App.Display(); 
	}
}

[cantide5ga]

Maybe I am missing something, but it seems that the above code will iterate through each item in myBullet[].

I'm not familiar with the library you are using, but no matter the method, each item will have to be dealt with one item at a time. The illusion comes when speed seemingly outputs it all at once or you are refreshing the screen from a buffer.

Can you explain what you see when you run your above program and what your expectations are?

[Dannehood]

Like I said the commented parts are where the problem lies so myBullet[i].sprite.Move(150 * App.GetFrameTime(), 0); needs to be applyed to all the objects but I cant just do
myBullet[0].sprite.Move(150 * App.GetFrameTime(), 0);
myBullet[1].sprite.Move(150 * App.GetFrameTime(), 0);
myBullet[2].sprite.Move(150 * App.GetFrameTime(), 0);
etc.. becuase they only get created once every second so from 0 - 1 this code will run and it will crash the program becuase there are no objects on those indexes yet.

same goes for App.Draw(myBullet[i].sprite);

So what I want is a bullet being created once every second and when it gets created it gets drawn and moves to the right (150 * App.GetFrameTime(), 0)

so I managed to created one every second but I'm not sure how to continue on from here. Sorry if I was unclear at first.

Also I'm using SFML.

[Vlykarye]

for (int i = 0; i < 10; i++)
{
    myBullet[i].sprite.Move(150 * App.GetFrameTime(), 0);
}

A simply loop should fix your problems. Of course, you will need to check if myBullet[i] actually contains a bullet.

[Dannehood]

It's already looped...

[Vlykarye]

No it's not. Lol. Or, at least it isn't in the code you posted.
If it is, then show me, and I'll tell you why it doesn't do what you want.

[Vlykarye]

Actually, nevermind. I'm too busy to be on all day. Here:

myBullet[i].sprite.Move(150 * App.GetFrameTime(), 0); // only applies to 1 object at a time :(

App.Clear(sf::Color(255,255,255,255));
App.Draw(myBullet[i].sprite); // only applies to 1 object at a time :(
App.Display(); 

Now. Don't chew me out if you already fixed this, but the reason why it only applies to 1 object is because you are looping your program, not the application of moving and drawing every object.

In fact, I see one while loop. And it has nothing to do with the objects. Everything else is in an if statement, or just the while block. So, all your awesome code that makes checks, moves, and draws the bullets happens only one time. Then the while block loops back around. If you want every bullet to change at once, then you need to redraw EVERY bullet, THEN redraw the screen (which is App.Display(); in this case). But your code is very specifically drawing one bullet, updating the scree, drawing the next bullet, updating the screen, and so on.