Re: Adding Items - C / C++ / MFC Discussion Boards

Re: Adding Items

Hamid_RT14-May-06 21:44

Hamid_RT

14-May-06 21:44

what is it sortentry* e?

whitesky

Re: Adding Items

VinayCool14-May-06 22:24

VinayCool

14-May-06 22:24

Hai,

it just Receives file names from other functions.

Re: Adding Items

Russell^'14-May-06 22:29

Russell^'

14-May-06 22:29

What error?
What line?

Re: Adding Items

VinayCool14-May-06 22:31

VinayCool

14-May-06 22:31

(VC++ Debug Library)

status.m_OUT.AddString(e->filename);

Re: Adding Items

Russell^'14-May-06 23:11

Russell^'

14-May-06 23:11

Check this:
status.GetSafeHwnd() ==NULL?
m_OUT is ==NULL?
m_OUT->GetSafeHwnd() ==NULL?
e->filename is CString? Empty?

Re: Adding Items

VinayCool15-May-06 0:24

VinayCool

15-May-06 0:24

hi,

how to check the values for status.GetSafeHwnd() ??

//m_OUT is ==NULL?

m_OUT will be empty when the dialog loads.

//e->filename is CString? Empty?
e value will alwys have a value its of CString type "C:\Dir\filename1"

Re: Adding Items

Russell^'15-May-06 0:39

Russell^'

15-May-06 0:39

You have to write code like this (according to your variable definitions)
and find out what is the object with the real problem.

CStatusDlg status;<br />
status.Create(IDD_INDEX_STATUS_DIALOG);<br />
<br />
ASSERT(status.GetSafeHwnd() != NULL);<br />
ASSERT(status.m_OUT.GetSafeHwnd() != NULL);<br />
ASSERT(e != NULL);<br />
ASSERT(e->filename != "");<br />
...<br />
<br />
status.m_OUT.AddString(e->filename);

You must add every test that could be a problem during execution, and, remember, use DEBUG mode, not RELEASE.

If there is a bug the program will stops at the line that point the problem.

Re: Adding Items

VinayCool15-May-06 1:04

VinayCool

15-May-06 1:04

Hi,

i have used f10 and check the exact line where the error generates

status.Create(IDD_INDEX_STATUS_DIALOG);

305: status.Create(IDD_INDEX_STATUS_DIALOG);
0040A83B push 0
0040A83D push 84h
0040A842 lea ecx,[status]
0040A848 call CDialog::Create (00409e36)

what will be cause ??
------------------------------

void Index :: readFiles (sortentry* e)
{
FILE *fp;
char *s;
static int fnum = 0;
int wds = 0;

if (e != NULL) {
readFiles (e->left);
fp = fopen (e->filename, "r");
fnum++;

CStatusDlg status;
status.Create(IDD_INDEX_STATUS_DIALOG);

if ((s = (char*)strrchr(e->filename, '/')) == NULL)
{
ASSERT(status.GetSafeHwnd() != NULL);
ASSERT(status.m_OUT.GetSafeHwnd() != NULL);
ASSERT(e != NULL);
ASSERT(e->filename != "");

status.m_OUT.AddString(e->filename);
}
else
{
ASSERT(status.GetSafeHwnd() != NULL);
ASSERT(status.m_OUT.GetSafeHwnd() != NULL);
ASSERT(e != NULL);
ASSERT(e->filename != "");
status.m_OUT.AddString(e->filename);
}

// cout <<" " << s+1;
insert (fnum, e->filename);
if (fp) {
wds = getWord (fp,fnum);
}
else {
cerr << "File: " << e->filename << " can't open" << endl;
return;
}
cout << " (" << wds << " words)" << endl;
fclose (fp);
free (e->filename);
readFiles(e->right);
delete e;
}
}

Re: Adding Items

Russell^'15-May-06 1:13

Russell^'

15-May-06 1:13

CStatusDlg is child of CDialog?
Check if IDD_INDEX_STATUS_DIALOG is the right id of a Dialog resource.

else follow the functions's call using F11, and, surely, you will find the point that give the error.

Re: Adding Items

Russell^'15-May-06 1:16

Russell^'

15-May-06 1:16

Possible error:

CStatusDlg status;<br />
status.Create(IDD_INDEX_STATUS_DIALOG);

->Use this:

CStatusDlg status(IDD_INDEX_STATUS_DIALOG);

Re: Adding Items

VinayCool15-May-06 1:24

VinayCool

15-May-06 1:24

Hi,

i did that i am getting compile time error

CStatusDlg status(IDD_INDEX_STATUS_DIALOG);

'CStatusDlg::CStatusDlg' : cannot convert parameter 1 from 'const int' to 'class CWnd *'
Conversion from integral type to pointer type requires reinterpret_cast, C-style cast or function-style cast

Re: Adding Items

Russell^'15-May-06 1:30

Russell^'

15-May-06 1:30

Then, follow the create function using F10 and F11.
Be sure that the ID (and resource object) IDD_INDEX_STATUS_DIALOG is correct.

Re: Adding Items

VinayCool15-May-06 1:51

VinayCool

15-May-06 1:51

Hai,

I have checked the Dialog ID its correct.

I am not able to trace with f11, how to do that ?? its keep on entering the loop its not breaking.

Re: Adding Items

Russell^'15-May-06 2:04

Russell^'

15-May-06 2:04

Simply press F11 in place of F10 when you want to show the code of a called function (in your case Create)

Re: Adding Items

Russell^'14-May-06 23:06

Russell^'

14-May-06 23:06

fp = fopen (e->filename, "r");<br />
...<br />
if ((s = (char*)strrchr(e->filename, '/')) == NULL)

mmm....

fp = fopen (e->filename, "r");<br />
...<br />
if ((s = (char*)strrchr(e->filename, '\\')) == NULL)

Is this a good correction?

Re: Adding Items

VinayCool14-May-06 23:26

VinayCool

14-May-06 23:26

Hi,

I have checked with out adding items it works corectly whn i add the additems to list box i am getting error.

Re: Adding Items

Eytukan15-May-06 0:03

Eytukan

15-May-06 0:03

can you please show the value of e->fileName ?

<marquee scrollamount="1" scrolldelay="1" direction="up" height="10" step="1">--[V]--

[My Current Status]

Re: Adding Items

VinayCool15-May-06 0:06

VinayCool

15-May-06 0:06

hi,

It hold the value like c:\path\dir\filename1 that it

Re: Adding Items

Russell^'15-May-06 0:16

Russell^'

15-May-06 0:16

Did you read this?
http://www.codeproject.com/script/comments/forums.asp?msg=1487775&forumid=1647#xx1487775xx[^]

Re: Adding Items

Eytukan15-May-06 0:17

Eytukan

15-May-06 0:17

what happens if you hard code it with "c:\\path\\dir\\filename1" ?

<marquee scrollamount="1" scrolldelay="1" direction="up" height="10" step="1">--[V]--

[My Current Status]

Re: Adding Items

Hamid_RT15-May-06 4:12

Hamid_RT

15-May-06 4:12

How to use readFiles can you show

whitesky

Re: Adding Items

_AnsHUMAN_ 14-May-06 21:00

_AnsHUMAN_

14-May-06 21:00

Check if you are binding the variables to the list box correctly. If affirmative you try building this piece of code in ReleaseMode.

Somethings seem HARD to do, until we know how to do them.
Wink | ;-)

_AnShUmAn_

Re: Adding Items

Maximilien15-May-06 3:04

Maximilien

15-May-06 3:04

 CStatusDlg status;
status.m_OUT.AddString(e->filename);

You cannot do this like that.

the m_OUT listbox will not be Created until you call either DoModal or Create, so it will crash because the HWND will not exists.

One way is to fill the listbox in the OnInitDialog of your dialog.

Maximilien Lincourt
Your Head A Splode - Strong Bad

Re: Adding Items

Hamid_RT15-May-06 4:13

Hamid_RT

15-May-06 4:13

i said but he writes it dosent work

whitesky

Re: Adding Items

David Crow15-May-06 6:21

David Crow

15-May-06 6:21

vinaycool wrote:
CStatusDlg status;
status.m_OUT.AddString(e->filename);

There are several things wrong here, namely status is a CStatusDlg object whose underlying window has not yet been created. At a minimum, you need to do something like:

CStatusDlg status;
status.DoModal();
...
BOOL CStatusDlg::OnInitDialog()
{
    CDialog::OnInitDialog();
    
    // now you can initialize m_OUT 
    m_OUT.AddString("1");
    m_OUT.AddString("Zoo");
}

Until CStatusDlg's window and the m_OUT control actually exist, calling AddString() will obviously result in an error.

"The largest fire starts but with the smallest spark." - David Crow

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.