在Android中用e.printStackTrace()

Is it a bad idea to use printStackTrace() in Android Exceptions?

up vote 35 down vote favorite 7	Is it a bad idea to use printStackTrace() in Android Exceptions like this? } catch (Exception e) { e.printStackTrace(); } java  android  exception  printstacktrace share improve this question asked  Oct 4 '10 at 12:26 jacknad 4,135 29 87 159
	add a comment

5 Answers

active oldest votes

up vote 22 down vote accepted	Yes, it is a bad idea. You should instead use Android's built-in log class specifically designed for these purposes: http://developer.android.com/reference/android/util/Log.html It gives you options to log debug messages, warnings, errors etc. share improve this answer answered  Oct 4 '10 at 13:28 Nailuj 10.9k 10 54 84
	add a comment

up vote 26 down vote

I believe this is what you need: catch (Exception e) { Log.e(TAG,Log.getStackTraceString(e)); } share improve this answer edited Nov 11 '11 at 22:25 sth 132k 34 214 315 answered  Oct 25 '11 at 17:12 Adil Atilgan 279 3 3

23   Nope. What the OP needs is Log.e(TAG, "Explanation of what was being attempted when the exception was thrown", e). Note the third parameter. Log.e(String,String,Throwable) gets the stacktrace string from the Throwable for you. Use the message parameter for something meaningful. –  spaaarky21  Jun 20 '14 at 23:27  @spaaarky21 You should make that an answer. –  AndreKR  May 5 '16 at 3:22 @AndreKR Someone already did. :) And I upvoted it. I just wanted to point out that this answer is promoting a bad practice, since it's rated so highly compared to correct answers. –  spaaarky21  May 5 '16 at 16:54 @spaaarky21 I learned from your answer that I can pass an Exception to android.util.Log.e(). I didn't learn that from any of the other answers. (I didn't click the link in Nailuj's answer.) –  AndreKR  May 5 '16 at 16:56 @AndreKR Thanks. And good point. I was referring to Ryan's answer but now that I look again, I see that it's using Log in an unusual way – using a Log instance (I assume?) and passing the level in as a parameter. I added an answer. –  spaaarky21  May 5 '16 at 18:09