search

Danted96 / ProgrammazioneWeb

0 stars 0 forks source link

consigli #1

Open De-Lac opened 6 years ago

De-Lac commented 6 years ago

sconsigliato utilizzare nomi non idonei alla funzionalità dell'oggetto che puntano

https://github.com/Danted96/ProgrammazioneWeb/blob/0e52a69011f2845384471564fa02c91b7fc930b1/routes/routing.js#L817

https://github.com/Danted96/ProgrammazioneWeb/blob/0e52a69011f2845384471564fa02c91b7fc930b1/routes/routing.js#L8

De-Lac commented 6 years ago

Le socket sono utilizzate per gli aggiornamenti real-time, in modo che invece ad essere il client che chiede al server ad intervalli regolari se ci sono stati cambiamenti, il client si "registra", e sarà il server poi a notificare i client "registrati" sul suo canale delle modifiche avvenute.

L'approccio è quindi diverso dalle API rest. Avrebbe comunque senso utilizzare le socket in caso di parti altamente dinamiche, come ad esempio l'ultima offerta in caso di asta, o la disponibilità corrente in caso di disponibilità altamente volatile

https://github.com/Danted96/ProgrammazioneWeb/blob/0e52a69011f2845384471564fa02c91b7fc930b1/zzCustom/socketGlobal.js#L1-L7

De-Lac commented 6 years ago

E' bene inserire controlli sui dati da inserire.

Per controllare che una variabile sia inizializzata, si può fare

if(prodotto.quantita)
   { /* true se quantita è != undefined && != false && != 0  */}

se lo 0 è un valore valido, si può fare

if(prodotto.quantita != undefined)
   { /* true se quantita è != undefined */ }

per controllare che una stringa non sia vuota, si può fare

if(prodotto.nome.trim() != "")
   { /* true se nome non è stringa vuota. trim rimuove eventuali spazi vuoti come prefisso */ }

per controllare che sia un numero, ad esempio si può fare così

p.s. evitare assolutamente i caratteri 'strani' per i nomi delle variabili, coma la à in quantità

https://github.com/Danted96/ProgrammazioneWeb/blob/0e52a69011f2845384471564fa02c91b7fc930b1/routes/routing.js#L638-L640