Originally Posted by earlyriser

I didn't know that C is better for char arrays than C++.

Originally Posted by Ancient Dragon

That's not true, character arrays can be handled equally by either language.

Originally Posted by joeprogrammer

If you're going to use char arrays, you might as well be using C. And no, I know it's not your fault, so I blame your teacher for that.

Originally Posted by WaltP

Seems to me the concept here is to learn to process c-strings, because they are necessary in C++.

earlyriser, stop using any and all functions that have a definition of string and use only char* functions. Help us save Joe's sanity.

Originally Posted by earlyriser

After reading a line in a text file I need to read another line which contains an INT. I thought that the bold line in this code will be ok to retreive the data, but it doesnt.

Help with Code Tags

C++ Syntax (Toggle Plain Text)

 
#include <iostream>
#include <fstream>
#include <string>
#include <cstdlib>
using namespace std;
 
struct Professeur
{
    char nom[20];
    int ancien;
};
 
 
class DossierProfesseur
{
 
 
    public:
        Professeur *tete;
        DossierProfesseur();
        ~DossierProfesseur();
};
 
 
DossierProfesseur::DossierProfesseur()
{
    Professeur *PCourant, *PTete=NULL;
    string line;
    ifstream myfile ("FP.txt");
    if (myfile.is_open())
    {
        while (! myfile.eof() )
        {
            getline (myfile,line);
            PCourant = new Professeur;
            strcpy (PCourant->nom, line.c_str());
            cout << PCourant->nom;
            [B]getline (myfile, PCourant->ancien);[/B]
        }
        myfile.close();
    }
 
    else cout << "Unable to open file";
}
 
 
void main () {
 
 
    DossierProfesseur *dossierProfesseur;
 
    dossierProfesseur= new DossierProfesseur;
}

#include <iostream> #include <fstream> #include <string> #include <cstdlib> using namespace std; struct Professeur { char nom[20]; int ancien; }; class DossierProfesseur { public: Professeur *tete; DossierProfesseur(); ~DossierProfesseur(); }; DossierProfesseur::DossierProfesseur() { Professeur *PCourant, *PTete=NULL; string line; ifstream myfile ("FP.txt"); if (myfile.is_open()) { while (! myfile.eof() ) { getline (myfile,line); PCourant = new Professeur; strcpy (PCourant->nom, line.c_str()); cout << PCourant->nom; [B]getline (myfile, PCourant->ancien);[/B] } myfile.close(); } else cout << "Unable to open file"; } void main () { DossierProfesseur *dossierProfesseur; dossierProfesseur= new DossierProfesseur; }

Originally Posted by Lazaro Claiborn

Being the nerd I am, I'll give my opinion on this particular issue. If you're going to use a structure data type, I'd just use malloc instead of new to allocate memory on the heap:

Help with Code Tags

c Syntax (Toggle Plain Text)

PCourant = (struct Professeur*)malloc(sizeof(struct Professeur));

PCourant = (struct Professeur*)malloc(sizeof(struct Professeur));

And for the string variable line, you might as well use a character array of say 100 characters or etc:

Help with Code Tags

c Syntax (Toggle Plain Text)

#define MAX_LINE 100
char line[MAX_LINE];

#define MAX_LINE 100 char line[MAX_LINE];