char targets[map_size][map_size] = {0};

In dem Array werden die Ziele mit folgendem Code mit einem x belegt:
(target_count_help ist zu Beginn 10)

while(target_count_help){
x = rand()%map_size+1;
y = rand()%map_size+1;
if(!targets[x-1][y-1]){
                       targets[x-1][y-1]='x';
                       target_count_help--;
                       }
}

Und jetzt der Part der mich verzweifeln lässt:

cin.getline(Try, 3);
try_x = Try[1];
try_y = Try[0];
cout<<"X: "<<try_x<<"  "<<"Y: "<<try_y-'A'<<endl;
cout<<targets[try_x][try_y-'A']<<endl;
cout<<targets[9][0]<<endl;

Die eingegebene Koordinate liegt in der Form Großbuchstabe/Zahl in Try[].
Wenn ich jetzt A9 eingebe, enthält die entsprechende Stelle im Array jedes mal "Y"! Bei anderen Werten werden plötlich Smileys ausgegeben.
Wie kann das sein, wenn ich das Array anfangs mit 0 initilisiert habe?!?
Wenn ich dann in der letzten Zeile direkt 9/0 ausgebe, ist das Feld wie gewünscht leer.
Hoffe Ihr wisst weiter

try_x mag jetzt meinetwegen '5' sein... welchem Ascii-Wert entspräche das denn?

Snip

?

try_x = Try[1]-'A';

So funktioniert es.
Wer das Spiel mal ausprobieren will, hier der Source
(chaotisch, weil ich zZ noch keine Funktionen benutze):

#include <cstdlib>
#include <iostream>
#include <stdlib.h>
#include <time.h>
#include <conio.h>
#include <windows.h>  

using namespace std;

void textcolor(int color)
{
	SetConsoleTextAttribute( ::GetStdHandle(STD_OUTPUT_HANDLE), color );
}

int main(int argc, char *argv[])
{
	char beep = 7;
	bool getroffen=false;
	char Try[3];
	int try_x;
	int try_y;
	bool test = false;
	const int tries = 10;
	int tries_count=tries;
	srand(time(NULL));
	const int map_size=5;
	const int target_count = 5;
	int target_count_help = target_count;
	char map[map_size][map_size] = {0};
	char targets[map_size][map_size] = {0};
	int x,y;
	char y_scale='A';
	int x_scale=1;

	cout<<"Finden Sie einen der "<<target_count<<" Schaetze!"<<endl<<endl;
	cout<<"  ";
	for(x_scale=0;x_scale<map_size;x_scale++){
		cout<<x_scale<<' ';    
	}
	cout<<endl;

	for(y=0; y<map_size; y++){
		cout<<y_scale<<' ';
		y_scale++;
		for(x=0;x<map_size;x++){
			cout<<'#'<<' ';
		}
		cout<<endl;
	}
	for(y=0; y<map_size; y++){
		for(x=0;x<map_size;x++){
			targets[x][y]=0;
		}
	}
	while(target_count_help){
		x = rand()%map_size+1;
		y = rand()%map_size+1;
		if(!targets[x-1][y-1]){
			targets[x-1][y-1]='x';
			target_count_help--;
		}
	}                                          
	if(test){                        
		for(y=0; y<map_size; y++)         {
			cout<<"  ";
			for(x=0;x<map_size;x++){
				cout<<targets[x][y]<<' ';
			}
			cout<<endl;
		}                  
	}              
	cout<<"\r\nZiel eingeben:"<<endl;
	while(tries_count>0 && !getroffen){
		cin.getline(Try, 3);
		try_x = Try[1]-'0';
		try_y = Try[0]-'A';
		//cout<<"X: "<<try_x<<"  "<<"Y: "<<try_y<<endl;
		if(targets[try_x][try_y] == 'x'){
			cout<<"\r\nGewonnen!"<<endl<<"Sie haben einen Schatz gefunden!\r\n"<<endl;
			getroffen = true;    

			for(y=0; y<map_size; y++){
				cout<<"  ";
				for(x=0;x<map_size;x++){

					if(x != try_x || y != try_y){
						cout<<targets[x][y]<<' ';
					}else{
						textcolor(12);
						cout<<'#';
						textcolor(15);
					}
				}
				cout<<endl;
			}                  
			cout<<endl;                        
		}        

		else{
			cout<<"keinen Schatz gefunden..."<<endl;
		}
		tries_count--;    
		if(tries_count && !getroffen) cout<<"Noch "<<tries_count<<" Versuche."<<endl;
	}
	if(!getroffen) cout<<"Verloren!"<<beep<<endl<<endl;
	system("PAUSE");
	return EXIT_SUCCESS;
}

//Edit: Code _etwas_ lesbarer formatiert

(chaotisch, weil ich zZ noch keine Funktionen benutze)

Ahja... aber die Formatierung soll in Ordnung sein?! *scnr

EDIT: Jetzt hat er es schon um einiges besser formatiert.

Webster schrieb:

(chaotisch, weil ich zZ noch keine Funktionen benutze)

Ahja... aber die Formatierung soll in Ordnung sein?! *scnr

Ist sie nicht. Ich komm mit devcpp nicht ganz klar...

for(y=0; y<map_size; y++){
                cout<<"  ";
                for(x=0;x<map_size;x++){

                    if(x != try_x || y != try_y){
                        cout<<targets[x][y]<<' ';
                    }else{
                        textcolor(12);
                        cout<<'#';
                        textcolor(15);
                    }
                }
                cout<<endl;
            }

die '{' gehört nicht hinter die schleife/bedingung! mag zwar
geschmackssache sein, aber man kanns echt schwer lesen besonders bei
bis zu 4 einrückebenen wo welche klammer hingehört.

versuchs so

for(y = 0; y < map_size; ++y)
{
    cout << "  ";
    for(x = 0; x < map_size; ++x)
    {
        if(x != try_x || y != try_y)
        {
            cout << targets[x][y] << ' ';
        }
        else
        {
            textcolor(12);
            cout << '#';
            textcolor(15);
        }
    }
    cout << endl;
}

ich greif mir mal grade ne stelle wo mans besonders drastisch sieht:

Sehe da kein Problem.
Du musst die Klammer nicht sehen - die Einrueckung sagt dir schon wo ein Block beginnt und wo er endet...

Ich finde hingegen, dass einige Leerzeilen mehr dem Code gutun würde, v.a. zwischen mehreren Funktionen und Variablen.

Lesen kann ichs auch, aber ich übersehe da eher etwas, als wenn ich die Blockweise nehme.

Ich will jetzt keine Diskussion, was besser ist. Das ist bestimmt ansichtssache, aber nehmt ihr den Stil einfach, weil er halt Standard ist und weit verbreitet, oder weil ihrs wirklich besser und schneller lesen könnt?

Wenn man viel unbekannten Code hat bleibt einem nichts anderes übrig als den vorliegenden Stil zu akzeptieren, und wenn man ihn bearbeiten muss sogar im gleichen Stil weiterzuschreiben.

aber zum thema
soweit ich weis kann man ein array nicht auf deine weise initialisieren oder?

sowas geht

int [] a = { 0, 1, 2, 3, 4 };
int [5] b = { 0, 1, 2, 3, 4 };

int [5] falsch = { 0 }; //  geht nicht

d.h. es geht schon nur ist nur der erste wert mit 0 belegt (oder halt die ersten 10 wenn du 10 zahlen reinschreibst

std::vector<char> DeinArray(5,'A')

So hast du einen Vector mit 5 Elementen, die bereits mit einem A initialisiert sind.

d.h. es geht schon nur ist nur der erste wert mit 0 belegt (oder halt die ersten 10 wenn du 10 zahlen reinschreibst

Nein eine Null reich um alles mit 0 zu initialisieren.

ISO IEC 14882 §8.5.1,7 schrieb:

If there are fewer initializers in the list than there are members in the aggregate, then each member not
explicitly initialized shall be value-initialized (8.5)

ISO IEC 14882 §8.5,5 schrieb:

To value-initialize an object of type T means:
— if T is a class type (clause 9) with a user-declared constructor (12.1), then the default constructor for T is
called (and the initialization is ill-formed if T has no accessible default constructor);
— if T is a non-union class type without a user-declared constructor, then every non-static data member
and base-class component of T is value-initialized;
- if T is an array type, then each element is value-initialized;
— otherwise, the object is zero-initialized

Find what:
\) \{

Replace with:
)\n{

Use: Regular Expressions

sowas geht

int [] a = { 0, 1, 2, 3, 4 };
int [5] b = { 0, 1, 2, 3, 4 };

int [5] falsch = { 0 }; //  geht nicht

Die Dimensionsklammern kommen aber nach dem Bezeichner.
Es reicht übrigens auch Folgendes, um das ganze Array mit 0 zu initialisieren:

int richtig[5] = {};