Discussion :

[hansaplast]

EDIT : je vous renvoit a la 3eme post de ma part, le pb ayant evolué, ce n'est pas la peine de lire le debut

------ FIN EDIT ------

Bonjours,

comme d'habitude, a chaque fois que je me debloque (au bout de deux jours), je tombe immediatement sur un nouveau pb qui me re-bloque deux jours...

donc, voila a quoi vont etre consacrés mes deux prochaines journées :

(petite precision, j'utilise wxwidgets, et ses classe wxSocketBase, et wxocketClient)

coté envoie de donnée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[8192] ;
sockBaseImg->Write(&buffer, 8192);

coté reception :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[8192];
socketImgCli->Read(&buffer, 8192);

--------------------------------------------------------------------
ceci ne fonctionne pas (mon applit qui doit devevoir les données se gele)

alors qu'en procedant ainsi, ca fonctionne :
=>cot" envoit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
wxUint32 tempBis = 1234;
sockBaseImg->Write(&tempBis, 4);

=> coté reception :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
wxUint32 i, temp;
socketImgCli->Read(&i, 4);

donc la seule difference est dans les données envoyées :
pour ce qui fonctionne : un entier sur 4 octets
pour ce qui ne fonctionne pas : un tableau de 8192 char...

[Marco85]

coté envoie de donnée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[8192] ;
sockBaseImg->Write(&buffer, 8192);

coté reception :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[8192];
socketImgCli->Read(&buffer, 8192);

Salut hansaplast,

Essaie ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[8192] ;
sockBaseImg->Write(buffer, 8192);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[8192];
socketImgCli->Read(buffer, 8192);

A mon avis, ça devrait déjà marcher mieux ...

Redis-moi ...

Marco85

[hansaplast]

merci de ta reponse, j'ai essayé, mais ca ne marche pas.

pour info, la doc de cette fonction :

wxSocketBase::Write
wxSocketBase& Write(const void * buffer, wxUint32 nbytes)

This function writes a buffer of nbytes bytes to the socket.

Use LastCount to verify the number of bytes actually written.

Use Error to determine if the operation succeeded.

Parameters

buffer

Buffer with the data to be sent.
nbytes

Number of bytes.

encore merci de ta reponse

[hansaplast]

pb resolut : il fallait sortir de ma boucle d'evenement :

pseudo code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
case input:
traitements
break

alors que il fallait faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
case input:
if (entete)
   traitement
else
   autre traitement
finIf
break

en gros, j'essayait de recuperer toutes les données dans ma boucle ou seules les premierees données avaient etes receptionnées : faut laisser le soft zn ressortir, puis y re-rentrer...

c pas super util, je met en delestage, dsl.

[hansaplast]

bon,, bein en fait, j'ai un pb :

j'arrive a envoyer, puis a recevoir mes données, mais si je les re-ecrit dans un fichier... bein... ca doit etre un traducteur tot-o-matic que j'ai fait...

test!!
:)

tt

et l'autre toto entend :

䆐Ĕ徘Ĕἦ@徘Ĕ

et la c'est pas cool!

voici mon extrait de code :

envoi des données :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
	wxFile *myfile ;
	myfile= new wxFile("./image/testtexte.txt");
	off_t taille = myfile->Length(); //taille en octet :)
 
	//on envoie la taille totale du fichier qui va suivre :
	char buffer[taille] ; // on crée un buffer de la taille du fichier
	wxUint32 tempBis;
	tempBis = taille;
 	sockBaseImg->Write(&tempBis, 4);
	sockBaseImg->Write(buffer, tempBis);

reception :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
if (GetImgCliEnCours() == false) 
				{
				 	 //on recoit le header
				 	 memoLog->AppendText("Recuperation de la taille du ficier\n");
				 	 socketImgCli->Read(&tailleImg, 4);//on recupere la taille du fic
					 tailleImgRestante = tailleImg;			 	 
				 	 SetImgCliEnCours(true);
		 	   		 buffImage = new char[tailleImg];
			    } else {
				  	 //on recoit l'image 
				  	 wxUint32 tempReaded;
				  	 char newBuffer[30000];
				  	 socketImgCli->Read(&newBuffer, tempReaded);				  	 
				  	 tempReaded = socketImgCli->LastCount();
				  	 tailleImgRestante -= tempReaded;
 
				  	 for (int i = 0; i < tempReaded ; i++) 
				     {
					   	 	  buffImage[i + tailleImg - ( tailleImg - tailleImgRestante ) ] = newBuffer[i];
				     }
				  	 if ( tailleImgRestante == 0) 
				  	 {
 	 		   		    wxFile *myfile ;
 	 		   		 	myfile= new wxFile("./image/testTxtRecue.txt", wxFile::write);
				  	 	myfile->Write(buffImage, tailleImg);
				     }

en gros : coté envoie, j'envoit un header contenant la taille, puis le fichier "en un coup"

et coté reception : je recoit la taille du fichier, puis je bascukle sur la reception du fichier (d'ou mon 1 er teste IF), reception qui se fait en plusieures fois. une fois tout recu, j'ecrit mon fichier...

ps : y'a quelques variables inutiles, j'ai essayé de les enlever... mais y'en a ptet encore...

[Marco85]

voici mon extrait de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
	wxFile *myfile ;
	myfile= new wxFile("./image/testtexte.txt");
	off_t taille = myfile->Length(); //taille en octet :)
 
	//on envoie la taille totale du fichier qui va suivre :
	char buffer[taille] ; // on crée un buffer de la taille du fichier
	wxUint32 tempBis;
	tempBis = taille;
 	sockBaseImg->Write(&tempBis, 4);
	sockBaseImg->Write(buffer, tempBis);

Salut hansaplast,

J'ai lu vite fais cette partie et il me semble que ton

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char buffer[taille];

n'est pas correct. Si tu déclares un tableau, sa taille doit être connue à la compilation, ce qui ne me semble pas être le cas. A mon avis, utilise wxString, c'est fait pour ... ou alors déclare dynamiquement ton tableau (en oubliant pas de le désallouer après ...).

Marco85

[hegros]

[quote="hansaplast"]merci de ta reponse, j'ai essayé, mais ca ne marche pas.

pour info, la doc de cette fonction :

wxSocketBase::Write
wxSocketBase& Write(const void * buffer, wxUint32 nbytes)

This function writes a buffer of nbytes bytes to the socket.

Use LastCount to verify the number of bytes actually written.

Use Error to determine if the operation succeeded.

Parameters

Exactement. La fonction attends une adresse.

Ceci est juste :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(buffer,X);

Ceci ne devrait pas être juste ( ce que tu fais toi )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(&buffer,X);

A la rigeur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(&buffer[0],X);

Ce n'est pas parce qu'un code fonctionne que tu n'as pas fais d'erreur de programmation.

Ton compilateur ne te renvoi aucune erreur et aucun warning ???

[Laurent Gomila]

Ceci est juste :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(buffer,X);

Ceci ne devrait pas être juste ( ce que tu fais toi )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(&buffer,X);

Pour un tableau statique, &buffer et buffer renverront la même valeur ; seul leur type change. Donc c'était correct.

[hansaplast]

je condfirme que la declaration dynamique a une influence, en effet, maitenant, je me retrouve avec :

x
Î x
Î

... un generateur de smiley!!

je continue a chercher

(apres avoir fait un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char *buffer = new char[taille] ;

)

[hansaplast]

une autre erreure : lors de la recopie du buffer recu dans le buffer contenant tout le fichier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
				  	 for (wxUint32 i = 0; i < tempReaded ; i++) 
				     {
					   	 	  buffImage[i + ( tailleImg - tailleImgRestante ) ] = newBuffer[i];
				     }
				     tailleImgRestante -= tempReaded;

auparavant, je faisait une inversion dnas tous les sens (tres bon algo de cryprage, je pense...:'( )

[hegros]

Ceci est juste :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(buffer,X);

Ceci ne devrait pas être juste ( ce que tu fais toi )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char buffer[X];
write(&buffer,X);

Pour un tableau statique, &buffer et buffer renverront la même valeur ; seul leur type change. Donc c'était correct.

J'ai pas vu ou que c'était statique ce tableau dans le code

[hansaplast]

le roi des boulets vous met la solution :'(

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

myfile->Read(buffer, taille);

j'avait oublié de lire le contenu du fichier...

[Laurent Gomila]

J'ai pas vu ou que c'était statique ce tableau dans le code

Statique dans le sens "pas dynamique", pas static.