Vecteur avec element vide, fait planter l'execution

Version imprimable

05/12/2008, 20h52
neismarspra

Vecteur avec element vide, fait planter l'execution
bon soir,

Est-ce-que quelqu'un pourrait me confirmer que dans le cas suivant,

vector<string> st;

avec 3 element: "jedna","dve",""

si j'essay de lire la dernier valeur comme ceci:
Code:

1 2 vector<string>::iterator::iter = st.end(); string tmp = *st;
alors a la lecture j'ai une erreur??

Si c'est bien le cas, j'aimerai savoir comment detecter ce cas.

merci, et bonne soire
05/12/2008, 21h02
mayayu
Bonsoir,

Je ne vois pas trop ce que tu essaies de faire, surtout avec le "main" après la lecture de la variable, mais ça ne m'étonne pas que ça plante.

Essaie ça, ça devrait marcher mieux :
Code:

1 2 string tmp = *(st.end());

Ca m'étonne que ce code compile car il n'est aucunement valide.
Le code suivant compile:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #include <string> #include <iostream> #include <vector> int main(int argc, char const *argv[]) { std::vector<std::string> v; v.push_back("toto"); v.push_back("titi"); v.push_back("tata"); std::cout<<v[v.size()-1]<<std::endl; return 0; }
Edit:

Citation:

Bonsoir,

Je ne vois pas trop ce que tu essaies de faire, surtout avec le "main" après la lecture de la variable, mais ça ne m'étonne pas que ça plante.

Essaie ça, ça devrait marcher mieux :

Code:

string tmp = *(st.end());

Ceci est faux car end() ne pointe JAMAIS sur quelque chose appartenant au vector. Il ne FAUT PAS déférencer end(), sinon boom.

05/12/2008, 21h08
neismarspra

merci.

1] le main() c'est une erreur.

2) Ca compile tres bien..

je vais devlopper:
je lit un fichier, et la derniere ligne est defois vide, d'ou le dernier element
ne contenant rien.
Et c'est quand j'essay de lire la valeur contenu dans ce dernier element en
utilisant string = *iterateur
que L'EXECUTION plante

des idees??

Citation:
Envoyé par Davidbrcz
Ca m'étonne que ce code compile car il n'est aucunement valide.
Le code suivant compile:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #include <string> #include <iostream> #include <vector> int main(int argc, char const *argv[]) { std::vector<std::string> v; v.push_back("toto"); v.push_back("titi"); v.push_back("tata"); std::cout<<v[v.size()-1]<<std::endl; return 0; }
Edit:

Ceci est faux car end() ne pointe JAMAIS sur quelque chose appartenant au vector. Il ne FAUT PAS déférencer end(), sinon boom.
Méa culpa!
Code:

1 2 string tmp = *(st.end()-1);

05/12/2008, 21h38
neismarspra
oui, mais en faissan cela, je lit la valeur precedente, c'est ca??

et moi la valeur precedente je l'ai lue au tour precedent
(ehh, oui ca veut rien dire, mais on se comprend?)

moi j'ai une boucle:
Code:

1 2 3 4 5 6 getline(ifstream, string buffer) vectoeur.push_back(buffer); //des fonction qui initialise iter et fin for(iter;iter!=fin;iter++) { string temp = *iter; if ......
Il arrive que fin, pointe sur le dernier element du vecteur qui est parcouru par iter, ET il arrive que ce dernier element, soit egalement la derniere ligne vide d'un fichier:
Code:

1 2 premiere ligne\n seconde ligne\n
on a dans ce cas un saut a la ligne en fin du fichier, qui est considerai comme une ligne par getline

Et si ces deux condition son rempli, alors ca rame 4seconde et execution est interompu.
Je suis bien sur que le plantage survient a la ligne

Code:

temp = *iter;

.

hmm??
05/12/2008, 22h03
Davidbrcz

Montre un code complet minimal et compilable avec le fichier approprié qui reproduit le problème.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
ifstream fileI(chemin, ios_base::in);
vector<string> fichier;
getline(ifstream, string buffer) vectoeur.push_back(buffer);
fileI.close();
 
vector<string>::iterator iter=fichier.begin();
vector<string>::iterator fin=fichier.begin();
// fonction qui initialise iter et fin
secpos(string, fichier, iter, fin); //(string sec, vector<string> & fichier, iterator & iter, iterator & fin)
/* cf le fichier, apres l'apele de secpos
iter pointe sur le premier element
fin pointe sur la dernier ligne, qui ne contien rien
*/
 
for(iter;iter!=fin;iter++)
   {
   string temp = *iter;  //la ici plantage quand iter=fichier.end() + cf note
   if (temp.find("=",1)!=string::npos){
         ect

fichier:

Code:

1
2
premiere ligne\n
seconde ligne\n

NOTE
je ne suis pas obligatoirement en fin de fichier, ce n'est que dans ce cas que y'a des problemes. Donc je ne peut faire une verification du genre:

Code:

1
2
3
4
5
for(iter;iter!=fin;iter++)
   {
   if (iter==fichier.end()) //resoud pas le problem
   string temp = *iter;
   if (temp.find("=",1)!=string::npos){

ca va comme CMI???
J'espere que ca devien comprehensible

Je ne vois pas l'utilité d'utiliser "secpos". Je pense que tu peux simplifier ton code comme ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 ifstream fileI(chemin, ios_base::in); vector<string> fichier; getline(ifstream, string buffer) vectoeur.push_back(buffer); fileI.close(); vector<string>::iterator iter; for(iter=fichier.begin();iter!=fichier.end();iter++) { string temp = *iter; if (temp.find("=",1)!=string::npos){ ect
Par contre ça m'étonne que ton code compile, il manquerait pas un ";" après la parenthèse fermante suivant de getline?

ouaaaa :aie:

desoler c'est while(xx) fichier.push_backxxx.

Bon question desisif: quelqu'un a compris ce que je veus, car des le premier message
le probleme me semble exposer de facon assez simple et juste.>>>>
Code:

1 2 3 4 5 vector<string> fichier ["un","deux",""] iter = fichier.begin(); string tmp = *iter; iter++; tmp = *iter; iter++; tmp = *iter; // ici plantage du programme, comment detecter, ou eviter ce probleme

05/12/2008, 23h50
neismarspra

don a mon avis, vau mieux laisser tomber. Merci a tous.
je penserai plus tot au code complet minimal, la prochaine fois.

Bon neismarspra, ça devient agaçant.
Davidbrcz t'as demandé du code compilable ce n'est pas pour rien.

Si l'on corrige les erreurs de ton code, on a ceci, qui marche sans problème :
Code:

1 2 3 4 5 6 7 8 9 10 std::vector<std::string> fichier; fichier.push_back("un"); fichier.push_back("deux"); fichier.push_back(""); std::vector<std::string>::iterator iter; iter = fichier.begin(); std::string tmp = *iter; iter++; tmp = *iter; iter++; tmp = *iter;
Les concepteurs de la stl seraient quand même de sacrés charlots si l'on ne pouvait pas accéder à une string vide dans un vecteur. :aie:

Donc il doit y avoir une erreur ailleurs dans ton code, mais tant que tu ne nous le montre pas, on ne peut pas deviner.

Edit : Je pense avoir compris ton problème. :yaisse2:
C'est une erreur classique, on se fait tous avoir au début. Tu penses que fichier.end() te renvoie un itérateur sur le dernier élément du vecteur. Et ben non ! En fait fichier.end() pointe juste après le dernier élément. C'est une sorte de marqueur pour indiquer la toute fin du vecteur en mémoire. Le dernier élément se trouve donc bien à (fichier.end()-1). C'est une petite différence avec fichier.begin() qui lui pointe directement sur le premier élément.

En résumé on a ça:
Code:

1 2 3 4 5 vecteur : | "un" | "deux" | "" | ^ ^ | | itérateur: begin() end()
Le plus simple reste tout de même de parcourir ton vecteur avec une boucle for, comme le montre emiaille ci-dessous. :king:

Je suis désolé mais ton code est encore faux, la déclaration de ton vector ne marche pas...

Pourquoi tu ne veux pas utiliser une boucle pour parcourir ton vecteur? Sinon comment tu veux faire pour détecter la fin (liée à la taille de ton vector)?

Essayes ce code , ça marche très bien :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 #include <vector> #include <string> using namespace std; int main() { vector<string> fichier; vector<string>::iterator iter; string tmp; fichier.push_back("un"); fichier.push_back("deux"); fichier.push_back(""); for(iter = fichier.begin(); iter!=fichier.end(); iter++) { tmp = *iter; cout << tmp << endl; } }

06/12/2008, 04h00
neismarspra

je doit tous dabord des excuses a vous tous (en dehors du code compilable: cf plus loin).

bon, dabord emiaille:
ben, faut dire qu'avec mes autre message, j'esperer que vous allez comprendre que je declare pas mon vector avec [ xx,xx,xx], mdr
mais que ca representer les 3 ligne lu dans le fichier.
Et je pense l'avoir dit,j'ai pas eu de probleme de compilation

Citation:

Pourquoi tu ne veux pas utiliser une boucle pour parcourir ton vecteur? Sinon comment tu veux faire pour détecter la fin (liée à la taille de ton vector)?

je retraite le probleme du debut (cf plus loin + je sais pas trop mais j'aime pas les break dans mes boucle for. Je les trouvent mieux a leur place dans une while).
Ca m'a paru plus clair apres 2h30 de transport en commun en rentrant du boulot.

puis Arzar,
ben merci encore, surtout pour la pressision du .end()

ET conclusion:
n'essayer pas de corriger un bug dans du code c++, quand vous avez 5ecran a surveiller au travail, et ne chercher pas toujours l'erreur la ou elle pretend etre, mais plutôt aux endroit ou le code est quasi indéchiffrable. (la fonction secpos) oui je sais je l'ai pas fourni.
06/12/2008, 10h00
mayayu

Lorsque l'énoncé d'un problème est exactement connu, le problème est résolu; ou bien c'est qu'il est impossible. La solution n'est donc autre chose que le problème bien éclairé.

Emile-Auguste Chartier