Discussion :

[pierreonxbox]

Bonjour,

J'ai développé une petite application qui utilise une connexion Url pour récupérer un fichier HTML présent sur un de mes serveurs web, puis qui extrait différentes images/fichier en fonction du contenu de cette page.

Hors lors de mes tests, il apparait que les performance que ce soit lors de la lecture de la page ou la copie des fichiers sont vraiment pas terrible.

Pour la lecture de la page, j'utilise le mode de chargement suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	URL url = new URL("http://www.monsite.com");
		InputStreamReader ipsr = new InputStreamReader(url.openStream());
		BufferedReader br = new BufferedReader(ipsr);
		String line = null;
		String lineRes="";
		while ((line = br.readLine()) != null) {
		      if(id==0){}else{
			lineRes+="\n"+line;
			}
		      id++;
		}

Le telechargement des images de la manière suivante

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
	URL   url  = new URL(tmp);
	URLConnection urlC = url.openConnection();
	InputStream is = url.openStream();
	FileOutputStream fos=null;
	fos = new FileOutputStream("/sdcard/IMG_/"+this.tab[IMG]+"/"+file);
	int oneChar, count=0;
	while ((oneChar=is.read()) != -1){
		fos.write(oneChar);
		count++;
	}
	is.close();
	fos.close();

Comment puis je améliorer les performances lors de la récupération des flux ?

Cordialement

[MrDuChnok]

Salut,

Concernant les performances, tu peux regarder du coté de DDMS, et surveiller un peu ce qu'il se passe au niveau mémoire.
Ensuite tu peux essayer de threader un peu tout ça afin de ne pas bloquer ton appli (si ce n'est pas déjà fait).
Enfin, quelques petites remarques sur ton code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
URL url = new URL("http://www.monsite.com");
InputStreamReader ipsr = new InputStreamReader(url.openStream());
BufferedReader br = new BufferedReader(ipsr);
String line = null;
String lineRes="";
if ((line = br.readLine() != null) {
  while ((line = br.readLine()) != null) {
     lineRes+="\n"+line;
  }
}

ça t'évitera de faire un test à chaque tour de boucle.
Ensuite, pour ton téléchargement d'image, au lieu de lire byte par byte, tu peux directement lire un buffer de byte, ça devrait être un petit peu plus rapide.

[adiGuba]

Salut,


Le problème de performance ne vient pas de l'InputStream, mais de la concaténation de String dans une boucle, qui produit un grand nombre d'objet temporaire.

Il faut utiliser un StringBuilder !


De plus, pourquoi lire le fichier ligne par ligne si tu veux le reproduire en entier. Autant le lire directement par bloc...

Enfin il manque les b]try/finally[/b] pour la libération propre des ressources...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	URL url = new URL("http://www.google.fr");
	StringBuilder builder = new StringBuilder();
	InputStreamReader reader = new InputStreamReader(url.openStream());
	try {
		char[] cbuf = new char[8192];
		int len;
		while ( (len=reader.read(cbuf)) >= 0 ) {
			builder.append(cbuf, 0, len);
		}
	} finally {
		reader.close();
	}
	String lineRes = builder.toString();

a++

[pierreonxbox]

Merci à tous les deux pour vos conseils, effectivement maintenant cela va mieux !