search

Roba1993 / Happy-Hour

Das DHBW Happy-Hour Projekt vom Kurs WWI2012F
MIT License
4 stars 0 forks source link

Backend Code Überarbeitung #130

Closed Roba1993 closed 9 years ago

Roba1993 commented 9 years ago

Todos:

Logging Level: DEBUG allgemeines Debugging (auffinden von Fehlern) INFO allgemeine Informationen (Programm gestartet, Programm beendet, Verbindung zu Host Foo aufgebaut, Verarbeitung dauerte SoUndSoviel Sekunden …) WARN Auftreten einer unerwarteten Situation ERROR Fehler (Ausnahme wurde abgefangen. Bearbeitung wurde alternativ fortgesetzt) FATAL Kritischer Fehler, Programmabbruch

Personen: @TobiasH1993 @stegmajo @Tabea-Kraus @Micash0711 @mighty190

Neram commented 9 years ago

Hilfsklasse für die Hashfunktion erstellt

Neram commented 9 years ago

Bitte bearbeitet den Code entsprechend unser festgesetzten Code Konventionen: https://github.com/Roba1993/Happy-Hour/wiki/Backend-Code-Konventionen Dort steht auch drin, dass die Fehlerausgabe mit einen Logeintrag erfolgen soll

Neram commented 9 years ago

Das Verschieben des Ablaufschemas in den richtigen Ordner habe ich übernommen

Neram commented 9 years ago

@mighty190 @TobiasH1993 @stegmajo bei euch sehe ich immer noch e.printStackTrace(); anstatt eines Log-Eintrags - bitte noch ändern. Hinzu kommt, dass die toString-Funktion in der Datenklasse bei den meisten von euch auch noch fehlt zuzuüglich dem Test dazu - bitte noch ergänzen

Roba1993 commented 9 years ago

@Neram @Tabea-Kraus @stegmajo @mighty190 @TobiasH1993 @Micash0711

Dieses Issu hätte zum Ende der Programmierphase abgeschlossen sein sollen! Bis heute Abend sind diese Punkte bitte abgearbeitet. Ansonsten wird im Bereich Abgaben notiert, das der Backend-Code nicht fertig ausgeliefert wurde!

Neram commented 9 years ago

von Micha, Tabea und mir wurden die Punkte abgearbeitet. Die anderen wurden vor Ende der Pahse nochmals persönlich daran erinnert, doch es ist bis jetztichts erfolgt.

stegmajo commented 9 years ago

Wieso müssen die H2DAO Klassen Funktionskommentare besitzen? Die vererbenden DAO Klassen haben doch schon Kommentare und die zeigt Eclipse für die überschreibende Funktion an.

stegmajo commented 9 years ago

Und wieso sollen wir die Modellklassen testen? Das sind doch reine Datenklassen, mit Settern und Gettern, da muss doch nichts getestet werden, auch nicht die toString Funktion?

Neram commented 9 years ago

Die Modellklassen sollen ordnungshalber getestet werden. Ist ja auch kein großer Aufwand. Aber damit sind die Tests vollständig.

Neram commented 9 years ago

Es soll ALLES kommentiert werden, damit überall steht WER es geschrieben hat und um WAS für eine Funktion es sich handelt und man nicht erst in eine andere Klasse reinschauen muss, um was sich dabei handelt. Und natürlich auch damit es überall einheitlich ist.

stegmajo commented 9 years ago

Ok, beides erledigt.

Neram commented 9 years ago

Ich habe die restlichen Code-Überabeitungen übernommen: Kommentare in die H2Dao Funktionen eingefügt, alle restlichen Loggerenträge eingefügt, Code an die Code-Konventionen angepasst

Es fehlt nocht die Test Klasse der Location-Model Klasse. @TobiasH1993 Und einige Getter und Setter sind noch in den H2DAO Funktionen drin @TobiasH1993, @Roba1993, muss das so sein oder können diese auch in eine seperate Klasse ausgelagert werden?

TobiasH1993 commented 9 years ago

Ich kümmere mich darum sobald ich Zeit dafür finde. Diese restlichen Dinge sind ja relativ schnell gemacht. Reicht es wenn ich mich morgen Abend daran setze?

TobiasH1993 commented 9 years ago

Alle Punkte erfolgreich abgearbeitet.