[SOLVED] Emm Wtf??? What's wrong??

Question

fun shippingCost(amount: Double, international: Boolean): Double {

var x = 0
when {
international == true && amount.toInt() <= 50 -> var x = amount.toInt() * 15 / 100
international == false && amount.toInt() < 75 -> var x = amount.toInt() * 10 / 100
international == false && amount.toInt() >= 75 -> var x = amount
international == true && amount.toInt() > 50 -> println("Invalid")
}
return (amount + x).toDouble()
}
fun main(args: Array<String>) {
val total = readLine()!!.toDouble()
val international = readLine()!!.toBoolean()
println(shippingCost(total, international))

}

It supose I must find the price of the orders, with a 10% for import add to total price, that's for < 75$ and 75+ is for free, so that's no international, in case to be international, the import is 15% with a max of 50$

Answer 1

Yes, this line:
var x = 0
is a declaration for an integer!
You have to use:
var x = 0.0
That's a double type

Answer 2

Coding Santa Thanks you so much for your help bro 👍👍👍 it's alright, but there's one case wrong, and i dont have any idea 🤣🤣 but don't worry I'll figure it out, thanks you so much 👌👍

Answer 3

Oooooooo 🤦🏻‍♂️🤦🏻‍♂️🤦🏻‍♂️🤦🏻‍♂️🤦🏻‍♂️🤦🏻‍♂️

Answer 4

You should not calculate with Integer and convert the return value to double. This way you will lose the dezimals.
And you are using much code for this.
As a hint, you can have look at this (AFTER your version is working)

https://code.sololearn.com/c6f3aa6fQAIX/?ref=app

Answer 5

Your func declaration is:

fun shippingCost(amount: Double,....

That's ok.
But then you are always using:
amount.toInt() or a.toInt() for your calculation. This way, the following will be happen: If the input for amount was originaly 23.45 you make 23 from that and calculate with this wrong value. The dezimals are cutted off.
And even if you convert the result to double, it's wrong.

Answer 6

Yes, Agnes Ahlberg is right.
One more hint: There is no need to convert anything in the whole code. You starts with double and boolean at the input. Use this for any calculation and return / print a double.
And the correct multiplier for 15% is 0.15. Because we are only intrested on the shipping cost portion.

Answer 7

Thanks you so much Coding Santa I'll try it man

Answer 8

fun shippingCost(a: Double, i: Boolean): Double {

var x = 0

when {
i == false && (a.toInt()) < 75 -> x = ((a.toInt())*10/100)
i == false && (a.toInt()) >= 75 -> x = 0
else -> x = a.toInt()*15/100 }

return x.toDouble();
}
fun main(args: Array<String>) {
val total = readLine()!!.toDouble()
val international = readLine()!!.toBoolean()
println(shippingCost(total, international))

}

I fix it a little, but I don't understand why it say error, I mean, I use a calculator, and when you find 15% of 199.9 is 29.985, but when the Code Playground does, it say 29, so I don't know what's happening :(

Answer 9

Some languages use integer division if only integers are involved 🤓 Try making something a Double, like 15.0/100, or something language appropriate 🍀🙂

Answer 10

It's just the Code Playground says error when I try to use the Double to the calc, i set it like this: a*15.0/100.0 but it says that an Int was expected 😐

Answer 11

You have to watch out for types. A Double is not an Int.

If you do:
var x = 0

Then x is of type Int. Then you cannot assign Double in the when.

When you do:
var x = 0.0
Then x is of type Double. Then you cannot assign x = 0 in the when block.

Check that all your types are correct and match throughout 🙂

I will leave it to Coding Santa now to help you out so that things don't get messy with too many cooks 🙂🍀

Answer 12

Ok... Coding Santa Agnes Ahlberg I'll try with 0.15

Answer 13

Coding Santa I don't know what to do man 🥲 look at this:

fun shippingCost(a: Double, i: Boolean): Double {

var x = 0

when {
i == false && a < 75.0 -> x = a*0.10
i == false && a >= 75.0 -> x = a*0.0
else -> x = a*0.15 }

return x
}

But it don't works, it says the same thing, that Int was expected 😐 They want i calcule with Int but then I'll lose the decimals so the result will be wrong 😐 pls help me and thanks for be pacient 🤣 I'm to slow for learn

Answer 14

Coding Santa Creo que ví x ahí algo de multiplicar x 1.15, que me dices?

I mean, I could use that? (i don't remember where I saw it)

Answer 15

I tried to use another variable, that use amount as a Integer, but it don't works :'(