[C/C++] Battaglia Navale in C

[P8257 WebMaster]

.. Mi riferisco ai blocchi while attorno ad ogni "getch()"

Bye

[Alhazred]

Beh, time1 lo prendo dalla memoria condivisa con time1 = p->timeStmp
quindi dovrebbe essere tutto ok, giusto?

[P8257 WebMaster]

Esatto.

Bye

[Alhazred]

Originariamente inviato da P8257 WebMaster
.. Mi riferisco ai blocchi while attorno ad ogni "getch()"

Bye

In effetti penso che basti il controllo per il timestamp solo in corrispondenza del primo getch(), ovvero quello per la colonna, se si arriva agli altri vuol dire che l'input è stato messo dall'utente e quindi è corretto.

[Alhazred]

Ah, web, dato che ci siamo, sai mica se e come posso mettere un'icona che piace a me al .exe che mi viene generato? Al momento avevo provato cliccando col tasto destro sull'eseguibile, ma in proprietà non c'è "cambia icona". Il fatto è che il file viene creato senza icona e gli viene assegnata quella standard da windows, quindi non c'è un'icona da cambiare, dovrei proprio mettercela.

[Alhazred]

Anche scritto così non mi piace molto

time(&tempo2);
while(tempo2 < tempo1 + 2)
{
	buffer = getch();
	while((c_col < 65) || (c_col > 74)) 
		c_col = getch();
	time(&tempo2);
}

Bisogna digitare 2 volte la coordinata per la colonna per fargliela accettare, la prima volta viene presa da buffer = getch() e persa.
Com'è il codice corretto per fare questo controllo?

Mettendo c_col al posto di buffer mi sembra che funzioni...

[Alhazred]

Altro problema, non va il controllo dei duplicati in Colpo().
Esempio.
1° colpo - C1
2° colpo - E4
3° colpo - C1 viene accettato

Altro esempio
1° colpo - D3
2° colpo - F3
3° colpo - F3 viene riconosciuto come già inserito
3° colpo bis - F3 viene riconosciuto come già inserito
3° colpo ter - D3 viene accettato

In pratica sembra non controlli il primo elemento dell'array che contiene i colpi sparati,
ma il ciclo for parte da 0...

[P8257 WebMaster]

Fare il debug di "colpo()" all'interno del sorgente diventa un'operazione piuttosto ostica dato che dovresti istanziare due volte il debugger per le due istanze .. ti suggerisco di debuggarla trasferendola in un sorgente a parte creato ad hoc, ovviamente, dato che il pezzo di codice che devi debuggare non ha bisogno né della memoria condivisa e né delle altre strutture dati, sostituiscile con semplici variabili locali...

Bye

[Alhazred]

Sto provando, ma non sono molto pratico col debugger, arrivato ad una istruzione printf mi appare una finestra che mi chiede dove si trova il file printf.c

[Alhazred]

Dato che al momento non riesco a fare il debug a causa della finestra sopra citata, che se non la accontento mi fa vedere il programma in assembly e non ci capisco una mazza, provo ad azzardare un'ipotesi, può essere colpa del break? In realtà non ci credo molto, però ho stampato il codice di Colpo() e leggendolo e facendo a mente i conti di ciò che dovrebbe accadere mi sembra corretto...

[Alhazred]

Ho trovato il problema , la variabile int colpo doveva essere globale, altrimenti viene riazzerata ogni volta che viene chiamata Colpo(), nell'array dei colpi viene occupata solo la posizione 0 e il controllo avviene solo sull'ultima coordinata inserita, ora il controllo è giusto (Y)

[LoryOne]

Per tagliare la testa al toro:
Per quanto riguarda lo svuotamento del buffer della tastiera fai così:

HANDLE hInput;
hInput = GetStdHandle(STD_INPUT_HANDLE);

e prima di effettuare il getch(), svuota il buffer con:
FlushConsoleInputBuffer(hInput);

Per quanto riguarda il controllo dei duplicati io ti consiglio di riscrivere la procedura, facendo particolare attenzione che un duplicato è tale solo se sono uguali riga e colonna.
Questo implica che devi controllare la riga (o la olonna) solo se la colonna (o la riga) coincide.

[LoryOne]

Ops, hai già risolto.

[Alhazred]

Si, ma la tua informazione su FlushConsoleInputBuffer(hInput); mi ha risolto un effetto collaterale apparso solo ora che i controlli sono ok.
In pratica dopo che era stato riconosciuta una coordinata già valida la successiva chiedeva solamente la riga, la colonna risultava vuota.

[Alhazred]

Originariamente inviato da LoryOne

...
Per quanto riguarda il controllo dei duplicati io ti consiglio di riscrivere la procedura, facendo particolare attenzione che un duplicato è tale solo se sono uguali riga e colonna.
Questo implica che devi controllare la riga (o la olonna) solo se la colonna (o la riga) coincide.

Intendi così?
La modifica è in neretto.

Count = 0;
for(Count = 0; Count < colpo + 1; Count++)
{
	existingCol = FALSE;
	existingRaw = FALSE;

	if(pArray[Count].colonna == coorx[0])
		existingCol = TRUE;

	if(existingCol == TRUE)
		if(pArray[Count].riga == atoi(tmpStr))
		         existingRaw = TRUE;
			
	if((existingCol == TRUE) && (existingRaw == TRUE))
		break;
}