Entrega Segunda Tarea

[mundomon]

No description provided.

[cristiantanas]

Comentarios Actividad 2

Ramon, ¡el trabajo está muy bien! No veo ninguna cosa "rara" en la implementación del CursorAdapter. Lo que quizá sí cambiaría seria la implementación del método showTasks() de la HomeActivity:

private void showTasks() {
     sqliteDatabase.openDatabaseForReadOnly();
     tasks = sqliteDatabase.returnTasks();
     CustomList taskAdapter = new CustomList(this, tasks);
     lvTasks.setAdapter(taskAdapter);
}

Estás creando una nueva instancia del adapter cada vez que se llama a este método, pero no hace falta. Se puede cambiar simplemente su contenido. Por ejemplo:

private void showTasks() {
     sqliteDatabase.openDatabaseForReadOnly();
     tasks = sqliteDatabase.returnTasks();
     if (taskAdapter != null) {
          taskAdapter.changeCursor(tasks);
     }
     else {
          taskAdapter = new CustomList(this, tasks);
          lvTasks.setAdapter(taskAdapter);
     }
}

En este caso, la variable taskAdapter seria una variable privada y en el onCreate tendrás que asignar el CursorAdapter a la lista, aunque esté vacío.

A parte de eso, en la Activity tienes las variables title y description que creo que no utilizas en ningún sitio.

[cristiantanas]

Comentarios Actividad 3

¡En general un muy buen trabajo! Sólo tengo un par de comentarios.

• La comprobación de si hemos de lanzar una notificación o no para una tarea debería tomarse en el momento de lanzar la notificación. En cambio, en tu caso, estás basándote en el estado de la tarea en el momento de crear la alarma. Se puede dar el caso de que el usuario modifique el estado de una tarea después de crearla.

• Si hay dos alarmas que se disparan en un intervalo corto de tiempo, sólo veríamos la notificación correspondiente a la segunda de ellas, ya que al utilizar siempre el mismo ID estaríamos actualizando cualquier notificación existente.