Replace e.printStackTrace() with Log.e(TAG, "Error on ...", e);

[EmmanuelMess]

Replace e.printStackTrace() with Log.e(TAG, "Error on ...", e);

[HoussemNasri]

I found 174 occurrences of printStackTrace(), I am thinking of adding helper methods to LogHelper for future use, otherwise I'll have to add private static final TAG = ... for each class, @EmmanuelMess what do you think?

/**
* Helper method to log exceptions
*
* @param object the object where the exception was caught, used to retrieve the class name
* @param ex     the exception
*/
public static void e(Object object, Exception ex) {
      Log.e(object.getClass().getSimpleName(), ex.toString());
}

[VishalNehra]

This make a whole lot of sense actually. I was already planning to have a form of abstraction over the logging. So that we can come to know of the silent errors from users, not just crash reports.
With this we can implement logic to- say, append logs in a txt file and keep it as reference for user reports.

[EmmanuelMess]

I would see if there is a library for this, that already does this.

[EmmanuelMess]

@HoussemNasri this implementation could lead to leaking of parent class instance if used in anonymous objects (like lambdas)

[puranjayK]

@EmmanuelMess I would like to work on this issue. Could you please assign it to me?

[ilihas786]

i want to contribute i am begineer in open source

[VishnuSanal]

@puranjayK Please confirm if @ilihas786 can take this over.

@ilihas786 We'll assign it to you if there's no reply or if they confirm that they're no longer working on this.

[nilerudra]

Is this issue open?

[VishnuSanal]

@VishalNehra is this still valid for we're using slf4j now?