V tabeli proga imata postaje in razdalje med njimi kar v nizih kot sezname z vejicami kot ločili, kar je ravno nekaj, čemur se hočeta z uporabo baze izogniti. Vidim sicer, da imata to bolje rešeno s tabelo progekraji, tako da če tabele proga ne potrebujeta, jo kar pobrišita (sicer pa uporabita tabelo progekraji).
V tabeli progekraji manjka referenca na tabelo postaja (ter morda tudi proga - smiselno bi bilo ohraniti ID, pa morda še kakšne osnovne podatke o progi).
Gesla v bazo shranjujeta v čistopisu, kar ni najbolj varno. Bolje je, če shranjujeta zgostitve gesel (poglejta si, kako so drugi to uredili).
Kot si @Globocnik0 napisal po e-pošti, bosta še uredila izgled aplikacije.
Sedaj, ko sta premaknila aplikacijo v mapo aplikacija (ne pa tudi mape views s predlogami), bi koristilo v README.md dodati navodila za poganjanje aplikacije.
Drugače pa aplikacija sedaj deluje, tako da bom dodal povezavo na repozitorij predmeta. Če se še nista, se lahko z @alenFMF domenita za termin zagovora. Če bosta do takrat še kaj spreminjala, mi javita, da posodobim povezavo.
Poleg #2 in #3 imam še sledeče pripombe:
proga
imata postaje in razdalje med njimi kar v nizih kot sezname z vejicami kot ločili, kar je ravno nekaj, čemur se hočeta z uporabo baze izogniti. Vidim sicer, da imata to bolje rešeno s tabeloprogekraji
, tako da če tabeleproga
ne potrebujeta, jo kar pobrišita (sicer pa uporabita tabeloprogekraji
).progekraji
manjka referenca na tabelopostaja
(ter morda tudiproga
- smiselno bi bilo ohraniti ID, pa morda še kakšne osnovne podatke o progi).aplikacija
(ne pa tudi mapeviews
s predlogami), bi koristilo vREADME.md
dodati navodila za poganjanje aplikacije.Drugače pa aplikacija sedaj deluje, tako da bom dodal povezavo na repozitorij predmeta. Če se še nista, se lahko z @alenFMF domenita za termin zagovora. Če bosta do takrat še kaj spreminjala, mi javita, da posodobim povezavo.