[C/C++] Battaglia Navale in C

[Alhazred]

E quest'altro è il processo che dovrebbe scrivere sulla memoria condivisa:

void scrivi(HANDLE mapping) {
	char colonna_c;
	int colonna_i, riga_s;
	t_Mossa *p;
	
	p = (t_Mossa*)MapViewOfFile(mapping, FILE_MAP_WRITE, 0, 0, 0);
	if(p == NULL)Errore_("Impossibile attaccare la memoria condivisa");
	puts("Inserire la lettera per la colonna: ");
	scanf("%s",colonna_c);
	colonna_i = ChartoI(colonna_c); //converto il carattere in intero
	
	//controllo dell'input per la colonna e scrittura
	if(colonna_i < 1 || colonna_i > 10) {
		do {
			printf("Input errato!\n");
			puts("Inserire la lettera per la colonna: ");
			scanf("%s",colonna_c);
			colonna_i = ChartoI(colonna_c);
		} while (colonna_i > 0  && colonna_i < 11);
	}
	strncpy(p->colonna, colonna_c, 0);

	puts("inserire un intero per la riga: ");
	scanf("%u", riga_s);
	
	//controllo dell'input per la riga e scrittura
	if(riga_s < 1 || riga_s > 10) {
		do {
			printf("Input errato!\n");
			puts("Inserire un intero per la riga: ");
			scanf("%u", riga_s);
		} while (riga_s > 0 && riga_s < 11);
	strncpy(p->riga, riga_s, 0);
	ExitProcess(0);
}

In neretto le righe a cui si riferiscono gli warnings.

Editato per non aggiungere post inutili.

[P8257 WebMaster]

L'hai compilato?

Attento a "p"...
Non so come hai implementato "t_Mossa" ma se è un typedef struct allora "p" è un puntatore a struttura che tu ricasti, probabilmente perché se non lo fai, il compilatore ti segnala che i tipi sono incompatibili ... ed assegni a MapViewOfFile, la quale ritorna LPVOID se non sbaglio, con la posizione di partenza dell'offset del mapping .. quindi come potrebbe essere un t_Mossa p?

Bye

[Alhazred]

Originariamente inviato da P8257 WebMaster
L'hai compilato?

Attento a "p"...
Non so come hai implementato "t_Mossa" ma se è un typedef struct allora "p" è un puntatore a struttura che tu ricasti, probabilmente perché se non lo fai, il compilatore ti segnala che i tipi sono incompatibili ... ed assegni a MapViewOfFile, la quale ritorna LPVOID se non sbaglio, con la posizione di partenza dell'offset del mapping .. quindi come potrebbe essere un t_Mossa p?

Bye

Ecco l'inizio:

#include <stdio.h>
#include <windows.h>

#define Errore_(x) {puts(x); ExitProcess(1);}

typedef struct {
	char colonna;
	int riga;
} t_Mossa;

Intanto ho compilato, tolto gli errori ed editato il codice postato in precedenza, adesso ho questi Warning:
battleship.c(59) : warning C4047: 'function' : 'char *' differs in levels of indirection from 'char '
battleship.c(59) : warning C4024: 'strncpy' : different types for formal and actual parameter 1
battleship.c(59) : warning C4047: 'function' : 'const char *' differs in levels of indirection from 'char '
battleship.c(59) : warning C4024: 'strncpy' : different types for formal and actual parameter 2
battleship.c(71) : warning C4047: 'function' : 'char *' differs in levels of indirection from 'int '
battleship.c(71) : warning C4024: 'strncpy' : different types for formal and actual parameter 1
battleship.c(71) : warning C4047: 'function' : 'const char *' differs in levels of indirection from 'int '
battleship.c(71) : warning C4024: 'strncpy' : different types for formal and actual parameter 2

[P8257 WebMaster]

Si, ci sono un po' di incongruenze di livello, tipiche dei puntatori (char * su char ad esempio)...

Attenzione ai tipi in return e ai parametri delle funzioni...

Bye

[P8257 WebMaster]

strncpy(p->colonna, colonna_c, 0);

colonna è un char!

Bye

[Alhazred]

Originariamente inviato da P8257 WebMaster
Si, ci sono un po' di incongruenze di livello, tipiche dei puntatori (char * su char ad esempio)...

Attenzione ai tipi in return e ai parametri delle funzioni...

Bye

Me lo ha detto anche il compilatore, ma non ne capisco il significato, cosa devo fare?

[P8257 WebMaster]

strncpy(p->riga, riga_s, 0);

Riga è un int...

Bye

[Alhazred]

[P8257 WebMaster]

Devi rispettare ciò che le funzioni vogliono e devi scaricare il loro valore di ritorno in un'entità adeguata.

Bisogna gaurdare sempre il prototipo delle funzioni...

ad esempio:

char *  strncpy ( char * dest, const char * src, sizet_t num );

Questa è la strncpy...
ritorna un puntatore a stringa di cui nel tuo caso puoi benissimo fregartene e lasciare che si perda... però vuole a sua volta un puntatore a stringa (char *) dest dove deve mettere il risultato, non puoi dargli un char o un int.. devi allocare una stringa almeno di 2 caratteri, poi vuole la stringa, in questo caso anche un singolo char va bene, da inserire e col size_t te la puoi cavare semplicemente col sizeof(src)...

Non voglio farti fare confusione, sii solo più attento ai tipi di dati e ai casting.. cerca di farne il meno possibile .. ti conviene e diminuisci i puntatori dove possibile.

Bye

[Alhazred]

Ok, non so come togliere questi warning.
Ho davanti un manuale di C, ma non so dove guardare.
Credo di aver capito che non posso assegnare un int o un char ad un tipo strutturato anche se lo assegno ad un campo del tipo in questione definito come int o char.
Come faccio scrivere i valori di colonna e riga in p?

[LoryOne]

Cio che voleva spiegarti WebMaster è che tu utilizzi una funzione che ha come valore di ritorno una stringa. Tu la utilizzi sia quando hai a che fare con una stringa, sia quando hai a che fare con un numero.

Cosa significa

t_Mossa *p;
	
	p = (t_Mossa*)MapViewOfFile(mapping, FILE_MAP_WRITE, 0, 0, 0);

Spiega a parole cosa imponi al compilatore di fare con queste due istruzioni.

[LoryOne]

p->colonna

cosa s'intewnde con questa notazione ?

[Alhazred]

Originariamente inviato da LoryOne
Cio che voleva spiegarti WebMaster è che tu utilizzi una funzione che ha come valore di ritorno una stringa. Tu la utilizzi sia quando hai a che fare con una stringa, sia quando hai a che fare con un numero.

Cosa significa

t_Mossa *p;
	
	p = (t_Mossa*)MapViewOfFile(mapping, FILE_MAP_WRITE, 0, 0, 0);

Spiega a parole cosa imponi al compilatore di fare con queste due istruzioni.

Con questa dovrei avere un puntatore di tipo t_Mossa che punta all'inizio della memoria condivisa.

p->colonna
cosa s'intende con questa notazione ?

Questa dovrebbe indirizzare sul campo colonna di p.

Cosa c'è di sbagliato? Scusate l'ignoranza, ma come già detto il C lo sto imparando ora.

[P8257 WebMaster]

Alhazred ha scritto:
Ok, non so come togliere questi warning.
Ho davanti un manuale di C, ma non so dove guardare.
Credo di aver capito che non posso assegnare un int o un char ad un tipo strutturato anche se lo assegno ad un campo del tipo in questione definito come int o char.
Come faccio scrivere i valori di colonna e riga in p?

Se t_Mossa è un tipo strutturato formato da un char e un int, allora non hai bisogno di funzioni intermedie per operare con i campi in quanto essi sono comunque dei tipi nativi.

Ti basta l'operatore di referencing e il gioco è fatto:

Se Colonna = 'A' e Riga = 1 allora:

t_Mossa p;

p -> Colonna = 'a';
p -> Riga = 1;

Semplicemente.

Se usi il tipo dati in questo modo, vedrai che spariranno un bel po' di warning, perché elimini la 'strncpy' usata in maniera errata.

Bye

[LoryOne]

Tranquillo, non ti sto dicendo che stai sbagliando.
Ora ti prego di dirmi cosa riscontri in questi due programmini:

Programmino 1:

typedef struct {
    char colonna;
    int riga;
} mossa;

main(void)
{
   mossa *p;
   p->colonna=97;
   p->riga=1;
}

Programmino 2:

typedef struct {
    char colonna;
    int riga;
} mossa;

main(void)
{
   mossa p;
   p.colonna=97;
   p.riga=1;
}

Ti dirò subito che il compilatore li compila senza problemi.