search

matteocontrini / comuni-json

🇮🇹 Database JSON comuni italiani (2020) con informazioni ISTAT + CAP
Other
499 stars 63 forks source link

Formato JSON per campo CAP #7

Closed andrea-capigiri closed 7 years ago

andrea-capigiri commented 7 years ago

Per una corretta formattazione del documento Comuni.json il campo CAP dovrebbe essere rappresentato in qualsiasi caso come array di stringhe

primax79 commented 7 years ago

concordo i json dovrebbero essere coerenti per tutti gli elementi

matteocontrini commented 7 years ago

D'accordo anche io, l'idea era non introdurre "breaking changes" nel caso qualcuno aggiornasse per via automatica i dati usando il file json. Ma si fa presto (?) a modificare la struttura se quello appena spiegato non è un problema così rilevante.

m47730 commented 7 years ago

Io sono uno di quelli che ha implementato la procedura automatica. Ho dovuto mettere qualche "if" per le situazioni in cui il file json non era coerente, se facessi la modifica semplicemente non dovrei cambiare nulla perchè le condizioni sarebbero sempre vere (o sempre false non ricordo). Per me è semaforo verde e mi immagino altri abbiano dovuto gestire le incoerenze in qualche modo