ZephLe 03/03/2009 à 19:06
Si je peux me permettre quelques remarques :
- Dommage de laisser apparaître dans ta lib la contrainte d'initialisation de cURL, ça aurait été plus élégant de la rendre transparente (automatique via un constructeur de ta classe "cURL" par exemple)
- Même remarque pour la désinitialisation, d'autant plus qu'il y a un risque d'oublier et de leaker, ça gagnerait à être automatisé dans un destructeur
- La méthode "init" ne fait pas très C++, pourquoi ne pas avoir mis ce code dans le constructeur de la classe ?
- De même, les fonctions internes utilisées pour le fonctionnement de ta classe (lock, unlock) pourraient devenir des méthodes statiques privées
- Il n'y a aucune méthode qui permette de passer des variables en POST dans la requête ? (la méthode "download" fait beaucoup de choses à la fois)
- Et plein de petits détails genre des "const" qui pourraient apparaître aux endroits judicieux, etc ^^