karelze / dhbw_mwi Goto Github PK

Mir sind mehrere Dinge aufgefallen:

1. Aktiver Tab wird bei Laden der Seite nicht gehighlighted. ☑️
2. Periode kann nach abschließen noch verändert werden. Ist es möglich den Tab auszugrauen?☑️
3. TransaktionsartId hat wert null bei festgeldverzinsung in der Buchungstabelle // erzeugt nullpointerexceptions bei wertpapier_verkaufen ☑️
4. Prüfen, dass Feld Manueller Kurs optional zu machen.:x:
5. Prüfen, ob berechneter Kurs angezeigt werden kann. ❓
6. Sicherstellen, dass Anwender einen Spread erfasst. Wenn Anwender keinen Wert explizit in den Spread eingibt, wird bei Spread null in die Datenbank eingetragen, das führt dazu, dass in Zeile 34 vom FloatingRateNoteModel eine NullPointerException fliegt. ☑️
7. Periode kann nach erneutem Aufruf wieder und wieder gepflegt werden. ❌
8. Inkonsistentes Layout.☑️
9. Überprüfung, ob Bewertungszeitpunkt für Floater und ETF korrekt ist.☑️
10. Manueller Kurs von z. B. 101 als 101 und nicht als 1.01 speichern. ☑️

Wertpapier hat keine Zuordnung zum Spiel, wodurch auch nicht zum Spiel gehörige Wertpapiere ausgewiesen werden im UI.

Eine Umsetzung analog Unternehmen ist notwendig.

Problem: bezugsgröße hat falschen Wert
z.B.: Kauf zu 10.000 €
Ausschüttung nächste Periode (5%): 10.000 * 5 % = 500 € (funktioniert)
Ausschüttung nächste Periode (5%): 10.000 + 500 (Ausschüttung von vorheriger Periode, da gleiche WertpapierID und die Portfoliofassade dieses auch zurückliefert) > 500 € -> falscher Wert
gleiches gilt für Verkaufen

Die Auswahl beim Wertpapierkauf-/ Verkauf ist falsch.
Aktien sind Stücknotiert d. h. in Euro notiert, während Anleihen Prozentnotiert sind.

In Periode pflegen ist es korrekt erfasst.

Es ist festzulegen, wie mit Pleiteunternehmen umzugehen ist. Werden diese Aktien vom Kauf ausgesetzt?

Diese ist in dem Fachkonzept Kauf / Rendite festzulegen.

Perioden müssen die Namen "Periode 0", "Periode 1", usw. als variable gespeichert bekommen.
Dies wird insbesondere bei der Periodenauswahl in der InvestmentÜbersicht benötigt, kann aber auch bei PeriodenPflegen verwendet werden.

Ich habe verschiedene Werte für den manuellen Kurs und den Spread einer Anleihe ausprobiert, jedoch wird in der DB immer der gleiche Kurs abgespeichert.

Anika wurde ohne Datenbank-Datei gestartet.

Spiel anlegen-Screen wird korrekt angezeigt
Datenbank-Datei wird erstellt, jedoch ohne Tabellen
=> Mehrere NullPointerExceptions und SQL-Errors

1. Prüfen, dass Feld Manueller Kurs optional zu machen.
2. Prüfen, ob berechneter Kurs angezeigt werden kann.

Excel-Datei von Carol Alexander kann nur Floater mit einer Restlaufzeit von 4 Jahren umgehen, nicht aber 10 Jahre.

Excel-Datei muss entsprechend angepasst werden.

Beschreibung:
Es ist möglich, alle aktiven Spiele zu löschen.
Wird dann kein neues Spiel auf aktiv gesetzt, lässt sich die Anwendung weiter nutzen, führt dann aber zu diversen Fehlern z. B. bei Aufruf von Periode pflegen.

Fix:
Sicherstellen, dass anderes Spiel aktiviert wird.
Sicherstellen, dass AktuelleSpieldaten aktuell gehalten werden.

derzeit wird die Transaktionshistorie angezeigt

Problem: Drucken verändert die UI-Node der Anzeige

Lösung: Node für Druck versuchen zu clonen und dann zu manipulieren.

Ich bin mir nicht sicher, ob es Sinn macht/korrekt ist, dass das Festgeld einen Kurs von 0€ hat. Sollte ein Teilnehmer sein Geld darin anlegen wollen, kauft er ja x*0€ und somit verändert sich sein Zahlungsmittelkonto ja nicht. Hab aber auch keine Option gefunden, den Kurs anzupassen.

Ist es möglich, dass die Tabs noch eine zum restlichen Design einheitliche Formatierung erhalten, sodass sie insgesamt etwas größer sind und ggf. in dem hellen grau sind?

Bislang werden die Anleihen mit einer risikolosen Zinskurve bewertet.

Hierbei ist es für den Planspiel-Charakter sinnvolller, eine risikobehaftete Zinskurve zu verwenden, da andernfalls Bonitätsveränderungen der Emittenten unberücksichtigt bleiben.
Diese Überlegung wurde bereits im Fachkonzept aufgestellt.

Das Vorgehen weicht dabei geringfügig von Alexander ab, die davon ausgeht, dass die Bonität dem des Referenzzinses entspricht.

Carol Alexander schreibt zu:

Example III.1.13: Add footnote: ‘The solution assumes the 60bp spread is not a reflection of a lower
than AA credit rating, If it is, then discounting should be not at LIBOR but at
LIBOR + 60bp and the price would simply be 100.’

Daraus folgt, dass auch mit risikobehafteter Zinskurve diskontiert werden kann.

Der Teilnehmer kann negative Were für die Anzahl eingeben. Wird ohne Fehlermeldung in DB übernommen.

• Buttons haben ungleiche Breite
• Button löschen ist nicht leserlich

Beschreibung:
PeriodeAnlegenController legt in doPerdioeAnlegen() für jedes Wertpapier, also auf für diese, die nicht zum SPiel gehören, einen neuen Kurs an

Problem: Wenn man bei Periode pflegen keinen Wert explizit in den Spread eingibt, wird bei Spread null in die Datenbank eingetragen, das führt dazu, dass in Zeile 34 vom FloatingRateNoteModel eine NullPointerException fliegt.
Lösung: evtl. An der Oberfläche sichergestellen, dass ein Spread eingetragen wird.

Für kein Feld (weder Anleihe noch Aktie) wird der Seminarleiter über Falscheingaben (z.B Buchstaben statt Zahlen) informiert
Die Eingabe einer korrekten Zahl wird im Insolvenzkurs nicht übernommen
Negative Werte für den Aktienkurs werden nicht abgefangen

Z.B. beim Initialisieren eines Spiels wir man aufgefordert eine Anleihe zu erstellen. Der dort eingegebene Spread wird nicht übernommen. In der Datenbank steht "null" für Spread.

Siehe bspw.

Periodenabschluss-Transaktionen z. B. Zinsbuchung Festgeld erscheint nicht in Buchungstabelle.
Ggf. Fehler bei Ermittlung der Portfoliopositionen?

@Franzhose Habe es eben nachgestellt. Bei mir wird auch keine Buchung gespeichert.
Werden dem Mal nachgehen. Danke für den Hinweis.

-> liefern zu viel zurück -> Refactoring

Vorschlag:
Da es pro Unternehmen nur eine Aktie und eine Anleihe geben kann, würde ich vorschlagen, den Wertpapiernamen so zu generieren:

wertpapier.setName(wertpapier.getUnternehmen().getName() + " | " + wertpapier.getWertpapierArt().getName()")

Alert weg wegen Login erfolgreich.

Laut Recherche von Max und Christian gibt es Probleme bei der Zusammenstellung eines Installers.

Findet ihr eigentlich den Namen TeilnehmerRepository und Teilnehmer auch irreführend?

Würde vorschlagen es zu Benutzer bzw. BenutzerRepository zu refactoren, da es besser abbildet, dass auch Nutzer mit anderen Rollen z. B. Spielleiter darin abgespeichert sind.

Text in Spalte kaufen/verkaufen nicht lesbar
evtl. Tabelle (und jede Spalte) breiter machen?
hier fehlt meines Erachtens noch etwas Feinarbeit :-)

Christian und ich haben etwas gespielt. Dabei ist uns aufgefallen dass man anteilige Aktien und Anleihen kaufen kann.
Richtig:

Aktien nur Ganzzahl von Mindestvolumen z. B. 1 Stück
Anleihen nur Mindestvolumen von z. B. 1.000 EUR bzw. Vielfache davon, also auch keine 1.050 EUR
Festgeld beliebig

Ggf. kannst du die Oberfläche auch anpassen, sodass unterschiedliche Felder angeboten werden, je nach Wertpapierart z. B. Anleihe heißt das dann Nominalvolumen, bei Aktien Stückzahl etc.

Ich bin nach anlegen der 4. Periode in die Depotübersicht. Dann hab ich eine IndexOutofbounds Exception erhalten. Weiß nicht genau, wie das Problem zu reproduzieren ist; bin zufällig darauf gestoßen. Bin beim debuggen auch nicht schlauer geworden

Nicht alle Teilnehmer im Dokument/Druck sichtbar
Drucktest.pdf

1. Beim Anmelden eines Teilnehmers kommt der Screen zum Bearbeiten der Informationen eines Teilnehmers. Danach gelangt man aber nicht mehr auf den Screen, weil es keinen entsprechenden Eintrag im Menü gibt.
2. Wenn Spiel != null, dann Teilnehmermenü anbieten

Startkapital, Zinsen und Ordergebühren können negativ sein

In welchem Schritt wird die Bestenliste mit den aktuellen Portfoliowerten aktualisiert?
Ich habe mit einem Teilnehmer in Periode 1 eine Aktie gekauft, deren Wert ich mehrere Perioden danach weiter erhöht habe. In der Bestenliste steht jedoch nur der Portfoliowert aus Periode 1, nach dem 1. Kauf.
(Der Wert müsste weit über 1.000€ sein.)

Kapitalmarktzins und Ordergebühr werden in Prozent eingegeben und sollen laut Spaltenname auch als Prozent in Tabelle "Periode" gespeichert werden. Tatsächlich werden die Werte aber als Hundertstel, also als Zahl ohne Einheit (Prozent) abgespeichert.
Bsp.: Hier wurden 6% und 3% eingegeben.

Odervolumen, Ordergebühren und Gesamtkosten werden nicht aktualisiert

1. Unternehmen: Leere Unternehmensnamen werden nicht abgefangen (Fehler: Abort due to constraint violation (NOT NULL constraint failed: Wertpapier.unternehmen_id))
2. Spiel kann ohne Aktien bzw. Anleihen initialisiert werden
3. Es können mehrere Aktien/Anleihen für ein Unternehmen angelegt werden
4. Beim Klicken auf den "-"-Button (Lösch-Button) wird nicht die entsprechende Zeile sondern die oberste Zeile entfernt
5. Fehler bei Anlegen der Aktien/Anleihen (wahrscheinlich durch gleichzeitiges Anlegen): " The database file is locked (database is locked)"

• Selbstständigkeitserklärung-
• Prozentnotierung / Stücknotierung beschreiben.
• Insolvenzkurs 🎉

Steps to reproduce:

• neues Spiel Anlegen
• Zwei Unternehmen Anlegen
• Wertpapiere Anlegen:
  • Für Unternehmen A und Unternehmen B jeweils Aktie und Anleihe anlegen
  • Speichern drücken

Mir ist aufgefallen, dass die Teilnehmer bereits Wertpapiere kaufen können, sobald die Periode gespeichert, jedoch noch nicht abgeschlossen ist.
Ist das so korrekt? Wenn ja, dann haben wir das Problem, dass der ETF-Kurs und der Kurs der Aktien zu diesem Zeitpunkt noch nicht berechnet werden.
Nach Abschluss der Periode durch den Seminarleiter sind die Kurse dann berechnet.
Vielleicht hab ich hier aber auch einen Denkfehler...

Der Teilnehmer erhält zwar eine Benachrichtigung darüber, dass er nicht angelegt werden kann, da der Name schon in der Datenbank existiert, bekomt aber direkt danach die Bestätigung, dass es doch funktioniert hat.
Nutzer ist dann doppelt in der DB vorhanden