sandboxgangster / Part-DB

Electronic Parts Database
24 stars 10 forks source link

Fix für #26: Intern jetzt 4 Nachkommastellen #40

Closed susnux closed 9 years ago

susnux commented 9 years ago

Intern wird hiermit dann mit 4 Nachkommastellen gearbeitet ( #26) und als default werden 3 angezeigt (ich denke ein guter Kompromiss, 4 brauchen die meisten nicht immer und könnten die Lesbarkeit verringern, 2 hingegen rundet zu stark, 3 scheint ganz gut). Außer dem habe ich in der Doku den Link auf 0.3.1 angepasst.

ubruhin commented 9 years ago

Danke! Ich hätte noch folgende Anmerkung:

Die Änderung vom money format string gefällt mir irgendwie nicht so. Erstens ist es schade, dass Zahlen mit nur 2 Nachkommastellen damit noch eine (unnötige) "0" angehängt bekommen, und bei Zahlen mit 4 Nachkommastellen die letzte Ziffer nicht sichtbar ist. Und Zweitens wird durch diese Änderung das Währungsformat des eingestellten Gebietsschemas (Locale) einfach überschrieben.

Wäre es nicht schöner,

Ich habe gerade noch gesehen, dass bei DigiKey sogar 5 Nachkommastellen verwendet werden. Wenn wir in Part-DB schon die Anzahl Nachkommastellen erhöhen, sollten wir vielleicht besser auch gleich auf 5 Stellen erhöhen?

susnux commented 9 years ago

Danke für dein Feedback! Ich sehe das auch so, vor allem das mit den 5 Nachkommastellen ist kein Problem, da die Gesamtzahl der digits um 3 gestiegen ist (ob 9 oder 8 macht vom Speicherverbrauch keinen unterschied also nimmt man 9) kann man die Nachkommastellen noch um eine erweitern ohne den maximalen Preis zu verringern.

Was das mit dem Format angeht, das sehe ich ein. Ich gucke mal wie man das lösen kann.

susnux commented 9 years ago

Hab die Funktion dementsprechend geändert (wie deine zweite Idee). Sollte kein "money format" gesetzt sein, wird es aus dem aktuellen Gebietsschema "gebaut":

PS: Irgendwie ist das komisch, weil die Dateien haben ein gemischtes Zeilenende, mache \n manche \n\r ... Daher der große git diff (ich habe "dos" Format genommen weil das in einer der READMEs stand).