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 |