C / C++ / MFC

26-Jan-05 6:54

Hi
I'm having some problems storing some values. Here is it:
I have a class with is derived from CObject. It just stores Variables !

<br />
CMyStoreClass : public CObject<br />
{<br />
CString m_strName;<br />
<br />
CUIntArray m_arFirstArray;<br />
CUIntArray m_arSecondArray;<br />
[...].br />
CUIntArray m_arSixthArray;<br />
}<br />

Then in my Dialog class I have a variable which shall store the CMyStoreClass Instances.

<br />
CPtrList m_Rules;<br />

To Add a Instance of CMyStoreClass:

<br />
CMyStoreClass store;<br />
store.m_strName = "Test";<br />
store.m_arFirstArray = 5;<br />
store.m_arSecondArray = 6;<br />
[...].br />
if(m_Rules.IsEmpty())<br />
m_Rules.AddHead(&store);<br />
else<br />
m_Rules.AddTail(&store);<br />

Everthing still works fine but now if i want to access the CMyStoreClass Objects again:

<br />
for(POSITION pos = m_Rules.GetHeadPosition();pos != NULL; )<br />
{<br />
CMyStoreClass* pStore = (CMyStoreClass*) m_Rules.GetNext(pos);<br />
ASSERT(pStore);<br />
if(pStore->m_strName != "Test") continue;<br />
}<br />

when my programm executes this lines it breaks and "strcmp.asm" opens and i get to "dodwords:".
I have no clue what's wrong. Anyone ? Any clue ?

MFG
RedDragon2kx D'Oh! | :doh:

Unix and C are the ultimate computer viruses.

David Crow26-Jan-05 7:04

26-Jan-05 7:04

Have you stepped into CString's != operator to see what is going on?

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

RedDragon2k26-Jan-05 7:07

26-Jan-05 7:07

I gonna try this, but if i try:

<br />
MessageBox(pStore->m_strName);<br />

it fails, szText is marked as <badptr> in Debugger Window. It says:
CXX0030: Error: expression cannot be evaluated char

Unix and C are the ultimate computer viruses.

David Crow26-Jan-05 7:13

26-Jan-05 7:13

RedDragon2k wrote:
it fails,

Which means what?

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

RedDragon2k26-Jan-05 7:15

26-Jan-05 7:15

pStore->m_strName {0x00000000 <bad ptr="">}
- ATL::CSimpleStringT<char> {0x00000000 <bad ptr="">} ATL::CSimpleStringT<char>
- m_pszData 0x00000000 <bad ptr=""> char *
CXX0030: Error: expression cannot be evaluated char

Unix and C are the ultimate computer viruses.

David Crow26-Jan-05 7:35

26-Jan-05 7:35

I'm wondering if the ATL forum would be of any help.

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

RedDragon2k26-Jan-05 7:40

26-Jan-05 7:40

I gonna post there, too. Thx for your help

Unix and C are the ultimate computer viruses.

David Crow26-Jan-05 7:45

26-Jan-05 7:45

I think I see what might be ailing you. Try this:

CMyStoreClass *store = new CMyStoreClass;
store->m_strName = "Test";
store->m_arFirstArray = 5;
store->m_arSecondArray = 6;
[...].if(m_Rules.IsEmpty())
    m_Rules.AddHead(store);
else
    m_Rules.AddTail(store);

BTW, you can use AddTail() exclusively rather than calling IsEmpty() and AddHead().

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

RedDragon2k26-Jan-05 7:50

Converting text to float (MFC)

26-Jan-05 7:50

Thx man, that helped. Big Grin | :-D

Unix and C are the ultimate computer viruses.

Visit me: www.programming.info.ms

Member 166473326-Jan-05 6:39

26-Jan-05 6:39

I'm trying to convert a Ctring I read from an editbox into a float. This is the code I've got so far;

CString val, txtEdit;<br />
float boxval = 0.2f;<br />
<br />
GetDlgItemText(IDC_EDITBOX,txtEdit);<br />
char *txt = (char *) (LPCTSTR) txtEdit;  <br />
  //where txtEdit is the Ctring containing the text read from an Edit box<br />
<br />
sscanf(txt, "%f", &boxval);<br />
val.Format(_T("%f"), boxval);<br />
AfxMessageBox(val);

Unfotunately it does not return the right float. It does return a float but it is truncated to its integer value. I.e. if I enter 2.3 it returns 2.00.

Could anyone help me here? What am I doing wrong?
What's the best way to convert a string into a float?
Thanks!

David Crow26-Jan-05 6:47

26-Jan-05 6:47

How about:

GetDlgItemText(IDC_EDITBOX, txtEdit);
double d = atof(txtEdit);

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

Member 166473326-Jan-05 6:56

26-Jan-05 6:56

Thanks, I had tried that too but have the exact same problem!
I should prob add that it works ok with text in define in the code but not with text I read from an edit box...
Really don't understand what I'm doing wrong???

David Crow26-Jan-05 7:00

26-Jan-05 7:00

What is the value of txtEdit after the call to GetDlgItemText()?

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

Member 166473326-Jan-05 7:05

26-Jan-05 7:05

When I display it in a messagebox its value is the decimal value read from the edit box.

David Crow26-Jan-05 7:13

26-Jan-05 7:13

Unless there is something else going on that is not being shown, the code snippet I provided will work.

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

Member 166473326-Jan-05 7:16

26-Jan-05 7:16

Actually I've only just realised that only the first digit is being converted. I.e. if I enter 2.3 I get 2.00 but if I enter 14.6 I get 1.00.
I must do something really wrong somewhere but I can't see where! Sigh | :sigh:

Thanks for your help!

Bob Ciora26-Jan-05 7:27

26-Jan-05 7:27

Something's definitely amiss here. If you don't make the call to GetDlgItemText but, instead, just set txtEdit to "2.334" (for example), boxval ends up being 2.334 (or whatever you ste in txtEdit). This works just fine...tested it m'self, I did.

So something's not right, and it's definitely something to do with the conditions under which GetDlgItemText is called. Maybe you're only responding to single keystrokes in the edit box, I'm not sure.

Some things to try testing:

* GetDlgItemText returns the number of characters copied. Store this returned value to an int and make sure it's returning the right number of characters.

* sscanf returns the number of arguments extracted from the string. So if your sscanf() isn't returning '1', then something's wrong.

Bob Ciora

Member 166473326-Jan-05 7:49

26-Jan-05 7:49

First thanks for the help! Much appreciated! Smile | :)

Here's my exact code now;

void CmySpinButtonCtrl::OnDeltapos(NMHDR* pNMHDR, LRESULT* pResult) <br />
        {<br />
	NM_UPDOWN* pNMUpDown = (NM_UPDOWN*)pNMHDR;<br />
<br />
<br />
        CString tmp, txtEdit="82.45";<br />
	float boxval;<br />
<br />
	char *txt = (char *) (LPCTSTR) txtEdit; <br />
	sscanf(txt, "%f", &boxval);<br />
	double d = atof(txt); <br />
<br />
	tmp.Format(_T("atof is %f"), d);<br />
	AfxMessageBox(tmp);<br />
	tmp.Format(_T("sscanf is %f"), boxval);<br />
	AfxMessageBox(tmp);<br />
<br />
<br />
	*pResult = 0;<br />
        }

Both Message boxes display the same value 8.0000.
This function is being called every time the spinner is clicked on.
I'm lost now! What am I doing wrong?

Bob Ciora26-Jan-05 8:03

26-Jan-05 8:03

Try putting a breakpoint at the start of the function, then single step through it (with F10). Use your Watch window to watch the values of tmp, txtEdit, boxval, and d as each line is executed. Validate the values at each step, and make sure you see the following:

  (code)                                 (output)

  CString tmp, txtEdit="82.45";          // tmp = "", txtEdit = "82.45"
  float boxval;                          // boxval is undefined at this point

  char *txt = (char *)(LPCTSTR)txtEdit;  // txt = "82.45"
  sscanf(txt, "%f", &boxval);            // boxval = 82.45
  double d = atof(txt);                  // d = 82.45

  tmp.Format(_T("atof is %f", d);        // what is tmp here?
// AfxMessageBox(tmp);                   // don't need this for testing
  tmp.Format(_T("sscanf is %f", boxval); // what is tmp here?
//  AfxMessageBox(tmp);                  // single step is your friend!

Definitely a stumper, but it's definitely worth single stepping.

As a side note on programming practice, you should do the following:
const char * txt = (LPCTSTR)txtEdit; instead of your original char * txt = (char *)(LPCTSTR)txtEdit;. You may run into unintended consequences if you force a const to a non-const. It's not hurting you here, but it could bite you in bad places. I'm curious to know if you get a compiler warning from your original statement.

Bob Ciora

Member 166473326-Jan-05 8:26

26-Jan-05 8:26

Thanks again for your precious help!

So using the debugger I've now realised that the line char *txt = (char *)(LPCTSTR)txtEdit; is the one causing the problem. In the debbuger the value for txt is 8.00. Funny the message box was returning the right charactere when reading from the edit box.. D'Oh! | :doh:

I have tried using char * txt = (char *)(LPCTSTR)txtEdit; instead but it gives me the error cannot convert from 'const unsigned short *' to 'const char *'.

Member 166473326-Jan-05 8:28

26-Jan-05 8:28

Sorry tried using const char * txt = (LPCTSTR)txtEdit; and got the error! Sigh | :sigh:

Bob Ciora26-Jan-05 8:59

26-Jan-05 8:59

I didn't say that was the line causing the problem Wink | ;)

I was just touching on coding practice. So have you single-stepped through the code and checked the values of the variables after each line is executed?

Bob Ciora

Member 166473326-Jan-05 9:05

26-Jan-05 9:05

No no, I stepped throught the code and that's where the problem is!
I've tried atof and sscanf again defining the char instead of the Cstring and they work fine! Smile | :)

All I need to do now is copy my Cstring into a char, unfortunately const char * txt = (LPCTSTR)txtEdit; doesn't work! Frown | :(

Thanks again!!! Smile | :)

Bob Ciora26-Jan-05 9:21