Problem on using VC++ 2008

[arunkr6]

Sorry I will explain it again.....

#include<ifstream>;
using namespace std;

CStringList parList;
fstream sourcefile;
CFileDialog FileDlg(FALSE, _T("txt"),_T("ParameterFile"), OFN_CREATEPROMPT|OFN_HIDEREADONLY | 
OFN_SHOWHELP|OFN_OVERWRITEPROMPT|OFN_LONGNAMES, _T("Save the file as(*.txt)|*.txt||"));

if (FileDlg.DoModal() == IDCANCEL)
return;
_TCHAR sFileName[MAX_PATH];

BeginWaitCursor();
UpdateData(true);

_tcscpy(sFileName, FileDlg.GetFileName());
sourcefile.open(sFileName,ios:out);

if(!sourcefile.is_open())
AfxMessageBox(_T("file for saving not opened"));
else
for (POSITION Pos1 = parList.GetHeadPosition(); Pos1 != NULL);
sourcefile<< (LPCTSTR)parList.GetNext(Pos1)+_T('\n');

sourcefile.close();

This is my code I've written for the function to save the data to a file.

The CStringList parList contains the datas(Which are enthered by users) from a ListBox(this has done on another function

already). I am trying to load it on the fstream sourcefile.

parList.GetNext(pos1)

returns the exact value entered.But it is not loading to the file.

When am opening the file,it has some Hex values there(hope, these r address).

How to fix it???

[Philip Nicoletti]

You have the following code ... does it even compiile ?

for (POSITION Pos1 = parList.GetHeadPosition(); Pos1 != NULL);
sourcefile<< (LPCTSTR)parList.GetNext(Pos1)+_T('\n');

[arunkr6]

Ya.. I've compiled it.....

This only returns the file with some unknown data..

[hoxsiew]

Is this a UNICODE build? Does the CString member of CStringList evaluate to CStringA or CStringW? Check with the debugger if you have to.

[Philip Nicoletti]

1) I do not think that it should compile

2) Do you really want a semi-colon at the end of the for statement.

[hoxsiew]

3) Why do you have a semi-colon at the end of the for statement?

Missed that. Good catch!

It should compile, but the loop ends at the semicolon.

Pos is normally incremented as part of GetNext().

Edit: It's missing a semicolon in the for(; loop so it should NOT compile. I stand corrected.

[arunkr6]

Sorry... That was my mistake while copying not in compilation...

I was copying it from the already posted codes..so there some smileys came n I just cleared without much care..

The code I've copied is

for(Position Pos1=ParList.GetHeadPosition();Pos1!=NULL;)

And it has compiled without any error..

I still conusing that my error is in part of fstream..

Shd I take any else precautions b4 using fstream??

[hoxsiew]

You need to determine what kind of string (wide or multi-byte) you are getting form you CStringList. Just casting the return value to LPCTSTR won't cut it if the fundamental type is wrong (wchar_t * v. char *).

[arunkr6]

It is CString I hope..

Sorry that, am not @ my work place nw...and I can't access internet from ther..

So can't confirm nw.. Bt the return value is a CString only. I've verified it by returning on CString..

done as

CString ss;
ss=parList.GetNext();
::AfxMessageBox(ss);

and it get's the exact data, which i needed..

??

[hoxsiew]

"CString" is a macro that can evaluate to either CStringA or CStringW depending on whether you are building a UNICODE app (whether _UNICODE is defined) or not.

[VictorN]

It is CString I hope..

Sorry that, am not @ my work place nw...and I can't access internet from ther..

Not good...
You was many times asked whether your current build is UNICODE or ANSI. You haven't answer yet...
Note that if it is a UNICODE one then you must use wfstream instead of fstream

[arunkr6]

I havn't defined it as UNICODE.

[hoxsiew]

Apps built by the AppWizard should default to UNICODE in VS2005 and later.

[arunkr6]

So what should I do further??

I need ASCI values.

So I have to change as wfstream right??

lemme compile it n get u the response...thank u...

[arunkr6]

thank u..

it got worked.. i jus changed 2 wfstream.... got the problems,,

thank u..all