while can't go to the 2nd record.

[Wen]

Hallo,
Can someone tell me why my while doenst read the second, third.... record?
(This's a part of my code.)
I've looked for it in the books and on internet, but juist don't see the
probleem.
TI@
Wen
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);

LeesMutatie( MutatieRec, klantNr);
while( MutatieRec.klan tNr !=HV) //HV is defined as 99999
{
cout<<MutatieRe c.klantNr;
LeesMutatie( MutatieRec, klantNr);
}
cin.get();

--

Met vriendelijke groet,

Wen

[Victor Bazarov]

"Wen" <We************ @hccnet.nl> wrote...

Can someone tell me why my while doenst read the second, third.... record?
(This's a part of my code.)
I've looked for it in the books and on internet, but juist don't see the
probleem.
TI@
Wen
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);

LeesMutatie( MutatieRec, klantNr);
while( MutatieRec.klan tNr !=HV) //HV is defined as 99999
{
cout<<MutatieRe c.klantNr;
LeesMutatie( MutatieRec, klantNr);
}
cin.get();

AFAICT, 'LeesMutatie' has not been defined in this program. Could it
be the source why it's not working? Post complete code, and not some
fragments that are impossible to validate.

[CrayzeeWulf]

Wen wrote:

Hallo,
Can someone tell me why my while doenst read the second, third.... record?
(This's a part of my code.)

Modify the code as show below. If it prints "Mutatie.klantN r is equal to
HV", then problem is not in the code fragment you provided.

// ---------------------------------------------------------
int*klantNr,*ac cess;
Client*MutatieR ec,*MasterRec;

if ( MutatieRec.klan tNr == HV ) {
cout << "Mutatie.klantN r is equal to HV" << endl ;
}

OpenBestand(acc ess);

if ( MutatieRec.klan tNr == HV ) {
cout << "Mutatie.klantN r is equal to HV" << endl ;
}
LeesMutatie(*Mu tatieRec,*klant Nr);
if ( MutatieRec.klan tNr == HV ) {
cout << "Mutatie.klantN r is equal to HV" << endl ;
}
while(*MutatieR ec.klantNr*!= HV)**//HV*is*defined*a s*99999
{
cout << MutatieRec.klan tNr;
LeesMutatie(*Mu tatieRec,*klant Nr);
}
cin.get();
// ---------------------------------------------------------

--
CrayzeeWulf

[Wen]

Well, I can't post all my code, it's really to much. But I'd select this
part, so that you can can see what's wrong with it.
Regards,
Wen

#include <iostream>
#include <ctime>
#include <string>
#include <cstring>
#include <fstream>

const int HV= 99999;

struct Client
{
int klantNr;
char soort[2];
char naam[26];
char adres[26];
char postcode[7];
char plaats[16];
int bankNr;
int giro;
char mutcode[2];
char tariefAfspr[2];
};

using namespace std;
void OpenBestand(int & access);
void LeesMutatie(Cli ent & MutatieRec, int & klantNr);

int main()
{
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);

LeesMutatie( MutatieRec, klantNr);
while( MutatieRec.klan tNr !=HV)
{
cout<<MutatieRe c.klantNr;
LeesMutatie( MutatieRec, klantNr);
}
cin.get();
}

void LeesMutatie(Cli ent & MutatieRec, int & klantNr)
{
ifstream mutatie ("mutatie.csv") ;
char puntkomma = ';';

mutatie >> MutatieRec.klan tNr >> puntkomma;
mutatie.getline (MutatieRec.soo rt, 2, ';');
mutatie.getline (MutatieRec.naa m, 26, ';');
mutatie.getline (MutatieRec.adr es, 26, ';');
mutatie.getline (MutatieRec.pos tcode,7, ';');
mutatie.getline (MutatieRec.pla ats, 16, ';');
mutatie >> MutatieRec.bank Nr >> puntkomma;
mutatie >> MutatieRec.giro >> puntkomma;
mutatie.getline (MutatieRec.mut code, 2, ';');
mutatie.getline (MutatieRec.tar iefAfspr, 2, ';');
if (mutatie.eof())
MutatieRec.klan tNr = HV;
}

void OpenBestand(int & access)
{
ifstream constant, klant, mutatie;

constant.open(" constant.csv");
if(constant.fai l())
{
cerr << "Het constantenbesta nd kon niet worden geopend" << endl;
access = 0;
}

klant.open("kla nt.dat", ios::binary );
if(klant.fail() )
{
cerr << "Het klantenbestand kon niet worden geopend" << endl;
access = 0;
}

mutatie.open("m utatie.csv");
if(mutatie.fail ())
{
cerr << "Het mutatiebestand kon niet worden geopend" << endl;
access = 0;
}
}
"Victor Bazarov" <v.********@com Acast.net> schreef in bericht
news:5IoIb.7985 6$VB2.162883@at tbi_s51...

"Wen" <We************ @hccnet.nl> wrote...

Can someone tell me why my while doenst read the second, third.... record? (This's a part of my code.)
I've looked for it in the books and on internet, but juist don't see the
probleem.
TI@
Wen
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);

LeesMutatie( MutatieRec, klantNr);
while( MutatieRec.klan tNr !=HV) //HV is defined as 99999
{
cout<<MutatieRe c.klantNr;
LeesMutatie( MutatieRec, klantNr);
}
cin.get();

AFAICT, 'LeesMutatie' has not been defined in this program. Could it
be the source why it's not working? Post complete code, and not some
fragments that are impossible to validate.

[Wen]

10001;A;Ahold;K ruidenierstraat 2;1234AA;Zaanda m;123456789;123 4567;M;X
10002;D;Albert Heijn;Industrie terrein
5;1111BN;Zaanda m;341259878;125 4444;V;Q
10003;E;Kruidva t;Holtenbroek 123;2300CB;Schi edam;331215171; 3459770;T;S
10004;F;Blokker ;Amstelveensweg 331;1111SH;Diem en;812355565;90 02352;M;H
10005;A;Houtman ;Rotterdamsvaar t 123;3800ET;Goud a;567568456;333 4444;V;P
10006;E;Visa;Ro dekruisweg 23;1156SH;Dieme n;80335658;7456 891;T;O
10007;B;Hema;Zi lverkruisweg 523;5056AA;Apel doorn;753356564 ;7456891;T;O
These are information form the "mutatie.cs v" file.

"Victor Bazarov" <v.********@com Acast.net> schreef in bericht
news:5IoIb.7985 6$VB2.162883@at tbi_s51...

"Wen" <We************ @hccnet.nl> wrote...

Can someone tell me why my while doenst read the second, third.... record? (This's a part of my code.)
I've looked for it in the books and on internet, but juist don't see the
probleem.
TI@
Wen
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);

LeesMutatie( MutatieRec, klantNr);
while( MutatieRec.klan tNr !=HV) //HV is defined as 99999
{
cout<<MutatieRe c.klantNr;
LeesMutatie( MutatieRec, klantNr);
}
cin.get();

AFAICT, 'LeesMutatie' has not been defined in this program. Could it
be the source why it's not working? Post complete code, and not some
fragments that are impossible to validate.

[CrayzeeWulf]

Wen wrote:

void LeesMutatie(Cli ent & MutatieRec, int & klantNr)
{
ifstream mutatie ("mutatie.csv") ;
char puntkomma = ';';

This will open the file "mutatie.cs v" again and again every time
LessMutatie() is called. So you will only read the first record.

Later,
--
CrayzeeWulf

[Wen]

ok, so I'd put the ';' away. I'll try that.
Thank you.
Wen

"CrayzeeWul f" <cr*********@no spam.gnudom.org > schreef in bericht
news:M4******** ***********@twi ster.socal.rr.c om...

Wen wrote:

void LeesMutatie(Cli ent & MutatieRec, int & klantNr)
{
ifstream mutatie ("mutatie.csv") ;
char puntkomma = ';';

This will open the file "mutatie.cs v" again and again every time
LessMutatie() is called. So you will only read the first record.

Later,
--
CrayzeeWulf

[Wen]

I'd try this, but it doens't work neither.
Regards,
Wen

void LeesMutatie(Cli ent & MutatieRec, int& klantNr)
{
ifstream mutatie ("mutatie.csv") ;
char komma;

mutatie >> MutatieRec.klan tNr >> komma;
mutatie.getline (MutatieRec.soo rt, 2, komma);
mutatie.getline (MutatieRec.naa m, 26, komma);
mutatie.getline (MutatieRec.adr es, 26, komma);
mutatie.getline (MutatieRec.pos tcode,7, komma);
mutatie.getline (MutatieRec.pla ats, 16, komma);
mutatie >> MutatieRec.bank Nr >> komma

MutatieRec.giro >> komma;

mutatie.getline (MutatieRec.mut code, 2, komma);
mutatie.getline (MutatieRec.tar iefAfspr, 2, komma);
if (mutatie.eof())
MutatieRec.klan tNr = HV;
}
"CrayzeeWul f" <cr*********@no spam.gnudom.org > schreef in bericht
news:M4******** ***********@twi ster.socal.rr.c om... Wen wrote:

void LeesMutatie(Cli ent & MutatieRec, int & klantNr)
{
ifstream mutatie ("mutatie.csv") ;
char puntkomma = ';';

This will open the file "mutatie.cs v" again and again every time
LessMutatie() is called. So you will only read the first record.

Later,
--
CrayzeeWulf

[CrayzeeWulf]

Wen wrote:

I'd try this, but it doens't work neither.
Regards,
Wen

void LeesMutatie(Cli ent & MutatieRec, int& klantNr)
{
ifstream mutatie ("mutatie.csv") ;

Wen,

Its not the comma. The problem is that you are creating an ifstream instance
named "mutatie" every time you call LessMutatie(). Every the first line of
this function is executed, you create an ifstream instance and open the
file "mutatie.cs v". At this point, the rest of the function reads the file
from the very beginning.

In order to fix this, you will have to redesign your code to make sure that
you do not open the file on every call to LessMutatie(). For example, you
can try opening the file outside LessMutatie() and pass it a reference to
an ifstream instance:

// -------------------------------------------------------------------------
void LeesMutatie(ifs tream& input_stream, Client & MutatieRec, int &klantNr);

int main()
{
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);
ifstream mutatie( "mutatie.cs v" ) ;

LeesMutatie( mutatie, MutatieRec, klantNr);
while( MutatieRec.klan tNr != HV )
{
cout << MutatieRec.klan tNr << ":" << MutatieRec.soor t << endl ;
LeesMutatie( mutatie, MutatieRec, klantNr);
}
cin.get();
}

void LeesMutatie(ifs tream& mutatie, Client & MutatieRec, int &klantNr)
{
char puntkomma = ';';
if ( ! mutatie ) {
std::cout << "mutatie kaput." << endl ;
exit(1) ;
}
mutatie >> MutatieRec.klan tNr >> puntkomma;
mutatie.getline (MutatieRec.soo rt, 2, ';');
mutatie.getline (MutatieRec.naa m, 26, ';');
mutatie.getline (MutatieRec.adr es, 26, ';');
mutatie.getline (MutatieRec.pos tcode,7, ';');
mutatie.getline (MutatieRec.pla ats, 16, ';');
mutatie >> MutatieRec.bank Nr >> puntkomma;
mutatie >> MutatieRec.giro >> puntkomma;
mutatie.getline (MutatieRec.mut code, 2, ';');
mutatie.getline (MutatieRec.tar iefAfspr, 2, ';');

if ( mutatie.eof() )
MutatieRec.klan tNr = HV;
}
// -------------------------------------------------------------------------

There are other problems with the original LessMutatie() besides the
reopening of "mutatie.cs v". But that is a separate issue that you will have
to figure out.

--
CrayzeeWulf

[CrayzeeWulf]

Wen wrote:

I'd try this, but it doens't work neither.
Regards,
Wen

void LeesMutatie(Cli ent & MutatieRec, int& klantNr)
{
ifstream mutatie ("mutatie.csv") ;

Wen,

Its not the comma. The problem is that you are creating an ifstream instance
named "mutatie" every time you call LessMutatie(). Every time the first line
of this function is executed, you create an ifstream instance and open the
file "mutatie.cs v". At this point, the rest of the function reads the file
from the very beginning.

In order to fix this, you will have to redesign your code to make sure that
you do not open the file on every call to LessMutatie(). For example, you
can try opening the file outside LessMutatie() and pass it a reference to
an ifstream instance:

// -------------------------------------------------------------------------
void LeesMutatie(ifs tream& input_stream, Client & MutatieRec, int &klantNr);

int main()
{
int klantNr, access;
Client MutatieRec, MasterRec;
OpenBestand(acc ess);
ifstream mutatie( "mutatie.cs v" ) ;

LeesMutatie( mutatie, MutatieRec, klantNr);
while( MutatieRec.klan tNr != HV )
{
cout << MutatieRec.klan tNr << ":" << MutatieRec.soor t << endl ;
LeesMutatie( mutatie, MutatieRec, klantNr);
}
cin.get();
}

void LeesMutatie(ifs tream& mutatie, Client & MutatieRec, int &klantNr)
{
char puntkomma = ';';
if ( ! mutatie ) {
std::cout << "mutatie kaput." << endl ;
exit(1) ;
}
mutatie >> MutatieRec.klan tNr >> puntkomma;
mutatie.getline (MutatieRec.soo rt, 2, ';');
mutatie.getline (MutatieRec.naa m, 26, ';');
mutatie.getline (MutatieRec.adr es, 26, ';');
mutatie.getline (MutatieRec.pos tcode,7, ';');
mutatie.getline (MutatieRec.pla ats, 16, ';');
mutatie >> MutatieRec.bank Nr >> puntkomma;
mutatie >> MutatieRec.giro >> puntkomma;
mutatie.getline (MutatieRec.mut code, 2, ';');
mutatie.getline (MutatieRec.tar iefAfspr, 2, ';');

if ( mutatie.eof() )
MutatieRec.klan tNr = HV;
}
// -------------------------------------------------------------------------

There are other problems with the original LessMutatie() besides the
reopening of "mutatie.cs v". But that is a separate issue that you will have
to figure out.

--
CrayzeeWulf