Unable to Call Function in VC++

Question

4.00/5 (1 vote)

See more:

Hi Frends,
I need help, I have a code in VC++.net 2008, at runtime one function is getting skipped automatically. any one know the reason please reply.

Thanx

MIDL

WriteTESTLog("TEST_LOGS_NIGHT",_T("Before FireMTURL::") + _T(minsat.MT_URL));
           
 resp = FireMTURL(DB_DATA.msisdn,msg_send,minsat.MT_URL,MT_SHORTCODE);
            
WriteTESTLog("TEST_LOGS_NIGHT",_T("After FireMTURL::") + _T(minsat.MT_URL) +_T(" # ") + _T(resp

));

1. before FireMTURL method log is written.
2. in above code whenever control reached at FireMTURL() method its getting closed and application is restarted.

Posted 22-Feb-11 20:49pm

sam_ckp

Updated 22-Feb-11 21:57pm

v3

Add a Solution

Comments

OriginalGriff 23-Feb-11 2:58am

You are going to have to give move info than that!
Perhaps a relevant code fragment would help?

[no name] 23-Feb-11 3:03am

Show the code.

aravindkrgec 23-Feb-11 3:20am

Let me know the nature of the function skipped. Shall you please explain the qn more clearly????

CPallini 23-Feb-11 3:36am

Code, Watson, we need code. And a detailed description of the scenario.

Olivier Levrey 23-Feb-11 4:05am

So the method is not skipped but makes your application crash: it's a big difference. What's inside FireMTURL?

sam_ckp 23-Feb-11 4:18am

CString FireMTURL(CString msisdn,CString mesg,CString MT_URL,CString MT_SHORTCODE)
{
WriteTESTLog("TEST_LOGS_NIGHT",_T("IN AIR BAL functionnnnnnnn------->"));
try
{

/* minsat.MT_URL.Replace("XXXX",msisdn);
minsat.MT_URL.Replace("YYYY",packs.MT_SHORTCODE);
minsat.MT_URL.Replace("ZZZZ",mesg);*/

mesg.Replace('*','%2A'); //replacing special characters in the messagess
mesg.Replace('#','%23');

if(mesg.GetLength()>=158)
{
WriteTESTLog("TEST_LOGS_NIGHT",_T("message greater than 160 characters"));

WriteNXLERRORLOG("MTURL_MAX_STRING",_T("message greater than 160 characters")+_T(packs.PackName));

CString error="";
error.Format("MT_URL_message greater than 160 characters-->%s",packs.PackName);

MessageBox(HWND_DESKTOP,(LPCTSTR)error,(LPCTSTR)minsat.TITLE,MB_OK);

return "MESG_MORE_THAN_160";
}

MT_URL.Replace("XXXX",msisdn);
MT_URL.Replace("YYYY",MT_SHORTCODE);
MT_URL.Replace("ZZZZ",mesg);

System::String^ s;
String ^managed_msisdn = gcnew String(msisdn);
String ^managed_URL = gcnew String(MT_URL);

CString no = managed_msisdn;
CString URL = managed_URL;

WriteTESTLog("TEST_LOGS_NIGHT",_T("MTURL_PRARAMSSSSSS")+_T(no)+_T("------>")+_T(URL)+_T("------")+_T(MT_SHORTCODE)+_T("------")+_T(mesg));

// WriteAIRLOG("PARAM_INFO",_T(no) + _T("#") +_T(URL) + _T("#")+ _T(logpath) + _T("#") + _T(circle));
s=URLFIRECPP::UrlFire::FireUrl(managed_URL);
CString response = s;

WriteTESTLog("TEST_LOGS_NIGHT",_T("MTURL_RESPONSE")+_T("------>")+_T(URL)+_T("------")+_T(response));

WritePSOLog("MT_URL",_T("MTURL_RESPONSE")+_T("------>")+_T(URL)+_T("------")+_T(response));
CString log="";
log.Format("msisdn-->%s#shortcode-->%s#mesg-->%s#url-->%s#response-->%s",no,MT_SHORTCODE,mesg,URL,response);

SendDBcommand("","MT_URL_FIRED",no,log);

if(response== "ok" ||response== "OK" )
{
CString bal = "success";
printf("%s",s);
WriteTESTLog("TEST_LOGS_NIGHT",_T("MTURL_RESPONSE-------->SUCCESS"));

return bal;
}
else
{
CString ret = "error";

WriteTESTLog("TEST_LOGS_NIGHT",_T("MTURL_RESPONSE-------->FAILED"));

return ret;
}
}
catch(CException *ex)
{
char str[1024];
CString error="";
ex->GetErrorMessage(str,1024);
error=str;
error=error + "#FireMTURL";
WriteNXLERRORLOG("ERROR",_T(error));

}
}

2 solutions

Solution 2

Hi,
Just as Olivier said, check the validity of the strings. Also check the value of resp, the return value. By the way did you debug the code to see where the actual problem is.....

Posted 22-Feb-11 22:56pm

aravindkrgec

Updated 22-Feb-11 22:57pm

v2

Add a Solution

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Olivier Levrey · Accepted Answer · 2011-02-22T22:46:00

OK I will do your job and ask a proper question (you should do it by yourself if you want a chance to have answers...).

Correct me if I am wrong. Your FireMTURL function looks like:

CString FireMTURL(CString msisdn, CString mesg, CString MT_URL, CString MT_SHORTCODE)
{
     //...
}

You are telling that your application crashes and restarts as soon as it tries to enter that function. Since you put lots of logs inside it, you should see the logs written, but there are none. So your function is not called.

If all this is correct, then I suppose there is a problem in the arguments:

1 - First of all, don't pass CString to your function, pass CString& instead. This will avoid calling copy-constructor.
2 - Before calling the function, check that all your CString parameters are valid. If one of them is corrupted, then the function call would probably crash due to the call of the copy-constructor.
3 - If one of the CString is corrupted, it is most likely that you accessed its internal data using the GetBuffer or GetBufferSetLength methods without calling ReleaseBuffer or ReleaseBufferSetLength after modification.

To use references, change your function declaration to:

CString FireMTURL(CString& msisdn, CString& mesg, CString& MT_URL, CString& MT_SHORTCODE)
{
    //...
}

To call that function, you can use your previous code without changing anything more:

resp = FireMTURL(DB_DATA.msisdn, msg_send, minsat.MT_URL, MT_SHORTCODE);

Unable to Call Function in VC++

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0

Unable to Call Function in VC++

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0

Existing Members

...or Join us