Aggiungere controlli sui dati

[simonaPr]

Lato server

• [x] ingredientCode
• [x] temperature
• [x] duration
• [x] data di inizio evento
• [x] id della città
• [x] timezone

[sablaireau]

Io avevo fatto dei controlli sulle date nel senso che febbraio non potrà mai avere 31 giorni. Inoltre avevo controllato se un mese finiva per 30 o 31. Non lo so se si devono fare questi controlli :(

[simonaPr]

no..questi non credo..perchè poi quando uso le funzioni java (new Date() e robe simili) se ne accorge da solo

[strawberry-code]

Richiedo un aggiornamento.

[simonaPr]

io direi che possiamo chiuderlo..

[sablaireau]

o.k. il controllo dell'id sul server?

[simonaPr]

io non lo farei (se no inutile avervi fatto scegliere l'id a voi sul client)...se l'id è sbagliato, la ricetta viene ignorata (come per twitter)

[sablaireau]

o.k. come vuoi :)

[simonaPr]

tu lo faresti?

[sablaireau]

se si potesse si, perché è comunque un attacco. Oppure visto che è un attacco, lo scartiamo come dici tu

[strawberry-code]

chissenefreeegaa!!! XD

[strawberry-code]

Se l'utente metterà una città non valida... beh? CAZZI SUOI! XD

[sablaireau]

le strade a mio avviso sono due :) ma fate voi :) l'importante che me lo dite cosi non faccio test fasulli di penetrazione al server :P XD

[strawberry-code]

Perdonatemi, lo chiudo. Non ha senso morirci sopra così.

[simonaPr]

ma più che altro..è che anche con twitter vale lo stesso discorso..anzi sarebbe ancora peggio in realtà, perchè l'utente normalemente non pensa di dover inserire "SimonaPrino" come destinazione...quindi è l'utente stesso che inconsapevolmente fa l'attacco ogni volta che inserisce una ricetta

[strawberry-code]

Fra si è arrabbiato e ha fatto il Ponzio Pilato, dicendo che se faranno un controllo su questa cosa CI FARANNO UN CULO COSI. Allora lo riapro. :(

[sablaireau]

aahahhahahahahahhahaha grazie :) per avermi venduto :P XD

[simonaPr]

mmm...io rimango della mia idea.. cmq se volete lo aggiungo..ma esattamente che controlli devo fare?

[sablaireau]

no allora lo chiudiamo e basta

[strawberry-code]

Io non lo farei. Ho espresso la mia idea.

[simonaPr]

no no..va bene..

[sablaireau]

no non fare i controlli

[simonaPr]

ma no...li faccio..aiutami solo a capire quali..

[strawberry-code]

Chiudere questo issue?

• [x] cri
• [x] fra
• [ ] simona

[simonaPr]

scarico il file e controllo che ci sia l'id?

[sablaireau]

chiudi, mi dice di chiudere l'iusse

[simonaPr]

li faccio, li faccio...

[simonaPr]

secondo te se mi tengo in una lista tutti gli id validi? è fattibile?

[simonaPr]

o sono troppi?

[sablaireau]

o.k. ma mi dispiace, quando ricevi il id della città controlla solo che esista e se la città è quella (se vuoi per la città) in caso di errore scarti la ricetta e mandi un errore del server (generico)

[simonaPr]

ok..ma secondo te è fattibile tenermi in una map<id, città> tutti quelli validi? O sono troppi?

[simonaPr]

mi daresti il file degli id validi?

[sablaireau]

o.k.

[simonaPr]

o dirmi dove posso trovarlo..

[simonaPr]

Sto pensando a questo issue...ma..mi chiedevo...tu come hai gestito il caso dei duplicato? Cioè..mi spiego: se "Turin" è presente in più posti del mondo (quindi corrisponde a più codici), quale codice prendi?

[sablaireau]

Noi gli stampiamo le cordinate geografiche :P Poi è l'utente che deve essere più specifico e meno :)

[simonaPr]

ok..quindi ne scegliete uno a caso tra quelli?

[sablaireau]

in realtà è il servizio a cui ci appoggiamo che ne prende una a caso. E ci passa molte informazioni molte delle quali non le stampiamo ma altre si in modo che l'utente sappia cosa stia facendo. Almeno l'idea di partenza era cosi, ora come ora non so che cosa è stato cambiato :(

[simonaPr]

ok..va bene, grazie! Tra l'altro questa mattina non mi funziona di nuovo quella Search...purtroppo capita un po' troppo spesso e, se ci capita quando siamo là, siamo fritti!

[sablaireau]

Al più gli mettiamo un icona che dice “sta ancora cercando ", a parte gli scherzi era l'unico modo per lavorare senza server e non scaricare 20Mbyte di roba o anche 8mytebe

[simonaPr]

sì sì..va bene..ma non capisco perchè cercando le stesse città, a volte funziona e a volte no...la banda è circa sempre la stessa...boh.. Questi sono gli errori più pericolosi, perchè se non va mai, almeno puoi sistemare il problema, ma così è un terno all'otto...ci affidiamo al caso

[sablaireau]

Io avevo trovato questo sistema, al tempo lavoravamo senza server :( e senza github, quindi mi sono dovuto ingegnare. Se volete potete cambiare, non ho problemi

[sablaireau]

Ti do ragione :) ma a questo punto dovremmo fare delle axaj al server

[simonaPr]

Ma più che altro è che non so se cambi la cosa...cioè...non so proprio cosa potrebbe causare questo problema! Non credo che sia sbagliato come avete fatto voi...anzi! Non so proprio perchè a volte funziona e a volte no (data una stessa città!! Non credo che il vostro metodo centri qualcosa)

[sablaireau]

Non so cosa dirti, so solo che avendo svilupatto questa cosa senza server di appoggio si è fatto quel che si è potuto :(

[simonaPr]

allora...con questo benedetto file avevo dei problemi..ho cercato di arrangiarmi in altri modi (interrogando direttamente il server openweathermap), ma non ho modo di vedere se funziona, perchè la search ha deciso di non funzionarmi più..cmq committo questa cosa..poi la testerò

[sablaireau]

va bene, io ho fatto il controllore e la pagina ma la pagina mi dai dei problemi :(

[simonaPr]

ok..mi dispiace..ma non è obbligatorio aggiungerlo!! Non stare ad impazzire!

[sablaireau]

mi sembra di essere scemo ahahhahahah

[simonaPr]

ma no..perchè???

ps: sto openweathermap è un servizio di merda!!!

[sablaireau]

hai ragione, è stata l'unica alternativa che trovai al posto di scaricare il file da 20Mbyte