search

openpkw / openpkw-weryfikator

Backend dla aplikacji weryfikującej wyniki wyborów
2 stars 8 forks source link

OW-U-20 Wrzucanie testowych wynikow wyborow (poziom kraju) #28

Closed SebastianCelejewski closed 8 years ago

SebastianCelejewski commented 8 years ago

Zakres zmian:

  1. Usunięcie generowania danych dla wykresów i dla tabelki dla poziomu kraju (wyłącznie dla poziomu kraju, dla poziomu okręgów i obwodów to będzie inne zadanie).
  2. Uporządkowanie kodu:
    • autoformatowanie wg szablonu OpenPKW
    • zmiana nazw metod na bardziej zwięzłe
    • zmiana nazw metod z "generate" na "get*"
    • dopisanie autorów kodu
    • dodanie "public" w metodach interfejsów
    • dodanie "private" dla pól w klasach
    • pola statyczne umieszczone są przed polami instancji klasy
    • i in.

Uwagi:

  1. W niektórych miejscach musiałem zrobił optymalizację dla pobierania danych z bazy danych. Normalne podejście Hibernate dawało czasy przetwarzania kilkanaście minut. Dodałem custom queries. Jednak rzecz jest do gruntownego uporządkowania.
  2. W bazie danych brakuje wielu istotnych rzeczy, np. liczby uprawnionych do głosowania.

Jak korzystać z nowych metod REST:

  1. W celu wygenerowania ok. 1% protokołów należy użyć następującego requestu: http://[host:port]/openpkw/database/generateVotes
  2. W celu usunięcia z bazy danych wszystkich protokołów należy użyć następującego requestu: http://[host:port]/openpkw/database/deleteVotes
SebastianCelejewski commented 8 years ago

Zauważone problemy po stronie frontendu:

  1. Zły URL do kontrolera służącego do pobierania danych dla okręgu (już poprawione na branchu master projektu openpkw-weryfikator-frontend, trzeba sobie ściągnąć).
  2. Wykres "Frekwencja wyborcza zbiorcza dla całego kraju" pokazuje takie same dane jak wykres "Liczba otrzymanych protokołów wyborczych dla całego kraju". W zwracanego przez web serwis JSONa wykresy powinny być inne.
  3. Po wybraniu okręgu nie pojawia się "Wykres prezentujący wyniki wyborcze w skali okręgu".
SebastianCelejewski commented 8 years ago

Zrobiłem optymalizację wydajnościową dla pobierania danych dla całego kraju.

pogos commented 8 years ago

Drobiazgi, które zauważyłem: JaxRsHelper - proponuje usunąć zduplikowany kod(wywołanie metody jako createClient(5000) ) oraz wyciągnąć przy okazji stałe SignService - zmiana w wyjątku wygląda na przypadkową zmiane

SebastianCelejewski commented 8 years ago

Dzięki. W SignService ewidentnie był to o jeden rename za daleko.

pogos commented 8 years ago

Proponuje mergować do mastera.