Skip to content

Conversation

@andre-bisa
Copy link

Avviso innanzitutto che non ho avuto tempo e modo di testare quanto scritto, ma vi invito ugualmente a "prendere l'idea" ed eventualmente a sistemare quanto scritto. Le modifiche apportate sono:

  1. Aggiunta soglia oltre la quale si viene SEMPRE avvisati
  2. Aggiunto un raggio dal "centro d'Italia" entro il quale si viene segnalati in caso di grandi eventi. Il punto centrale d'Italia l'ho calcolato facendo (latitudineMaggiore + latitudineMinore) / 2 e (longitudineMaggiore + longitudineMinore) / 2

Ho inoltre inserito un FIXME su un possibile errore logico: se una persona non è interessata ai grandi eventi (magnitudo >= 5) e questo dovesse essere proprio dove la persona si trova (o comunque all'interno del suo raggio di segnalazione), questo non verrebbe notificato. A dire il vero non sono sicuro al 100% di questo bug, ma ve lo segnalo per controllarlo. 

Spero di aver fatto cosa gradita :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant