C / C++ / MFC

7-Jun-05 4:06

i want to create an array whose size is dynamic. Each node of the array should be an array of characters of length 20.And then i want to send this as an argument in a function.
This is what i have done.....

	<br />
char *m_strTemp[10] = new char[count];<br />

After getting the value of "count" i start filling the array.

Now to send it as an argument of a function i declared the function as follows.

void CIdentifyDlg::SetVariable(char *str[10])<br />
{<br />
	char *tempStr[10] = new char[count];<br />
        tempStr = str;<br />
}<br />

The function is called as follows

<br />
SetVariable(m_strTemp[10]);<br />

This is giving error.
Can anyone please help me out. Hope my question is clear....

David Crow7-Jun-05 4:26

David Crow

7-Jun-05 4:26

nripun wrote:
This is giving error.

I give up. What's the error?

"Ideas are a dime a dozen. People who put them into action are priceless." - Unknown

Steen Krogsgaard7-Jun-05 4:26

Steen Krogsgaard

7-Jun-05 4:26

nripun wrote:
char *m_strTemp[10] = new char[count];

here you allocate an array of 10 char pointers and assign the char pointer returned by new to element zero (m_strTemp[0]). It's not an array of 10 (valid) pointers to char arrays.

nripun wrote:
char *tempStr[10] = new char[count];
tempStr = str;

same issue as above. Furthermore, in the second line you set the pointer tempStr to point to the same location as str points to (which is an array of 10 char pointers, where the first one points to a valid array). You do not copy the data pointed to by str into the array pointed to by tempStr.

nripun wrote:
This is giving error.

which error?

I would use containers instead, either MFC containers or STL ones.

Cheers
Steen.

"To claim that computer games influence children is ridiculous. If Pacman had influenced children born in the 80'ies we would see a lot of youngsters running around in dark rooms eating pills while listening to monotonous music"

Cedric Moonen7-Jun-05 4:28

7-Jun-05 4:28

Why don't you use std::string (from the Standard Template Library) or CString (from MFC) instead ?

It will look like this:

void CIdentifyDlg::SetVariable(std::string* StringsArray, int nCount)<br />
{<br />
// Retrieve each string:<br />
for (int i=0;i<nCount;i++)<br />
{<br />
std::string TempString = StringsArray[i];<br />
// Do things with TempString<br />
}<br />
}<br />

nripun7-Jun-05 5:04

7-Jun-05 5:04

can you please tell me how to store numbers in a CString object??

Cedric Moonen7-Jun-05 5:09

7-Jun-05 5:09

Why do you want to store numbers ???? The problem is exactly the same as a standard string except you don't need to manage the memory yourself...

But if you need to send integers then use an integer array instead (int*) that is much more easy than store the numbers into char strings (and how would you do that ???)

nripun7-Jun-05 5:24

7-Jun-05 5:24

This is what i want to do..........

<br />
sprintf(m_strTemp[i],"%d)\t%lf%% Match",i+1,m_final);

But m_strTemp is CString. So it gives the following error.

sprintf' : cannot convert parameter 1 from 'class CString' to 'char *'

Cedric Moonen7-Jun-05 6:25

7-Jun-05 6:25

Use Format() from CString, it will gives you exactly the same results:

m_strTemp[i].Format("%d)\t%lf%% Match",i+1,m_final);

nripun7-Jun-05 20:26

7-Jun-05 20:26

Thank You very much ............ Big Grin | :-D

John M. Drescher7-Jun-05 6:23

John M. Drescher

7-Jun-05 6:23

nripun wrote:
SetVariable(m_strTemp[10]);

How about this?

SetVariable(m_strTemp);

John

pausing a for loop

Lilli Alexis7-Jun-05 4:01

Lilli Alexis

7-Jun-05 4:01

I am programming a C++ MFC GUI application. During every execution of a for loop in my app I want the application to pause for about 1 second. How can I do this? I have been screwing around with WM_TIMER and SetTimer and KillTimer but I just can't get it to pause. Any help/examples would be a great help.
Thanks!
Lilli

Cedric Moonen7-Jun-05 4:06

7-Jun-05 4:06

You mean Sleep(1000) ?
But if your loop is in the main thread, your application will freeze also during 1 second...

David Crow7-Jun-05 4:28

David Crow

7-Jun-05 4:28

I'm sensing a potentially bad design here. Why exactly do you need a one-second pause?

"Ideas are a dime a dozen. People who put them into action are priceless." - Unknown

Blake Miller7-Jun-05 4:31

Blake Miller

7-Jun-05 4:31

Yes, as if the souls of a million maintenance programmers were crying out in agony...