OVH Cloud OVH Cloud

erreur avec while eof ?

15 réponses
Avatar
giovanni
voila le while(!fichier.eof()) se fais mais mal me fais que pause me affiche
et le
while(!fichier2.eof()) lui affiche les printf...
je sais pas pourquoi
donc ce programme compare une ligne (fichier) avec toute les autre ligne du
(fichier2) ensuite passe a la ligne suivante et refais la meme chose merci

#include <cstdlib>
#include <iostream>
#include <fstream>
#include <stdlib.h>
using namespace std;

int main()
{
ifstream fichier;
int nb_0, nb_1, nb_2, nb_3, nb_4, nb_5, nb_6, nb_7;
int ok_1, ok_2, ok_3, ok_4, ok_5, ok_6, ok_7, ok_8, ok_9;



ifstream fichier2;
int nb_00, nb_11,nb_22, nb_33,nb_44, nb_55, nb_66, nb_77;

ok_1=0;
fichier.open("text.txt", ios::in);
fichier2.open("text2.txt", ios::in);
if(fichier.bad())
return(1); // errore
if(fichier2.bad())
return(1); // errore
while(!fichier.eof()){

fichier >> nb_0 >> nb_1 >> nb_2 >> nb_3>> nb_4 >> nb_5 >> nb_6>>
nb_7 ;

while(!fichier2.eof()){
fichier2 >> nb_00 >> nb_11 >> nb_22 >> nb_33 >> nb_44 >> nb_55 >> nb_66 >>
nb_77;


printf(" lign a %d", nb_11);
printf(" %d", nb_22);
printf(" %d", nb_33);
printf(" %d", nb_44);
printf(" %d", nb_55);
printf(" %d", nb_66);
printf(" %d\n", nb_77);


printf(" ligne b %d", nb_1);
printf(" %d", nb_2);
printf(" %d", nb_3);
printf(" %d", nb_4);
printf(" %d", nb_5);
printf(" %d", nb_6);
printf(" %d\n", nb_7);

if (nb_11==nb_1) ok_1=ok_1+1;
if (nb_11==nb_2) ok_1=ok_1+1;
if (nb_11==nb_3) ok_1=ok_1+1;
if (nb_11==nb_4) ok_1=ok_1+1;
if (nb_11==nb_5) ok_1=ok_1+1;
if (nb_22==nb_2) ok_1=ok_1+1;
if (nb_22==nb_3) ok_1=ok_1+1;
if (nb_22==nb_4) ok_1=ok_1+1;
if (nb_22==nb_5) ok_1=ok_1+1;
if (nb_33==nb_3) ok_1=ok_1+1;
if (nb_33==nb_4) ok_1=ok_1+1;
if (nb_44==nb_5) ok_1=ok_1+1;
if (nb_44==nb_4) ok_1=ok_1+1;
if (nb_44==nb_5) ok_1=ok_1+1;
if (nb_55==nb_5) ok_1=ok_1+1;
printf(" ok=%d\n",ok_1);

ok_1=0;
}
system("pause");

}
system("pause");


fichier2.close();
// chiudere file
fichier.close();

return(0);
}

5 réponses

1 2
Avatar
kanze
Michel Michaud wrote:
Dans le message 41f69fdd$0$22651$,
std::vector<int> Valeurs, Valeurs2;
std::ifstream Fichier("test.txt");
std::ifstream Fichier2("test2.txt");

int Valeur;
while(Fichier>>Valeur) {Valeurs.push_back(Valeur);}
while(Fichier2>>Valeur){Valeurs2.push_back(Valeur);}


Il est plus direct (et plus savant ?) de faire

vector<int> valeurs(istream_iterator<int>(ifstream("test.txt")),
istream_iterator<int>());

etc.


Tout dépend du cahier de charges, mais j'ai du mal à voir un cas
où le code ci-dessus serait acceptable. Au minimum, on aimerait
un message d'erreur si on n'arrive pas à ouvrir "test.txt".

--
James Kanze GABI Software http://www.gabi-soft.fr
Conseils en informatique orientée objet/
Beratung in objektorientierter Datenverarbeitung
9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34


Avatar
kanze
Michel Michaud wrote:
Dans le message
, Gabriel Dos

"Michel Michaud" writes:
Il est plus direct (et plus savant ?) de faire


je ne sais pas si c'est savant, parce que cela ne fait aucun
contrôle sur l'entrée :-)


Tiens, tu coupes les messages un peu trop serrés maintenant ?
Je remets ce que tu as coupé et qui confirmait ce que tu dis :

Ça n'est pas toujours avantageux,


(mais le code original ne contrôlait pas plus les erreurs...)


Et tu veux l'encourager dans ses erreurs:-).

En fait, tant qu'à en parler, aussi bien dire que le mieux est
peut-être une solution intermédiaire, qui exposerait
clairement le flux qu'on peut alors tester :

ifstream f("test.txt");
// if (f.whatever()) si désiré

vector<int> valeurs(istream_iterator<int>(f),
istream_iterator<int>());


Et ça lui sert à quoi, de déclarer une fonction extern qu'il
n'appelle jamais ?

// Autres if si désirés, f est toujours disponible...

Et j'ajoute par ailleurs, qu'il est fort possible qu'une
solution avec boucles explicites (et contrôles d'erreurs)
soient plus utiles dans un programme « sérieux »...


Tout dépend. Je suis assez sceptique en ce qui concerne
l'utilisation des istream_iterator sur des types de base, pour
lire un fichier extern. En revanche, pour lire un stringstream
qu'on a extrait d'un fichier extern, ou pour lire un type
utilisateur (disons une entrée dans un fichier de propriétés,
avec une conversion implicite en std::map< std::string,
std::string >::value_type), surtout avec un streambuf filtrant
pour traiter les commentaires et les lignes de continuation, je
n'en dis pas. (Mais je crois que le posteur initial n'en est pas
là.)

--
James Kanze GABI Software http://www.gabi-soft.fr
Conseils en informatique orientée objet/
Beratung in objektorientierter Datenverarbeitung
9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34



Avatar
kanze
giovanni wrote:

je vais esayer donc ce que je cherche a faire c est qu il
controle premiere ligne du fichier 1 avec ligne 1 du fichier 2
me dire s il y a des difference ou equivalence ensuite ligne 1
du fichier 1 toujours mais ligne du fichier 2 etc jusqua fin
du fichier 2 ensuite reprendre ligne 2 du fichier 1 et
comparer a nouveau sve ligne 1 du fichier 2 etc ... donc
compare mais ligne pas vraiment fichier a fichier..


[Si tu essayais Un peu plus de ponctuation et des
majuscules, et un peu moins de fautes d'orthographe, la
lecture de tes postes serait énormement facilitée.]

La question la plus importante alors, c'est est-ce qu'il
pourrait être raisonable de tenir les fichiers (ou au moins un
des fichiers) en mémoire, parce que sinon, ça risque de ne pas
aller très vite, et il faudrait considérer des algorithmes
spéciaux pour l'accellérer. Si on peut se permettre de garder un
des fichiers en mémoire, quelque chose du genre :

std::vector f1( (std::istream_iterator< Line >( fichier1 )),
(std::istream_iterator< Line >()) ) ;
Line tmp ;
while ( fichier2 >> tmp ) {
if ( std::find( f1.begin(), f1.end(), Matcher( tmp ) ) ) {
// ...
}
}

me semble la solution indiquée. Après, reste à toi de définir
Line et Matcher comme il faut (mais ça ne doit pas être trop
difficile).

La seule faiblesse que je vois avec cette solution, c'est
qu'elle s'arrêtera sur la première ligne avec une erreur de
format, plutôt que de le sauter. C'est possible de résoudre ce
problème dans le operator>> pour Line ; c'est un peu un abus, au
moins dans le cas général, mais c'est parfois acceptable dans
des tout petits programmes comme celui-ci.

--
James Kanze GABI Software http://www.gabi-soft.fr
Conseils en informatique orientée objet/
Beratung in objektorientierter Datenverarbeitung
9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34

Avatar
giovanni
ok je vais voir en tout cas merci du coup de main si j'ai un prob je ferais
un news mais je pense cette fois y arriver...
je suis debutant en c++ mais en pascal ou vb je me debrouille un peu mieux
que c++ et comme gratuit en plus dev c++ ....
a écrit dans le message de news:

giovanni wrote:

j'ai essayer de mettre ca en memoire le prob maintenant avec
tableau comment savoir la taille max? que je peu cree et
utiliser la memoire dynamique comment faire comme je sais pas
la taille du fichier petit fichier ou gros fichier


D'abord, il faudrait apprendre se servir du news -- on ne cite
pas tout un message, y compris les signatures, pour poser une
question qui n'a rien à voir ; quand on répond à un message, on
répond en-dessous la partie citée, en coupant tout ce qui n'est
pas nécessaire pour que la réponse soit compréhensible.

Ensuite, évidemment qu'on ne sait pas d'office la taille du
fichier. En C++, l'idiome le plus courant consiste à mettre les
données dans un std::vector, au moyen de push_back. Mais c'est
difficile à donner une règle générale ; en fait, tout dépend de
ce qu'on veut faire avec ces données. En fait, je dirais que la
première décision à prendre quand on écrit un programme, c'est
sur les données qu'on veut traiter et comment les organiser.

--
James Kanze GABI Software http://www.gabi-soft.fr
Conseils en informatique orientée objet/
Beratung in objektorientierter Datenverarbeitung
9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34

Avatar
Michel Michaud
Dans le message ,
Michel Michaud wrote:
vector<int> valeurs(istream_iterator<int>(f),
istream_iterator<int>());


Et ça lui sert à quoi, de déclarer une fonction extern qu'il
n'appelle jamais ?


:-(

On ne peut quand même pas donner toutes les réponses, il faut que
le lecteur puisse travailler un peu... :-)

--
Michel Michaud
http://www.gdzid.com
FAQ de fr.comp.lang.c++ :
http://www.cmla.ens-cachan.fr/~dosreis/C++/FAQ/


1 2