Excercise 1 - 2016.10.27 - Balázs Ujlaki

[bujlaki]

Please review

[cristiantanas]

¡Muy buen trabajo! Sólo algunos comentarios:

• Los ficheros del directorio .idea no se deberían incluir en el sistema de control de versiones. Son ficheros específicos de la instalación de Android Studio y generarían conflicto si alguien se intenta bajar los cambios. Simplemente habría que incluir el directorio en el fichero .git_ignore.
• También es una buena práctica comprobar que las Activities realmente implementen las interfaces que definimos en los Fragments. Por ejemplo:

((NewTodoFragmentActionListener) activity).onSave();

Si la Activity no implementa la interfaz NewTodoFragmentActionListener tendremos una excepción no controlada. Podríamos resolver esto en el onAttach del Fragment:

@Override
public void onAttach(Context context) {
     super.onAttach(context);
     if(context instanceof Activity) {
          activity = (Activity) context;
         // Assumiendo que tenemos una variable de tipo NewTodoFragmentActionListener
         try { 
              this.listener = (NewTodoFragmentActionListener) activity; 
         } catch (ClassCastException exception) { 
              throw new ClassCastException("The activity does not implement the interface."); 
         }
     }
}