Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make decimal number parsing more resilient #176

Open
cmb69 opened this issue Aug 13, 2017 · 0 comments
Open

Make decimal number parsing more resilient #176

cmb69 opened this issue Aug 13, 2017 · 0 comments

Comments

@cmb69
Copy link
Member

cmb69 commented Aug 13, 2017

Ich habe gerade heute gelernt, dass eigentlich nur zwei Dezimaltrennzeichen weit verbreitet sind, nämlich . und ,. Selbst PHPs number_format() verlässt sich darauf. Daher wäre es, entgegen meiner bisherigen Bedenken, grundsätzlich doch möglich, bezüglich der Eingabe von Dezimalzahlen auch Dezimalkommata zu berücksichtigen. Das einfache str_replace(',', '.', $number), wie bei Wellrad zu finden, wäre mir zu lax, da 1.000,00 damit zu 1.000.00, und später zu 1.00 konvertiert würde. Aber solche potentiellen Tausendertrennzeichen könnte man explizit berücksichtigen.

Es sollte eigentlich nicht aufwendig sein, alles so umzuprogrammieren, dass:

  • bei der Eingabe als Dezimaltrennzeichen sowohl . wie , erlaubt ist, und Tausendertrennzeichen einfach ignoriert werden
  • bei der Ausgabe immer eine Dezimalzahl mit den sprach-konfigurierten Trennzeichen erzeugt wird

Allerdings sind wir schon in der RC-Phase, und da ist es wohl besser alles zu lassen wie es ist, es sei denn, die Probleme mit Dezimalzahlen reißen nicht ab.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant