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

Transaktionskosten in "Bezahlte Entgelte" einbeziehen #149

Merged
merged 2 commits into from Jun 27, 2014

Conversation

Projects
None yet
2 participants
@nistude

nistude commented Jun 21, 2014

Bin mir nicht sicher, ob das gewollt ist, aber ich fänd's hilfreich die Transaktionskosten in der Performance-Berechnung bei den "Bezahlte Entgelte" einbezogen zu haben. HTH.

@buchen

This comment has been minimized.

Owner

buchen commented Jun 22, 2014

Den Wunsch habe ich auf jeden Fall schon mehrfach gelesen. :-)

In der Performance-Berechnung gilt momentan:

Anfangswert
plus Kurserfolge
plus Erträge
minus Entgelte
minus Steuern
plus Performance neutrale Bewegungen (=Einlagen-Entnahmen)
= Endwert

Mit Deiner Änderung gilt das jetzt nicht mehr.
Die Entgelte z.B. von einem Verkauf tauchen doppelt auf: einmal weil der Kurserfolg durch die Gebühren reduziert wird, und einmal bei den Entgelten selbst. Das gleiche gilt für Gebühren bei Einlieferungen, etc.

Jetzt könnte man natürlich die Gebühren aus den anderen Positionen rausrechnen. Der Kurserfolg würde sich dann um die Kauf und Verkaufsgebühren vergrössern. Das habe ich bisher nicht gemacht weil auch Einstandswert und Marktwert in der Vermögensübersicht die Gebühren enthalten. Und da gehören sie m.M.n. dazu. Und die Werte zwischen den Ansichten müssen stimmig sein - ansonsten zweifelt man schnell an allen Rechnungen.

Alternativ könnte man auch einen Punkt "Transaktionskosten" unter "Kurserfolge" hinzufügen. Da werden dann die "enthaltenen Gebühren" angezeigt. Einfacher zu erweitern. Aber kein einzelner Punkt "Gebühren".

Wie siehst Du das @nistude ?

  1. Egal?
  2. Gebühren aus anderen Positionen rausrechnen?
  3. Separater Punkt für solche "enthaltene Gebühren"?
@nistude

This comment has been minimized.

nistude commented Jun 23, 2014

Ich kann Deinen Punkt leider nicht nachvollziehen, @buchen. Mit den Daten meines Portfolios ergeben sich keine erkennbaren Unterschiede zwischen den beiden Programmversionen (bei den vielen Zahlen hab ich's aber evtl. auch einfach nur übersehen). Wo im Code würde mein Diff denn dafür sorgen, daß die Gebühren doppelt berücksichtigt würden?

@buchen

This comment has been minimized.

Owner

buchen commented Jun 23, 2014

In dieser Beispieldatei habe ich eine Kauftransaktion mit 10 Euro Gebühren eingetragen. Die tauchen dann natürlich auch unter Gebühren auf (rot), reduzieren aber auch den Kurserfolg um 10 Euro (blau). Darum geht auch die Rechnung nicht mehr auf: 0 Anfangswert + 920 Kurserfolge - 10 Gebühren + 7758 Einlagen != 8678 Endwert

bildschirmfoto 2014-06-23 um 22 15 47

Im Prinzip entsteht das, weil unter #addCapitalGains im ClientPerformanceSnapshot Transaktion#getAmount gerufen wird - und die enthält die Gebühren.

Mit den Daten meines Portfolios ergeben sich keine erkennbaren Unterschiede zwischen den beiden Programmversionen

Vielleicht liegen die gebührenrelevanten Buchungen nicht im Berichtszeitraum? Andrerseits würdest Du dann ja keine Gebühren sehen...

@buchen buchen added the enhancement label Jun 23, 2014

@nistude

This comment has been minimized.

nistude commented Jun 24, 2014

Ok, jetzt kann ich's nachvollziehen. Dank Dir für das Beispiel.

Ich bin ein Freund einfacher, klarer Zahlen und plädiere von daher für die Anzeige reiner Kursgewinne ohne Kosten und der Extra-Anzeige der Kosten. Das erscheint mir grad angesichts der Begriffswahl klarer.

Wenn Du keine Einwände hast, dann würde ich die Tage (spätestens am Wochenende) schauen, daß ich den Code entsprechend angepaßt bekomme.

@buchen

This comment has been minimized.

Owner

buchen commented Jun 25, 2014

+1

Eventuell reicht es ja PortfolioTransaction#getLumpSumPrice aufzurufen?!?

@nistude

This comment has been minimized.

nistude commented Jun 25, 2014

Die "Performance-Berechnung" müßte jetzt passen. Danke für den Tip! Ich würde allerdings noch die "Wertpapier Performance" entsprechend anpassen wollen. Da würde ich die Kosten auch rausnehmen wollen. Was meinst Du?

@nistude

This comment has been minimized.

nistude commented Jun 25, 2014

Hab ich jetzt auch fix eingebaut. Das müßte in meinen Augen soweit konsistent sein.

@buchen

This comment has been minimized.

Owner

buchen commented Jun 26, 2014

Die ersten beiden Changes kann ich so mergen.

Bei dem Einstandswert/-kurs nach FIFO stimmt die Beschreibung dann nicht mehr. Allerdings sehe ich, dass die für den Kurs sowieso verrutscht ist. Und auch noch Rechtschreibfehler. :-( Das kann ich am WE fixen.

Dazu gibt es noch eine weitere Implementierung - die müsste man anpassen. Bei Gelegenheit will ich das auch mal refactorn - in der Vermögensaufstellung den SecuritPerformanceRecord zu verwenden.

bildschirmfoto 2014-06-26 um 06 31 13

Bei der Berechnung des Deltas möchte ich eigentlich den Wert inklusive der Gebühren sehen. Was nutzt es mir zu wissen, dass ich ohne Gebühren einen Gewinn gemacht hätte? Zumindest bei kleineren Positionen fällt das ins Gewicht, auch bei vielen (Ver-)Kaufsbuchungen. Und dann gilt das selbe ja für Steuern (meist auch nicht explizit erfassbar).

Man könnte eine weitere Spalte einfügen - dann hat der Benutzer die Wahl: mit oder ohne Gebühren.

bildschirmfoto 2014-06-26 um 06 31 55

@nistude

This comment has been minimized.

nistude commented Jun 26, 2014

Tut mir leid, da war ich zu schnell und hab nicht recht verstanden, was es mit den Posten auf sich hat. Nach Deiner Erklärung ist der Status Quo an der Stelle sinnvoll und ich ziehe 8252751 zurück.

buchen added a commit that referenced this pull request Jun 27, 2014

Merge pull request #149 from nistude/portfolio_transaction_fees
Include transactions costs into "paid fees"

@buchen buchen merged commit 8a4edb0 into buchen:master Jun 27, 2014

buchen added a commit that referenced this pull request Jun 29, 2014

@nistude nistude deleted the nistude:portfolio_transaction_fees branch Aug 5, 2014

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