| Kryteria oceny | Objaśnienie | Maks. liczba punktów | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | Liczba punktów | Komentarze | ||||||||||||||||||
| grupa G1 | grupa G2 | grupa G3 | grupa G4 | grupa G5 | grupa G6 | grupa G7 | grupa G8 | grupa G9 | grupa G10 | grupa G11 | grupa G12 | grupa G13 | grupa G14 | grupa G15 | grupa G16 | grupa G17 | grupa G18 | grupa G19 | grupa G20 | grupa G21 | grupa G22 | grupa G23 | grupa G24 | grupa G25 | ||||||||||||||||||||||||||||
| Zadania | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Diagram klas | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Identyfikacja klas | Dobór klas, czytelność nazw. | 4 | 4 | 4 | 4 | 3 | Brak rozróznienia pomiędzy typem produktu a jego instancją (np. partią towaru). Model nie podtrzymuje tego, że partie towaru tego samego typu mogą być w różnych lokalizacjach | 3 | Librarian jest aktorem, jego dane niekoniecznie będą przechowywane (w taki sposób) | 4 | 3 | Można zarezerwować, ale nie ma rezerwacji | 4 | 2 | Tych operacji mogło być wiecej. Jeżeli składamy wniosek - to wniosek też jest klasą dziedziny, jeżeli są zdefiniowani odbiorcy, to też są klasą, podobnie zlecenie przelweu… | 2 | Po co te opony. Zlecenie powinno mieć pozycje odnoszące się do Towar. Czym jest ten zasób? | 3 | W klasach dziedziny nie ma interface. Interface też nie bardzo ma atrybuty | |||||||||||||||||||||||||||||||||
| Atrybuty | Jak wyżej. | 2 | 2 | 1 | stan zlecenia? Opłata; czy zablokownao środki? Numer karty? | 2 | 2 | 2 | 2 | 2 | naprawdę wypożyczone: Książka? | 2 | 2 | 2 | 2 | |||||||||||||||||||||||||||||||||||||
| Związki | Poprawność związków i krotności,nazwy ról | 4 | 0 | use nie bardzo; brak nazw rol; sala MA miejsce, rezerwacja raczej nie ma seansu, ale dotyczy (asocjacja), to samo z miesjcem. Bilet powinien być powiązany z miejsem i seansem | 2 | brak ról, Order - location from & to (opcjonalnie by) | 4 | 3 | 3 | Asocjacje Librarian są nieuzasadnione. | 4 | 2 | krotności? Wielu autorów | 2 | Złe kierunki agregacji, krotności. | 3 | role zamiast atrybutów… | 3 | role. Raczej towar ma produkty; krotności | 4 | ||||||||||||||||||||||||||||||||
| Suma | 10 | 6 | 7 | 10 | 8 | 8 | 10 | 7 | 8 | 7 | 7 | 9 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||
| Przypadki użycia | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Kompletność | Czy możliwa funkcjonalność jest pokryta przypadkami użyca. Ocena związków. Udział aktorów. | 4 | 3 | Co to za dziwna strzałka? Jak zarezerwuj i zapłac to może extends? | 2 | Co robi w systemi klient w ramach getintocar/getoutofcar? System bankowy no pobierania opłat? GPS (aktor). | 4 | Rozumiem, że jest to system wewnętrzny, bez np. interfejsu webowego dla klienta…. | 2 | Temat sugerował inne funkcje. Co to miałoby być wprowadź dane. Zawsze wprowadzamy jakieś dane. | 2 | Rejestracje wypożyczeń i zwrotów | 4 | 3 | Można dodać kilka PU i aktorów, np.. Monity i powiadomienia. Jak kilent płaci karę w systemie. | 4 | 4 | 3 | zmiana lokalizacji produktu? Definiowanie lokalizacji? | 4 | ||||||||||||||||||||||||||||||||
| Forma scenariusza | Czytelność i jakość przebiegów, zidentyfikowny aktor, prewraunki, itd.. Opisać 3 przypadki użycia. | 9 | 7 | Klien zgłasza się, kasjer prosi. W systemowych PU tego nie ma. Kasjer wprowadza dane, interakcja z klientem to bzinesowy PU (jak przepis dla kasjerki zapytać o kod pocztowy i kartę lojalnościową) | 4 | Pkt 7 - jedzie po pasażera?
Kierowca realizuje zamówienie - naprawdę taki trigger? A nie po zabraniu
klienta? Może powinien potwierdzić
rozpoczęcie kursu Nie było PU dokonuwania płatności. Klient po ciemku w taksówce realizuje przelew |
9 | 5 | Ten klient jest trochę nieszczęśliwy, bo klient komunikuje się z działem sprzedaży, a nie magazynem, ale niech będzie. Po rejestracji wpłaty? Systemowy PU powinien zamknąć się w jednej sesji. Logowanie nie było na diagramie. Magazynier jest tym pracownikiem i PU miał pokazywać stopniowe odznaczanie, że zamówienie się kompletuje.... | 3 | CRUD Definiowanie ksiązki to to nie jest. Nie było obsługi klienta. System wewnętrzny jest złą nazwą (i niespójną z diagramem) Gdyby CRON powiadamiał o czymś - miałoby to sens. Trudno księgować niewniesione opłaty. | 8 | Normalnie definiowanie jest interpretowane jako CRUD. Rozumiem, że nie ma usuwania (jest zamykanie) ale gdyby zachodziła konieczność zmiany pól? | 6 | Wypożyczenie wymaga zebrania dwóch danych - książki i czytelnika. Założenie karty - a jak zgubi? Nie było przypadku wielu autorów. Zazwyczaj dodanie egzemplarza wiąze się z nadaniem mu numeru (i np.. Wydrukowaniem naklejki) | 6 | Zamawianie towaru nie jest zbyt realistyczne. Zwykle koszyk? Może np. samochód lub komputer, ale zwykle można dobrać wyposażenie lub kupić kilka towarów. Pobierz dane klienta - przekazujemy dane klienta dostawcom? To byłby raczej interfejs typu web service (aktorem jest stystem, a nie człowiek). | 8 | Czy użytkownik nie powinien wskazać konta (z diagramu wynika, że może być ich wiele?) | 5 | Realizacja zlecenia - magazynier powinien zebrać produkty, czyli stopniowo odkreślać elementy listy…. Mieszacie Panowie pojęcia, które sami wprowadziliście (Produkt vs towar) | 8 | Przy podzadaniach - pętla lub PU? | |||||||||||||||||||||||||||||
| Scenariusze alternatywne | Czy wystarczająco szczegółowe; czy pokrywają możliwe do zidentyfikowania warunki rozgałęzień? | 3 | 3 | Zasady numeracji | 2 | ad3.a.3.Kierowcazgłasza w aplikacji niestandardową sytuację. - nie było kierowcy? | 3 | 2 | 1 | 3 | 2 | 2 | 3 | 2 | 3 | |||||||||||||||||||||||||||||||||||||
| Suma | 16 | 13 | 8 | 16 | 9 | 6 | 15 | 11 | 12 | 15 | 10 | 15 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||
| Diagram stanów | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Diagram stanów parsera | 4 | 0 | to miało być zdecydowanie coś innego | 1 | 3 | gubi a/b a/*…*/ | 1 | 2 | a/b a/* */ niezbyt dokłdne przejścia definiujące komentarz | 4 | 3 | pętla w identifier; naprawdę 3 jest identyfikatorem? | 4 | 4 | 1 | 4 | ||||||||||||||||||||||||||||||||||||
| Diagram stanów dla systemu | 3 | 3 | 3 | 2 | 2 | 3 | 3 | 3 | 0 | Jak coś, co nie jest widoczne (poza systemem) może mieć stan? To jak stan praprawnuka - nieurodzony->urodzony. | 4 | Ok. Raczej jest to diagram zlecenia przelewu. | 0 | Nie chcemy diagramu stanów aktora | 3 | |||||||||||||||||||||||||||||||||||||
| Suma | 7 | 3 | 4 | 5 | 3 | 5 | 7 | 6 | 4 | 8 | 1 | 7 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||
| Diagramy czynności (systemowy i biznesowy) | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Systemowy | Logika przetwarzania, czytelność, rozgałęziania, tory | 4 | 3 | Próba diagnozy jest niezbyt systemowa | 4 | 4 | 4 | 4 | Decyzja bez przepływu wyjściowego? | 4 | Wolałbym inny PU | 4 | 4 | 4 | 4 | Niechętnie widzę logowanie | 4 | |||||||||||||||||||||||||||||||||||
| Biznesowy | także równoległość | 4 | 2 | Join (gruba kreska) łącząca przebiegi xor - deadlock. Wisząca czynność | 3 | Równoległość klient czeka, kierowca jedzie, wiszące czynnosci … | 1 | Stworzenie umowy nie jest biznesowym przypadkiem. Inny poziom granulacji. Stworzenie umowy jest raczej czynnością | 2 | W biznesowym żadnych alternatywnych ścieżek? | 3 | Brak końca! | 3 | Biznesowy to nie bardzo jest… W biznesowym czyności w przybliżeniu odpowiadają systemowym przypadkom użycia. | 3 | To nie jest poziom biznesowy | 2 | Zamów w tej wersji nie jest biznesowym PU. Problemem też są wiszące czynności bez kontynuacji i forki bez joinów | 4 | 3 | Biznesowy PU nie powinien rozróżniać zelecnia do systemu i realizacji przez system. Po prostu drukujemy, kompletujemy, itd.. Systemowe PU powinny stać się czynnościami | Niestety nieczytelny | ||||||||||||||||||||||||||||||
| Suma | 8 | 5 | 7 | 5 | 6 | 7 | 7 | 7 | 6 | 8 | 7 | 4 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||
| Realizacja PU (1xdiagram klas 2xdiagram sekwencji) | Proszę wklejać wektorowe rysunki (wklej specjalnie-obiekt visio) | |||||||||||||||||||||||||||||||||||||||||||||||||||
| Model klas | Czy wszystkieklasy są obecne. Podział na klasy interfejsu, przetwarzania i modelu. Relacje | 5 | 4 | Do anulowania brak | 4 | creates? Raczej relacje OrderRepo MA Order, role zamiast atrybutów… | 5 | 4 | Czy naprawdę jest związek pomiędzy klientem i formularzem danych? | 5 | 5 | 5 | OK, ale miał być osobny diagram klas dziedziny i osobny z dodanymi elementami. | 4 | obiekty powinny być połączone asocjacjami, a nie zależnościami (a formularz zakładania jets niepołącozny) | 4 | 5 | |||||||||||||||||||||||||||||||||||
| Ogólna postać | Składnia, reprezentacja graficzna, logika; czy obiekty należą do zidentyfikowanych klas? | 4 | 3 | anulownaie | 4 | 4 | 2 | Skąd się wziął Produkt, chyba powinien być wyszukany. Jeśli tworzymy obiekty to <<create>> i leżą niżej na osi czasu | 4 | 3 | Powinien utworzyć obiekt TeamMember (na 1. diagramie) | 3 | ReportFailure - w interfejsie, Założenie karty - nie bardzo rozumiem, skąd się bierze czytelnik i po co te dwa kroki. | 4 | 2 | składnia ALT , jakie regiony są dopuszczalne? Odróżnić wywołania i komunikaty zwrotne... | 4 | |||||||||||||||||||||||||||||||||||
| Czytelność | Czy reprezentacja zachowania jest zrozumiała, jeżeli nie, czy są objaśnienia | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | ||||||||||||||||||||||||||||||||||||||||
| Spójność z modelem klas | Czy wszystkie obiekty są reprezentowane. Czy komunikaty są metodami. | 2 | 2 | 2 | 2 | 2 | 2 | Raczej tak | 2 | 2 | 2 | 1 | dyskusyjne. Komunikaty zwrotne wyglądają jak wywołania metod, więc formalnie -- NIE | 2 | ||||||||||||||||||||||||||||||||||||||
| Reprezentacja przebiegów alternatywnych | Czy istnieje | 2 | 1 | Powinien być alt w alt. Linia oddzielająca… | 2 | 2 | 2 | 2 | 2 | 2 | 2 | 1 | 2 | |||||||||||||||||||||||||||||||||||||||
| Suma | 14 | 11 | 13 | 14 | 11 | 14 | 13 | 13 | 0 | 13 | 9 | 14 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||
| Projekt serwisu | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Opis słowny | Czego dotyczy | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||||||
| Klasy dziedziny | Obiekty modelujące dane przetwarzane przez system; związki między klasami. | 2 | 1 | User <ma>---> File | 1 | relacje ! | 1 | Czym mają się różnić linie przerywane od ciągłych. Skoro licytujemy to powinien powstać obiekt BID? | 0 | 2 | 1 | To przpomina model DB, a gdzie produkty? | 0 | 2 | 0 | |||||||||||||||||||||||||||||||||||||
| Klasy projektu | Identyfikacja klas dla stron HTML, skryptów, DB, relacje, metody i atrybuty | 4 | 3 | DataBase powinna być reprezentowana przez klasę. Jeżeli jest to File - to inna metoda (i brakuje obiektu dziedziny) | 4 | 3 | Co jest asocjacją a co depenedency? | 4 | 4 | 4 | 3 | GUI? | 4 | 4 | ||||||||||||||||||||||||||||||||||||||
| Związek pomiędzy klasami widoku i obiektami dziedziny | Jeżeli widok prezentuje obiekty dziedziny lub ich kolekcje, czy jest to widoczne | 1 | 0 | 0 | 1 | 0 | 1 | 1 | 0 | 1 | product --- page product | 0 | ||||||||||||||||||||||||||||||||||||||||
| Diagram sekwencji | Czytelność, zupełność, składnia, zgodność z technologią | 4 | 4 | 4 | 4 | 3 | Jeżli się uaktualnia to powinno być to widoczne (np. wywołanie włąsnej metody) | 4 | 4 | 4 | 4 | 4 | A tu diagramy ok | |||||||||||||||||||||||||||||||||||||||
| Reprezentacja scenariuszy alternatywnych | Czy istnieje | 2 | 2 | 0 | 2 | 2 | 2 | 2 | 2 | 0 | 2 | |||||||||||||||||||||||||||||||||||||||||
| Spójność z modelem obiektowym | Czy nazwy komunikatów odpowiadają metodom | 2 | 2 | z jednym wyjątkiem…. | 2 | 2 | 1 | Nie zawsze | 2 | 2 | 2 | 1 | addToBasket w controller | 1 | ||||||||||||||||||||||||||||||||||||||
| Suma | 15 | 12 | 11 | 13 | 10 | 15 | 14 | 11 | 0 | 12 | 11 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||
| Suma punktów z zadań | 70 | 50 | 50 | 63 | 47 | 55 | 66 | 55 | 30 | 63 | 45 | 49 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | 0 | ||||||||||||||||||||||||||