releasing the application

**Oliver Twesten** · February 11th, 2004, 04:48 PM

I do not know if anyone already pointed out the article "Surviving the release version" by Joseph M. Newcomer:

http://www.codeproject.com/debug/survivereleasever.asp

I think this one is very advisable.

Oliver.

**andreshs1** · February 11th, 2004, 04:59 PM

Thank you for the tip Oliver, I have read that article few times today, and I identify the problems in my code, which are the initialization of the variables, and I since I am fairly a new programmer I am going mad trying to find the way to initializate the variables

at present the problem I have is that one of the member variables of one of the classes is a struct

Code:

struct USERDET
{
  BOOL newUser;
  char userName[25];
  char passWord[25];

  USERDET()//initialisation as Mr Masur suggested
  {
    newUser = FALSE;
    memset(userName, 0, sizeof(userName));
    memset(passWord, 0, sizeof(passWord));
  }
};

that struct is initialized on the struct definition but do I need to initialized also in the class constructor? if so, how do I do it??

at the moment without initializing the struct on the class constructor when I try to access the values of the struct these values are set and I cant modify them such as by

Code:

strcpy(m_userDet.userName,userDlg.m_userName);
		strcpy(m_userDet.passWord,userDlg.m_passWord);
		
nothing gets copied accross no even like fallow

strcpy(m_userDet.userName,"hello");

this should copy hello into the m_userDet.userName but it doesn't copy anything

Any ideas why it doesn't work

Thanks

**gstercken** · February 11th, 2004, 05:03 PM

Originally posted by Oliver Twesten
I do not know if anyone already pointed out the article "Surviving the release version" by Joseph M. Newcomer:

http://www.codeproject.com/debug/survivereleasever.asp

I think this one is very advisable.

Yes, the FAQ mentioned also links to it - as a last resort. That article is indeed very good and comprehensive - the FAQ covers the most frequent causes for release build problems, and points to that detailed article in the case everything else fails.

**Andreas Masur** · February 11th, 2004, 05:09 PM

Originally posted by andreshs1
When the dialog doModal return I copy the values that it got from the CEdit boxes...

Well...one thing I noticed is that your given example should not even compile. With the following line

Code:

CUserDlg userDlg = new CUserDlg();

you create the dialog on the heap (using a pointer)...later on, however, you are accessing the dialog as it was created on the stack...

Code:

rtnDlg = userDlg.DoModal();
if (rtnDlg == IDOK)
{
  strcpy(m_userDet.userName,userDlg.m_userName);
  strcpy(m_userDet.passWord,userDlg.m_passWord);
		
  //stablish connection to server
  if (!cConnect->GetConnected(m_userDet))
  {
  }
]

Other than that...could you post your project here (without debug and release directory)?

**gstercken** · February 11th, 2004, 05:10 PM

Originally posted by andreshs1
that struct is initialized on the struct definition but do I need to initialized also in the class constructor? if so, how do I do it??

I don't understand what you mean by "initialized on the struct definition", but the code suggested by Andreas uses the struct's ctor for initialization. Which other class constructor are you referring to?

Originally posted by andreshs1
at the moment without initializing the struct on the class constructor when I try to access the values of the struct these values are set and I cant modify them such as by

Code:

strcpy(m_userDet.userName,userDlg.m_userName);
		strcpy(m_userDet.passWord,userDlg.m_passWord);
		
nothing gets copied accross no even like fallow

strcpy(m_userDet.userName,"hello");

this should copy hello into the m_userDet.userName but it doesn't copy anything

Any ideas why it doesn't work

Note that those problems have nothing to do with the struct members being initialized or not - the strcpy you showed should definitely copy the strings to the struct members, regardless of the previous content. So the problem must be something else - are you sure you are looking at the same instance of the struct in both cases? And since the struct instance seems to be a member of a class - are you sure you a looking at the same instance of that class?

**Andreas Masur** · February 11th, 2004, 05:12 PM

Originally posted by andreshs1

Code:

struct USERDET
{
  BOOL newUser;
  char userName[25];
  char passWord[25];

  USERDET()//initialisation as Mr Masur suggested
  {
    newUser = FALSE;
    memset(userName, 0, sizeof(userName));
    memset(passWord, 0, sizeof(passWord));
  }
};

that struct is initialized on the struct definition but do I need to initialized also in the class constructor? if so, how do I do it??

No, you don't...the constructor will be automatically called while constructing the enclosed class (in your case the dialog).

Originally posted by andreshs1
strcpy(m_userDet.userName,"hello");

this should copy hello into the m_userDet.userName but it doesn't copy anything

What? That is basically impossible. The above line will definitely copy the word 'hello' into 'userName'. Somehow I get the impression that you are copying to a local (-> different) structure than the one enclosed by the dialog class...

**andreshs1** · February 11th, 2004, 05:19 PM

ok, I am posting the application, I have left break points where the problem is detected, hope you manage to understand, thanks in advance for all your help

once the application is launched, go the the menu and click connect

Thanks

**gstercken** · February 11th, 2004, 05:26 PM

Oops, a VC++ .net project... Sorry, I won't be able to participate in the debugging party then...

**Andreas Masur** · February 11th, 2004, 05:29 PM

Well...I do not have any development IDE installed at the moment, and I just saw that you are using .NET. Thus, I simply looked over the code...

Run your application within the debugger and step into the constructor of 'CTCPChatClientDlg' and take a look what the value of 'm_userDet.passWord' and 'm_userDet.userName' is after doing the 'strcpy()'.

The following does not make sense to me and should not make it through the compiler...

Code:

CUserDlg userDlg = new CUserDlg();//here were I create the instance of the dlg

'new' returns a pointer...hence

Code:

CUserDlg *userDlg = new CUserDlg();//here were I create the instance of the dlg

would be correct...

Run your application within the debugger and step into the function 'OnConnectMenu()' and take a look what the value of 'm_userDet.passWord' and 'm_userDet.userName' is after doing the 'strcpy()'.

**Andreas Masur** · February 11th, 2004, 05:31 PM

Originally posted by gstercken
Oops, a VC++ .net project... Sorry, I won't be able to participate in the debugging party then...

What...you need a physical debugger to debug code??

**gstercken** · February 11th, 2004, 05:33 PM

Originally posted by Andreas Masur
What...you need a physical debugger to debug code??

Well, we're all getting older...

**andreshs1** · February 11th, 2004, 05:38 PM

ok, lets see

on the constructor the values are ok, but I tried to to a test such as

Code:

CTCPChatClientDlg::CTCPChatClientDlg(CWnd* pParent /*=NULL*/)
	: CDialog(CTCPChatClientDlg::IDD, pParent)
{
	m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME);
	//structure definition
	m_socket = 0;
	m_userList.status = 0;
	m_userList.UserId= 0;
	m_dlgCounter = 0;
	m_userDet.newUser = 0;
	strcpy(m_userDet.passWord,"test");//the values here are not modified, why???? 
//the new value should be "test"!!!!!
	strcpy(m_userDet.userName,"test");
		
}

Thank you, Mr Masur and Mr gstercken

**gstercken** · February 11th, 2004, 05:42 PM

OK, OK, just a little doubt... You are debugging the release version, right? Remember (from the FAQ) that the current line you see in the debugger doesn't always exactly correspond to the actual line being executed (due to optimizations)? Try stepping a few lines further and see if the values change.

**Andreas Masur** · February 11th, 2004, 05:45 PM

Originally posted by gstercken
OK, OK, just a little doubt... You are debugging the release version, right? Remember (from the FAQ) that the current line you see in the debugger doesn't always exactly correspond to the actual line being executed (due to optimizations)? Try stepping a few lines further and see if the values change.

Is he still? That would of course explain some things...

**andreshs1** · February 11th, 2004, 06:44 PM

I am very very sorry guys, you are right, I didn't remember that sometime the line I see is not updated and I wasn't continueing with the debbuging.

Sorry and Thanks!!

another problem I now have is that I use a variable which stores the value of the created socket, but I can only use it once because the second time the values is gone, any ideas why??

Code:


	ioctlsocket(m_socket,FIONREAD,&size);//here the value of socket it is fine

	while(size > 0)
	{
		rtn = recv(m_socket,buff,strlen(buff),0);//the value of socket disappears, 
                                                   //and doesn't receive anything
		if(rtn == SOCKET_ERROR)
		{
			AfxMessageBox("****"+m_socket);
		}
		str += buff;
		size = size - rtn;
	}
	strcpy(buff,str);
	ptrPacket = (PACKET*)buff;

Any suggestion?

Thanks

Thread: releasing the application

Thread Tools

Display

Posting Permissions