Game not working with 1 more thing in struct?

[Bingocat4]

I'm making a shooter game and decided I needed another thing in the Bullet struct, so I added it, and all of the sudden the game doesn't work anymore. Even if I add a variable in the struct that does nothing in the program, it doesn't work. I've figured out that the problem is that the server can't understand the player's packet for some reason and just gives it random numbers. If I add one other thing to any other struct the program works fine. It's just if I add something to the Bullet struct it crashes from huge random numbers from the player. I don't think the struct is too big or anything. The player struct is the biggest and doesn't crash the server if I add something to it. Code was too big, so I deleted some. Also, I had to delete a ton of code to get it under the 8000 length limit, so I probably messed up the parenthesis.

struct Player
{
	int x, y, ID, weapon, type, frame;
	int gunDis, gunOffX, gunOffY;
	float dir, health;
	bool ready;
	string name, verify;
	ENetPeer *peer;

	int hi;
};

///////////////////BULLET STRUCT HERE////////////////
struct Bullet
{
	float x, y, speed, dir, damage;
	int radius, playerID, life;
	int type, explosive, collidedPlayer;
	bool active;
	//int hi;
	//bool playerCollision;
};
/////////////////////////////////////////////////////

struct Item 
{
	int x;
	int y;
	int type;
};

int main () {

	al_init();
	if (!enet_initialize())
		cout << "Initialized ENet\n";

	ENetAddress address;
	ENetHost * server;
	ENetEvent event;

	address.host = ENET_HOST_ANY;
	address.port = 12345;
	server = enet_host_create(&address, 32, 1, 0, 0);

	if (address.port != NULL)
		cout << "Binded to port " << address.port << endl;
	if (server != NULL)
		cout << "Successfully created ENet Server\n";

	if (server == NULL) {
		cout << "ERROR: Could not create ENet Server " << endl << "Press any key to exit...\n";
		system("pause>nul");
		exit(EXIT_FAILURE);
	}

	ALLEGRO_EVENT_QUEUE *event_queue = NULL;
	ALLEGRO_TIMER *timer = NULL;

	event_queue = al_create_event_queue();
	timer = al_create_timer(1.0 / 60.0);

	al_register_event_source(event_queue, al_get_timer_event_source(timer));

	al_start_timer(timer);

	while (!done) {
		ALLEGRO_EVENT ev;
		al_wait_for_event(event_queue, &ev);

		if (ev.type == ALLEGRO_EVENT_TIMER) {
			UpdateBullets(bullets, players, server);
			UpdateGame(gameState, countDown, startTimer, startTimerStart, players, bullets, items, server, resetCount, resetTimerStart, resetTimer);
			
		}

		while (enet_host_service (server, & event, 0) > 0) {
			switch (event.type) {

				case ENET_EVENT_TYPE_RECEIVE:
					{
						//printf("Recieved a packet containing %s\n", event.packet->data);

						if(event.packet->data[0] == 'P') {
							char message[100];
							int messageCount = 0;
							int x;
							int y;
							float dir;
							int ID;
							int frame;

							for(unsigned int i = 1; i < event.packet->dataLength; ++i) {
								message[i] = event.packet->data[i];
							}
							string coords = message;

							istringstream ss(coords);
							string token;

							while(getline(ss, token, ',')) {
								if (messageCount == 1) {
									x = atoi(token.c_str());
								}
								else if (messageCount == 2) {
									y = atoi(token.c_str());
								}
								else if (messageCount == 3) {
									dir = atof(token.c_str());
								}
								else if (messageCount == 4) {
									ID = atoi(token.c_str());
								}
								else if (messageCount == 5) {
									frame = atoi(token.c_str());
								}
								messageCount++;
							}

							cout << "Size: " << players.size() << " X: " << x << " Y: " << y << " ID: " << ID << " Frame: " << frame << endl;

							players[ID].x = x;
							players[ID].y = y;
							players[ID].dir = dir;
							players[ID].frame = frame;
						}
						else if (event.packet->data[0] == 'N') {

							char message[100];
							int messageCount = 0;
							int ID;
							float x;
							float y;
							int type;
							string name;
							string verify;

							for(unsigned int i = 0; i < event.packet->dataLength; ++i) {
								message[i] = event.packet->data[i];
							}
							string coords = message;

							istringstream ss(coords);
							string token;

							while(getline(ss, token, ',')) {
								if (messageCount == 1) {
									ID = atoi(token.c_str());
								}
								else if (messageCount == 2) {
									x = atoi(token.c_str());
								}
								else if (messageCount == 3) {
									y = atoi(token.c_str());
								}
								else if (messageCount == 4) {
									name = token;
								}
								else if (messageCount == 5) {
									type = atoi(token.c_str());
								}
								else if (messageCount == 6) {
									verify = token;
								}
								messageCount++;
							}
							players[ID].name = name;
							players[ID].x = x;
							players[ID].y = y;
							players[ID].weapon = BAZOOKA;
							players[ID].type = type;
							players[ID].ready = false;
							players[ID].verify = "Verified";
							if (players[ID].type == 0) {
								players[ID].gunOffX = 45;
								players[ID].gunOffY = 21;
							}
							else if (players[ID].type == 1) {
								players[ID].gunOffX = 45;
								players[ID].gunOffY = 15;
							}

							players[ID].gunDis = 20;
			
							cout << "Name: " << name << " ID: " << ID << " X: " << x << " Y: " << y << " Type: " << type << endl << endl;
						}
						else if(event.packet->data[0] == 'M') {
							char message[100];
							int messageCount = 0;
							float dir;
							int ID;

							for(unsigned int i = 1; i < event.packet->dataLength; ++i) {
								message[i] = event.packet->data[i];
							}
							string coords = message;

							istringstream ss(coords);
							string token;

							while(getline(ss, token, ',')) {
								if (messageCount == 1) {
									dir = atof(token.c_str());
								}
								else if (messageCount == 2) {
									ID = atoi(token.c_str());
								}
								messageCount++;
							}
							players[ID].dir = dir;
						}
						else if (event.packet->data[0] == 'B') {

							char message[100];
							int messageCount = 0;
							int ID;

							for(unsigned int i = 0; i < event.packet->dataLength; ++i) {
								message[i] = event.packet->data[i];
							}
							string coords = message;

							istringstream ss(coords);
							string token;

							while(getline(ss, token, ',')) {
								ID = atoi(token.c_str());

								}
							}
						}
						if(event.packet->data[0] == 'R') {
							char message[100];
							int messageCount = 0;
							int ID;

							for(unsigned int i = 1; i < event.packet->dataLength; ++i) {
								message[i] = event.packet->data[i];
							}
							string coords = message;

							istringstream ss(coords);
							string token;

							while(getline(ss, token, ',')) {
								ID = atoi(token.c_str());
							}
							players[ID].ready = true;
						}

						enet_packet_destroy (event.packet);
					}
					break;

				case ENET_EVENT_TYPE_DISCONNECT:
					printf ("%s disconected.\n", event.peer -> data);
					event.peer -> data = NULL;
					for (unsigned int i = 0; i < players.size(); i++) {
						if(players[i].peer->address.host == event.peer->address.host && players[i].peer->address.port == event.peer->address.port) {
							players.erase(players.begin() + i);
							cout << "Size: " << players.size() << endl;
							break;
						}
					}
					break;
			}
		}
		string playerPacket = "";
		for (unsigned int i = 0; i < players.size(); i++) {
			char packet[256];
			string buf;
			sprintf_s(packet, sizeof(packet), "P,%i,%i,%i,%f,%s,%f,%i,%i,%i;", players[i].ID, players[i].x, players[i].y, players[i].dir, players[i].name.c_str(), players[i].health, players[i].type, players[i].frame, players[i].weapon);
			buf = packet;
			playerPacket = playerPacket + buf;
		}
		const char *packetBuf = playerPacket.c_str();
		ENetPacket *p = enet_packet_create((char*)packetBuf, strlen(packetBuf)+1, ENET_PACKET_FLAG_UNSEQUENCED);
		//printf("Sent a packet to client containing %s\n", playerPacket);
		enet_host_broadcast(server, 0, p);
	}
		

	enet_host_destroy(server);
}

[admkrk]

This is the only place in all the code you posted that has Bullet:

///////////////////BULLET STRUCT HERE////////////////
struct Bullet

Where do you even use it?

[Bingocat4]

I had to delete like half of the code to get it under the limit. I deleted all the functions. That's why you don't see updating bullets and stuff. But it doesn't even matter if the function is doing anything. If I add int Test to the strict, the program doesn't work right. If I add anything to any other struts, it works fine. If you need the whole code, I'll put it in a couple replies.

[admkrk]

Well if adding something to Bullet breaks the program, the problem must be related to Bullet somehow either directly or indirectly. I don't know anything about Allegro or the server stuff, that shouldn't matter. Just because adding stuff to other parts doesn't break it is irrelevant and if nothing else should point out even more that there is something wrong with Bullet and/or how you are using it. There is no reason to post 1000 lines of code, just narrow it down to where the problem might be.

[Bingocat4]

I tried commenting out all the update functions and everything that has to do with bullets, but still adding any new variable to the Bullet struct causes the server to not be able to figure out the players position, and just gives it random numbers, and crashes itself. I can't find any reason why it would make that happen...

[Bingocat4]

Okay... I changed nothing with it, but for some reason it decided it wanted to stop being weird and works now.