Votre avis sur ma fonction : cpp_sscanf (plus rapide que sscanf)

Bonjour à tous,
La fonction sscanf du C n'étant pas sécurisée, et donc presque inutilisable, j'ai décidé de coder un équivalent [extensible] en c++11 moderne.


Prototypes.

Code:

---

1 2 3

(1) template <class... Outs> string::const_iterator cpp_sscanf::sscanf( string input, string pattern, char delimiter, Outs&... output_arguments );

---

Code:

---

1 2 3

(2) template <class Iterator, class... Outs> Iterator cpp_sscanf::sscanf( Iterator begin, Iterator end, string pattern, char delimiter, Outs&... output_arguments );

---

Fonctionnalités.

- La fonction sscanf parse une chaine (input) de caractère selon un format fourni (pattern) et en extrait des valeurs (output_arguments).
- Elle retourne un itérateur vers la position où le parsing s'est arrêté.
- Les types pris en charge pour la sortie sont extensibles, via des type_traits.
- Grâce au c++, cette version est plus rapide que celle du C.
- Plus rapide que les std::istream (qui sont abominablement lents), et surtout, plus lisible.
- Bonne gestion des erreurs grâce aux exceptions.
- Testée.

Mini exemple.

Code:

---

1 2	cpp_sscanf::sscanf( "j'ai 10 ans" , "j'ai _ ans" , '_' , mon_age );

---

Exemple.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#include "cpp_sscanf.h" #include "parser_trait.h" #include <string> using namespace std;   int main(){ string chaine_a_tester = ...; string pattern = "Julien : _ ans, Diane : _ ans"; double age_julien; double age_diane;   cpp_sscanf::sscanf( chaine_a_tester , pattern , '_' , age_julien, age_diane );   cout << "l'age de Julien est : " << age_julien << endl; cout << "l'age de Diane est : " << age_diane << endl; }

---

Sources.

Les sources, les tests et le readme sont là :
https://bitbucket.org/Gauss_/library...anf/?at=master

Votre avis.

Qu'en pensez-vous ? Peut-on améliorer le code ? Cela vous semble-t-il utile ?

Salut,

ça a l'air sympa, mais ça ne permet que de scanner 1 variable dans une chaîne ?
Personnellement je n'y vois pas d'intérêt par rapport à l'utilisation d'une regexp, que je connais et est plus "paramétrable"/"utilisable".
En tous cas le code est plutôt agréable à lire. ;)

Bonjour,
Pas qu'une seule variable heureusement !

Cette fonction permet de lire autant de variables que nécessaire dans la string (input) fournie, tout en vérifiant que le format est valide, selon le modèle défini par le pattern.

Les regex sont en effet le candidat de choix en c++ :roll:. Personnellement, je les trouve assez fastidieuses à utiliser, et moins lisibles qu'un appel à sscanf. Voici un exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

// Avec Regex : // string input = ...; string decimal = R"(-?\d*\.?\d+(?:e[+-]?\d+)?)"; string entier = R"(-?\d+)";   regex pattern{ "commande : " +entier+ " paquets de " +decimal+ "g\n" };   smatch sm; regex_match(input,sm,pattern);   if(sm.size() != ...){ //erreur de parsing }   int quantite; double grammes; //recupérer les valeurs

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

// Avec sscanf : // string input = ...; string pattern = "commande : _ paquets de _g\n"; int quantite; double grammes;   try{ sscanf(input, pattern, quantite, grammes); } catch(CouldNotParseArgument& e){ //... }

---

Note :
J'ai édité les prototypes de mon premier message pour rajouter la ligne (que j'avais omise):

Code:

---

template <class... Outs>

---

Pourquoi tu fais des regexp aussi compliquées aussi ? regex_match(input, sm, regex("commande: (.*?) paquets de (.*?)g)); est largement suffisant et correspond à ton scanf.

Citation:

---

Envoyé par Bousk

Pourquoi tu fais des regexp aussi compliquées aussi ? regex_match(input, sm, regex("commande: (.*?) paquets de (.*?)g)); est largement suffisant et correspond à ton scanf.

---

Ah je ne suis pas un expert avec les regex...
Mais comment ensuite vérifier que l'entrée n'est pas invalide du type : input = "commande: 52abc paquets de 10.4g" ?
(Par exemple dans un test unitaire pour le format de sortie)

Code:

---

regex_match(input, sm, regex("commande: ([0-9]*) paquets de ([0-9]*)g));

---

permet de vérifier que tu n'entres que des nombres. Pour plus d'infos sur les regex : http://regexone.com/ par exemple