HN Explorer

Ask HN: Help me refactor this code please (Ruby)

1 points by grover_hartmann ↗ HN
It's basically a vending machine simulation, I want to refactor it into two classes, a model (VendingMachine) and a Controller. I'm not sure what else to do. Please help.

https://gist.github.com/anonymous/a978dcd30383f3e618e5

6 comments

[ 3.2 ms ] story [ 25.2 ms ] thread
grover_hartmann
So I would like to rename VendingMachine to Controller and have a VendingMachine model where I move #vend and #refund to it.

I'm also not very happy with #menu being this large in LoC. What would you guys recommend?

BlackjackCF
Having a vending machine model where it interacts with the balance and price is a good call.

I would also think about renaming either your refund method or your refund variable. It gets a little confusing.

You should also think about the fact that your refund method's case statement doesn't have a default. That means if someone doesn't put in either "y" or "n", you're screwed. Same situation with menu. You should always code for the unhappy path, meaning that you should think through for when someone decides to screw with your program.

In your menu case statement, I don't understand why you can't just do this:

case input when 1 name = "Coffee" price = @items[:coffee] ... #end of case statement

vend(name, price).

That'll DRY up your code at least a little bit.

[deleted]
(comment deleted)
technion
Think about types. You call .to_i on the balance, so it can only initially be an integer. But you have items costing 1.50.