converting from float or double to QString results in output being 0

[Dean21]

@Kent-Dorfman I did try #define amplfication_multiplier 10.0 instead of #define amplification_multiplier 10, but it made no difference to the final output

[Dean21]

@jsulm I tried below is the line of code I used, if I used the line you mentioned i get a load of different errors so I removed the QChar part

msg_as_Qstring.remove(("[\u0000]."));

but it still has it attached on the end here is the output

Topic:  "Light_Level" , Msg:  "5.73628\u0000"
test value is "5.73628\u0000"
Qstring as float  "5.73628\u0000"
before amp 0
amp multiplier 10
msg as qstring to double 0
msg as qstring conv "0"

[JonB]

@Dean21 said in converting from float or double to QString results in output being 0:

msg_as_Qstring.remove(("[\u0000]."));

How/what do you expect it to match with that . at the end?

Just chop off the last character, no regular expression, no? Did you find void QString::chop(qsizetype n) ?

If you want to find out whether this will make any difference try

msg_as_Qstring = "5.73628";

first and then you will know!

[JonB]

@Kent-Dorfman said in converting from float or double to QString results in output being 0:

so what is (float)5.7 / (int)10?

Umm, that would be 0.57(not 0).... ! Dividing a float by an int does not convert to int! :)

[Kent-Dorfman]

I stand corrected. I was certain the type of the denominator governed the result. Too much time in embedded kernel domain. D'Oh!

[jsulm]

@Dean21 said in converting from float or double to QString results in output being 0:

msg_as_Qstring.remove(("[\u0000]."));

Why did you put [,] and dot there?
I suggested this:

msg_as_Qstring.remove(QChar("\u0000"));

[Kent-Dorfman]

if in fact the conversion is being broken by the unicode null terminator then consider an intermediate conversion thru (char*) instead.

QString num(byteArray.data());

since char arrays have no defined length the constructor shoudl use the null as a string terminator, not as part of the string (as is the case when converting from a container that has an internal size field).

[Dean21]

@jsulm sorry the "." was a type but i did try both methods and get a lot of errors about invalid conversion

[JonB]

@Dean21
Already said. Don't know why you don't do the following (takes 10 seconds):

1. Try msg_as_Qstring = "5.73628"; and see if it works now. If it does
2. msg_as_Qstring.chop(1);.

Then you will know whether that is the issue....

[Dean21]

@JonB I just did sorry didnt reply sooner I had to wait this worked great, thank you for your help and everyone else who helped me with this

[JonB]

@Dean21
Glad it's working for you now. That's the steps I would have taken to get it working.

@Kent-Dorfman said earlier

I question whether removing non-numeric char at end of QSTring is necessary for the numeric conversion problem since all the conversion code I'm familiar with scans digits until it finds an char that can't be part of the number, and then returns everything up to that point.

But, for whatever reason, conversions to numeric from QString state:

Warning: The QString content may only contain valid numerical characters which includes the plus/minus sign, the character e used in scientific notation, and the decimal point. Including the unit or additional characters leads to a conversion error.

So that's how they do it --- all characters in the string must be valid. You could use:

bool ok;
double val = msg_as_Qstring.toDouble(&ok);
if (!ok)
    qDebug() << "Conversion failed!";

[Dean21]

@JonB I see that makes sense, thanks for the additional info and help :)

[Kent-Dorfman]

@JonB said in converting from float or double to QString results in output being 0:

Warning: The QString content may only contain valid numerical characters which includes the plus/minus sign, the character e used in scientific notation, and the decimal point. Including the unit or additional characters leads to a conversion error.

In which case RTFM is the rule of the day! but I'm lazy.