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

Saldo Sortierung inkorrekt #652

Closed
megabugman opened this issue Nov 11, 2016 · 1 comment
Closed

Saldo Sortierung inkorrekt #652

megabugman opened this issue Nov 11, 2016 · 1 comment
Labels
bug

Comments

@megabugman
Copy link

@megabugman megabugman commented Nov 11, 2016

wenn eine Aktie 2x am gleichen Tag gekauft wird, stimmt die Sortierung des Saldo nicht.
saldo
Erwartungsgemäß sollte 70,90 oben sein, da nach Datum sortiert wurde.
Dh wenn vom 3t letzten Saldo (100,40) der Kaufbetrag abgezogen wird (14,75) dann sollte nicht 70,90 rauskommen.

@buchen buchen added the bug label Nov 12, 2016
@buchen buchen added this to the 2016 Q3 Short-List milestone Nov 12, 2016
@buchen
Copy link
Owner

@buchen buchen commented Nov 13, 2016

Danke. Habe ich behoben. Das Problem ist, dass ich tatsächlich nur das Datum habe. Für die Berechnung des Saldos brauche ich aber eine stabile Sortierreihenfolge. Dafür habe ich noch Betrag und Typ hinzugenommen. Die sind aber in diesem Fall identisch. Jetzt habe ich noch den Hashcode hinzugenommen. Damit sollte es jetzt wirklich stabil sein.

@buchen buchen closed this Nov 13, 2016
buchen added a commit that referenced this issue Nov 13, 2016
The calculation of the balance needs a stable sort order. The initial
idea of date, amount, and type being "good enough" to separate transactions
turns out to be wrong as indicated by this issue. Including the hashCode
as last resort.

Issue: #652
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.