Discussion :

[umeboshi]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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?

[3DArchi]

Bonjour,
Examinons la ligne suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[3DArchi]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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