Problème avec getline

Voila, j'ai quelques problème avec getline.
Je lit ce fichier :

Citation:

---

[Melia/Lignes=5]
Pré sentation.
Sphere of Kseira, qu’est-ce que c’est ?
C’est un projet RPG 2D iso, débuté le 1 janvier 2006. Un projet assez simple,
même s'il va nous pendre du temps, beaucoup de temps.
Pas de date de prévu bien sur, vu que nous sommes tous amateur dans l'équipe.
[/Melia]
[Solas/Lignes=3]
- Et sa parle de quoi le jeu ? & Sur le monde de Martalya vivaient, il y a bien longtemps, les Anciens.
Ils ont bâtis toute la civilisation humaine, mais aussi ce que l'on nomme a présent "La barrière des Anciens".
Celui qui essaye de la traverser se fait immédiatement soustraire le Kseira de son corps.
[/solas]
[/END]

---

Et que je lance ce prog :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39

#include <iostream>   #include <string> #include <fstream>   using namespace std;   int main(int argc, char *argv[]) {   string buffer; int pos1 = 0, pos2 = 0; ifstream f("dialogues.dlog"); string avatar;     f >> buffer;   avatar = buffer.substr(buffer.find("[")+1, buffer.find("/")-1);   cout <<endl << avatar << " dit : " << buffer << endl; while (buffer != "[/" + avatar + "]") {   getline(f, buffer);   cout << buffer << endl;   f >> buffer; }       cout <<endl; system("PAUSE"); return 0;   }

---

Le programme oublie d'afficher a chaque fois le premier mot de chaque ligne !!!

Si vous pouviez m'aider

acieroid

Code:

---

1 2 3 4 5 6 7 8 9

while (buffer != "[/" + avatar + "]") {   getline(f, buffer);   cout << buffer << endl;   //f >> buffer; }

---

Tu lis une ligne puis tu la met dans buffer.
Ensuite tu lit le premier mot et tu le met dans buffer, c'est pour ça qu'à la ligne suivante il te manque le premier mot.

C'est bon, j'ai trouvé, merci, mais j'ai une erreur quand avec les pointeur string

Voila mon code entier :

Citation:

---

#include <string>
#include <fstream>
#include <iostream>
#include <stdlib.h>
#include <sstream>

using namespace std;

void lire(string *line);

int main(int argc, char *argv[])
{


string * line;
lire(line);

cout << line[0] ;


cout <<endl;
return 0;

}


void lire(string *line)
{
string buffer;
ifstream f("dialogues.dlog");
string avatar;
int n_lignes = 0;

while (getline(f, buffer))
{
avatar = buffer.substr(buffer.find("[")+1, buffer.find("/")-1); //On lite l'avatar

istringstream line_read(buffer.substr(buffer.find("Lignes=")+7, buffer.find("/")-1) ); //On lit le nb de lignes

line_read >> n_lignes;


line = new string[n_lignes];

//cout << avatar << " dit : " << endl;
for(int i = 0; (getline(f, buffer) && buffer != "[/" + avatar + "]"); i++)
{
line[i]= buffer;
//cout << "\t" <<line[i]<< endl;


}
}
}

---

Quand tu as un problème n'hésites pas à donner le(s) message(s) d'erreurs qui va/vont avec.

Tu a un problème avec les pointeurs, c'est évident. N'hésites pas à réviser un petit cours sur les pointeurs, ça fait toujours du bien au début... et même après :P


I. Allocation dynamique -> destruction de l'espace
- tu as un new ici:

Code:

---

line = new string[n_lignes];

---

-> tu dois donc avoir forcément un delete quelque part pour libérer la mémoire que tu as alloué dynamiquement.


II. Passage par pointeur dans une fonction:
En général quand on veut travailler sur un pointeur dans une fonction et pouvoir le récupérer en dehors de celle-ci, on utilise le retour de la fonction et non un paramètre.
Et ce n'est pas pour rien:

Code:

---

1 2	string * line; lire(line);

---

- tu crée un pointeur vers un string, il n'est pas initialisé, l'adresse vers laquelle il pointe peu être n'importe quoi.
- tu passes ce pointeur à une fonction.

Code:

---

void lire(string *line)

---

- La fonction crée un nouveau pointeur vers string et l'initialise avec le paramètre, c'est à dire dans ce cas n'importe quoi. Cette variable (line) est interne à la fonction.

Code:

---

line = new string[n_lignes];

---

- cette fois tu initialises la variable line avec une adresse correcte, car récupérée d'une allocation dynamique de mémoire.


PROBLEME: la variable line de main n'est pas modifiée. (en effet les deux line sont deux variables différentes qui contiennent une adresse (elles peuvent être vu comme des entiers).


SOLUTIONS:
- Tu utilises le retour de la fonction lecture et non un paramètre. (simple)
- Tu utilises un double pointeur de string pour pouvoir modifier le line du main. (moins simple ;))

Citation:

---

Envoyé par MatRem

SOLUTIONS:
[...]
- Tu utilises un double pointeur de string pour pouvoir modifier le line du main. (moins simple ;))

---

je n'ai fait que survoler le sujet mais face aux pointeurs :

Code:

---

std::vector< std::string >

---

STL
pas de new, pas de delete, pas de C :P

Sauf que là acieroid ne veux pas un tableau de chaine mais juste une chaine.

C'est vrai qu'il n'est même pas vraiment nécessaire d'utiliser les pointeurs, on pourrait retourner, directement une chaine, ou bien passer un paramètre par référence...

Y'a plein de solutions, mais je suis resté sur celle qu'il a commencé, c'est à dire les pointeurs, qu'il est bon de connaitre...:)

Oui mais manipuler un pointeur nécessite bcp de précautions. Tu fais bcp de new dans ton programme, mais aucun delete... En plus l'allocation dynamique est plus lente.
http://c.developpez.com/faq/cpp/?pag...ture_par_ligne