marcbudde94 / GDW_WS19-20_Juwig_Leistenschneider_Budde

1 stars 0 forks source link

MS2 #6

Open Marjuw opened 4 years ago

Marjuw commented 4 years ago

Hi Nathalie,

Wir haben nun noch einige Veränderungen vorgenommen.

  1. Dazu gehört die Rest Tabelle, in der wir eine Erklärung und ein Codebeispiel hinzugefügt haben.
  2. Außerdem haben wir uns auf einen Teil unseres Domänenmodells beschrenkt. Die aktualisierung liegt auch im Reiter "Domänenmodell" in unserem Wiki vor.
  3. Unsere geplante Anwendungslogik und die Erläuterung dazu liegt nun auch im Wiki.
  4. Zudem haben wir unsere Pup/Sub und Topics noch einmal aktualisiert.
  5. Zuletzt wurde eine Arbeitsmatrix erstellt.

Wir sind den Bewertungsbogen für den MS2 nun mehrmals durchgegangen. Es wäre sehr nett wenn Sie sich die aktualisierten Dokumente anschauen könnten und Feedback dazu geben , damit wir nicht am kommenden Montag zu sehr in Zeitstress geraten. Ist die geplante Anwendungslogik so genügend und korrekt beschrieben ? Bei der Pup/Sub Kommunikation haben wir uns gedacht, dass ein User sowohl benachrichtigt werden könnte, wenn irgend ein anderer User Lebensmittel abzugeben hat, oder auch nur auf einen bestimmten User bezogen. Die Arbeitsmatrix haben wir nun auf die Bereiche Wiki, ext.Webservice, Proofe of Concept (PoC), Pup/Sub, Rest Tabelle und Anwendungslogik/ Domänenmodell unterteilt. Genügt das so ?

PS: Das Proof of Concept befindet sich noch in Bearbeitung und wird die Tage fertig dokumentiert. Der Code befindet sich aber bereits im Repository.

nathaliekhn commented 4 years ago

Hallo Zusammen,

  1. Eure Restmodellierung sieht soweit gut aus, jedoch hab ich eine kleine Anmerkung: /user/{userID}/lebensmittel?kohlenhydrate<=value Warum verwendet ihr hier "<="? Als Uri ist diese Angabe falsch. Wenn ihr damit angeben wollt, dass nur Lebensmittel ausgegeben werden sollen, die weniger oder genau die Anzahl an Kohlenhydrate haben, die an Stelle von Value angegeben werden, müsst ihr dies im Code so umsetzen. Aber in der Uri wird immer nur "=" verwendet, ansonsten ist es formell falsch. Dies bitte an allen Stellen ändern, an denen ihr <= o.Ä. verwendet.

  2. Domänenmodell ist soweit auch ok. Ihr könntet es noch schriftlich erklären.

  3. Eure Anwendungslogik ist ausreichend.

  4. Bei eurer Pub/Sub und Topic Modellierung verstehe ich folgendes Topic nicht: "alle verfügbaren Lebensmittel". Bitte nochmal genauer erklären, damit ich euch bezüglich dessen weiterhelfen kann. Zusätzlich verstehe ich diese Anmerkung nicht: "-> Get?istAbzugeben=true". Solche Anmerkungen habt ihr bei allen Pub/Sub-Modellierungen erwähnt, jedoch weiß ich nicht, was ihr damit meint. Bitte auch nochmal näher erläutern. (Gerne hier im Issue)

  5. Bei der Arbeitsmatrix bitte überall %-Zahlen angeben, da nicht deutlich wird, was mit viel, mittel oder wenig gemeint ist. Die Arbeitsmatrix bitte weiter fortführen. Ihr könnt auch die Dokumentation innerhalb des Wikis als Kriterium in der Arbeitsmatrix erwähnen.

Ich hoffe ich konnte euch weiterhelfen