Discussion :

[atha2]

Bonjour,

Je viens de reprendre un projet et je suis tombé sur le code suivant :

Code :

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

1
2
3
4
5
6
7
8
 
int[] readBuffer = new int[100];
int len = 0;        
// we read each byte of the stream
while (this.inputStream.available() > 0 && (readBuffer[len] = this.inputStream.read()) != -1) 
{
     len++;
}

Il y a t'il une raison particulière de préciser en dure (100 plutôt que this.inputStream.available()) la taille du tableau ?
Surtout que dans la plupart des cas, il ne sera rempli que d'une 30aine d'éléments.
Pour info, l'InputStream est celui d'un socket.

[thelvin]

Déjà, ce n'est pas comme ça qu'on lit un InputStream, mais plutôt comme ça :

Code :

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

1
2
3
4
5
6
int nbRead = 0;
int thisRead = this.inputStream.read(readBuffer);
while(thisRead > 0) {
  nbRead += thisRead;
  thisRead = this.inputStream.read(readBuffer, nbRead, readBuffer.length - nbRead);
}

Ensuite pour répondre à ta question :

- available() ne sert à rien, et notamment pas à estimer la taille de tableau qui serait utile.
Il n'y a aucune raison, jamais, d'appeler available(). Créer cette méthode était une idée sans intérêt. Elle n'est là que pour compatibilité et elle n'est pas deprecated parce qu'elle n'est pas nuisible.

- Il reste donc à estimer la taille nécessaire du buffer d'une façon ou d'une autre. Pourquoi 100 ? Pourquoi pas.
Si la taille des données est plutôt de l'ordre de la trentaine d'octets, peut-être qu'un tableau de 50 suffirait. Ou peut-être pas. Il faut faire en fonction de ton cas.