Discussion :

[Janitrix]

Salut à tous. Je fais une application qui nécessite de scanner le réseau local pour trouver d'autres ordinateurs disposants de l'application. Mon problème, c'est qu'en dessous d'un timeout de 1000 millisecondes, il ne me trouve plus des ordinateurs pourtant connectés. Voilà le code test :

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
 
try {
	InetAddress localAddress = InetAddress.getLocalHost();
	String hostIp = localAddress.getHostAddress();
	System.out.println(hostIp);
	int number =
		Integer.parseInt(hostIp.substring(hostIp.lastIndexOf(".") + 1));	        String rootIpPart =
		hostIp.substring(0, hostIp.lastIndexOf(".") + 1);
	System.out.println(rootIpPart);
	for(int i = 1; i < 255; i++) {
		if(i == number) {
			continue;
		}else {
		        InetAddress distant = 
				InetAddress.getByName(rootIpPart + i);
		        if(distant.isReachable(1000)) {
	                       System.out.println(rootIpPart + i + " : Reachable");
			}else {
	                       System.out.println(rootIpPart + i + " : Unreachable");
			}
		}
	}
}catch(Exception e) {
	e.printStackTrace();
}

Si je lance le code ci-dessus, ça marche. Mais dès que je change ceci

Code :

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

distant.isReachable(1000)

par cela

Code :

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

distant.isReachable(850)

il ne me trouve plus mes pc !
C'est très dérangeant, car attendre 1 seconde entre chaque test devient vite insupportable... Est-il techniquement possible de diminuer ce temps d'attente tout en faisant en sorte que les ordinateurs bien présents puissent avoir le temps de répondre ?!
Merci.

[Sanguko]

Peut être qu'en parallelisant les tests, en utilisant un pool de threads par exemple, tu pourrais gagner sur le temps global.

[Janitrix]

J'y ai pensé mais ça deviendrait vite le bordel, tant au niveau du code que du processus. Il n'y a pas une méthode plus orthodoxe ? Merci.

[Janitrix]

Bon j'ai fait ce que tu m'as proposé, et en effet, j'y gagne énormément en vitesse. Par contre, je trouve le code pas très "propre" :

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
 
try {
        InetAddress localAddress = InetAddress.getLocalHost();
	String hostIp = localAddress.getHostAddress();
	System.out.println(hostIp);
	int number =
		Integer.parseInt(hostIp.substring(hostIp.lastIndexOf(".") + 1));	final String rootIpPart =
		hostIp.substring(0, hostIp.lastIndexOf(".") + 1);
	System.out.println(rootIpPart);
	for(int i = 1; i < 255; i++) {
		if(i == number) {
			continue;
		}else {
			final int itemp = i;
			new Thread(new Runnable() {
				public void run() {
					try {
				                InetAddress distant = 
							InetAddress.getByName(rootIpPart + itemp);
						if(distant.isReachable(1000)) {
						      System.out.println(rootIpPart + itemp + " : Reachable");
						}else {
							System.out.println(rootIpPart + itemp + " : Unreachable");
						}
					}catch(Exception e) {
						e.printStackTrace();
				}
			}
	}).start();
 
}
	}
}catch(Exception e) {
		e.printStackTrace();
}

Bref c'est le bordel.. Vous auriez pas une idée pour alleger ce monstre ?
Merci.

[Sanguko]

Je n'en connais pas assez en réseau pour savoir si la solution que tu as choisie est la meilleure.
Pour la gestion des threads, et puisque tu es en java 5, tu devrais jeter un coup d'oeil de ce coté et également du coté de la classe ThreadPoolExecutor

[Janitrix]

Merci de tes réponses, je vais voir ça.