Problem med skoluppgift i Java.

jag tror att du missat att dra av summan på valör som du beräknar. dvs om du ska få tillbaks 255:- ocj beräknar hur många hundralappar du ska få tillbaks så är det två hundralappar. men dom två hundralapparna ska dras ifrån "tillbaka" innan du beräknar 50-lappar och så vidare.
hoppas jag inte lurat dig nu.

Skickades från m.sweclockers.com

Problemet är att 50 kr inte delas jämnt i 20 lappar.

krona = tillbaka % 20;
tia = krona / 10;

dvs efter tillbaka%20 kommer du inte alltid ha rätt summa att dela upp tiorna i de fall då en femtiolapp ska ges i retur.

hmmm, det blir ju någon sort nedräkning om jag förstår det hela rätt. men om du tar och använder krona istället för tillbaka.

hundraLapp = tillbaka / 100;
krona = krona % 100;
femtioLapp = krona / 50;
krona = krona % 50;
tjugoLapp = krona / 20;
krona = krona % 20;

om jag tänkt rätt nu

Skickades från m.sweclockers.com

orsaken är att du beräknar antalet tior på att du tar mod 20 på totala antal kronor du ska få tillbaks. dvs 280 mod 20 blir noll i rest. därför måste det räknas ned

Skickades från m.sweclockers.com

/*
* Klistra in, kompilera och kör
* Det här programmet kan använda alla valutor. Det är bara att modifiera listan i metoden getCashback();
*/

public class Machine {

public static class Money{
public Money(int unit, int value){
this.unit = unit;
this.value = value;
}

public int unit;
public int value;
}

/**
* @param args
*/

public Money[] getCashback(int value){
Money[] cashback = {
new Money(100,0),
new Money(50,0),
new Money(20,0),
new Money(10,0),
new Money(5,0)
};

for(int i=0; i<=cashback.length-1; i++){
cashback[i].value = value / cashback[i].unit;
value = value % cashback[i].unit;
}

return cashback;
}

public void printCashback(int paid, int total){
print(paid - total, getCashback(paid - total));
}

public void print(int back, Money[] cashback){
System.out.println("Tillbacka: " + back);

for(int i=0; i<cashback.length; i++){
System.out.print((i>0 ? "," : "") + cashback[i].value + ":" + cashback[i].unit);
}
}

public Machine(){

}

public static void main(String[] args) {

int total = 220;
int paid = 475;

Machine machine = new Machine();

machine.printCashback(paid,total);
}

}

Vet inte varför man återupplivar en 2 veckor gammal tråd men men.
Valutor är inte samma sak som valörer btw
Lite overkill struktur för något som tar någon minut att lösa med en loop som ger samma resultat.
Typ:

static int[] values = {500,100,50,20,10,5,1};
public static void main(String[] args) {
		int total = 220;
		int paid = 475;
		for(int i = 0, change = paid - total; i < values.length; i++, change= i != 0 ? change%values[i-1] : change){
			System.out.println(String.format("Change (%3d): %d", values[i], change/values[i]));
	}
}

Yeye hoppas tråden får förbli död nu iaf ;<

Fyra viktiga saker:

1- Tack är alltid välkommen ...
2 - Ny kunskap dör aldrig
3 - och viktigast .. din lösning skulle inte överleva ett enkelt MVC överblick ... den blandar logik med presentation
4 - igen ... tack är alltid välkommen

1. Inte min tråd, finns inget att tacka för.
2. Eh... ok?
3. Kul att du lärt dig en akronym, sen hur MVC kommer in i den här kontexten är för mig ett mysterium utan dess like. Du ska inte skapa lite Spring dependency injection, cache och transaktionshantering etc, maven integration med egen artifactory samt deploya hans växeluppgift till en fullfjädrad EE-container m.h.a.hudson samt få ut miljöspecifik config via puppet i en redundant zabbix-övervakad servermiljö medan du ändå håller på?
4. Fortfarande inte TS.-.-

Sen så stavas det "tillbaka" och inte "tillbacka" btw, samt klassmedlemmar skall inte vara public såvida de inte är konstanter

Puss sötis<3