[C/C++] Battaglia Navale in C

[Alhazred]

E' successo di nuovo

[Alhazred]

Premessa
Ho creato la memoria condivisa in questo modo nel main:
mapping = CreateFileMapping(INVALID_HANDLE_VALUE,NULL,PAGE_R EADWRITE,0,0,"shared_mem");

e la attacco in controlla() così:
p = (t_Mossa *)MapViewOfFile(mapping, FILE_MAP_WRITE, 0, 0, 0);

p è dichiarato t_Mossa *p; e l'HANDLE mapping è passato a controlla() come parametro.

Domanda
E' corretto leggere il contenuto della memoria condivisa in questa maniera?
colonna_c = p->colonna;
riga = p->riga;

[P8257 WebMaster]

A me sembra corretto...

Il problema della "scrivi()" potrebbe essere che in essa hai aggiunto una chiamata ad AggiornaCampo con diversi parametri?...

Bisognerebbe vedere il codice fin'ora prodotto....

Bye

[Alhazred]

Il codice di "Scrivi()" è postato nella pagina precedente, comunque l'errore lo da anche se metto tutta la funzione come commento facendola precedere da /* e seguire da */, in pratica anche togliendola il problema rimane... può essere colpa del compilatore?

[P8257 WebMaster]

Si, ho visto scrivi .. ed appunto per questo suggerivo che sarebbe utile vedere tutto il codice fin'roa prodotto...

Bye

[Alhazred]

Ecco il codice, ho segnato il punto che genera l'errore, è in fondo.

[P8257 WebMaster]

Ma ovviamente scusa .. la AggiornaCampo vuole un parametro che tu non gli dai nella chiamata:

void AggiornaCampo(char bArray[10][10])

chiamata:

AggiornaCampo();

Tra l'altro mi sembra che questo parametro non venga usato nella funzione, quindi se fai:

void AggiornaCampo(void)

L'errore sparisce...

Bye

P.S.: Mancano i prototipi, in C i prototipi sono obbligatori.

[Alhazred]

Perché però se non tocco Scrivi() non ho errori e il programma funziona?

[P8257 WebMaster]

Perché evidentemente non viene ricompilato l'oggetto del sorgente C, ma viene solo linkato.

Bye

[Alhazred]

Ok, comunque mi hai convinto che posso anche toglierlo il parametro in AggiornaCampo.

[P8257 WebMaster]

Alhazred ha scritto:
Ok, comunque mi hai convinto che posso anche toglierlo il parametro in AggiornaCampo.

.. Più che convincerti di toglierlo, volevo sottolineare che le chiamate devono essere coerenti con gli eventuali parametri ...

Un consiglio: creati un header coi prototipi, oppure metti i prototipi all'inizio del file appena sotto i #define...

Bye

[Alhazred]

Originariamente inviato da P8257 WebMaster


.. Più che convincerti di toglierlo, volevo sottolineare che le chiamate devono essere coerenti con gli eventuali parametri ...
...

L'ho tolto perché tanto per ricreare il campo va a leggere una variabile globale (cArray[10][10]) sarebbe inutile passargliela di nuovo, giusto?

[P8257 WebMaster]

Alhazred ha scritto:

L'ho tolto perché tanto per ricreare il campo va a leggere una variabile globale (cArray[10][10]) sarebbe inutile passargliela di nuovo, giusto?

(Y) ...

Bye

[Alhazred]

Quando parte, la prima istanza, dovrà creare la memoria condivisa, il semaforo (fin qui ok) e restare in attesa finché non parte la seconda per poi decidere a chi toccherà cominciare e comunicarglielo.
Come posso far capire alla prima istanza che la seconda è partita?
Io avrei pensato ad una cosa del genere:
- P1 parte, crea la memoria condivisa e il semaforo con 0 token liberi
- P1 lancia una WaitForSingleObject(sem, INFINITE) per attendere che il semaforo si liberi
- P2 parte, sa che esistono sia la memoria condivisa che il semaforo
- P2 rilascia un token su sem (si può fare se prima non lo occupa?)
- P1 accede al semaforo e quindi sa che P2 è partito

E' una soluzione valida?

[Alhazred]

Per ora questo è il main

int main(int argc, char*argv[]) {
	HANDLE hOut;
	int i = 0, j = 0, istanza;

	HANDLE mapping, sem;
	STARTUPINFO si;
	PROCESS_INFORMATION pi;

	//Inizializzazione della griglia di gioco
	for(i=1; i<11; i++)
		for(j=1; j<11; j++)
			cArray[i][j] = ' ';	

	memset(&si, 0, sizeof(si));
	memset(&pi, 0, sizeof(pi));
	si.cb = sizeof(si);

	mapping = CreateFileMapping(INVALID_HANDLE_VALUE,NULL,PAGE_READWRITE,0,0,"shared_mem");
	if(mapping == INVALID_HANDLE_VALUE) //se è vera, questa è la prima istanza del programma
		istanza = 1;
	else istanza = 2;
//qualcosa tipo il case of	
{
		//Creazione memoria condivisa
		mapping = CreateFileMapping(INVALID_HANDLE_VALUE,NULL,PAGE_READWRITE,0,0,"shared_mem");
		if(mapping == INVALID_HANDLE_VALUE)Errore_("Errore nella creazione della memoria condivisa");

		//Creazione semaforo
		sem = CreateSemaphore(NULL,0,1,"semaforo");
		if(sem == INVALID_HANDLE_VALUE)Errore_("Errore nella creazione del semaforo");

		WaitForSingleObject(sem,INFINITE);
	    
	sem = OpenSemaphore(SEMAPHORE_ALL_ACCESS, FALSE, "semaforo");
	if(sem == INVALID_HANDLE_VALUE)Errore_("Errore nell'apertura del semaforo");

	hOut = GetStdHandle(STD_OUTPUT_HANDLE);
    
	posizione.X = 0;
         posizione.Y = 24;
	SetConsoleCursorPosition(hOut,posizione);
}

L'int istanza mi dirà se quella in esecuzione è la prima o la seconda, dato che ci sarà da eseguire del codice diverso a seconda dell'istanza, esiste una struttura tipo il Case of che usavo ai tempi del Pascal? Come è fatta in C?