You never want to have e.printStackTrace in production code.  It almost always means that you're not handling an exception correctly.  Usually the answer is to `throw new RuntimeException(e)` so you can propagate the error.  If you really want to swallow the error without propagating it, you can use Log.e() and add a comment that says `// intentionally ignored` or something to inform people that you are deliberately swallowing the error.