C / C++ / MFC

13-Oct-07 5:13

Hi Chandu
Thank you for your nice suggession. I tried to follow what you said. I have a member variable of CListBox pointer and I have created dynamic listbox in OnInitDialog() but I think DoDataExchange called before calling OnInitDialog and does not find the window handle and crushes.
Here is what I did: Please let me know if I did anything wrong based on your suggesstion. ====================
1.declared in header file
protected:

<br />
<br />
	HICON m_hIcon;<br />
	CListBox* m_dynalist;// dynalic listbox member<br />
	// Generated message map functions<br />
	//{{AFX_MSG(CTestdynDlg)<br />
	virtual BOOL OnInitDialog();<br />
	afx_msg void OnPaint();<br />
	afx_msg HCURSOR OnQueryDragIcon();<br />
	afx_msg void OnButton1();<br />
	afx_msg void OnSelchangeList();// function for static listbox<br />
	afx_msg void OnSelchangeDynaList();// manually added function for dynamic list box<br />
<br />
	//}}AFX_MSG<br />
<br />
	DECLARE_MESSAGE_MAP()<br />
<br />
2. In .cpp file<br />
void CTestdynDlg::DoDataExchange(CDataExchange* pDX)<br />
{<br />
	CDialog::DoDataExchange(pDX);<br />
	//{{AFX_DATA_MAP(CTestdynDlg)<br />
	DDX_Control(pDX, IDC_LIST, m_list);// static listbox<br />
	DDX_Control(pDX, IDC_LISTTEST, *m_dynalist);//dynamic listbox<br />
	DDX_Control(pDX, IDC_EDIT, m_edit);<br />
	DDX_Text(pDX, IDC_EDIT, m_editstr);<br />
	//}}AFX_DATA_MAP<br />
<br />
}<br />
<br />
BEGIN_MESSAGE_MAP(CTestdynDlg, CDialog)<br />
	//{{AFX_MSG_MAP(CTestdynDlg)<br />
	ON_WM_PAINT()<br />
	ON_WM_QUERYDRAGICON()<br />
	ON_BN_CLICKED(IDC_BUTTON1, OnButton1)<br />
	ON_LBN_SELCHANGE(IDC_LIST, OnSelchangeList)//static listbox<br />
	ON_LBN_SELCHANGE(IDC_LIST, OnSelchangeDynaList)// dynamic listbox<br />
<br />
	//}}AFX_MSG_MAP<br />
<br />
END_MESSAGE_MAP()<br />
<br />
// handler for dynamic listbox for selection change<br />
void CTestdynDlg::OnSelchangeDynaList() <br />
{<br />
	// TODO: Add your control notification handler code here<br />
	CString  str;<br />
	int isel = m_dynalist->GetCurSel();<br />
	m_dynalist->GetText(isel,str);<br />
	m_editstr = str;<br />
	m_edit.SetWindowText(str);<br />
	<br />
}<br />

msc

chandu00413-Oct-07 5:50

Re: dynamic Listbox control [modified]

13-Oct-07 5:50

wait for some time may be 30 minutes,
i shall have my dinner, refer my backups and reply exactly.
meanwhile try it ur self.

Suggestion to the members:
prefix your main thread subject with [SOLVED] if it is solved.
chandu.

Mark Salsbery13-Oct-07 7:06

13-Oct-07 7:06

s196675m wrote:
I think DoDataExchange called before calling OnInitDialog

It's called BY CDialog::OnInitDialog().

You need to do your creation before calling the base class
OnInitDialog(), and also check for a valid pointer in your
DoDataExchange() override...

void CTestdynDlg::DoDataExchange(CDataExchange* pDX)
{
   CDialog::DoDataExchange(pDX);
   //{{AFX_DATA_MAP(CTestdynDlg)
   DDX_Control(pDX, IDC_LIST, m_list);// static listbox
   <font color="Red">if (m_dynalist)</font>
   <font color="Red">{
      //**EDIT** Actually DDX_Control isn't needed.  The control is already subclassed
      //   when it is created
</font>      <font color="Red">//DDX_Control(pDX, IDC_LISTTEST, *m_dynalist);//dynamic listbox</font>
   <font color="Red">}</font>
   DDX_Control(pDX, IDC_EDIT, m_edit);
   DDX_Text(pDX, IDC_EDIT, m_editstr);
   //}}AFX_DATA_MAP
}

BOOL CTestdynDlg::OnInitDialog()
{
   <font color="Red">// create m_dynalist here!</font>

   <font color="Red">// Now call the base class OnInitDialog()</font>
   CDialog::OnInitDialog();
}

Last modified: 55mins after originally posted --

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

chandu00413-Oct-07 7:24

13-Oct-07 7:24

if mark's suggestion, worked for you, then its fine.
byt in my application, i have not taken a pointer *m_dynalist, instead, i took normal variable(in my case, array).it worked quite comfortably for me.

s196675m wrote:
DDX_Control(pDX, IDC_LISTTEST, *m_dynalist);//dynamic listbox

s196675m wrote:
ON_LBN_SELCHANGE(IDC_LIST, OnSelchangeDynaList)// dynamic listbox

in both the above cases, instead of IDC_LISTTEST, i have used some unique number like, for example, 10001
and i have used the same number while creation in the initdialog.

what i observe in your case is, you are declaring it as pointer and you are not creating memory for it using new.

if iam right, try creaitng the memory and see or use normal variable instead of pointer.
any more clarifications required?

Suggestion to the members:
prefix your main thread subject with [SOLVED] if it is solved.
chandu.

s196675m13-Oct-07 7:57

13-Oct-07 7:57

Hi I tried mark's suggession but still no luck.
Yes I have create memeory using new and I also used unique number defined for IDC_LISTTEST. I am scratching my head for last two days even can not make it work. I will be glad if you can help me to make it work
I need to understand this concept, otherwise I can not go forward.

Here is updated code:

<br />
void CTestdynDlg::DoDataExchange(CDataExchange* pDX)<br />
{<br />
	CDialog::DoDataExchange(pDX);<br />
	//{{AFX_DATA_MAP(CTestdynDlg)<br />
	DDX_Control(pDX, IDC_LIST, m_list);<br />
	if (m_dynalist){<br />
		DDX_Control(pDX, IDC_LISTTEST, *m_dynalist);<br />
	}<br />
	DDX_Control(pDX, IDC_EDIT, m_edit);<br />
	DDX_Text(pDX, IDC_EDIT, m_editstr);<br />
	//}}AFX_DATA_MAP<br />
<br />
}<br />
<br />
BEGIN_MESSAGE_MAP(CTestdynDlg, CDialog)<br />
	//{{AFX_MSG_MAP(CTestdynDlg)<br />
	ON_WM_PAINT()<br />
	ON_WM_QUERYDRAGICON()<br />
	ON_BN_CLICKED(IDC_BUTTON1, OnButton1)<br />
	ON_LBN_SELCHANGE(IDC_LIST, OnSelchangeList)<br />
	ON_LBN_SELCHANGE(IDC_LISTTEST, OnSelchangeDynaList)<br />
<br />
	//}}AFX_MSG_MAP<br />
<br />
END_MESSAGE_MAP()<br />
<br />
BOOL CTestdynDlg::OnInitDialog()<br />
{<br />
<br />
	m_dynalist = new CListBox;<br />
<br />
	m_dynalist->CreateEx( WS_EX_LEFT | WS_EX_LTRREADING | WS_EX_TOOLWINDOW,<br />
	_T("LISTBOX"),<br />
	NULL,<br />
	WS_CHILD | WS_BORDER,<br />
	CRect(0, 0, 100, 70),<br />
	this,<br />
	IDC_LISTTEST,<br />
	NULL);<br />
	CDialog::OnInitDialog();<br />
<br />
	// Set the icon for this dialog.  The framework does this automatically<br />
	//  when the application's main window is not a dialog<br />
	SetIcon(m_hIcon, TRUE);			// Set big icon<br />
	SetIcon(m_hIcon, FALSE);		// Set small icon<br />
<br />
	// dynamic list control<br />
	m_dynalist->MoveWindow(130,200, 110, 100);<br />
<br />
	m_dynalist->AddString("oone");<br />
	m_dynalist->AddString("TTwo");<br />
<br />
	m_dynalist->UpdateData(FALSE);// false true both same effect here<br />
	m_dynalist->ShowWindow(SW_SHOW);<br />
	<br />
	// TODO: Add extra initialization here<br />
	// static list control<br />
	m_list.InsertString(0,"one");<br />
	m_list.InsertString(1,"two");<br />
	m_list.InsertString(2,"three");<br />
<br />
<br />
	return TRUE;  // return TRUE  unless you set the focus to a control<br />
}<br />
void CTestdynDlg::OnSelchangeDynaList() <br />
{<br />
	// TODO: Add your control notification handler code here<br />
	CString  str;<br />
<br />
	int isel = m_dynalist->GetCurSel();<br />
	m_dynalist->GetText(isel,str);<br />
	m_editstr = str;<br />
	m_edit.SetWindowText(str);<br />
	<br />
}<br />
<br />
<br />

msc

Mark Salsbery13-Oct-07 8:03

13-Oct-07 8:03

What's not working?

Check out my previous post....I changed a line...

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

chandu00413-Oct-07 8:09

13-Oct-07 8:09

do as mark said, or otherwise, if your application is light weighted, means, no dependencies like database etc,
and if it is in visual c++ 6.0, then mail me i shall see it.
chandu004@yahoo.com.
after mailing please indicate it here.

Suggestion to the members:
prefix your main thread subject with [SOLVED] if it is solved.
chandu.

s196675m13-Oct-07 8:50

13-Oct-07 8:50

I have sent my project files to you. Please have a look. I am using VC++6.0
I really appreciate you help.

chandu00413-Oct-07 9:54

13-Oct-07 9:54

received your source code.
but it is still downloading.
you would have deleted teh debug folder and zipped it.
any way,
i will go through it and come back.

Suggestion to the members:
prefix your main thread subject with [SOLVED] if it is solved.
chandu.

Mark Salsbery13-Oct-07 8:10

13-Oct-07 8:10

s196675m wrote:
m_dynalist->UpdateData(FALSE);// false true both same effect here

It may be true that it has the same effect in this case but that's not true in general.

First, you should be passing FALSE when calling UpdateData() from OnInitDialog.

Second, CDialog::OnInitDialog() Calls UpdateData(FALSE) for you....it's redundant to call
it again unless you've actually made a modification that effects DDE - you haven't here.

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

s196675m13-Oct-07 8:52

13-Oct-07 8:52

Hi Mark
If you want I can send you the source file.
Thank you and appreciate your help

Mark Salsbery13-Oct-07 10:02

13-Oct-07 10:02

Sure. Without the object code (delete the debug folder) Smile | :)

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Mark Salsbery13-Oct-07 10:45

Re: dynamic Listbox control [modified]

13-Oct-07 10:45

Fixed.

I sent back one file - search for "//MARK" to see the changes I made.

One change was the one I modified in my first reply - the DDX_Control() call isn't
necessary since the control is already subclassed.

Second change: LBS_NOTIFY style added to the listbox control so you
get your selchange notifications.

Third change: m_dynalist->UpdateData() call removed. That's useless,
and does nothing Smile | :)

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

s196675m13-Oct-07 10:52

13-Oct-07 10:52

Hi mark
Thank you for your big help.
Your three line comment is more than my two days of work.

I appreciate your help.
Thank you again.

-- modified at 17:14 Saturday 13th October, 2007

Mark Salsbery13-Oct-07 10:56

13-Oct-07 10:56

s196675m wrote:
Thank you

You're welcome Smile | :)

I sent another email in reply to the other topic...

Cheers,
MArk

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

chandu00413-Oct-07 11:02

13-Oct-07 11:02

you are very dynamic sir.
by the time i downloaded it, copied it onto other system, and unzipped it, you fired the answer.
hats off.

Mark Salsbery13-Oct-07 11:19

13-Oct-07 11:19

Heh. Well (s)he sent me a project that built fist try (after conversion to
VS2005 format) so that made it easy. Smile | :)

Plus there wasn't thousands of lines of code to weed through.

Mark

Mark Salsbery
Microsoft MVP - Visual C++
Java | [Coffee]

Visual C++ and Linux

Sangeetha_J12-Oct-07 20:51

Sangeetha_J

12-Oct-07 20:51

Hi,

I had developed one file manipulation tool using Visual C++ 6.0.. But the customer wishes, that the same exe, if runs under Linux and Sun Solaris, it could be better. Is there any ways to do this?

Thanks in advance,

With Regards,
Sangeetha.

led mike12-Oct-07 20:56

led mike

12-Oct-07 20:56

There is no way for the same .exe to be loaded and executed by Linux or Solaris.

Sangeetha_J12-Oct-07 20:59

Sangeetha_J

12-Oct-07 20:59

Hi mike,

Is there any option to develop the application using VC 6.0 to run in Windows, linux as well as in Solaris?

With Regards,
Sangeetha.

http://www.google.com/search?hl=en&q=cplusplus+%22cross+platform+development%22&btnG=Search[^]

led mike12-Oct-07 21:04

led mike

12-Oct-07 21:04

Sangeetha_J12-Oct-07 21:05

Sangeetha_J

12-Oct-07 21:05

thank u mike. i will search and get back to u

With Regards,
Sangeetha.