TCP/IP String lent

**Belegkarnil** · 23/06/2006, 08h35

Bonjour, je fais un serveur et un client sous TCP/IP. J'envoie une très longue chaîne de caractères, et cela prend beaucoup de temp. Y a t'il un moyen de raccourcir? Soit un moyen d'envoyer plsu vite les données, ou compresser la chaine (par exemple avec l'agorithme zip)...?

Merci d'avance

**sozie9372** · 23/06/2006, 08h54

Montre ton code d'envoi

Tu peux peut être utiliser un tableau de bytes, tu gagneras du temps...

**Belegkarnil** · 23/06/2006, 13h01

Je veux bien, mais il n'y a rien de spétial. Je vais chercher dans informations dans une base de données Access. Je concatène toutes ces informations en une string, et je l'envoie. Quand je fais copier coller de la String dans word, j'obtient 70pages.

Côté client :

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
27
28
 
  	public void connection(String requete) {
    	Socket socket;
    	DataInputStream userInput;
    	PrintStream theOutputStream;
   try{
    	socket = new Socket(serveur, port);
    	try {
      		BufferedReader in = new BufferedReader(new InputStreamReader(socket.getInputStream()));
      		PrintStream out = new PrintStream(socket.getOutputStream());
      		out.println(pseudo+";"+passe+";"+requete);
      		String reponce = "";
      		boolean continuer=true;
      		do{
				String temp = in.readLine();
				reponce+=temp+"\n";
				if(temp==null)continuer=false;
			}while(continuer);
      		rep=reponce;
    	} catch (Exception e) {
      		e.printStackTrace();
    	}finally{
			try{socket.close();
			}catch(IOException e){}
		}
	}catch(UnknownHostException uhe){uhe.printStackTrace();
	}catch(IOException ioe){ioe.printStackTrace();}
  	}

Côté serveur :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
 
import java.net.*;
import java.io.*;
import java.security.*;
import java.nio.channels.*;
class TCPIP{
	static ServerSocketChannel serverSocketChannel = null;
	static ServerSocket serverSocket = null;
	static Socket clientSocket = null;
	static boolean fin = false;
	public static void main(String arg[]){
		try{
     		System.out.println("Lancement du serveur");
			SocketAddress address = new InetSocketAddress(666);
			serverSocketChannel = serverSocketChannel.open();
			serverSocket = serverSocketChannel.socket();
			serverSocket.bind(address);
			while(!fin){
				try{
					clientSocket = serverSocket.accept();
					traitements();
				} catch(ClosedByInterruptException IException){
					fin = true;
				}
			}
		} catch(IOException IException){
			IException.printStackTrace();
		} finally {
			try{
				if(serverSocket != null) serverSocket.close();
			} catch(Exception IException){
			}
			try{
				if(serverSocketChannel!=null) serverSocketChannel.close();
			} catch(Exception IException){
			}
		}
	}
	public static void traitements() {
		try{
			System.out.println("Connection établie");
			String messageRecu = "",messageEnvoye,
			ip= ""+clientSocket.getInetAddress();
			//if(ip.equals("/127.0.0.1")) fin=true;
			BufferedReader in = new BufferedReader(new InputStreamReader(clientSocket.getInputStream()));
			PrintStream out = new PrintStream(clientSocket.getOutputStream());
			messageRecu = in.readLine();
			String pseudo = messageRecu.substring(0,messageRecu.indexOf(";")),
					passe = messageRecu.substring(messageRecu.indexOf(";")+1,messageRecu.indexOf(";",1+messageRecu.indexOf(";"))),
					requete = messageRecu.substring(passe.length()+pseudo.length()+2);
			messageEnvoye = "";
			String idOK;
			idOK=Access.verifieID(pseudo,Codeur.MD5(passe),"SELECT * FROM autorisation");
			if(!idOK.equals("false")){
				int numero = Integer.valueOf(idOK);
					idOK=Access.verifieID(pseudo,Codeur.MD5(passe),"SELECT * FROM autorisation");
					messageEnvoye=toAString(Access2.newEnleveNull(Access2.newEleveAvecLeTout(Access2.newIdEleves(""+numeroProf))));//,numeroProf
			}else messageEnvoye += "ERREUR";
				out.println(messageEnvoye);
				clientSocket.close();
				System.out.println("Connection fermée");
		}catch (Exception e) {
			e.printStackTrace();
		}
	}
	public static String toAString(String donnee[][]){
		String retour="";
		for(int i=0;i<donnee.length;i++){
			retour+=((char)18);
			for(int j=0;j<donnee[i].length;j++)
				retour+=((char)8)+donnee[i][j];
		}
		return retour;
	}
}

Voila puisqu'il faut absolument le code

**Janitrix** · 23/06/2006, 13h09

Quelques remarques :

traitements();

: pourquoi l'avoir déclaré static si c'est pour l'appelé ainsi ?
Ensuite à mon avi, la lenteur proviens de ça :

Code :

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

1
2
3
4
5
for(int i=0;i<donnee.length;i++){
			retour+=((char)18);
			for(int j=0;j<donnee[i].length;j++)
				retour+=((char)8)+donnee[i][j];
		}

C'est pas très propre ton code, mais je suis pas là pour te faire la leçon (en plus si tu voyais le mien

) mais il me semble que cette "double" boucle
est un peu "caca"

.
Enfin voila. La méthode toAString() est également en static, alors que ne l'utilise pas en t'en que static. Cela n'a pas grande importance, mais bon...

**sozie9372** · 23/06/2006, 13h20

Bon déjà, tu peux faire :

Code :

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

1
2
3
4
5
 
String temp;
while((temp = in.readLine) != null) {
	reponse+=temp+"\n";
}

Au lieu de

Code :

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

1
2
3
4
5
6
 
      		do{
				String temp = in.readLine();
				reponce+=temp+"\n";
				if(temp==null)continuer=false;
			}while(continuer);

Je cherche pour le reste...

**le y@m's** · 23/06/2006, 13h27

Envoyé par sozie9372

Bon déjà, tu peux faire :

Code :

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

1
2
3
4
5
 
String temp;
while((temp = in.readLine) != null) {
    reponse+=temp+"\n";
}

L'utilisation d'un StringBuilder (ou StringBuffer si < Java 5) serait plus appropriée

.
cf Comment concatener des chaînes de caractères ?

**sozie9372** · 23/06/2006, 13h31

Je voulais savoir, si tu envoies de petites données, ca mets du temps ?
Sinon, je pense toujours que la solution d'un tableau de bytes en plusieurs fois serait judicieuse...

**adiGuba** · 23/06/2006, 18h43

Envoyé par le y@m's

L'utilisation d'un StringBuilder (ou StringBuffer si < Java 5) serait plus appropriée

.
cf Comment concatener des chaînes de caractères ?

100 % D'accord ! Et à mon avis ton principal problème vient de là !!!

L'utilisation de l'opérateur + (ou +=) sur des String dans une boucle est un anti-pattern, car il implique la création de nombreuses chaines temporaires. Ainsi si tu tentes de lire un fichier de 1 Mo par ligne de 100 caractères, tu auras allouer/désallouer plus de 700 Mo de mémoire...

De même je te conseille d'utiliser un bloc try/finally par ressource que tu ouvres, cela te permettra de libérer les ressources proprement quoi qu'il arrive... Et si tu utilises un try/catch fais le remonter d'un cran plutôt que d'utiliser un try/catch/finally qui t'oblige à traiter un try/catch supplémentaire.

Cela pourrait donner :

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
    public void connection(String requete) {
        try {
            Socket socket = new Socket(serveur, port);
            try {
                BufferedReader in = new BufferedReader(new InputStreamReader(
                        socket.getInputStream()));
                try {
                    StringBuilder buf = new StringBuilder();
                    String line;
 
                    while ( (line=in.readLine()) != null ) {
                        buf.append(line).append("\n");
                    }
 
                    rep = buf.toString();
                } finally {
                    in.close();
                }
            } finally {
                socket.close();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

a++

**Belegkarnil** · 24/06/2006, 06h53

Oui, si je met des petites données, cela est très rapide. La lenteur ne vient pas du code (à la limite, ca ralenti d'une seconde si je n'ai pas fait le plus court possible), mais du paquet de données. J'envoie une String qui équivaut à 70pages de word. La lenteur provient donc des données. Comment les compacter...?
Pour envoyer en byte, je ne sais pas comment faire.

**adiGuba** · 24/06/2006, 10h35

Envoyé par Belegkarnil

La lenteur ne vient pas du code (à la limite, ca ralenti d'une seconde si je n'ai pas fait le plus court possible), mais du paquet de données. J'envoie une String qui équivaut à 70pages de word. La lenteur provient donc des données. Comment les compacter...?

As-tu essayé avec un StringBuffer/StringBuilder à la place de la concaténation de chaine ?
70pages de Word ? Ca équivaut à quoi en Mo ??? Cela ne doit être bien méchant...
As-tu essaysé d'envoyer les données via FTP (ou un autre protocole) ? Est-ce que c'est aussi lent qu'avec ton programme ?

Envoyé par Belegkarnil

Pour envoyer en byte, je ne sais pas comment faire.

Il suffit d'utiliser un byte[] suffisamment grand (1024 par exemple), et de lire directement avec la méthode read(byte[]) de l'InputStream. Mais ce n'est interréssant que si tu renvoit directement les données plutôt que de tout chargé en mémoire...

a++

**Belegkarnil** · 24/06/2006, 16h51

Non, je n'ai pas essayer avec un autre protocole. Ni avec les StringBuffer/StringBuilder . Quel est l'avantage avec des StringBuffer/StringBuilder ? Merci

**le y@m's** · 24/06/2006, 16h57

Envoyé par Belegkarnil

Quel est l'avantage avec des StringBuffer/StringBuilder ?

Déjà dit :

Comment concatener des chaînes de caractères ?

**orelero** · 25/06/2006, 00h16

Quel est l'avantage avec des StringBuffer/StringBuilder ?

Je crois que ça permet un traitement plus rapide vu qu'on ne recréer pas à chaque fois une autre chaine de caractere dés qu'il y a une modification

.
J'ai vu quelque part que c'est ça qu'il faut systématiquement utiliser du coté serveur quand il y a un traitement lourd de chaines de caractères.

**Belegkarnil** · 26/06/2006, 14h17

En fait, le serveur envoie 1Ko toute les minutes environs.

**adiGuba** · 26/06/2006, 14h20

Envoyé par Belegkarnil

En fait, le serveur envoie 1Ko toute les minutes environs.

Et ? Ton problème est résolu ? Tu en es où ???

a++

**Belegkarnil** · 26/06/2006, 17h17

Et bien c'est toujours aussi lent. J'envoie en une fois une String, et donc le serveur devrait envoyer + ou - 300Ko en une fois ce qui me semble ne doit pas prendre longtemps. Mais le serveur n'envoie que 1Ko / minute + ou -

**adiGuba** · 26/06/2006, 17h46

Envoyé par Belegkarnil

Et bien c'est toujours aussi lent. J'envoie en une fois une String, et donc le serveur devrait envoyer + ou - 300Ko en une fois ce qui me semble ne doit pas prendre longtemps. Mais le serveur n'envoie que 1Ko / minute + ou -

Tu es sûr que tu n'aurais pas un problème réseau ? Tu as essayé avec un autre protocole ?
Quel est ton code désormais ???

a++

**Belegkarnil** · 26/06/2006, 21h43

Mon code n'est pas beacoup changé, mais comment puis-je changer de protocole? Est-ce que certains ports ont une vitesse limitées? Lorsque je calcule le temps pour préparer ma chaine à envoyé, il ne faut même pas une seconde, mais l'envoie dure plus de temps.

**sozie9372** · 26/06/2006, 23h11

Je trouve ca un peu long tout de même...

**Belegkarnil** · 27/06/2006, 08h13

Le temp de pour préparer toutes les données? Oui mais j'ai du travailler avec une base de données mal faite, le directeur de mon école ne voulait pas qu'on la change. Donc je dois passer par plusieurs tables pour associer des données, trier, et redemander des données... Mais si ca prend une seconde, ca ne dérange pas trop par à l'envoie de données qui peut prendre 10minutes -30minutes

Voila pourquoi je demande de l'aide.

TCP/IP String lent

Entrée/Sortie Java

Discussions similaires

Partager

Partager