[C++] Packets

[.Infinite]

Ich versuche gerade einen simplen Packetbot zu schreiben... Meine Funktion fürs Empfangen sieht momentan so aus:

void packets::Receive(stringstream &response)
{
	char ReceivedByte;
	while(recv(Socket, &ReceivedByte, 1, 0) > 0)
	{
		response << ReceivedByte;
	}
};

Nur leider dauert das Empfangen der Antwort auf eine simple GET Anfrage ca. 15 Sek. Lässt sich das irgendwie effizienter machen?

[ms​]

Mehr als nur 1 Byte pro recv-Call holen.

[.Infinite]

Aber wenn ich mir jetzt zb. immer 5 bytes hole, und von der Antwort fehlen nur noch 4 bytes. Dann wirft recv() einen Fehler und mir fehlen die letzten 4 byte oder?

[ms​]

Der zurückgegebene Wert gibt an, wie viele Bytes angekommen sind. Das 3. Argument gibt nur an, wie viele Bytes maximal geholt werden sollen.

[.Infinite]

Oh, da hab ich mich wohl vertan... Danke!

[MrSm!th]

Du solltest aber den Fall bearbeiten, dass 0 Bytes empfangen wurden. Ein Rückgabewert von 0 bedeutet, dass die Verbindung geschlossen wurde.

[.Infinite]

Der Fall wird doch mit der Schleife behandelt. Der Wert kann doch eig. nur gleich Null sein, wenn alle Daten übertragen würden und die Verbindung geschlossen wurde oder größer als 0(bzw. hier 1), wenn er noch am empfangen ist...

€: Habs jetzt mit mehreren Bytes(255 ist meine ich maximum) gemacht, bringt aber nichts. Ist genauso langsam wie vorher... Aber bei kurzen Antworten ist es unter einer Sekunde. Nur wenn ich ne komplette Seite Anfrage dauert es lange.

[MrSm!th]

Dann mach halt bei ner kompletten Seite noch größere Abfragen.
Da kannst du ruhig ein ganzes KB abfragen bei Seiten mit 80+ KB.

Die Verbindung kann auch durch einen Fehler getrennt werden und dann solltest du richtig reagieren und nicht einfach das ganze als vollständige Antwort betrachten ;f

[.Infinite]

Ich machs jetzt so:

string packets::Receive()
{
	string response;
	char ReceivedBytes[1024];
	while(recv(Socket, ReceivedBytes, sizeof(ReceivedBytes), 0) > 0)
	{
		response.append(ReceivedBytes);
	}
	return response;
};

Allerdings dauert es immer noch ewig...



Komisch ist jedoch, dass erst teilweise das Paket und dann nur noch zufällige Zeichenfolgen ankommen. Außerdem gibt die Konsole Piptöne aus. Das war vorher nicht...

€: Array nicht nullterminiert?

[MrSm!th]

Du solltest es natürlich nicht einfach ausgeben.

Ein Http Packet hört mit einer Leerzeile auf, ob danach noch ein \0 Byte folgt, weiß ich nicht.

Häng halt sicherheitshalber immer eins dran.

Wie groß ist denn die Seite, die du lädst?
Du kannst es btw auch mit der WinInet Api machen.

[.Infinite]

Warum nicht ausgeben?

Die Seite ist ca. 80kb groß.

[MrSm!th]

Dann liegt es wohl einfach daran, dass du C++ nutzt und der String immer wieder verlängert werden muss.

Sicher, dass es 80kb ohne Bilder etc. sind?

[.Infinite]

Jep. 84kb. Die Startseite von e*pvp...

Es liegt anscheinend tatsächlich daran, dass der String immer wieder verlängert werden muss. Wenn ich es sofort ausgebe klappts super schnell...
Gibt es also eine bessere Möglichkeit dafür? Ich kann ja schlecht ein 10MB großes char Array erstellen, in das die empfagenen Bytes verschoben werden.


€: dass nur Müll ankam lag an meinem Packet und daran, dass immer 1kb in response verschoben wurde, auch wenn weniger ankommt...

Jetzt sieht es also so aus:

string packets::Receive()
{
	string response;
	char ReceivedBytes[1024];
	int BytesReceived = 1;
	while(BytesReceived > 0)
	{
		BytesReceived = recv(Socket, ReceivedBytes, sizeof(ReceivedBytes), 0);
		if (BytesReceived < sizeof(ReceivedBytes))
		{
			response.append(ReceivedBytes, BytesReceived);
		}
		else
		{
			response.append(ReceivedBytes);
		}
	}
	return response;
};

[MrSm!th]

Die Startseite von Epvp enthält auch Grafiken.

Ein Browser cached die Daten in nem File.
Das geht vermutlich schneller, als jedes mal den Speicher zu relozieren.
Am Ende kannst du ja alles wieder aus dem File in einen String laden, dann musst du nur einmal den Speicher allozieren.

[.Infinite]

Ähh. Normalerweise ist es doch so, dass im Quelltext der Seite nur die Links zu den Bildern sind und nicht die Bilder selber.
Der Browser fragt dann jedes Bild seperat vom Server an. Aber ich brauche das ja nicht zu machen. Mich interessiert ja nur der Text...

Das mit dem Cachen ist ne gute Idee. Bin gerade dran, habe aber bis jetzt keine große Verbesserung erreicht, das Zeitraubende ist dann wieder, die Datei auszulesen (geht das auch nicht Zeilenweise?).