InputStream.read (byte [], lunghezza 0) si arresta in anticipo?

Question

Ho scritto qualcosa per leggere un flusso di richiesta (contenente dati gzip) da una HttpServletRequest in arrivo ('richiesta' di seguito), tuttavia sembra che il normale metodo di lettura di InputStream non legga effettivamente tutto il contenuto?

Il mio codice è:

InputStream requestStream = request.getInputStream(); 
if ((length = request.getContentLength()) != -1) 
{ 
    received = new byte[length]; 
    requestStream.read(received, 0, length); 
} 
else 
{ 
    // create a variable length list of bytes 
    List<Byte> bytes = new ArrayList<Byte>(); 

    boolean endLoop = false; 
    while (!endLoop) 
    { 
     // try and read the next value from the stream.. if not -1, add it to the list as a byte. if 
     // it is, we've reached the end. 
     int currentByte = requestStream.read(); 
     if (currentByte != -1) 
      bytes.add((byte) currentByte); 
     else 
      endLoop = true; 
    } 
    // initialize the final byte[] to the right length and add each byte into it in the right order. 
    received = new byte[bytes.size()]; 
    for (int i = 0; i < bytes.size(); i++) 
    { 
     received[i] = bytes.get(i); 
    } 
} 

Quello che ho trovato durante i test è che a volte la parte superiore (per quando una lunghezza del contenuto è presente) sarebbe solo smettere di leggere in parte attraverso il flusso di richiesta in ingresso e lasciare il resto della matrice di byte 'ricevuti' vuota. Se faccio in modo che esegua sempre la parte else dell'istruzione if, legge bene e tutti i byte previsti vengono inseriti in 'received'.

Quindi, sembra che ora possa lasciare il mio codice solo con quel cambiamento, ma qualcuno ha idea del perché il normale metodo "read" (byte [], int, int) "ha smesso di leggere? La descrizione dice che potrebbe fermarsi se è presente una fine del file. Potrebbe essere che i dati gzippati capita di includere dei byte che corrispondono a qualsiasi firma?

Answer 1

È necessario aggiungere un ciclo while nella parte superiore per ottenere tutti i byte. Il flusso tenterà di leggere tanti byte come può, ma non è richiesto di tornare len byte in una volta:

viene effettuato un tentativo di leggere quanti byte len, ma un numero inferiore può essere letto , possibilmente zero.

if ((length = request.getContentLength()) != -1) 
{ 
    received = new byte[length]; 
    int pos = 0; 
    do { 
     int read = requestStream.read(received, pos, length-pos); 

     // check for end of file or error 
     if (read == -1) { 
      break; 
     } else { 
      pos += read; 
     } 
    } while (pos < length); 
} 

EDIT: mentre fissa.

Answer 2

È necessario verificare la quantità di buffer riempita. È garantito solo per darti almeno un byte.

Forse quello che volevi era DataInputStream.readFully();