Discussion :

[Ceubex]

Bonjour,

J'utilise le code suivant pour télécharger le code de ma page :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
try {
			URL u=new URL("http://monurl");
			InputStream  is = u.openStream();
			DataInputStream dis= new DataInputStream(new BufferedInputStream(is));
				 while ((s = dis.readLine()) != null) {
					 code+=s+"\n";
				 }
 
		} catch (MalformedURLException e) {
			e.printStackTrace();
		} catch (IOException e) {
			e.printStackTrace();
		}

Mon soucis c'est que des fois cela prend beaucoup de temps voir se bloque et que pendant ce temps l'utilisateur ne peut rien faire car le logiciel est figé. J'aimerais que l'utilisateur puisse annuler son action et stopper le téléchargement. Mais comment faire ?

[sinok]

Toute opération bloquante doit être réalisée dans un Thread séparé.

Cf http://gfx.developpez.com/tutoriel/j...ing-threading/

[adiGuba]

Salut,


De plus le code de lecture est particulièrement affreux :

• Il n'y a pas de try/finally pour la fermeture par flux
• Tu fais une lecture ligne par ligne alors que tu n'en as aucune utilité (un lecture par bloc serait plus adapté)
• Tu recrées une String par concaténations, ce qui peut exploser la mémoire (un StringBuilder ou StringBuffer serait plus approprié)

a++

[Ceubex]

Merci sinok
adiGuba qu'entends tu par "lecture par bloc"

[adiGuba]

adiGuba qu'entends tu par "lecture par bloc"

Ben une lecture avec un buffer de taille fixe. Dans ton cas il n'y a aucun intérêt à lire par ligne, ce qui peut augmenter les traitements si les lignes sont très courtes...

De plus tu ne précises pas non plus l'encodage : c'est celui du système qui est pris en compte et qui pourrait te poser problème. Il serait préférable de le spécifier ou d'essayer de le récupérer depuis l'URLConnection si possible.

Sinon ton code de copie devrait plutôt ressembler à ceci :

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
URL u=new URL("http://monurl");
Reader reader = new InputStreamReader( u.openStream(), "utf-8" );
try {
	// Le StringBuilder qui contiendra toutes les données
	StringBuilder sb = new StringBuilder();
	// Le buffer de lecture :
	char[] cbuf = new char[8192];
	// La quantité d'élément lu à chaque iteration (peut varier)
	int len;
 
	// boucle de lecture / copie 
	while ( (len = reader.read(cbuf)) >= 0 ) {
		sb.append(cbuf, 0, len);
	}
	// Copie en String :
	code = sb.toString();
} finally {
	reader.close();
}