Problem with CString's Format

[ProgMaster]

I've used the Cstring format to convernt a CTime into a a CString, yet I get an error message, here's the code:

void CTimerDlg::OnTimer(UINT_PTR nIDEvent)
{
CString m_sTime;
// TODO: Add your message handler code here and/or call default
CTime CurTime = CTime::GetCurrentTime();

m_sTime.Format ("%d,%d,%d", CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());

UpdateData(FALSE);

CDialog::OnTimer(nIDEvent);
}

The Error message is : error C2664: 'void ATL::CStringT<BaseType,StringTraits>::Format(const wchar_t *,...)' : cannot convert parameter 1 from 'const char [9]' to 'const wchar_t *'

How do I do this?

[Meetee]

I've used the Cstring format to convernt a CTime into a a CString, yet I get an error message, here's the code:

void CTimerDlg::OnTimer(UINT_PTR nIDEvent)
{
CString m_sTime;
// TODO: Add your message handler code here and/or call default
CTime CurTime = CTime::GetCurrentTime();

m_sTime.Format ("%d,%d,%d", CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());

UpdateData(FALSE);

CDialog::OnTimer(nIDEvent);
}

The Error message is : error C2664: 'void ATL::CStringT<BaseType,StringTraits>::Format(const wchar_t *,...)' : cannot convert parameter 1 from 'const char [9]' to 'const wchar_t *'

How do I do this?

I think const wchar_t * typed parameter should be passed to CurTime.GetHour() function and you have passes const char [9] types parameter. Which causes such an error

[weaknessforcats]

m_sTime.Format ("%d,%d,%d", CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());

If you are using <tchar.h> this should be:
[code=c]
m_sTime.Format (TEXT("%d,%d,%d"), CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());

[ProgMaster]

If you are using <tchar.h> this should be:
[code=c]
m_sTime.Format (TEXT("%d,%d,%d"), CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());

Thank you very much, the code now works perfectly, but could you please tell me what caused the error, becuase I took the code directly from a tutorial about Visual C++. And what does the command "TEXT(..)" do?

[weaknessforcats]

Thank you very much, the code now works perfectly, but could you please tell me what caused the error, becuase I took the code directly from a tutorial about Visual C++. And what does the command "TEXT(..)" do?

TEXT is a macro. If _UNICIODE is defined TEXT becomes L:

m_sTime.Format (L"%d,%d,%d", CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());
 

The L means the string is wchar_t characters and not char characters.

If _UNICODE is not defined, the TEXT becomes nothing.

This allows your code to switch between ASCII and UNICODE without having to change the code.

There are lot of things to worry about. Start here.

[ProgMaster]

TEXT is a macro. If _UNICIODE is defined TEXT becomes L:

m_sTime.Format (L"%d,%d,%d", CurTime.GetHour(), CurTime.GetMinute(),CurTime.GetSecond());
 

The L means the string is wchar_t characters and not char characters.

If _UNICODE is not defined, the TEXT becomes nothing.

This allows your code to switch between ASCII and UNICODE without having to change the code.

There are lot of things to worry about. Start here.

Thank you very much, the page was very helpul. I'm still a beginner in C++ , and after programming Visual Basic for so long, many things in C++ seem very confusing.