A3 alarm notifications

[nicolashechim]

Objetivo

Familiarizarse con el uso de alarmas y notificaciones.

Resultados

- Notificación generada por un broadcast receiver a través de un evento que emite una alarma programada:

[cristiantanas]

Comentarios Actividad 3

El trabajo está muy bien :). Sólo tengo un par de comentarios.

• La variable isAlarmSet despista mucho... básicamente porque no estás comprobando que haya una alarma seteada sino que estás comprobando que el usuario haya seteado la fecha :p. La podríamos llamar isDateSet por ejemplo.

• Para saber si tenemos que lanzar la notificación o no, deberíamos tener en cuenta el estado de la tarea en el momento de lanzar la notificación. En tu caso estás basando esta lógica en base al momento de crear la notificación. Si modificamos el estado de la tarea a posteriori, la notificación seguiría apareciendo en el momento de finalización de la tarea.

• Quizá seria mejor utilizar el identificador de la tarea como identificador para lanzar la notificación ya que según esta implementación, si hay una segunda alarma que se dispara al poco tiempo de la primera, sólo veríamos la segunda notificación.