[C/C++] Battaglia Navale in C

[Alhazred]

Originariamente inviato da P8257 WebMaster
Prova ad usare "%c" e stampare la riga carattere per carattere...

La mia paura è che non avendo precedentemente inizializzato l'array i suoi campi restino "sporchi", come posso inizializzarlo però?
Va bene così?

for(i=0;i<numeroNavi;i++){
   mArray[i].colonna = ' ';
   mArray[i].riga = 0;
}

[NS-1]

...visto che diverse volte hai utilizzato in modo "strano" la & (e commerciale), vorrei dirti che:

- quando passi un vettore non hai bisogno di mettere la & se devi puntare direttamente al primo dei suoi elementi...

- la & serve per passare l'inirizzo di memoria di una variabile.

infatti:
char *ptrCarattere = &Stringa[6] // assegni al puntatore l'indirizzo

spero di esserti stato d'aiuto..

[P8257 WebMaster]

Non è "paura".. è certezza.. che siano sporche se non le inizializzi esplicitamente .. e questo vale anche per tutte le variabili che userai ..

Il codice postato è corretto... naturalmente mArray deve avere "numeroNavi" elementi.

Bye

[Alhazred]

Non riesco proprio a farmi stampare i valori corretti delle righe, allego tutto il codice aggiornato.
Il codice che dovrebbe stampare colonna e riga è alla fine di AggiornaCampo.

[P8257 WebMaster]

La riga di stampa corretta è questa:

printf("\n\r%c - %d ", mArray[n].colonna, mArray[n].riga);

Perché usi l'operatore '&' .. la printf non ha bisogno dell'indirizzo di memoria dell'oggetto ... è questo che da problemi .. oltre al fatto che vada stampato con "%c" e non con "%s"...

Usi sempre tanti puntatori e tante referenze ... attenzione ..

Bye

[Alhazred]

Se tolgo quei puntatori, in esecuzione ottengo il messaggio "... la memoria non poteva essere Read!"

Edit: con l'istruzione da te proposta il messaggio d'errore non appare più, ma la riga è sempre uguale a -35

[P8257 WebMaster]

Ecco la versione corretta della "posizionnanavi' che ora gestisce le colonne e valorizza correttamente il t_mossa .. sostituiscila nel codice e, se ti va, commenta le modifiche che ho apportato.
Attenzione perché non ho gestito il solito 13, occupatene tu, quando l'utente batte INVIO per saltare l'inserimento della seconda cifra delle righe...

void PosizionaNavi(){
   int Count = numeroNavi, nave = 1;
   char c_col = ' ';
   char c_raw = ' ';
   char coorx[3];
   char tmpStr[3];
   coorx[0] = '\0';
   tmpStr[0] = '\0';

   for(i=0;i<numeroNavi;i++) {
      mArray[i].colonna = ' ';
      mArray[i].riga = 0;
   }

   printf("Inserire le coordinate per %i navi.\n",numeroNavi);
   
   while(Count > 0) {
   printf("Nave %i:    Colonna_", nave);
      
      // A-J
      while((c_col < 65) || (c_col > 74)) // invio = conferma
         c_col = getch();
      coorx[0] = c_col;
      
      mArray[nave-1].colonna = coorx[0];
      printf("%c    Riga_", c_col);
      c_col = ' '; // reset

      // 0-9
      while(((c_raw < 48) || (c_raw > 57))) // invio = conferma
         c_raw = getch();
      printf("%c", c_raw);
      tmpStr[0] = c_raw;
      c_raw = ' '; // reset
		
      // 0-9 o ENTER per non dare coordinata a cifra singola
      while(((c_raw < 48) || (c_raw > 57)) && (c_raw != 13)) // invio = conferma
         c_raw = getch();
      printf("%c", c_raw);
      tmpStr[1] = c_raw;
      c_raw = ' '; // reset
      printf("\n\r");		
      mArray[nave-1].riga = atoi(tmpStr);
      Count --;
      nave++;
   }
}

Bye

[P8257 WebMaster]

Alhazred ha scritto:
Se tolgo quei puntatori, in esecuzione ottengo il messaggio "... la memoria non poteva essere Read!"

Edit: con l'istruzione da te proposta il messaggio d'errore non appare più, ma la riga è sempre uguale a -35

Era un problema della posizionanavi, che ho corretto....

Bye

[P8257 WebMaster]

ERRATA CORRIGE

... Non preoccuparti del "13".. viene gestito correttamente, se insirisci righe ad una sola cifra...

Bye

[Alhazred]

Ok, grazie

[Alhazred]

Ho pensato che un giocatore potrebbe ripensare alla posizione di una nave, quindi vorrei chiedere alla fine una conferma sulle posizioni inserite, se il giocatore risponde di sì si va avanti, altrimenti viene ripetuta la richiesta di inserimento delle coordinate.
Ho provato a scrivere questo codice, ma credo di avere dei problemi nella gestione della risposta. Ho provato utilizzando un while come per le coordinate, ma non mi riesce, poi ho scritto questo e non funziona lo stesso

while(!corretto){
		while(Count > 0) {
			printf("Nave %i:    Colonna_", nave);
      
			// A-J
			while((c_col < 65) || (c_col > 74))
				c_col = getch();
				coorx[0] = c_col;
      
			mArray[nave-1].colonna = coorx[0];
			printf("%c    Riga_", c_col);
			c_col = ' '; // reset

		    // 0-9
			while(((c_raw < 48) || (c_raw > 57)))
				c_raw = getch();
			printf("%c", c_raw);
			tmpStr[0] = c_raw;
			c_raw = ' '; // reset
		
			// 0-9 o ENTER per non dare coordinata a cifra singola
			while(((c_raw < 48) || (c_raw > 57)) && (c_raw != 13))
                           c_raw = getch();
			printf("%c", c_raw);
			tmpStr[1] = c_raw;
			c_raw = ' '; // reset
			printf("\n\r");		
			mArray[nave-1].riga = atoi(tmpStr);
			Count --;
			nave++;
		}
		printf("Confermi le posizioni delle navi? (S/N) ");
		conferma = getchar();
		if((conferma != 'N') || (conferma != 'S'))
			do {
				fflush(stdin);
				printf("\nConfermi le posizioni delle navi? (S/N) ");
				conferma = getchar();
			}while((conferma != 'N') || (conferma != 'S'));
		if(conferma == 'S')
			corretto = TRUE;
                  else{
                       for(i=0;i<numeroNavi;i++) {
			mArray[i].colonna = ' ';
			mArray[i].riga = 0;
			}
                       Count = numeroNavi;
                       navi = 1;
                  }
	}

l'inizio è uguale a quello postato in precedenza.

[Alhazred]

Mi stavo impicciando con l'uso dei comparatori, ho risolto.

[Alhazred]

Ho scritto questa funzione che controlla se si sono inserite una o più volte le stessa coordinate in modo che si debbano inserire necessariamente tutte posizioni diverse per le navi:

BOOL Ripetizioni(){
	int n = 2;
	int quante = 0;
	BOOL esiste = FALSE;
	//Controlla la 1° coordinata con le seguenti 4
	for(n = 2; n<numeroNavi; n++)
		if(mArray[0].colonna == mArray[n].colonna && mArray[0].riga == mArray[n].riga)
			quante++;
	n = 2;
	//Controlla la 2° coordinata con le seguenti 3
	for(n = 2; n<numeroNavi; n++)
		if(mArray[1].colonna == mArray[n].colonna && mArray[1].riga == mArray[n].riga)
			quante++;
	n = 3;
	//Controlla la 3° coordinata con le seguenti 2
	for(n = 3; n<numeroNavi; n++)
		if(mArray[2].colonna == mArray[n].colonna && mArray[2].riga == mArray[n].riga)
			quante++;
	//Controlla la 4° coordinata con la 5°
	if(mArray[3].colonna == mArray[4].colonna && mArray[3].riga == mArray[4].riga)
		quante++;

	if(quante > 0) esiste = TRUE;
	return(esiste);
}

Funziona, ma vorrei renderla parametrica in base a quanto vale numeroNavi, ovvero in base a quante navi si devono inserire sul campo. Così come l'ho scritta dovrei modificarla se aumentassi o diminuissi numeroNavi.
Come posso fare?

[P8257 WebMaster]

Per renderla parametrica dovresti utilizzare un'array di appoggio che incrementi ogni volta .. memorizzando in essa le coordinate correnti da controllare ed utilizzando un unico ciclo ... personalmente la vedo poco efficento come soluzione.. ti consiglio di controllarle al momento dell'inserimento.. nella "posizionaNavi" bastano un paio di righe di codice per farlo...

Bye

[Alhazred]

Credo anch'io che mettere il controllo direttamente in PosizionaNavi sia meglio, stavo solo cercando un modo di non impantanarmi troppo con i cicli while.