Problème pour rechercher/convertir string vers int

Bonjour,

J'aimerai réaliser une fonction qui vérifie une date saisie en string, pour cela je récupère les données avec substr et les transforme en int avec atoi().
Voici ma fonction modifiée pour vérifier les données:

Code:

---

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

cout<<"Date de fin (ex: 01/01/2009) : "; getline(cin,dateFin); // Récupère la date saisie   bool dateValide(string dateFin) { char *j,*m,*a; j = (char*)dateFin.substr(0,2).c_str(); m = (char*)dateFin.substr(3,2).c_str(); a = (char*)dateFin.substr(6,4).c_str();   cout<<atoi(j)<<" "<<atoi(m)<<" "<<atoi(a)<<endl; }

---

Pour l'instant je récupère le mois et l'année (m,a) mais la valeur de jour est la même que m.
Cela semble concerner uniquement les premiers caractères de dateFin, car si je mets substr(6,4) pour j, j'obtiens l'annee.
J'ai essayé en assignant directement "01/02/2009" à dateFin sans passer par la saisie et j'ai la même erreur...

Qu'en pensez-vous? d'où peut venir le problème?

Bonjour,
Examinons la ligne suivante :

Code:

---

j = (char*)dateFin.substr(0,2).c_str();

---

Elle se décompose en :
1/ Appel de substr -> construction d'un std::string temporaire , appelons sub1, résultat de la fonction
2/ Appel de c_str sur sub1
3/ destruction de la std::string temporaire sub1.
-> le pointeur c_str ne désigne plus quelque chose de valide !
En gardant ton principe, ce doit être :

Code:

---

1 2 3 4

std::string str_jour(dateFin.substr(0,2)); const char *j = str_jour.c_str(); //...

---

Quelques remarques sur ton code :
1/ Ne fais pas de cast sauvage ( j = (char*)...), mais utilises le type approprié : const char *j;
2/ Le C++ t'offre des moyens de faire des conversions avec des solutions C++ : cf FAQ.

Merci pour ta réponse :)
En effet ça fonctionne mieux sans les cast sauvages ^^
Mais j'ai finalement utilisé istringstream comme tu me l'a conseillé et ça fonctionne bien.

Code:

---

1 2 3 4 5 6

int j,m,a; string dateFin;   getline(cin,dateFin); // Récupère la date saisie 'jj mm aaaa' istringstream iss(dateFin); // Créé un flux à partir de la chaine à convertir iss>>j>>m>>a; // Récupère le jour, le mois et l'année en int

---

J'espère l'avoir bien utilisé, en tout cas ça fonctionne, je mets en résolu.
Merci encore :ccool:

Citation:

---

Envoyé par umeboshi

Merci pour ta réponse :)
En effet ça fonctionne mieux sans les cast sauvages ^^
Mais j'ai finalement utilisé istringstream comme tu me l'a conseillé et ça fonctionne bien.

Code:

---

1 2 3 4 5 6

int j,m,a; string dateFin;   getline(cin,dateFin); // Récupère la date saisie 'jj mm aaaa' istringstream iss(dateFin); // Créé un flux à partir de la chaine à convertir iss>>j>>m>>a; // Récupère le jour, le mois et l'année en int

---

J'espère l'avoir bien utilisé, en tout cas ça fonctionne, je mets en résolu.
Merci encore :ccool:

---

Tu reste tributaire d'un format de chaîne assez fixe. Tu peux regarder soit du côté des expressions régulières soit du côté de Boost.Date_Time pour une approche moins maison :mrgreen: