[C++/CURL] HttpDownloader

[Schlüsselbein]

Gleichmal vorweg: Die Klasse ist unvollständig, teilweise vielleicht auch buggy, erst frisch fertig geworden und äußerst häßlich ohne großartige Strukturierung oder Kommentare in eine Datei gehakt.
Dennoch möchte ich sie mit euch teilen.

Beschreibung: Ich muss für ein Projekt Dateien via Http (später vielleicht auch noch FTP, weswegen es vielleicht noch eine Abstrakte Downloaderklasse geben wird) laden. Dazu benutze ich CURL, dessen typisches C-Interface halt nicht wirklich gut in die C++ Welt passt. Aus diesem Grund ist ne kleine HttpDownload-Klasse erstanden, welche die C-Callbacks in Einklang mit der C++ Welt bringt. Sowohl Userdaten, als auch Callbacks hab ich versucht, typsicher zu machen.

Hier jedenfalls mal die (unvollständige!) Klasse:


Verwendung:
Die Klasse benötigt als template-Parameter die Typen für die Userdaten von write-callback und progress-callback (beide standardmäßig void).
Die Signatur für die Callbacks seht ihr im angeführten Beispiel. Der erste Parameter vom Typ bool& besimmt, ob die Callbacks für dieses Event weiter abgearbeitet werden sollen (standardmäßig true). Aus dem write-callback einfach ein false zurückgeben um den Download zu beenden (ist noch als quick&dirty-Lösung implementiert).

#include <iostream>
#include "HttpDownloader.h"

int main()
{
	int totalSize = 0;
	int counter = 0;

	HttpDownloader<int, void> dl("http://www.elitepvpers.com/forum/images/bullet/misc/logostrip_l.gif", &counter);

	auto f = [&totalSize](bool&, size_t size, char*, int* user) { 
		totalSize += size;
		++(*user);
		return true;
	};

	auto pf = [](bool&, double now, double total, void*) {
		std::cout << "received " << now << " of " << total << " bytes\n";
	};

	dl.add_data_callback(f);
	dl.add_progress_callback(pf);

	dl.start();
	std::cout << "received " << totalSize << " bytes\n";
	std::cout << "data callback called " << counter << " times\n";
}

Wie gesagt, wirds noch ne viele, viele Änderungen geben - ggf steig ich auch komplett auf boost::signal um.


Vielleicht kann ja doch jemand was damit anfangen,




Gruß

[Delinquenz]

Dazu benutze ich CURL, dessen typisches C-Interface halt nicht wirklich gut in die C++ Welt passt.

Wieso nicht cURLpp? Die VS-Projektmappe ist zwar recht unaufgeräumt, aber insgesamt ein recht ordentlicher C++ Wrapper.

Naja, vielleicht wäre das auch ein bisschen Overload für deinen Verwendungszweck - nur ein Vorschlag^^

[Schlüsselbein]

Wieso nicht cURLpp? Die VS-Projektmappe ist zwar recht unaufgeräumt, aber insgesamt ein recht ordentlicher C++ Wrapper.

Weil ich mich einfmal damit rumgeschlagen hab und es mir einfach sowas von garnicht gefallen hat. Ich brauch ausserdem keine komplexen Requests, weswegen mir n dünner Wrapper überm C-Interface reicht.

Trotzdem nichts gegen die Leute, denen cURLpp gefällt.


Edit: Habs mir das Projekt nach deinem Post nochmal angeschaut und es scheint tot zu sein. Die Versionen, die ich finde, sind 4-5 Jahre alt.

[Delinquenz]

Edit: Habs mir das Projekt nach deinem Post nochmal angeschaut und es scheint tot zu sein. Die Versionen, die ich finde, sind 4-5 Jahre alt.

Habe mich vor kurzem reingearbeitet und auch wenn es recht alt ist, ist es immer noch wesentlich angenehmer als mit der libcurl zu arbeiten. Jo, die letzte Version ist anscheinend 3,5 Jahre alt, jedoch funktioniert sie (bis jetzt) super mit der neusten libcurl Version 7.29.0.

Aber wie gesagt, für deinen Zweck reicht denke ich deine Klasse vollkommen - wollte nur drauf hinweisen.

[Schlüsselbein]

Habe mich vor kurzem reingearbeitet und auch wenn es recht alt ist, ist es immer noch wesentlich angenehmer als mit der libcurl zu arbeiten

Kann ich so nicht unterstreichen. Bin auch der Meinung, dass C in nem C++-Source so gut wie nie verwendet werden soll, aber libcurl bietet ein äußerst angenehmes Interface. Ich (und auch viele andere) haben sich mit curlpp vom Kompilieren bis zum Benutzen rumgeschlagen. Ich will nicht bestreiten, dass curlpp nicht auch gut sein kann - bestimmt ist es dank RAII, Operator-Overloading und andere C++-Schmankerl sogar auch sehr gut zu benutzen.

Dennoch würde ich von einem Projekt, dass Tod ist (wenn immer möglich), die Finger lassen. Support gibt es dann keinen mehr und auch schwerwiegende Bugs können sich noch irgendwo verstecken (fällt das dann zu spät oder/und beim Kunden auf, kann das Geschrei groß werden).

Auch ist hier ja nicht garantiert, dass alle Features jetzt oder in Zukunft funktionieren, da es ja keinen Support mehr gibt.

Aber wie gesagt, für deinen Zweck reicht denke ich deine Klasse vollkommen - wollte nur drauf hinweisen.

Danke dir dafür (wirklich), ich hatte das Projekt schon vergessen (verdrängt?).

Ne richtig gut gelungene Netzwerk-Lib (die auf modernes C++ setzt) ist neben boost::asio auch (wirklich einen Blick wert!).




Gruß

[Delinquenz]

Ne richtig gut gelungene Netzwerk-Lib (die auf modernes C++ setzt) ist neben boost::asio auch netlib (wirklich einen Blick wert!).

Ich belebe diesen Thread mal wieder, da ich mir wirklich Mühe gegeben habe, die cpp-netlib zu benutzen. Leider benötigt aktuelle Stable Version die statischen Boost-Bibliotheken und ist voll mit Bugs wie hier, die immer noch nicht in der Stable Version gefixt wurden. Die Git Version buggt auch noch an etlichen Stellen (bestimmte Bibliotheken wurden nicht mitgelinkt) und leider habe ich das nicht auf die Reihe bekommen, das Ganze zu kompilieren. Im Arch Repo gibts die Library auch nicht und das AUR liefert nur die Header Dateien.

Sorry, aber cURLPP konnte mir bisher einen wesentlich zuverlässligeren Dienst als cpp-netlib erweisen. boost::asio scheidet aus, da es sich dabei eher um eine Lowlevel-Bibliothek handelt als um eine HTTP-Bibliothek.