Lecture dans un fichier

Lecture erronée

23 septembre 2007 à 19:57:10

Bonjour !

Je suis en train de codé une fonction qui lit le nombre de lignes et le nombre de colonnes dans un fichier et qui insére ensuite le contenu du fichier dans une matrice :

#include "fichier.h"
void ouvrirFichier(char* nom_fichier, int& nbLignes, int& nbColonnes, char** Matrice)
{
ifstream fichier ; // Objet de type ifstream
fichier.open(nom_fichier); //Ouverture en lecture seule
//Lecture de la taille du fichier
fichier >> nbColonnes;
fichier >> nbLignes;
allegro_message("%d, %d", nbLignes, nbColonnes);
//Allocation du tableau par niveau
Matrice = new char*[nbColonnes];
for(int i=0; i< nbColonnes; i++)
Matrice[i] = new char[nbLignes];
//Initialisation des valeurs du tableau
for(int i=0; i< nbColonnes; i++)
for(int j=0; j< nbLignes; j++)
Matrice[i][j] = '0';
// Lecture du tableau, caractère par caractère
for(int j=0; j<nbLignes; j++)
for(int i=0; i<nbColonnes; i++)
fichier >> Matrice[i][j] ;
fichier.close(); //Fermeture du fichier
}

Le petit fichire texte :

10
8
0000000000
0000000000
0000000000
0000000000
0000000000
0000000000
0000000000
0000000000

Mais lorsque je demande à l'ordinateur quels nombres il à entré dans nbLignes et nbColonnes (à l'aide de allegro_message, similaire à cout, sauf dans une fenêtre), il me dit 0, 320 o_O

Je ne comprends pas ?
Merci de votre aide !

neuneutrinos

23 septembre 2007 à 20:10:41

simplifi toi la vie :lol:

1) va à la ligne (en lisant ligne par ligne ou à chaque '\n'
2)indiquer la colone (lire caractere par caractere )
3)puis enregistrer

cookie_au_chocolat

23 septembre 2007 à 20:14:55

nan nan, j'ai envie de faire comme ça

Sinon, dans ma fonction où est déclaré mes variables nbLignes et nbColonnes, je les ais toutes deux initialisées à 0, et maintenant, j'ai dans allegro message 0, 0.
Donc, cela prouve que mon fichier >> ***** ne fais rien

neuneutrinos

23 septembre 2007 à 20:31:29

void ouvrirFichier(char* nom_fichier, int& nbLignes, int& nbColonnes, char** Matrice)

j'aurais plutôt mis

void ouvrirFichier(const char* nom_fichier, int nbLignes, int nbColonnes, char** Matrice)

//Lecture de la taille du fichier
fichier >> nbColonnes;
fichier >> nbLignes;

ca ne lit pas la taille du fichier (ca lit 10 et 8)

allegro_message("%d, %d", nbLignes, nbColonnes);

ca renvoie un nombre à l'ecran?
c'est une de tes fonctions?(si oui je voudrais bien l'a voir

parce qu'un cout serait mieux en console)

en fait, je n'arrive pas à cerner ton raisonnement...

cookie_au_chocolat

23 septembre 2007 à 21:01:17

Au fait, le fil conducteur de "mon" idée (elle n'est pas de moi), c'est que le programme lit le nombre de lignes et de colonnes du fichier.
Ensuite, j'alloque la Matrice avec ces nombres (pour que la matrice ais le même nombre de lignes et de colonnes du fichier, pour que le niveau soit entièrement retranscrit).

Je code mon jeu avec allegro, une librairie de developpement de JV, alternative à SDL (je la préfere, me demander pas pourquoi :-°

)

Si je met ton prototype de fonction, je ne pourrais pas "écrire" mes valeurs dans mes variables, donc cela serais inutile, et j'ai besoin de mes variables dans toutes mes fonctions

Merci quand même

MatteX

24 septembre 2007 à 19:37:08

1) au lieu de me faire un char** et 2 variables j'aurai créer une classe Matrice qui contient ça

2) fait tes validation d'ouverture et de lectures.

3) char * beurk... utilise les merveilleux outils de la STL : string

bool lire_fichier( const std::string & nomFichier, Matrice & mat )
{
int nombre_tampon;
std::string chaine_tampon;
std::ifstream ifs( nomFichier.c_str() );
// si le fichier n'est pas ouvert, rien ne sert de continuer
// On aurait pu faire un throw aussi
if( !ifs.is_open() )
return false;
if( !(ifs >> nombre_tampon) )
return false;
mat.setNbLignes( nombre_tampon);
if( !(ifs >> nombre_tampon) )
return false;
mat.setNbColonnes( nombre_tampon);
//...
}

Bon dans mon exemple il y a beacoup de répétition de "return false" mais au moins il y a validation. Personnellement j'aurais créer une fonction "int lire_un_nombre( ifstream & )" qui aurait lancé une exception en cas de mauvaise lecture.

Bonne continuation en C++!