Pregled kode

[kle-pra]

Pozdravljeni,
najprej hvala za vaš trud in poslano nalogo.
Se mi zdi kar v redu rešeno in prikazano kar nekaj znanja. Spodaj je moj pregled poslanega - imamo pa tukaj bolj Angular in Javo, tako da je pregled bolj splošen.

Frontend

Všeč mi je

• SPA framework in razbito na logične komponente,
• všečen GUI
• super, da se validira čas začetka in konca
• frontend validacija
• error handling za napake z backenda

Kaj bi lahko bilo boljše

• boljša uporaba strong type v typescript, veliko je any, npr. lahko bi se naredil nov type ali interface za definicijo reservation
  • POPRAVEK: vidim, da je interface Reservation na frontendu, tako da se lahko tole ignorira.
• dobro bi bilo, da se štarta z zadnjo verzijo Reacta ter ostalih dependencijev (obstaja priročen node tool ncu za check + update)
• dodatek kakšnega osnovnega testa bi izboljšalo celovitost rešitve in demonstriralo poznavanje testiranja
• linter vrne nekaj napak, ki bi jih bilo dobro počisititi (vscode eslint plugin navadno že opozori)

Backend

Všeč mi je

• logično razbito: routes, controller functions
• dodatna backend valicija poleg fronted, saj se frontend lahko zaobide
• preverjanje tako sosledja kot zasedenosti v db
• uporaba ENV spremenljivk za DB credentials, saj je zelo dobro da skrivnosti niso v SCM.
• uporaba parameterized queries za db, ki prepreči mysql injection

Kaj bi lahko bilo boljše

• strong type za reservation (mogoče skupen type ali interface s frontendom, ker je oboje v typescript), tako npr. result query-jev ne bi rabil biti any
• eslint errorji
• kakšen osnovni test
• namesto console.log mogoče uporaba kake logging knjižnice.

Splošno

Prednosti

• lepo urejen README
• veliko git commitov, se vidi postopek izdelave, uporaba git
• uporaba eslint, gitignore
• mysql, uporabljamo tudi mi

Kaj bi lahko bilo boljše

• v package.json delujoča script za pognati backend
• na node 22 (long term support oz. LTS verziji) mi ni uspelo pognati tudi s flagom ali ts-node, na 23 pa deluje kot omenjeno. Bolj kot opomba tole, ker je omenjen flag v README.

Ker je pri nas poudarek na Angularju in Javi, me zanima, ali imate tudi kaj izkušenj z Javo oz. ali ste se pripravljeni spoznati in delati s temi tehnologijami?

Lp, Klemen

[fr33bits]

Pozdravljeni,

hvala za povratne informacije. Z Javo in Angular sem že programiral (sicer bolj splošno na fakulteti in na žalost trenutno nimam projekta pri roki za pokazati), v vsakem primeru pa nimam nobenih zadržkov glede dela in učenja s temi tehnologijami. Ker so projekti na CV-ju načeloma enostavnejši se mi je (glede na vso režijo, ki jo s sabo prinaša) Angular zdel malce overkill.

Na podlagi povratnih informacij sem implementiral naslednje spremembe:

• Vse any tipe sem strožje tipiziral (tako v frontendu kot v backendu), torej any tipov ni več.
  • Tip Reservation sem premaknil skupaj z drugimi datotekami v novo mapo common in ga poskusil uporabiti v api/controllers/reservations.ts datoteki, vendar TypeScript oz. db.query() za result očtino zahteva tip mysql2.ResultSetHeader in ne sprejme Reservation oz. Reservation[]. Izjema je funkcija reservations vendar tam mislim, da je sprejetje Reservation posledica tega, da je v pozivedi izpuščen values parameter, saj če se tega doda (kot []) to povzroči isto napako. Kljub vsemu pa uporaba Reservation tu ni potrebna za izključitev vseh any tipov.
• Mislim, da so ESLint opozorila sedaj odpravljena.
• React in ostale odvisnosti sem posodobil na zadnje verzije.
• Še enkrat sem odnamestil Node verzijo 23.6 in namestil Node LTS verzijo 22.13.1. Z ukazom node --experimental-strip-types app.ts mi je ponovno uspelo zagnati API strežnik na Windows (tokrat tudi 11) in macOS. ts-node pa očitno res nič ne spremeni oz. ne deluje in sem zato te omembe zbrisal. Druga testna okolja pa ob tem trenutku nimam tako, da težko rečem, zakaj vam --experimental-strip-types opcija ni delovala.
• V package.json sem uredil ukaz start za zagon backenda, vendar to vključuje le zagon app.ts in ne MySQL Community Server strežnika, saj se mi po (sicer površnim) iskanjem zdi, da to preko CLI ni mogoče oz. vključuje druge odvisnosti ali posebnosti glede na posamezen operacijski sistem oz. platformo. (Prej je bil v package.json pod start napisan ukaz ts-node app.ts vendar ga namerno nisem oglaševal ravno zaradi komplikacij, ki jih ima Node.js pri poganjanju TypeScript kode.)
• Kljub temu pa sem dodal novo npm skripto setup-db za avtomatski set-up podatkovne baze namesto tistih pet bolj zamudnih korakov.
• Ker mi je do prejšnjega roka zmanjkalo časa za pisanje (unit) testov sem jih (tudi glede na te povratne informacije) sedaj dodal naknadno, tako za frontend kot za backend, vendar ker mi ni uspelo ustvariti mock database ta test za backend ne deluje ravno dobro (lahko popravim kasneje ampak trenutno mi zmanjkuje časa).
• Backend logging se sedaj izvaja s Pino knjižnico.

[kle-pra]

Hvala za popravke, zgleda veliko popravljenega in tudi v redu popravljeno. Glede baze in skripte, super, nisem ciljal, da naj še db pripravi, samo backend API server, ampak fajn.
Glede Node 22 s flagom, sem testiral na Ubuntu, nisem se preveč spuščal potem v podrobnosti, ampak kot rečeno, na 23 je delalo OK. Super tudi za teste, tudi če ni vse.

Še dodatno vprašanje, s kakim ORM ste se že srečali? (v Node svetu je to recimo Drizzle, pri nas tudi to uporabljamo, ampak za Javo)?

Žal trenutno ni bilo vodje, pride drugi teden in se takrat javimo z več info za naprej in razgovora, najverjetneje kar v ponedeljek.
LP

[fr33bits]

Pozdravljeni,

hvala za hiter odgovor. ORM na žalost še nisem uporabljal, edino slišal sem za TypeORM oz. približno vem kaj je (ORM).