KRILab 8. Labor eredmények eredmények

Kikerültek az első és nyolcadik mérés eredményei, ami a publikus eredménytáblázatban is frissül. A részletes pontozás megjegyzésekkel alább látható. Kérem azokat, akik még nem töltötték fel az utolsó feladat megoldásukat, sürgőssen pótolják!

Az eredménytábla: https://docs.google.com/spreadsheets/d/1Fv92hg7ro-jCKmFxTZV309r_eWMLAObPhJERLKlURC0/pubhtml?gid=889418965&single=true


1. mérés:

  Dokumentáció Többszállúság Munkafolyamat Implementáció Megjegyzés
Pont Max 2 Max 3 Max 3 Max 2 10  
theoldsquirrels 2 3 3 2 10 Végül szépen elkészült, érdemes volt rászánni azt a pár napot.
csapat15 1 3 1 1,5 6,5 A dokumentáció egy összevisszaság, ilyet kár volt írni. Nincs behivatkozva a laborra elkészített kód. (Doksi-1) A munkafolyamatban az üzenetek mergelése során listákban keresitek meg az üzenetek párjait, és ha nem találjátok visszateszitek a sor végére!!! (Munkafolyamat -2)  A Github repo is kaotikus (implementáció -0,5 pont)
Required 2 3 3 2 10 Szép megoldás
517 1 3 3 2 9 Szép megoldás, de a többszálúság és a munkafolyamat pont nincs dokumentálva.
kg_pt 1,5 3 3 0,5 8 Nincs behivatkozva a laborra elkészített kód, ami el törik a verziók során. (?, Doksi-0,5 Implementáció-0,5)  Miért random generáltok ID-t? Laboron is megmondtam hogy ez így nem jó! (Implementáció-1)
TheCakeIsALie 1 2 3 2 8 Miért lockoltatok? Szerinte mennie kellene lock nélkül is. Továbbá nem szinkronizáljátok a szálakat merge-nél, és nem írjátok miért. (szálkezelés -1) Dokumentációban elég gyenge, a munkafolyamatról és a többszáluságról nem  írtok semmi hasznosat (dokumentáció -1)
datB 2 3 2 2 9 Szép dokumentáció! Megoldás is szép lenne, ha nem nem a queue-kban keresnétek az ID-nek megfelő párokat, hanem egyszerűen két új map-et használnátok, amiben lehet ID alapján keresni. Ez így könnyebben hatékonyabb, és karbantarthatóbb. (Amikor megérkezik egy dokumentum, akkor megnézi hogy a másik mapben benne van-e a párja. Ha igen, akkor kiveszi és beteszi a feladatsorba a párost, ha nem akkor beteszi a hozzá tartozó mapbe.) (Munkafolyamat -1pont)
Girls 1,5 3 3 1,5 9 Jó lett volna, ha megjelölitek az összes laborra elkészült verziót gitben. (Doksi-0,5 Implementáció-0,5)
includetr 1,5 3 2 1,5 8 Jó lett volna, ha megjelölitek az összes laborra elkészült verziót gitben. (Doksi-0,5 Implementáció-0,5) A merge kód elég zavaros, ti is  a queue-kban keresitek az ID-nek megfelő párokat. Jobb lenne ha egyszerűen két-három új map-et használnátok, amiben lehet ID alapján keresni. Ez így könnyebben hatékonyabb, és karbantarthatóbb. (Amikor megérkezik egy dokumentum, akkor megnézi hogy a másik mapben benne van-e a párja. Ha igen, akkor kiveszi és beteszi a feladatsorba a párost, ha nem akkor beteszi a hozzá tartozó mapbe.) (Munkafolyamat -1pont)
HostBusters 2 3 3 2 10 Gratulálok a szép Scalas megoldáshoz és a lambda kifejezésekhez!
NullPointerException 0,5 3 3 2 8,5 A dokumentációban keveset írtatok a többszállú megvalósításról. Egyébként egyszerű, elegáns megoldás!

8. mérés:

  Doksumentáció Metamodel Kódgenerálás Glue kód megjegyzés
Csapat Max 2 Max 3 Max 3 Max 2 Max 10  
theoldsquirrels 2 3 3 2 10 Miért van metamodellben Eoperation??? Ettől függetlenül szép megoldás.
csapat15 2 2 1,5 1 6,5 Egy enumba fixen bele vannak téve azok a folyamat lépései, a kódgenerátor ezeken switch-case művelettel generál ki minden kódot. Glue-kódot is generálják.
Required 1 3 3 2 9 Doksi hiányos, csak bevezető mondatok vannak minden fejezethez, semmi konkrét.
517 0,5 3 3 2 8,5 Doksi erőssen hiányos, csak bevezető mondatok vannak minden fejezethez, semmi konkrét. [Metamodellt xmi-ból kellett olvasnom :(] A kódgenerátor egyébként szépszép
kg_pt 2 3 3 2 10 Szép megodás 
TheCakeIsALie 2 3 3 2 10 Nagyon szép megoldás
datB 2 3 2,5 1 8,5 Szebb lett volna, ha a BlockingQueue manipulációt is generáljátok
Girls 2 3 3 2 10 Szép megodás!
includetr 2 3 3 2 10 Nem tettétek bele a dokumentációba a metamodell ábráját, egyébként szép
HostBusters 2 2 2 1 7 A kódgenerátor nem általános, az csomópontok implementációja és a munkafolyamat szerkezete nehezen változtatható.
NullPointerException 2 3 3 2 10 Nem tettétek bele a dokumentációba a metamodell ábráját, egyébként szép