Anhängen an Puffer in C++

Question

Ich versuche, den Inhalt einer Karte effizient in eine Zeichenfolge zu konvertieren, um über Socket zu senden. Ich habe dies so weit ...

char buffer[1024]; 
for (iter = my_mapy.begin(); iter != my_map.end();iter++) 
{ 
    sprintf(buffer, "%s|%ld|%d", buffer, iter->first, iter->second); 
} 

Während dies funktioniert, ich habe mich gefragt, ob es ineffizient ist. Google sucht nach dem effizientesten Weg, um int/long/doubles in string zu konvertieren, was in sprintf resultiert, weshalb ich es verwende. Aber ich bin besorgt, dass der Inhalt des Puffers immer wieder kopiert wird, während ich nur bis zum Ende anhängen möchte. Ist das richtig, und wenn ja, gibt es einen besseren Weg, dies zu tun? Leistung und Geschwindigkeit haben Priorität 1.

Danke!

Answer 1

Sie haben Recht; Die hier vorgeschlagene Lösung kopiert den Puffer jedes Mal. Um es besser zu machen, müssen Sie den Rückgabewert von sprintf verwenden.

char buffer[1024]; 
char* end_of_buffer = buffer; 
std::size_t remaining_space = sizeof(buffer); 

for (auto iter = my_map.begin(); iter != my_map.end(); iter++) 
{ 
    int written_bytes = snprintf(end_of_buffer, remaining_space, "|%ld|%d", iter->first, iter->second); 

    if (written_bytes > 0) { 
     end_of_buffer += written_bytes; 
     remaining_space -= written_bytes; 
    } else { 
     perror("Something is wrong with the buffer"); 
    } 
} 

Hinweis, nebenbei gesagt, dass ich snprintf verwendet, die Spur des Puffers des Restlänge hält. Sie sollten immer anstelle der unsicheren Version verwenden. Ihre Anwendung wird sich weiterentwickeln, und durch große Kreativität finden Sie einen Weg, diesen Puffer überlaufen. Inzwischen kommt die Sicherheit ohne zusätzliche Kosten.

(Ich meine nichts für ungut, natürlich.)

Answer 2

Sie std::ostringstream s verwenden sollten, sind sie effizienter und C++ - ish:

#include <sstream> 

std::ostringstream oss; 
for (iter = my_mapy.begin(); iter != my_map.end();iter++) 
{ 
    oss << iter->first << "|" << iter->second; 
    //oss.str() returns the string in which everything was stored. 
} 

Danach werden Sie noch operator<< Sachen am Ende des ostringstream anhängen können.

Answer 3

Sie sollten wahrscheinlich stattdessen stringstream verwenden, aber wenn Sie mehr Effizienz wollen, dann nehme ich an, Sie können gerade char * für Ihren Puffer verwenden, müssen Sie nur sicherstellen, dass Sie es wachsen, es sei denn Sie wissen, es wird nicht über 1024 Bytes.

Answer 4

Sie haben den Puffer an sprintf als Eingabe und Ausgabe übergeben. Das ist undefiniertes Verhalten.