Discussion :

[porcher]

Bonsoir a tous
Voila j'ai un fichier texte sur lequel j'effectue des traitements.
Une partie de mon programme pose probleme.
En fait la fonction affiche() ne peut s'effectuer 2 fois de suite correctement.La seconde n'affiche rien ce qui est normal.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
#include<fstream>
using namespace std;
 
fstream f ("personnel.txt",f.in | f.out );
void affiche(){
      char *courant;
      while(f.getline(courant,100)) {
                 cout<<courant<< "  nbre de caract. lus: ";
                 int n=f.gcount();cout<<n<<endl;
      }
          // ici un 'f.clear();' fait buguer le programme en 
          // annonçant que l'exe doit fermer!!!  
      cout<<"etat: "<<f.rdstate()<<endl;// affiche 6 donc failbit 
                                        // et eofbit activés ce qui est normal
      f.seekg(0);cout<<"position pointeur: "<<f.tellg()<<endl;// affiche -1 
}
 
main() {
    affiche();  //ok pas de probleme
    affiche();  //probleme évidemment car good()==false donc pas d'entree ds le while
    f.close();
}

Est t'il possible de remédier à ce probleme sans avoir à fermer le fichier et a le rouvrir pour que cela marche?
Comment réactiver goodbit a la fin de affiche() etant donné que je ne peux pas compter sur l'instruction clear() pour le faire???!!
D'avance merci

[Luc Hermitte]

Il faut bien un clear pour pouvoir remettre à zéro les bits d'état du flux. Et ce n'est pas lui qui fait bugguer le programme mais ta mauvaise utilisation de la mémoire.

Je sais que tu insistes pour utiliser les char *, mais c'est une erreur. Non seulement le code est moins souple (Dans la FAQ de Bjarne Stroustrup tu trouveras un article sur le pourquoi le C++ doit être enseigné comme un langage à part entière, regarde bien les deux codes présentés en parallèle), mais en plus il est plus facile de commettre des erreurs.

De même que dans tes fonctions de convertions, tu n'alloues pas correctement la mémoire que doit utiliser ton programme, et tu ne la libères pas non plus.

[porcher]

ok Merci... j'ai donc du taf en perspective....
Si je m'entete a utiliser les char *, c 'est que j'ai pas encore étudié la classe string.
Toujours est t'il que je ne vois pas ce qui fait buguer mon programme
Faut que j'alloue de la mémoire dynamiquement ?
Et puis pourquoi deleter dans le cas de petites applications puisque le compilateur le fait à la fin du programme?

[porcher]

Encore merci monsieur Hermitte,c'etait bien un probleme d'alloc

[Luc Hermitte]

Présenter les char * avant les std::string est un boulet historique que se coltinent certains livres et cours sur le C++. Ce n'est ni ce qu'il y a de plus simple à utiliser, ni ce qui va vraiment servir en industrie -- les classes de gestion des chaines de caractères étant fortement recommandées.
Bref.

Sinon oui, il faut TOUJOURS allouer la mémoire dans laquelle on veut stocker des choses.
Dans un cas pareil, savoir si il faut ou non se souvenir des diverses lignes peut influencer sur le fait que l'on aurait utilisé un tableau statique de caractères plutôt qu'un tableau dynamique (-ment alloué).
Mais ces approches sont super limitaives (lignes à dimensions fixes si on veut le code simple à maintenir) et rapidement fausses (vu qu'en C++ new[] doit lever une exception s'il ne peut allouer de mémoire ; ce qui devient rapidement inmaintenable quand on n'enveloppe pas nos resources).

Pour la libération, petit programme ou pas, c'est une bonne habitude à prendre que de libérer les ressources que l'on alloue. Pourquoi apprendre à pisser des lignes fausses quand on peut apprendre à pisser des lignes justes et plus simples ?
(Et ce n'est pas le compilo, mais seulement certains OS qui récupèrent la mémoire (et je crois bien uniquement cette ressource ci))

[porcher]

ok merci de ces explications.
En effet je potasse le bouquin de Dellanoy.
(Qui n'est pas terrible d'ailleurs!Maintes sujets dont celui de l'allocation dynamique sont succintement abordés.De plus il faut connaitre le C avant de le lire et Monsieur Delannoy se garde bien de le dire dans son intro!!
Je ne connais pas le C j'ai juste fait de l'ADA.)
Merci pour ces considerations.
Mais alors

Sinon oui, il faut TOUJOURS allouer la mémoire dans laquelle on veut stocker des choses.

Si je déclare

Code :

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

int j;

il faut que l'alloue de la memoire par

Code :

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

j=new int;

Ou c'est fait automatiquement ???[/code][/quote]

[porcher]

pardon je délire

int j;j=new int;

C'est

Code :

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

int j;int *p=&j;

La memoire pour p est allouée.Basta.Je vais me coucher....

[moldavi]

Si je déclare
Code:
int j;

il faut que l'alloue de la memoire par
Code:
j=new int;

Ou c'est fait automatiquement ???

Oui c'est fait automatiquement.

Un "int" a une taille fixe que tu peux retrouver avec sizeof. Comme ton compilateur connaît la taille d'un "int", ben il le fait tout seul comme un grand. Et comme c'est lui qui a alloué ce "int", c'est lui qui va le désalloué encore tout seul comme un grand.

Par contre si tu alloues explicitement de la mémoire avec l'opérateur new, alors la oui tu dois libérer ce que tu as alloué avec delete. [/quote]

[moldavi]

OUPS...

suis trop long pour répondre aux messages...