Ich habe folgendes Problem: Ich wollte einfach
ein kleines Schiffe versenken mit C++ programmieren das in der Eingabeaufforderung läuft.
Dafür habe ich 2 char Arrays, 10 hoch 10 breit(system_player[10][10]), angelegt die ich, mit 2 for-schleifen, als "Untergrund"
mit einfachen Punkten: '.' fülle und dann wieder mit for-schleifen
ausgeben lasse.
Jetzt kommt aber irgentwas komplett durcheinander und die komplette Koordinaten-
reihe für die y = 0 ist ist dort wo y = 9 sein sollte und koordinate (0|0) vom gegnerischen Feld liegt bei (9|9) vom Feld des Spielers also irgentwie total
verrrutscht. Der Rest stimmt so aber ich frage mich was da falsch läuft vielleicht
könnt ihr mir ja weiterhelfen. Ich benutze zurzeit Ein Linux OS und Code::Blocks
Hier die Schleifen zum füllen und ausgeben der Arrays.

//Füllen der arrays

for (int y = 10; y != 0; y--)
{	
	for (int x = 0; x != 10; x++)
	{
		system_player[x][y] = ground_player;
		system_enemy[x][y] = ground_enemy;
	}
}

//ausgeben der arrays
clearscreen; //definiert als: #define clearscreen cout<<"\033[2J\033[1;1H";
for(int y = 10; y != 0; y--)
{
	cout<<y-1<<" ";

	for (int x = 0; x != 10; x++)
	{
		cout<<system_player[x][y]<<" ";
	}

	cout<<"      ";

	cout<<y-1<<" ";

	for (int x = 0; x != 10; x++)
	{
		cout<<system_enemy[x][y]<<" ";
	}
	cout<<endl;
}

cout<<"  0 1 2 3 4 5 6 7 8 9         0 1 2 3 4 5 6 7 8 9 "<<endl;

//Füllen der arrays

for (int y = 10; y != 0; y--)
{	
	for (int x = 0; x != 10; x++)
	{
		system_player[x][y] = ground_player;
		system_enemy[x][y] = ground_enemy;
	}
}

Du zählst außen von Zehn bis Eins und innen von Null bis Neun...

cout<<system_player[12][12]

erzeugt auch die ausgabe '.'
obwohl system_player folgendermaßen deklariert wurde:

char system_player[10][10];

Hääh was habe ich jetzt an arrays nicht verstanden??

Hääh was habe ich jetzt an arrays nicht verstanden??

C++ ist kein Ponyhof. Wenn du über die Grenzen eines Arrays zugreifst, dann passiert eben genau das, ohne jede Warnung. Was die Folge davon ist, ist jedoch undefiniert. Insbesondere kann das auch mal gut gehen^*. In dem Fall bekommst du eben die Daten die zufällig an der Stelle hinter dem Array stehen.

^*: Das Programm ist natürlich trotzdem falsch.

Meinst du mein gesamtes Programm ist falsch [...] ?

Wie wär's wenn du's mal posten würdest?

// main.cpp
#include "field.h"

int main()
{
    C_field Field;
    Field.show_menu();
}

//field.h

#include <string>
#include <iostream>
#include <vector>
#include <stdio.h>
#include "player.h"
#define clearscreen cout<<"\033[2J\033[1;1H";

using namespace std;

class C_field
{
	private:
	vector<C_player>players;
	char position;
	char menu_choice;
	char ground_player;
	char ground_enemy;
	char system_player[10][10];
	char system_enemy[10][10];

	public:
	C_field();
	string get_users_name();
	void show_menu();
	void show_battlefield();
	void refill_system(string unit);
	void show_option_menu();
	void show_battlebeginUI();
	void show_battleUI();
	void show_shipsetUI();

};

//field.cpp

#include "field.h"

C_field::C_field()
{
	C_player enemy("Enemy");
	players.push_back(enemy);
	C_player player(get_users_name());
	players.push_back(player);

	ground_player = '.';
	ground_enemy = '.';

	for (int y = 10; y != -1; y--)
	{
		for (int x = 0; x != 11; x++)
		{
			system_player[x][y] = ground_player;
			system_enemy[x][y] = ground_enemy;
		}
	}

}

string C_field::get_users_name()
{
	string user_name;
	cout<<"Bitte Namen eingeben: ";
	cin>>user_name;
	return(user_name);
}

void C_field::show_menu()
{
	bool run = true;

	while(run)
	{

		clearscreen;

		cout<<"B + Enter: Spielen"<<endl
			<<"O + Enter: Optionen"<<endl
			<<"Q + Enter: Beenden"<<endl
			<<">";
			menu_choice = cin.get();

			switch(menu_choice)
			{
				case 'b':
				{
					show_battlebeginUI();
					break;
				}
				case 'o':
				{
					show_option_menu();
					break;
				}
				case 'q':
				{
					run = false;
					break;
				}
			}

		}
}

void C_field::show_option_menu()
{
	bool run = true;

	while(run)
	{

		clearscreen;

		cout<<"P + Enter: Untergrund des Spielers aendern: "<<ground_player<<endl
			<<"E + Enter: Untergrund des Gegners aendern: "<<ground_enemy<<endl
			<<"N + Enter: Eigenen Namen aendern: "<<players[1].name<<endl
			<<"G + Enter: Namen des Gegners aendern: "<<players[0].name<<endl
			<<"Q + Enter: Zureuck"<<endl
			<<">";
			cin>>menu_choice;

			switch(menu_choice)
			{
				case 'p':
				{
					cout<<"Gewuenschtes Zeichen eingeben: ";
					cin>>ground_player;
					refill_system("player");
					break;
				}
				case 'e':
				{
					cout<<"Gewuenschtes Zeichen eingeben: ";
					cin>>ground_enemy;
					refill_system("enemy");
					break;
				}
				case 'n':
				{
					cout<<"Neuen Namen fuer den Spieler eingeben: ";
					cin>>players[1].name;
					break;
				}
				case 'g':
				{
					cout<<"Neuen Namen fuer den Gegner eingeben: ";
					cin>>players[0].name;
					break;
				}
				case 'q':
				{
					run = false;
					break;
				}
			}

	}
}

void C_field::refill_system(string unit)
{
	if(unit == "player")
	{
		for (int y = 10; y != -1; y--)
		{
			for (int x = 0; x != 11; x++)
			{
				system_player[x][y] = ground_player;
			}
		}
	}
	else if(unit == "enemy")
	{
		for (int y = 10; y != -1; y--)
		{
			for (int x = 0; x != 11; x++)
			{
				system_enemy[x][y] = ground_enemy;
			}
		}
	}
}

void C_field::show_battlebeginUI()
{
	bool run = true;

	while (run)
	{
		clearscreen;
		show_battlefield();
		cout<<"B + Enter: Spiel Beginnen"<<endl
			<<"O + Enter: Optionen"<<endl
			<<"Q + Enter: Spiel Verlassen"<<endl
			<<">";
			cin>>menu_choice;

			switch(menu_choice)
			{
				case 'b':
				{
					show_shipsetUI();
					break;
				}
				case 'o':
				{
					show_option_menu();
					break;
				}
				case 'q':
				{
					run = false;
					refill_system("player");
					refill_system("enemy");
					break;
				}
				case '`':
				{
					int x,y;
					cin>>x;
					cin>>y;
					system_enemy[x][y] = '*';
					break;
				}
				default:
				{
					cout<<"Keine gueltige Eingabe!";
					cin.ignore();
					break;
				}
			}
	}
}

void C_field::show_battlefield()
{
	clearscreen;
	for(int y = 10; y != 0; y--)
	{
		cout<<y-1<<" ";

		for (int x = 0; x != 10; x++)
		{
			cout<<system_player[x][y]<<" ";
		}

		cout<<"      ";

		cout<<y-1<<" ";

		for (int x = 0; x != 10; x++)
		{
			cout<<system_enemy[x][y]<<" ";
		}
		cout<<endl;
	}

	cout<<"  0 1 2 3 4 5 6 7 8 9         0 1 2 3 4 5 6 7 8 9 "<<endl;
}

void C_field::show_shipsetUI()
{

	bool run = true;
	int x,y;

	while(run)
	{
		show_battlefield();
		cout<<"Bitte Schiffe platzieren dazu zuerst eine X- und dann eine Y-Koordinate eingeben"<<endl
			<<"Tanker: "<<players[1].num_tanks<<endl
			<<"Kriegsschiffe: "<<players[1].num_warships<<endl
			<<"Rettungsboote: "<<players[1].num_savers<<endl
			<<">X: ";
			cin>>x;
			cout<<endl<<">Y: ";
			cin>>y;

	}
}

//player.h

#include <iostream>
#include <map>
#include <string>

using namespace std;

class C_player
{
	public:
	string name;
	int num_ships, num_tanks, num_warships, num_savers;
	map<string, C_ship> ships;
	C_player(string namee);
};

//player.cpp

#include "player.h"

C_player::C_player(string namee)
{
	name = namee;
	num_ships = 0;
	num_warships = 2;
	num_tanks = 2;
	num_savers = 2;
}

Ich möchte in diesem Zeitpunkt der Blöße darauf hinweisen das, ich weder beruflich, noch ausbildungs/studientechnisch mit C++ und Programmierung an sich
zu tun habe sondern programmiere weil ich es liebe meine Ideen in Prorammen
verwirklichen zu können und wirklich ALLES machen zu können von sinnlos bis
tatsächlich nütlich. Es ist eine Beschäftigung für mich und die Eleganz meiner
Programmierung und die sinnvolle ausführung liegen allein in meinem Ermessen und ich bin lediglich an der Lösung meines Problems interessiert.
Ich hoffe ihr könnt mit dem obigen Code etwas anfangen und mir Helfen.
Vielen Dank.

'C_ship': undeclared identifier

map<string, c_ship>ships;

rausnehmen ich hatte das jetzt schon gemacht tut mir Leid.

C_field::C_field()
{
    C_player enemy("Enemy");
    players.push_back(enemy);
    C_player player(get_users_name());
    players.push_back(player);  // einen vector fuer konstant 2 Spieler? rly?
 
 
    ground_player = '.';  // Initialisierungsliste
    ground_enemy = '.';   // verwenden!
 
    for (int y = 10; y != -1; y--)  // zählt von 10 bis 0 ... die Dimension geht aber nur von 0 bis 9
                                    // warum zählst du rückwärts?
    {
        for (int x = 0; x != 11; x++)  // zählt von 0 bis 10 ... die Dimension geht aber nur von 0 bis 9
        {
            system_player[x][y] = ground_player;
            system_enemy[x][y] = ground_enemy;
        }
    }

    // warum nicht gleich std::memset() ?
 
}

Gleiche Kacke in C_field::refill_system() , ähnliche Kacke in C_field::show_battlefield() .

// warum nicht gleich std::memset() ?

Warum ::memset wenn es std::fill gibt?

du darfst auf Arrays nur innerhalb der Indexgrenzen zugreifen, d.h. wenn du ein Array mit 10 Elementen dafiniert hast, dann darfst du nur die Indizes 0 bis 9 benutzen:

for (int y = 0; y < 10; ++y)
{
    for (int x = 0; x < 10; ++x)
    {
        system_player[x][y] = ground_player;
        system_enemy[x][y] = ground_enemy;
    }
}

Deine Erklärung bzgl. "Rückwärtszählen" ist außerdem falsch (weil es hier doch gar nicht auf die Darstellung ankommt)!

Und deine anderen Schleifen auch entsprechend anpassen!!!

Und wenn du bei der Darstellung in show_battlefield() die Anzeige umdrehen willst (also oben die letzte Zeile des Arrays), dann entweder als Index '9-y' benutzen oder aber

for (int y = 9; y >= 0; --y)

verwenden.

Außerdem als Tipp: keine Magic-Numbers verwenden, sondern definiere dir 2 Konstanten:

const int MaxX = 10;
const int MaxY = 10;

und benutze diese überall in deinem Programmcode!

Swordfish schrieb:

// warum nicht gleich std::memset() ?

Warum ::memset wenn es std::fill gibt?

Weil ich C-geschädigt bin.