Arduino-Function merkwürdig

[Frank E. (Firma: Q3) (qualidat)]

Die folgende Funktion soll die Anzahl der zurückzugebenden Münzen in 
einem Münzwechsler auf Basis MK4 ausrechnen. Übergeben wird der Betrag 
in Cent, zurückgegeben wird ein String, der durch Punkte getrennt die 
Anzahl der 2€ und 1€ und 50ct-Münzen enthält. Vorbedingung: Der Betrag 
muss durch 50ct teilbar sein. Problem: Die Funktion liefert nur bis 
950ct brauchbare Ergebnisse, ab 1050 kommt "error". Aber warum? Danke 
für Tips.

String mk4_sumout(int sum)
{
  String ret = "error";
  int mod = sum % 50;

  if(mod == 0)
  {
    int c2 = 0;
    int c1 = 0;
    int c5 = 0;
    while(sum>0)
    {
      bool fnd = false;
      if(!fnd && sum>=200){sum -=200;c2++; fnd=true;}
      if(!fnd && sum>=100){sum -=100;c1++; fnd=true;}
      if(!fnd && sum>=50){sum  -=50; c5++;  fnd=true;}
    }
    ret = String(c2)+"."+String(c1)+"."+String(c5);
  }
  return ret+"."+String(mod);
}

Vielleicht bin ich auh nur ein wenig überarbeitet ... :-(

[Rolf M. (rmagnus)]

Frank E. schrieb:
> Problem: Die Funktion liefert nur bis 950ct brauchbare Ergebnisse, ab 1050
> kommt "error".

Und bei 1000?

error kann ja nur kommen, wenn diese Bedingung nicht gilt:

> int mod = sum % 50;
>   if(mod == 0)

Ich sehe keinen Grund, warum das bei 1050 passieren sollte. Bist du denn 
sicher, dass deine Eingabe korrekt funktioniert? Vielleicht nimmst du 
nur 3 Zeichen entgegen, und es steht nur 105 drin statt 1050 oder so.

[Diek (Gast)]

Was liefert das denn?

return ret+"."+String(mod);

[cppbert (Gast)]

debug doch einfach durch?

nur so als Tip geht auch ohne Schleife

    int sum = 1050;  
    int c2 = sum / 200;
    sum = sum % 200;
    int c1 = sum / 100;
    sum = sum % 100;
    int c5 = sum / 50;

[donvido (Gast)]

Also das hier funktioniert wohl so wie es soll:

Zumindest hier: https://www.onlinegdb.com/online_c++_compiler#

#include <iostream>
#include <string>

using namespace std;

string mk4_sumout(int sum){

  string ret = "error";
  int mod = sum % 50;
  if(mod == 0)

  {

    int c2 = 0;
    int c1 = 0;
    int c5 = 0;
    
    while(sum>0)
    {
      bool fnd = false;
      if(!fnd && sum>=200){sum -=200;c2++; fnd=true;}
      if(!fnd && sum>=100){sum -=100;c1++; fnd=true;}
      if(!fnd && sum>=50){sum  -=50; c5++;  fnd=true;}
    }
    ret = to_string(c2)+"."+to_string(c1)+"."+to_string(c5);
  }

  return ret+"."+to_string(mod);

}

int main()
{
    cout<<mk4_sumout(1050);
    return 0;
}

[Peter D. (peda)]

int mod = sum % 50;
sum /= 50;
int c5 = sum & 1;
sum /= 2;
int c1 = sum & 1;
int c2 = sum / 2;

[Geldwexler (Gast)]

Das tät mich ja anpesten wenn das Ding fast nur 2er ausspuckt, und max. 
einen 1er und einen halben.
Das sollte schon von der Verfügbarkeit abhängen und allgemein ein bissel 
gemischt sein.

[Rolf M. (rmagnus)]

Geldwexler schrieb:
> Das tät mich ja anpesten wenn das Ding fast nur 2er ausspuckt, und max.
> einen 1er und einen halben.

Warum? Ich würde mich darüber freuen, nicht unnötig viele Münzen als 
Wechselgeld zu bekommen.

[Frank E. (Firma: Q3) (qualidat)]

Rolf M. schrieb:

> Ich sehe keinen Grund, warum das bei 1050 passieren sollte. Bist du denn
> sicher, dass deine Eingabe korrekt funktioniert? Vielleicht nimmst du
> nur 3 Zeichen entgegen, und es steht nur 105 drin statt 1050 oder so.

DANKE! Genau das war die Ursache: Der übergebene Wert stammt aus einem 
URL-Parameter eines GET-Statements. Und genau hier wurde wohl durch eine 
ungeschickte String-Operation die Null hinten abgeschnitten. DANKE!

[Frank E. (Firma: Q3) (qualidat)]

Geldwexler schrieb:
> Das tät mich ja anpesten wenn das Ding fast nur 2er ausspuckt, und
> max.
> einen 1er und einen halben.
> Das sollte schon von der Verfügbarkeit abhängen und allgemein ein bissel
> gemischt sein.

Ja, daran wird gearbeitet.

Weniger deshalb, weil es irgend jemanden "anpisst", sondern um die 
Vorräte an Münzen möglichst gleichmäßig zu nutzen. Der erste Schritt ist 
der, bei Rückgabebeträgen unter 5 Euro auf die 2er zu verzichten. 
Ähnlich könnte man es mit den 1ern halten, wenn die Rückgabesumme z.B. 
kleiner 2.50 Euro ist ...