problema con if

[jekkost]

ciao a tutti, avrei un problema con un ciclo if.

ho un oggetto (nave) composto da quattro pezzi(o caselle).

if(	campoDaGioco[indiceRiga][indiceColonna] ==1){
     //sostituisco l'1 della nave con un 6.
			campoDaGioco[indiceRiga][indiceColonna] = 6;
		  //inizializzo una variabile contatore h_6;
			//ho inizializzato fuori dall if--> int h_6=0;
			h_6=h_6+1;
			if (h_6<4){
			System.out.println("Colpita!Ma non mi hai affondata!");	
			}
			else if(h_6==4){	
			System.out.println("Colpita e affondata!");
			}
		}

il problema è che mi restituisce colpita ma non affondata per 4 volte(quando invece alla 4 volta dovrebbe dirmi colpita e affondata), e se (la 5° volta) gli ridò la posizione dell'ultima casella, il programma non mi risponde, ma prosegue ripetendomi l'struzione della riga come se nulla fosse.
Non capisco dove sbaglio.

Grazie a quanti mi aiuteranno.

[LeleFT]

Java viene trattato nel forum Java.

Sposto.


Ciao.

[LeleFT]

Da questo commento

//ho inizializzato fuori dall if--> int h_6=0;

Deduco che ogni volta crei una nuova variabile h_6, la inizializzi a 0 e poi, se l'if è soddisfatto la incrementi di uno... ma avendola, appunto, inizializzata a 0, al massimo varrà sempre 1 e mai 2, 3 o 4.

Verifica che il problema sia effettivamente questo (l'inizializzazione di tale variabile va fatta una sola volta, in questo caso, non ogni volta).

Eventualmente, se non è così, posta un po' più di codice così capiamo meglio.


Ciao.

[jekkost]

ciao LeleFT e grazie per la risposta ho provato a spostare l'inizializzazione ma mi da sempre lo stesso problema.
Posto il metodo (semi)completo (in quanto ho trascurato le altre navi)

public void setColpo(int ColpoRiga, int ColpoColonna) throws Exception {
			
			int	indiceRiga= ColpoRiga-1;
		int indiceColonna= ColpoColonna-1;
		if (ColpoRiga >= 0 & ColpoRiga <= getRighe() & ColpoColonna >= 0 & ColpoColonna <= getColonne()) {
		if (campoDaGioco[indiceRiga][indiceColonna] == 0) {

		System.out.println("Acqua, non ci sono navi in questa casella!");

		} else { 
		if(	campoDaGioco[indiceRiga][indiceColonna] ==1){
			campoDaGioco[indiceRiga][indiceColonna] = 6;
		  //inizializzo una variabile contatore h(6);
			h_6=h_6++;
			if (h_6<4){
			System.out.println("Colpita!Ma non mi hai affondata!");	
			}
			else if(h_6==4){	
			System.out.println("Colpita e affondata!");
			}
		}

[LeleFT]

La variabile h_6 va dichiarata e inizializzata fuori da quel metodo (magari nel costruttore), altrimenti ricadi sempre nello stesso problema... ogni volta che viene chiamato il metodo setColpo() la variabile viene re-inizializzata a zero e non è ciò che vuoi.

Dichiara la variabile a livello di classe, inizializzala a 0 nel costruttore. Il metodo setColpo() deve solo controllarla ed, eventualmente, incrementarla.


Ciao.

[jekkost]

grazie mille, ora va!!!