Un modo più elegante per usare recv() e vettoriale <unsigned char>

Question

Finora, ho questo codice di esempio:

... 
int nbytes =0; 
vector<unsigned char> buffer; 
buffer.resize(5000); 
nbytes = recv(socket, &buffer[0], buffer.size(),0); 
//since I want to use buffer.size() to know data length in buffer I do 
... 
buffer.resize(nbytes); 

E 'un po' un altro modo, di conoscere la lunghezza dei dati nel buffer senza utilizzare ridimensionare () due volte? Perché non è possibile ricevere dati nel vettore che non viene ridimensionato alla dimensione corretta. Penso che il metodo reserve() non faccia l'allocazione, secondo la documentazione di C++ STL. E un'altra domanda: usare questo tipo di tecnica è una perdita di memoria sicura?

Answer 1

Questa tecnica è a prova di perdite, abbastanza pulita e preferibile. L'utilizzo di std::vector è il modo consigliato di implementare un buffer a lunghezza variabile in C++.

Se non tutti i dati si inseriscono nel vettore, nessun problema, ridimensionalo a dimensioni maggiori e passa l'indirizzo della sezione che segue la parte già occupata.

L'utilizzo di reserve() non è una buona idea - non influisce sui ritorni di , quindi perdi praticità e probabilmente non otterrai alcun vantaggio.

Answer 2

Questo codice va bene. La differenza tra resize e reserve è che resize modifica il valore restituito da size (e crea effettivamente nuovi oggetti inizializzati di default), mentre lo reserve non lo fa (assegna solo più memoria).

A seconda di come si elaborano i dati, è possibile lasciare il secondo resize fuori, e lo fa con un ciclo come questo:

for (vector<unsigned char>::iterator it = buffer.begin(); 
    it != buffer.begin() + nbytes; 
    it++) 
{ 
    // process each byte 
} 

Così si può solo leggere i dati che è stato effettivamente scritto, e ignorare il riposo. Ciò significa che devi solo impostare la dimensione del vettore una volta, e poi non cambiarla mai. In generale, finché lavori con gli iteratori, non è necessario ridimensionare il vettore, poiché l'intervallo di dati valido sarà sempre [buffer.begin(), buffer.begin() + nbytes).

Answer 3

Non ci credo [un modo più elegante?]. Fondamentalmente, è necessario disporre di più di un numero sufficiente di caratteri nel buffer per recv molti byte; quindi, una volta letti, se si desidera che il buffer contenga solo i byte ricevuti, è necessario ridimensionarli verso il basso. Quello che hai mostrato è probabilmente simile a come mi avvicinerei alle cose.

Sei corretto che reserve non è sufficiente. Non è possibile scrivere su elementi che non esistono e sono stati assegnati in anticipo solo agli archivi.

Answer 4

Non c'è molto che tu possa fare, non puoi conoscere la dimensione del post prima della chiamata recv.

po 'di pulizia:

std::vector<unsigned char> buffer(5000); 
int result = recv(socket, buffer.data(), buffer.size(), 0); 
if (result != -1) { 
    buffer.resize(result); 
} else { 
    // Handle error 
} 

Answer 5

C'è un modo più elegante quando si utilizza il protocollo TCP: utilizzare un'intestazione del messaggio.
Si conosce la dimensione (fissa) dell'intestazione, quindi è possibile leggerlo in un buffer di dimensioni fisse.
All'interno dell'intestazione, esiste una dimensione del messaggio, quindi è possibile utilizzarlo per allocare il buffer e leggere il numero di byte specificato nell'intestazione.

Attenzione che usando questo, potresti aver bisogno di alcune tecniche per i casi secolari come la frammentazione per i messaggi lunghi.