[C/C++] Battaglia Navale in C

[P8257 WebMaster]

Basta stampare il codice ascii col '%c'....

Bye

[Alhazred]

Sto testando il layout della griglia e se sono giusti i comandi per prendere le coordinate e ristampare la griglia aggiornata... ma maledetto "... la memoria non poteva essere READ!"
Non riesco a farlo andare, esce sempre questo errore, sono certo che non è il programma, mi usciva anche quando cercavo di vedere un DivX ed ho risolto togliendo tutti i codec e rimettendoli, ma qui come faccio?
Vi allego il codice della griglia, provate a compilarlo e vedete se a voi funziona per favore.

[Alhazred]

Facendo copia-incolla per risistemare il codice mi sono accorto di aver fatto un errore:

scanf("%s",i); <--- questa diventa scanf("%s",r);
i = ChartoI(r);
cArray[i][j] = 'x';

[NS-1]

scusa ma "r" non mi sembra un array... perchè non scrivi:

scanf("%s", &r);

fammi sapere

[LoryOne]

Nel codice che hai postato ho notato subito alcuni errori piuttosto rilevanti:

1) Lo scanf fa esplodere tutto.
Segui il consiglio di NS-1
2) Hai scambiato le righe con le colonne
3) La colonna DEVE essere maiuscola, altrimenti non funziona.

Poi ho notato:

1) Che utilizzi "%s" al posto di "%c" quando devi immettere 1 solo carattere
2) Che a='a' nel caso in cui a sia definito char, si può anche scrivere come a=97
Non capisco perchè hai implementato CharToI (Assolutamente superfluo)
3) Il putchar ripetuto aumenta le righe di codice inutilmente:
Es:
putchar(196);
putchar(196);
putchar(196);
putchar(194);

lo si può riscrivere così:
printf("%c%c%c%c",196,196,196,194);

-Cerca di fare a meno di utilizzare nuove funzioni il cui scopo sia raggiungibile attraverso l'uso di quelle già utilizzate.

NON sono critiche ma consigli.
Piano piano si affinano le tecniche a tutto vantaggio dell'esperienza del programmatore.

[P8257 WebMaster]

Va in crash di brutto brutto brutto ...

I consigli dati sono esatti, uno lo aggiungo anch'io:
Acquisisci l'imput come carattere o, addirittura come stringa .. è più sicuro .. vedi thread " ...non capisco più nulla! (--- C ---)" di NS-1...

Bye

[P8257 WebMaster]

Il motivo del crash .. è che non permetti alla scanf di immagazzianre un oggetto nella modalità adatta .. infatti se non gli passi l'esatto indirizzo di "j".. lei va a scrivere "dove gli capita" .. e, tanto per cambiare, quel "dove gli capita" è un'area di memoria che "non poteva essere read o write."

int j;
scanf("%d", &j);

Altra cosa:
Perché scrivi:

printf("%s","     A   B   C   D   E   F   G   H   I   J\n");

Basta scrivere:

printf("     A   B   C   D   E   F   G   H   I   J\n");

La printf capisce che vuoi stampare una stringa...

Bye

[Alhazred]

Originariamente inviato da LoryOne
1) Lo scanf fa esplodere tutto.
Segui il consiglio di NS-1
2) Hai scambiato le righe con le colonne
3) La colonna DEVE essere maiuscola, altrimenti non funziona.

Provvederò.

Poi ho notato:
1) Che utilizzi "%s" al posto di "%c" quando devi immettere 1 solo carattere

Pensavo potesse essere quello a generarmi l'errore dato che mi era già capitato, all'inizio avevo messo %c, ma evidentemente mi sono dimenticato di rimettercelo prima di postarvi il codice.

2) Che a='a' nel caso in cui a sia definito char, si può anche scrivere come a=97
Non capisco perchè hai implementato CharToI (Assolutamente superfluo)

All'inizio del 3d si era parlato di questa cosa, mi è stata consigliato ChartoI, forse con l'evolversi del programma è effettivamente diventato inutile.

3) Il putchar ripetuto aumenta le righe di codice inutilmente:
Es:
putchar(196);
putchar(196);
putchar(196);
putchar(194);

lo si può riscrivere così:
printf("%c%c%c%c",196,196,196,194);

Questa non la sapevo.

NON sono critiche ma consigli.
Piano piano si affinano le tecniche a tutto vantaggio dell'esperienza del programmatore.

Non preoccuparti, so bene che quello che mi dite è per aiutarmi e per questo vi ringrazio

[P8257 WebMaster]

Alhazred ha scritto:

All'inizio del 3d si era parlato di questa cosa, mi è stata consigliato ChartoI, forse con l'evolversi del programma è effettivamente diventato inutile.

Avevo consigliato "ChartoI" perché credevo avesse bisogno di un rapporto colonna riga 1:1 .. in pratica la chartoi fa un casting formale e una semplice operazione "C - 64" così da eguagliare come numerazione le righe e le colonne...

Chiaramente ora è da valutare l'utilità effettiva...

Bye

[Alhazred]

Adesso va! (Y)

[Alhazred]

Originariamente inviato da P8257 WebMaster


Avevo consigliato "ChartoI" perché credevo avesse bisogno di un rapporto colonna riga 1:1 .. in pratica la chartoi fa un casting formale e una semplice operazione "C - 64" così da eguagliare come numerazione le righe e le colonne...

Chiaramente ora è da valutare l'utilità effettiva...

Bye

Difatti l'ho tolto.

[NS-1]

... ti consiglio di utilizzare gets() al posto di scanf()...

[NS-1]

gets()

Syntax:
#include <stdio.h>
char *gets( char *str );

The gets() function reads characters from STDIN and loads them into str, until a newline or EOF is reached. The newline character is translated into a null termination. The return value of gets() is the read-in string, or NULL if there is an error.

[P8257 WebMaster]

Si, molto più sicuro .. attenti che "gets" vuole una stringa .. anche se si legge 1 solo carattere .. comunque restituisce un puntatore a stringa....

Bye

[Alhazred]

Sto pensando ora a come tenere memorizzati i dati relativi alla griglia di ciascun giocatore.
Avevo pensato una cosa di questo tipo:

char *griglia;
griglia = malloc(dimensioni per contenere cArray[10][10])

La prima volta che parte il programma si riserva un'area di memoria per la griglia, quando parte la seconda volta si riferirà alla stessa area di memoria dato che anche questo cercherà di allocarla con lo stesso nome?
Se è così, come faccio a fare in modo che le 2 istanze del programma abbiano aree di memoria diverse?
Quanta memoria devo allocare per contenere l'array di 100 caratteri?
Basta fare il conto 1 carattere = 1 byte => 100 caratteri = 100 bytes o devo tener conto di qualcos'altro?