client - serveur reception du message parfois erroné

[cosmoff]

Bonjour,

voila j'ai crée un client TCP sur mon smartphone qui transmet les entiers 1,2,3 jusqu'à 10000, et mon serveur recois les messages et les enregistre dans un fichier texte. Le serveur écrit chaque entier qu'il recoit sur le fichier texte puis fait un retour chariot. Ainsi, j'aurais donc un ficher texte de la forme :
1
2
3
...
Le problème est que parfois je me retrouve avec des retours chariots, par exemple si j'émet 555, le serveur écrit sur le fichier texte
5
55

au lieu d'écrire tout simplement
555

j'ai aussi des fois des symboles bizarre qui apparaissent sur mon fichier texte.

Avez vous une idée du probleme et comment je peux le résoudre. Ce qui m'étonne c'est qu'en général ca marche mais dans quelque cas j'ai parfois des retour chariot et des symboles bizarre.

Merci d'avance pour votre aide
-----

[Jack]

Sûrement une erreur dans ton code

[cosmoff]

le probleme est que je ne vois pas l'erreur :

int main(int argc, char const *argv[])
{
	ofstream fichier("points.txt", ios::in | ios::trunc);
	TcpServer SocketTcp(15000); // on active le serveur avec comme argument le numéro de port
	string messageRecu;

	bool valeurAscii = true;

	if(fichier){

		while(1){
			messageRecu = SocketTcp.lire();
			
			valeurAscii = true;

			for (int i = 0; i < messageRecu.size()-1; ++i) // on vérifie si les characteres du string sont un chiffre
			{
				if(((int)messageRecu[i] < 48) && ((int)messageRecu[i] > 57)){// alors ce n'est pas un chiffre du code ascii
					valeurAscii = false;
					break;
				}

			}


			if(valeurAscii == true){
				if(((int)messageRecu[messageRecu.size()-1] != 10) && ((int)messageRecu[messageRecu.size()-1] != 13))//il n'y pas de retour chariot
					fichier << messageRecu << std::endl;
				else fichier << messageRecu;
			}
				
			
		}
			
		fichier.close();
	}

et la fonction lire() est :

string TcpServer::lire(){
	
	char buffer[256] ="";
	recv(csock, buffer, 256, 0);
	string messageRecu = buffer;
	return messageRecu;
}

arrives tu a voir une erreur ?

[Jack]

C'est mieux avec du code

Tu as essayé avec un debugger pour voir ce qui arrive réellement?
Qui est à l'origine de la classe TcpServer?
Il aurait peut-être été plus simple de gérer des fichiers binaires.

[A voir en vidéo sur Futura]

[pm42]

A la louche c’est la lecture par block de 256. A la fin du block on écrit dans le fichier avec un endl et il tombe parfois entre 2 chiffres. Ceci dit j’ai lu vite et je suis à un endroit avec peu de wifi donc je ne peux pas facilement creuser.
Le conseil du debugger est celui à suivre.

[jacknicklaus]

j'ai un doute sur la méthode Tcpserver::lire. Le buffer est seulement initialisé à zéro sur son premier caractère, par une chaine vide. Aucune garantie pour le reste du buffer. Tout dépend alors du comportement de la fonction recv.
Pour en avoir le coeur net, faire une initialisation complète à zéro de tout le buffer (memset).

par ailleurs, pour confirmer un éventuel problème de gestion des fins de blocs, est-ce que les caractères bizarres n'arriveraient pas tous les 256 octets, par hasard ?