@sdetweil said in MMM-Cryptocurrency:
@tippon has nothing to do w javascript
how do u get from the text value to the proper number?
write the steps down
I’m not sure. I know it’s multiplying my number of coins from the wallet section in config.js by the price result from the API, and it’s rounding the figures to the number of decimal points I specified, but I’m getting lost.
I think I’ve spotted the problem now though, and it seems to be working. These two lines:
var myPrice=currentCurrency.price.replace("£","");
var myPrice=myPrice.replace(".", "").replace(",", ".");
strip out the pound sign, then the decimal point, then change the thousands separator to a decimal point.
Removing the pound sign is at least part of what changes the amount from text to a number (sorry, I’m figuring it out as I’m typing), but removing the decimal point for the low value coins under £1 multiplies their value by 10. e.g. Doge goes from 0.039 to 0039, or 39p. Swapping the thousands separator to a decimal point does the opposite for the large value coins, turning Bitcoin’s £43,000 into £43.000, or £43. I’ve changed it to this:
var myPrice=currentCurrency.price.replace("£","");
var myPrice=myPrice.replace(",", "");
Now the pound symbol is still removed, so the coin’s value is still changed from text to a number, and the thousands separator is removed for the same reason, but without changing the value. I think that the original contributor was European, so uses dots instead of commas for number separation, and changing to GBP messed that up.
@sdetweil said in MMM-Cryptocurrency:
@sdetweil get rid of leading no decimal
remove thousands separator
if it contains the decimal separator, use string.parseFloat, otherwise use string.parseInt
once u have the right numbers it should work
Now that my brain’s caught up with what you were saying, I think I’ve got it right. I have no idea how to use string.parseFloat or string.parseInt though, and no idea how to check which numbers the API sends. That can be my project for tomorrow though, it’s 2am here, and my brain is melting :beaming_face_with_smiling_eyes:
Thank you for helping me, and giving me a shove in the right direction :thumbs_up: :slightly_smiling_face: