Komplettering på uppgift / modulus / växel

Vad händer om du nu matar in så att du ska få tillbaka t.ex. 95 kr?

Är uppgiften specifikt för att använda modulus i slututräkningen? Användningen är intressant, men inte alls så som jag skulle löst det rent spontant (genom att räkna ner kvarvarande växel efter antalet av varje valör).

Ett övrigt tips är att se till att dina tester har bra täckning, som i detta fallet att testa att samtliga valörer räknas rätt (vilket borde kunna göras genom en enda körning med växel som innebär att man får tillbaka av varje valör). Nu har du nog fixat felet i dett orginalinlägg men antar att felet var att e.g. 10-kronan aldrig var testad

tusenlapp = växel / 1000;
             femhundralapp = (växel % 1000) / 500;
             hundralapp = (växel % 500) / 100;
             femtiolapp = (växel % 100) / 50;
             tjugolapp = (25 % 50) / 20;
             tiokrona = (5 % 20) / 10;
             femkrona = (växel % 10) / 5;
             enkrona = växel % 5;

Nej, det är inte detaljerna som är fel utan hela metoden Du måste som sagt ta hänsyn till hur mycket växel som faktiskt är kvar, d.v.s. du behöver uppdatera växel efter varje steg.

tusenlapp = växel / 1000;
             femhundralapp = (växel % 1000) / 500;
             hundralapp = (växel % 500) / 100;
             femtiolapp = (växel % 100) / 50;
             tjugolapp = (25 % 50) / 20;
             tiokrona = (5 % 20) / 10;
             femkrona = (växel % 10) / 5;
             enkrona = växel % 5;

Nej. Du behöver hela tiden hålla reda på hur mycket växel som återstår.
Själv skulle jag gjort nåt i stil med

  tusenlapp = växel/1000;
  växel = växel % 1000;
  femhundralapp = växel/500;
  växel = växel % 500;
  hundralapp = växel/100;

Osv.

För mig så gör modulos ont i huvudet för detta

Modulos visar ju "resten" dvs det som blev kvar när du kollade valören..... Jag hade tittat på liknande mönster

tusenlapp = ( växel / 1000);
växel = växel - (tusenlapp * 1000);

femhundralapp = (växel / 500);
växel = växel - (femhundralapp*500);

hundralapp = (växel / 100);
växel = växel - (hundralapp*100);

Jag föredrar lösningar som är rakt på och inte försvårar det hela

Då dina "lappar" är definierade som INT så kan de endast bli heltal, vilket passar bra i detta läge

mvh Lazze

Det är korrekt, och betyder exakt samma sak som

tusenlapp = (växel/1000);
växel = växel % 1000;

(förutsatt heltalsaritmetik vid division, annars blir bägge varianterna fel)

Det senare är dock lite kortare och enklare att skriva.

IMHO så är min kod mycket enklare att läsa för någon som inte sett den tidigare.

Tror det ända egentliga problemet med lösningen du lämnade in sker när en valör inte är jämt delbar med nästa.

Ta t.ex. 60kr växel
50lapp:
(60%100)=60 -> korrekt
(60%100)/50=1 -> korrekt, en femtiolapp

20lapp:
(60%50)=10 ->korrekt, vi ska ha kvar 10 efter att ha dragit 50-lappen
(60%50)/20=0 -> korrekt, vi ska inte få någon 20-lapp

10krona:
(60%20)=0 -> fel, vi har ju dragit en 50-lapp och borde ha en rest på 10 efter det, men räknar fortfarande bara bort 20-lappar från det ursprungliga talet, som vi här verkar anta att vi redan gett tillbaka 3 av
(60%20)/10=0 -> STUPID MACHINE STOLE MA MONEYS!

Du kan komma runt det genom att successivt räkna ner växel-variablen (som andra föreslagit, och som jag också spontant hade löst det, då kan du också undvika att använda modulus övh. om du vill). Men du borde fortfarande kunna använda modulus även så som du gjort utan att räkna ner växel-variablen, men du måste göra flera stycken modulus för 10-kronan.

E.g.

10krona:
(60%50%20)=(10%20)=10 -> korrekt, 10 kronor kvar efter att vi dragit 50- och 20-lappar
(60%50%20)/10=1 -> korrekt, det motsvarar en 10-krona

Gjorde det verkligen det? Resten efter 100lappar dragits blir 89kr. Det borde bli 1 50-lapp, 1 20-lapp, 1 10-krona, 1 5-krona, 4 1-kronor.

89/50=1 <- japp, en femtiolapp
(89%50)/20=1 <- japp, en tjuga
(89%20)/10=0 <- nope, fel

(89%50%20)/10 = (39%20)/10 = 19/10 = 1 <- korrekt, en tia