:cant convert parameter 2 from ''char [20]' to LPWSTR

Question

2.00/5 (2 votes)

See more:

void CDeptStore2Dlg::OnBnClickedNewitem()
{
	// TODO: Add your control notification handler code here
	CNewStoreItemDlg dlg;
	srand( (unsigned)time(NULL) );
	char strNumber[20];

	int number1 = rand() % 999;
	int number2 = rand() % 999;

	sprintf(strNumber, "%d-%d", number1, number2);
	dlg.m_ItemNumber = strNumber;

	if( dlg.DoModal() )
	{
		LVITEM lvItem;

		lvItem.mask = LVIF_TEXT;
		lvItem.iItem = 0;
		lvItem.iSubItem = 0;
		lvItem.pszText = strNumber;
		this->m_StoreItems.InsertItem(&lvItem);
	}
}

i had copy of this code in list control from functionx website..but gives an error...
ERROR ::cant convert parameter 2 from ''char [20]' to LPWSTR

[Edit] Included code block, removed some spelling [/Edit]

Posted 8-Mar-11 0:43am

tabassum malik

Updated 8-Mar-11 0:46am

Ryan Zahra

v2

Add a Solution

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

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

Olivier Levrey · Answer 1 · 2011-03-08T02:14:00

The problem comes from these lines:

C++

   char strNumber[20];
   ...
   sprintf(strNumber, "%d-%d", number1, number2);
   ...
   dlg.m_ItemNumber = strNumber;
}

You didn't show us the type of m_ItemNumber but I guess it is a CString.
You have two options:

1- Change your project settings so that CString class will use ANSI strings.
To do that, display the project properties, then select General, then change the Character Set field to Not Set. Do that for all configurations (Release and Debug).

2- If you don't want to change your project settings, then you need to use the appropriate types and functions. Change the lines I mentionned into:

C++

    TCHAR strNumber[20];
    ...
    _stprintf(strNumber, _T("%d-%d"), number1, number2);
    ...
    dlg.m_ItemNumber = strNumber;
}

And make sure the type of m_ItemNumber is CString.
This will work wether your project uses wide char strings or not.

Ryan Zahra · Answer 2 · 2011-03-08T00:48:00

Solution 1

Modify this line

<br />
wsprintfA(strNumber, _T("%d-%d"), number1, number2);<br />

Posted 8-Mar-11 0:48am

Ryan Zahra

Updated 8-Mar-11 0:56am

v3

tabassum malik · Answer 3 · 2011-03-08T00:52:00

Solution 2

Again it gave 2 error ..
ist is b4 i wrote..
2nd is sprintf:cant convert parameter to from 'const wchar_t[6]' to const char*'

Posted 8-Mar-11 0:52am

tabassum malik

Comments

Ryan Zahra 8-Mar-11 6:56am

My bad...use wsprintfA instead of sprintfA

Richard MacCutchan 8-Mar-11 7:07am

sprintf() does not take wide char arrays, use swprintf(). See this link: http://msdn.microsoft.com/en-us/library/ybk95axf.aspx