Wechselgeld 5 besten Kombinationen

[Ein ehemaliger Benutzer]

@codinglea ich habe dir weiter oben aufgezeigt, wo fehler sind. hast du die schon behoben bekommen?

[Belli]

Die hat doch nun einen funktionierenden Code von WOB bekommen ...
In ihrem Thread wegen Vertauschung von Buchstaben in Strings war sie mit der gleichen Situation (Code kam von Wutz) dann zufrieden, behauptete später auch, sie hätte den Code verstanden, obwohl der Code dort nur einen Teil ihrer Anforderung erfüllte, einen anderen, wesentlichen Teil aber nicht!

Das lässt meiner Meinung nach auf mangelnde Eigeninitiative oder mangelndes Interesse am programmieren allgemein schließen.

[codinglea]

@Wade1234
Ja, habe ich.

[Ein ehemaliger Benutzer]

@codinglea
dann müsste es jetzt ja eigentlich laufen, sofern du keine beträge wie 10050 o.ä. eingibst, weil sich das programm da erst einmal aufhängt.

mögliche verbesserungen wären jetzt erst einmal folgende:

• jeweils ein einzelnes printf für ein ergebnis mit verständlichem ausgabestring, d.h. printf("Anzahl 500 Euro: %i\n", a);, printf("Anzahl 200 Euro: %i\n", b); usw. und immer einen zeilenumbruch dazwischen.
• verständliche variablennamen, d.h. statt a, ... , o und max_a, ... , max_o schreibst du bspw. anzahl500euro, ..., anzahl1cent und max_anzahl500euro, ... , max_anzahl1cent. du kannst dafür übrigens "suchen und ersetzen" verwenden: dazu drückst du strg-h und gibst dann ein, welcher text durch welchen text ersetzt werden soll. vorsicht: "alle ersetzen" kann schief gehen.
• verwendung von konstanten: statt int o_wert = 1;, ... , int a_wert = 50000; schreibst du #define CENT1 1, ... , #define EURO500 50000

achja und den code dann bitte!

[Ein ehemaliger Benutzer]

@Belli sagte in Wechselgeld 5 besten Kombinationen:

Das lässt meiner Meinung nach auf mangelnde Eigeninitiative oder mangelndes Interesse am programmieren allgemein schließen.

oder auf fehlende sozialkompetenz von euch. ihr seid alle teamleiter in irgendwelchen unternehmen, ja?

[Belli]

@Wade1234 sagte in Wechselgeld 5 besten Kombinationen:

@codinglea
dann müsste es jetzt ja eigentlich laufen, sofern du keine beträge wie 10050 o.ä. eingibst, weil sich das programm da erst einmal aufhängt.
...
achja und den code dann bitte!

Du willst jetzt nicht allen Ernstes diesen Code mit den zig verschachtelten Schleifen weiter voran treiben, nachdem WOB hier eine kurze knackige allgemeine Lösung gepostet hat?

[Ein ehemaliger Benutzer]

@Belli sagte in Wechselgeld 5 besten Kombinationen:

@Wade1234 sagte in Wechselgeld 5 besten Kombinationen:

@codinglea
dann müsste es jetzt ja eigentlich laufen, sofern du keine beträge wie 10050 o.ä. eingibst, weil sich das programm da erst einmal aufhängt.
...
achja und den code dann bitte!

Du willst jetzt nicht allen Ernstes diesen Code mit den zig verschachtelten Schleifen weiter voran treiben, nachdem WOB hier eine kurze knackige allgemeine Lösung gepostet hat?

um ehrlich zu sein: doch! und zwar einfach deshalb, weil sie sonst nichts lernt.

[codinglea]

@Wade1234
Danke!!!

[Belli]

Warum glaubst Du, kann sie aus dem Code von WOB nichts lernen?
Da ist Rekursion drin - wo sie bekanntermaßen Schwierigkeiten hat - und vor allen Dingen kann man da wunderschön dran sehen, wie leicht man es sich und anderen machen kann, wenn man seine Variablen vernünftig benennt.

[wob]

@codinglea sagte in Wechselgeld 5 besten Kombinationen:

@Wade1234
Danke!!!

Sorry, aber warum zur Hölle willst du diesen unleserlichen Code, den die Wade einreden will, weiter anschauen und benutzen?! Sobald du nur eine Sache dort ändern willst, wie zum Beispiel einen neuen Geldschein einführen oder einen bisherigen löschen willst, musst du ungeheuer viel Code ändern.

Hast du wenigstens mal versucht, meine gepostete Lösung zu verstehen? Ich sage jetzt nicht, dass sie das nonplusultra ist, aber du kannst dort wenigstens beliebig die verfügbaren Scheine angeben und bekommst dann, ohne weiteren Code ändern zu müssen, alle Lösungen.

[codinglea]

@wob
Ich habe es versucht...
Aber ich verstehe ihn nicht komplett und will ihn nicht einfach übernehmen.

[wob]

@codinglea sagte in Wechselgeld 5 besten Kombinationen:

Aber ich verstehe ihn nicht komplett und will ihn nicht einfach übernehmen.

Dann wäre doch die Frage, wo du in dem Code Probleme hast.

[Ein ehemaliger Benutzer]

@Belli sagte in Wechselgeld 5 besten Kombinationen:

Warum glaubst Du, kann sie aus dem Code von WOB nichts lernen?

weil das noch zu viel ist und weil da z.b. keine kommentare und auch sonst keine erklärungen bei waren. einfach nur "hier haste den code, den kannst du dann mit copy & paste in dein projekt einfügen!" und sowas ist einfach schlecht.

außerdem ist das hier falsch:

    const int scheine_size = sizeof(scheine)/sizeof(scheine[0]);
    int auswahl[scheine_size];

unter c++ geht das soweit ich weiß, aber unter c macht man sowas nicht!

Da ist Rekursion drin - wo sie bekanntermaßen Schwierigkeiten hat - und vor allen Dingen kann man da wunderschön dran sehen, wie leicht man es sich und anderen machen kann, wenn man seine Variablen vernünftig benennt.

ich habe zuerst darauf hingearbeitet, dass da erst einmal ein grundverständnis für iterative lösungsfindung vorhanden ist und ein funktionierendes programm entsteht, und jetzt arbeite ich darauf hin, dass das programm etwas verständlicher und leichter lesbar wird.

[wob]

@Wade1234 sagte in Wechselgeld 5 besten Kombinationen:

außerdem ist das hier falsch: (...)
unter c++ geht das soweit ich weiß, aber unter c macht man sowas nicht!

Weil? Das 2. Array ist genauso lang wie das erste. Es ist ja nicht so, dass scheine_size eine Variable von außen wäre.

Und unter C++ würde ich das gerade nicht so machen, sondern ein std::vector oder ein std::array nehmen.

[Ein ehemaliger Benutzer]

@wob sagte in Wechselgeld 5 besten Kombinationen:

@Wade1234 sagte in Wechselgeld 5 besten Kombinationen:

außerdem ist das hier falsch: (...)
unter c++ geht das soweit ich weiß, aber unter c macht man sowas nicht!

Weil? Das 2. Array ist genauso lang wie das erste. Es ist ja nicht so, dass scheine_size eine Variable von außen wäre.

Und unter C++ würde ich das gerade nicht so machen, sondern ein std::vector oder ein std::array nehmen.

naja du erzeugst da ein dynamisches array ("variable length array") und dynamisch ist eben fast immer doof. also was ich meine ist eben, dass du damit unnötig overhead produzierst, weil der speicher eben zur laufzeit angefordert wird und nicht während des copiliervorgangs.

[TGGC]

@Wade1234 Du laberst Unsinn! Auch iterativ kann man das Problem in paar Zeilen lösen und nicht mit deiner Schrott-15fach Verschachtelung. Reine Zeitverschwendung sich mit dem Ansatz zu beschäftigen, da kein normaler Mensch das als "Lösung" ansieht und man daraus auch keine Lösung ableiten kann.

[Ein ehemaliger Benutzer]

@TGGC sagte in Wechselgeld 5 besten Kombinationen:

@Wade1234 Du laberst Unsinn! Auch iterativ kann man das Problem in paar Zeilen lösen und nicht mit deiner Schrott-15fach Verschachtelung. Reine Zeitverschwendung sich mit dem Ansatz zu beschäftigen, da kein normaler Mensch das als "Lösung" ansieht und man daraus auch keine Lösung ableiten kann.

ja vielleicht, aber DU hast jedenfalls in den mittlerweile 170 beiträgen außer dümmlichen pöbeleien überhaupt nichts beigetragen - so wie jetzt ja auch schon wieder.

[Belli]

@Wade1234
Ich finde Deinen Einsatz und Deine Hilfsbereitschaft super, wertvoll für jeden Anfänger, wirklich!
Leider bist Du selbst aber ein bisschen lernunwillig und beratungsresistent, das finde ich dann wieder nicht so gut ...

[Ein ehemaliger Benutzer]

@Belli es sagt ja niemand, dass nicht auch noch eine rekursive lösung erarbeitet werden kann, aber zuerst einmal sollte dafür die iterative lösung funktionieren. unübersichtlichkeit und mangelende praxisrelevanz hin oder her!

[Ein ehemaliger Benutzer]

OT:

Es ist grandios sich diesen Thread durchzulesen...
popcorn faktor sondergleichen