Discussion :

[tooney]

Bonjour à tous!!!

Voici mon problème :

Je réalise un chat, avec donc clients / serveur. A la connexion d'un client, le serveur lui envoie un message de bienvenue sur plusieurs lignes du style :

Serveur >>> Bienvenue sur le chat
Serveur >>> Il y a actuellement 4 connectés
Serveur >>> Vous n'avez pas encore de pseudo
etc....

Le seul problème, c'est que mon client ne reçoit pas tjs les données, et c'est complétement aléatoire. Des fois il va afficher une ligne, des fois toutes les lignes.

Ce n'est pas un problème au niveau du serveur, car j'ai testé avec telnet, et ça marche impecc...

Voici un extrait du code de mon client :

Pour la réception des données :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
  // Recevoir les données du serveur sur la socket
  private String recevoir() {
    String texte = "";
    try {
      BufferedReader input = new BufferedReader(new InputStreamReader(this.serveur.getInputStream()));
      texte = input.readLine();
    }  
    catch (IOException e) {}
    flux.afficheFluxEntrée(texte);
    return texte;
  }

Utilisation de la méthode recevoir() dans le Client :

Code :

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

1
2
3
4
5
6
 
    while (true) {
      String données = recevoir();
      if (données != "")
        traitementDonnées(données); //Traitement des données reçues
    }

Au début je pensais que mon traitement de données était trop long, et que l'envoi de données était trop rapide pour qu'elles soient toutes traitées par le client. Malgré tout, mon traitement est relativement restreint, et je pense qu'il est possible que ça vienne plutôt de la méthode readLine(), car j'ai lu qu'elle était deprecated...

Quelqu'un aurait-il une solution pour moi?

[al85]

fais un flush apres chaque ecriture dans le flux pour vider le buffer et forcer l'ecriture.

[Wookai]

Hello !

Après vérification, readLine() n'est pas deprecated, et je l'ai souvent utilisée, il me semble pas qu'il y ait de problème avec... C'est vrai qu'un flush devrait peut-être arranger les choses, mais le problème vient peut-être du fait que tu ne fermes ni ton BufferedReader ni le InputStreamReader.

Mais ma question est la suivante : pourquoi crées-tu un nouveau stream et un nouveau reader sur ton socket à chaque lecture, plutôt que d'en créer un à l'ouverture du socket et de garder toujours le même ? Peut-être qu'en faisant ainsi tu n'aurais pas ce problème...

[tooney]

fais un flush apres chaque ecriture dans le flux pour vider le buffer et forcer l'ecriture.

Je fais bien sûr un flush après chaque écriture dans le flux (flush automatique). J'utilise :

Code :

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

PrintWriter output = new PrintWriter(serveur.getOutputStream(), true);

Je vais essayer la solution de Wookai de créer un nouveau stream uniquement à l'ouverture de la socket et je vous tiens au courant.

DD

[tooney]

Merci Wookai, il semblait effectivement qu'il n'aimait pas trop l'ouverture intempestive d'un nouveau reader!!! J'ai corrigé le problème comme tu me l'as suggéré, à savoir d'ouvrir le reader une seule fois à l'ouverture de la connexion, et ensuite de faire uniquement un input.readLine(), et pour l'instant pas de souci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  private void connexionServeur(String[] args) {
 
    int port;
    try {
      port = Integer.parseInt(args[1]);
      this.serveur = new Socket(args[0], port);
      output = new PrintWriter(this.serveur.getOutputStream(), true);
      input = new BufferedReader(new InputStreamReader(this.serveur.getInputStream()));
 
    }
    catch (UnknownHostException e) {}
    catch (IOException e) {}
  }

Merci beaucoup Wookai!!!!

[Wookai]

De rien ! Content d'avoir pu aider...