MihutBalasanu / VM

0 stars 0 forks source link

Make payment method #4

Open nicolaeabacioaiei opened 6 years ago

nicolaeabacioaiei commented 6 years ago

https://github.com/MihutBalasanu/VendingMachine/blob/a81ba569d48e6805a15cc6bc71f6f6d133b31b9f/20181021_VendingMachine/src/VendingMachine.java#L85

Aici e bine. Pe final nu vad de ce ai avea nevoie sa apelezi din nou displayPayment(). Vad ca ce se introduce le tii in coinStockVM. Eu as sugera sa aduni efectiv suma intr-o variabila si gata. Nu are sens sa tii minte si ce fel de monezi a introdus. Te intereseaza doar valoarea totala. Iar daca trebuie sa ii dai valoarea aia inapoi pur si simplu o sa apelezi metoda de a da rest pentru valoarea respectiva.

nicolaeabacioaiei commented 6 years ago

Ar trebui sa ai un singur stock pentru monezi care sa fie coinStock. Nu ai nevoie sa tii pentru buyer si pentru VM separat. Te intereseaza doar cel al vm-ului. Ce introduce utilizatorul o sa tii intr-o variabila..unde te intereseaza doar suma totala introdusa