Lesson 13: availableBorrowsETH.toString() * 0.95 * (1/daiPrice.toNumber())

[ManuWeb3]

Hi all,

In aaveBorrow.js script, what exactly is the purpose of appending .toString() and .toNumber() to availableBorrowsETH.toString() and 1/daiPrice.toNumber() ?

I ran the script without .toString() and .toNumber() and got the correct outputs.

So, what do these conversion functions: toString() and toNumber() do here ?

Script Github link:
https://github.com/PatrickAlphaC/hardhat-defi-fcc/blob/main/scripts/aaveBorrow.js

P.S. I understand the overall formulae, just do not know the reason of these conversions made.

[krakxn]

They serve to make the logic more definitive (i.e. conclusive) and as type-conversions are not computationally intensive, it does not hurt to use it as a "precaution" and hence, acts like a mini conditional.

An additional point to be made is that it so because we are using an external contract, but from my rationale the above is the primary reason as to why we do this.

[ManuWeb3]

Ok.
I just changed availableBorrowsETH.toString() to .toNumber() and it threw below error:

But, when I changed 1/daiPrice.toNumber() to .toString(), daiPrice did not throw any error.

Why is so ??

[krakxn]

There are specific reasons which vary in every other scenario, in this context is as so:

As evident from the above SS, it results in an overflow that is precisely why it was converted to a string. Similarly for the latter.

[ManuWeb3]

Ok, seems I'll better understand this nitty-gritty when I do a complete course on JS which I'm anyway going to start after this course.

thanks buddy.

[krakxn]

Good luck!