Android : lecture efficace d'un flux d'entrée

Question

Je fais une requête HTTP get à un site web pour une application Android que je suis en train de réaliser.

J'utilise un DefaultHttpClient et j'utilise HttpGet pour émettre la requête. Je reçois la réponse de l'entité et à partir de celle-ci, j'obtiens un objet InputStream pour obtenir le html de la page.

Je parcours ensuite la réponse en procédant comme suit :

BufferedReader r = new BufferedReader(new InputStreamReader(inputStream));
String x = "";
x = r.readLine();
String total = "";

while(x!= null){
total += x;
x = r.readLine();
}

Cependant, c'est horriblement lent.

Est-ce inefficace ? Je ne charge pas une grosse page web - www.cokezone.co.uk pour que la taille du fichier ne soit pas trop importante. Existe-t-il une meilleure façon de procéder ?

Merci

Andy

Answer 1

Le problème dans votre code est qu'il crée beaucoup de lourdes String des objets, en copiant leur contenu et en effectuant des opérations sur eux. Au lieu de cela, vous devez utiliser StringBuilder pour éviter de créer de nouvelles String à chaque ajout et pour éviter de copier les tableaux de caractères. L'implémentation pour votre cas serait quelque chose comme ceci :

BufferedReader r = new BufferedReader(new InputStreamReader(inputStream));
StringBuilder total = new StringBuilder();
for (String line; (line = r.readLine()) != null; ) {
    total.append(line).append('\n');
}

Vous pouvez maintenant utiliser total sans le convertir en String mais si vous avez besoin du résultat sous forme de String il suffit d'ajouter :

String result = total.toString() ;

Je vais essayer de mieux l'expliquer...

• a += b (ou a = a + b ), où a y b sont des chaînes de caractères, copie le contenu de les deux a et b vers un nouvel objet (notez que vous copiez également l'objet a qui contient le accumulé String ), et vous effectuez ces copies à chaque itération.
• a.append(b) donde a est un StringBuilder , ajoute directement b contenu à a de sorte que vous ne copiez pas la chaîne accumulée à chaque itération.

Answer 2

Avez-vous essayé la méthode intégrée pour convertir un flux en chaîne ? Elle fait partie de la bibliothèque Apache Commons (org.apache.commons.io.IOUtils).

Votre code serait alors cette seule ligne :

String total = IOUtils.toString(inputStream);

Sa documentation peut être trouvée ici : http://commons.apache.org/io/api-1.4/org/apache/commons/io/IOUtils.html#toString%28java.io.InputStream%29

La bibliothèque Apache Commons IO peut être téléchargée ici : http://commons.apache.org/io/download_io.cgi

Answer 3

Je pense que c'est assez efficace... Pour obtenir un String à partir d'un InputStream, j'appellerais la méthode suivante :

public static String getStringFromInputStream(InputStream stream) throws IOException
{
    int n = 0;
    char[] buffer = new char[1024 * 4];
    InputStreamReader reader = new InputStreamReader(stream, "UTF8");
    StringWriter writer = new StringWriter();
    while (-1 != (n = reader.read(buffer))) writer.write(buffer, 0, n);
    return writer.toString();
}

J'utilise toujours l'UTF-8. Vous pourriez, bien sûr, définir le jeu de caractères comme un argument, en plus de InputStream.

Answer 4

Qu'en est-il de ceci. Il semble donner de meilleures performances.

byte[] bytes = new byte[1000];

StringBuilder x = new StringBuilder();

int numRead = 0;
while ((numRead = is.read(bytes)) >= 0) {
    x.append(new String(bytes, 0, numRead));
}

Edit : En fait, cela englobe à la fois Steelbytes et Maurice Perry.

Answer 5

Peut-être qu'au lieu de lire "une ligne à la fois" et de joindre les chaînes de caractères, essayez de "lire tout ce qui est disponible" afin d'éviter le balayage pour la fin de ligne, et d'éviter également les jonctions de chaînes de caractères.

ie, InputStream.available() y InputStream.read(byte[] b), int offset, int length)