Help with Crazy Error Message

[lisal]

I'm running the program below and I enter in the file name and the output file and then a message box appears that reads...

The instruction at "0x00406375" referenced memory at "0xdddddddc". The memory
could not be "read"
Click OK to terminate the application
Click CANCEL to debug the application

*Here is the code That I'm using*

#include <iostream>
#include <fstream>
#include <string>
#include <stdio.h>

using namespace std;

#define USE_SAVED_MARKER 0


void ParseOnly(ifstream in, ofstream out);

void main()
{

ifstream in;
ofstream out;

string path = &quot;c:\\projects\\qrt4\\&quot;;
string text = &quot;.txt&quot;;

string filename;
string outfile;

cout << &quot;Enter desired filename: &quot;;
cin >> filename;
filename = path + filename + text;

in.open(filename.c_str());


cout << &quot;Enter output file: &quot;;
cin >> outfile;

outfile = path + outfile + text;
out.open(outfile.c_str());

ParseOnly(in, out);
}

void ParseOnly(ifstream in, ofstream out)
{
...
...
...
}


If anyone can tell me why i'm getting this problem and possibly how to fix it would be greatly appreciated! Thanx...

 

[jtm111]

Hi:

I pasted your code into Microsoft VC++ and it compiled and ran without errors.

Is the problem with your function ParseOnly?

 

[jpoggio]

Debug MFCs fills memory bytes with 0xDC when space is allocated (e.g. before calling the constructor of an object)
and with 0xDD when memory is freed (e.g. after the destructor of the object exits.)

maybe, check your objects life cycle ?

...or flame me if you don't use MCFs

Hope it helps.
Jacques.

 

[Zyrenthian]

pass the input and output files by reference. By passing them as you are... you are redeclaring them in the function. By doing that you will need to reopen the file if I am not mistaken.

Matt

 

[Miros]

put
cout << &quot;Step n&quot;;
between the existing lines in ParseOut to find out which line is crashing (use a different # for n in each statement).

I also think changing the declaration of ParseOut to
void ParseOnly(ifstream &in, ofstream &out)
would improve things for you.

Rose/Miros