Discussion :

[houssine91]

Salut,
j'ai fais un programme de tchat avec les sockets et il fonctionne , mais le problème c'est qu'il consomme presque 26% du processeur , j'ai constaté que les portions du programme responsables de ce problème sont les suivantes:

le 1er threads pour la réception des messages:

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
public void run() {
		// TODO Auto-generated method stub
		while(!stop){
			try {
				if((MsgRecu = in.readLine())!=null){
				System.out.println(MsgRecu);
				fenetre.recu(MsgRecu);
				}
			} catch (IOException e) {
				e.printStackTrace();
				try {
					in.close();
				} catch (IOException e1) {
					e1.printStackTrace();
				}
				break;
			}
		}
	}

Le 2ème thread pour l'émission

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public void run() {
		while(!stop&&message!=null){	
		if(envoyer){
			message=fenetre.envoyer();
		out.println(message);
		out.flush();
		envoyer=false;
		}
	}

out.close();
	}

Est ce que quelqu'un peut me proposer une solution ,pour empcher le processus de travailler tant que aucun message n'est encore arrivé?

[Sharcoux]

Tu n'es peut-être pas obligé de vérifier les messages non-stop. Ton utilisateur peut peut-être tolérer un délai d'une seconde par exemple. Tu peux donc peut-être mettre ton Thread en pause quelques centaines de millisecondes après avoir lu tous les messages reçus. Tu recommences ensuite. Ca c'est si tu n'as pas de moyen d'être averti quand un message est arrivé.

Quant à l'émission, tu peux te contenter d'émettre seulement au moment où un message est envoyé. Pas besoin de vérifier en permanence puisque c'est toi qui contrôle.

D'une manière générale, il vaudrait mieux que tu essayes de baser ton code sur une logique de Listener plutôt que sur une logique procédurale.

[houssine91]

Tu n'es peut-être pas obligé de vérifier les messages non-stop. Ton utilisateur peut peut-être tolérer un délai d'une seconde par exemple. Tu peux donc peut-être mettre ton Thread en pause quelques centaines de millisecondes après avoir lu tous les messages reçus. Tu recommences ensuite. Ca c'est si tu n'as pas de moyen d'être averti quand un message est arrivé.

Quant à l'émission, tu peux te contenter d'émettre seulement au moment où un message est envoyé. Pas besoin de vérifier en permanence puisque c'est toi qui contrôle.

D'une manière générale, il vaudrait mieux que tu essayes de baser ton code sur une logique de Listener plutôt que sur une logique procédurale.

j'ai basé le code sur les listener et maintenant et il ne consomme pas beaucoup , merci

[Sharcoux]

Content d'avoir pu t'aider.
Passe en résolu stp.

[houssine91]

Content d'avoir pu t'aider.
Passe en résolu stp.

le problème est de la classe de l'émission, la solution est :

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
public void run() {
		while(true){
		this.suspend();
		if((message=fenetre.envoyer())!=null){
		out.println(message);
		out.flush();
		}
		}
	}
 
	// si on appui sur la bouton envoi
	@SuppressWarnings("deprecation")
	public void actionPerformed(ActionEvent arg) {
		this.resume();	
	}