Почему крашится приложение android, при вызове okhttp?

Question

[run182]

Вот такой лог:

D/CONNECT:: START
D/AndroidRuntime: Shutting down VM
E/AndroidRuntime: FATAL EXCEPTION: main
                  Process: com.example.run18.flirtodrom, PID: 5385
                  java.lang.NullPointerException: println needs a message
                      at android.util.Log.println_native(Native Method)
                      at android.util.Log.d(Log.java:143)
                      at com.example.run18.flirtodrom.PlaceItem$GetProductDetails$1.run(PlaceItem.java:107)
                      at android.os.Handler.handleCallback(Handler.java:751)
                      at android.os.Handler.dispatchMessage(Handler.java:95)
                      at android.os.Looper.loop(Looper.java:154)
                      at android.app.ActivityThread.main(ActivityThread.java:6119)
                      at java.lang.reflect.Method.invoke(Native Method)
                      at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
                      at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

А вот такой код:

class GetProductDetails extends AsyncTask<String, String, String> {
        @Override
        protected void onPreExecute() {
            super.onPreExecute();
            pDialog = new ProgressDialog(PlaceItem.this);
            pDialog.setMessage("Loading product details. Please wait...");
            pDialog.setIndeterminate(false);
            pDialog.setCancelable(false);
            pDialog.show();
        }
        protected String doInBackground(String[] params) {
            runOnUiThread(new Runnable() {
                public void run() {
                    Log.d("CONNECT: ", "START");
                    try {
                        //GET
                        OkHttpClient client = new OkHttpClient();
                        Request request = new Request.Builder().url(url_db + "?action=get_place_by_id&id=" + pid).build();
                        Response response = client.newCall(request).execute();
                        String result = response.body().string();
                        Log.d("CONNECT: ", "OK");

                        /*JSONArray productObj = new JSONArray(result);
                        try {
                            if (1 == 1) {
                                JSONObject product = productObj.getJSONObject(0);

                                txtName = (EditText) findViewById(R.id.inputName);
                                txtPrice = (EditText) findViewById(R.id.inputPrice);
                                txtDesc = (EditText) findViewById(R.id.inputDesc);

                                txtName.setText(product.getString(TAG_NAME));
                                txtPrice.setText(product.getString(TAG_PRICE));
                                txtDesc.setText(product.getString(TAG_DESCRIPTION));
                            } else {
                                // продукт с pid не найден
                            }
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }*/
                    } catch (Exception e) {
                        Log.d("CONNECT: ", e.getMessage());
                    }
                }
            });
            return null;
        }
        protected void onPostExecute(String file_url) {
            pDialog.dismiss();
        }
    }

Пробовал комментить execute() - не выкидывает. Сервер данные отдает без ошибок. На предыдущем активити тоже есть аналогичный вызов okhttp, он проходит без ошибок.

Answer 1

[ipc_ngs]

По номеру строки можно определить, в каком месте ломается: PlaceItem.java:107 .
Скорее всего внутри catch при выполнении Log.d("CONNECT: ", e.getMessage()); из-за нулевого e.getMessage().
Оставьте там только e, и тогда увидите, какой именно эксепшн возникает при выполнении execute().

Comments

[run182]

Понял свою ошибку, сделал e.toString(). Спасибо за ответ!

Answer 2

[Rou1997]

} catch (Exception e) {
    Log.d("CONNECT: ", e.getMessage());
}

Кто так пишет?
Мало того, что это может привести вот к таким проблемам, вдобавок это неэффективно так как сообщения может не быть вовсе, или быть бесполезным.
Сам бог велел писать так:

} catch (Exception e) {
   e.printStackTrace();
}

Если этого недостаточно, то пишите свою util функцию, но тут уже хорошо думать надо.

Comments

[Artem Gapchenko]

Бог говнокода - может быть. А адекватный разработчик знает про уровни логирования, фильтрацию лог-сообщений, подмену того ресурса, в который будет писаться лог и printStackTrace() для вывода лога не использует.

[Rou1997]

Artem Gapchenko: Как ни странно, я тоже про все это знаю, но чаще всего использую именно printStackTrace(), по уровням логирования и фильтрации - ничто не мешает при необходимости сделать так.

catch (Exception ex) {
Log.i("FoobarActivity", "Exception");
e.printStackTrace();
}

Здесь и уровень и фильтрация, но это лучше, чем:

catch (Exception ex) {
SomeUntrustedUtil.logDebug("FoobarActivity", e);
}

Подменять выходной поток логирования никогда не приходилось - зачем?
И еще. Адекватному разработчику в идеале хорошо бы еще и уметь работать с кодом от неадекватных, в связи с чем в таких случаях перспективнее писать утилиты для рефакторинга (например, плагины для IDE), и тем самым решать проблему, а не уходить от нее, как при написании SomeUntrustedUtil для себя.

[Artem Gapchenko]

1) Это не лучше. Во-первых вы залогировали исключение на неправильном уровне (i - для информативных сообщений, а не для ошибок), во-вторых ваше сообщение на уровне логирования info ("Exception") успешно теряет всю нужную информацию, в-третьих задачу "Отфильтровать все сообщения из лога, происходящие в FoobarActivity, на уровне логирования info/warn/error/etc" ваш код как не решал, так и не решает.
2) Нужно написать свою собственную систему сбора крэшей, которая на отладочных версиях приложения пишет ошибку в стандартный поток, а на release версиях - в отдельный файл, содержимое которого периодически отсылается на удалённый сервер для сбора и анализа. Логгер (не стандартный андроидовский, а вообще как сама идея абстракции для логирования событий, происходящих в приложении) позволит быстро отделить одно от другого, printStackTrace() - нет.
3) Как связан пассаж про утилиты, плагины и SomeUntrustedUtil с тем, что я написал - не понятно.

[Rou1997]

Artem Gapchenko:

(i - для информативных сообщений, а не для ошибок)

Сарказм ON
Спасибо, не знал!
Сарказм OFF

вы залогировали исключение на неправильном уровне

Да это я тут "от балды", вы еще придеритесь к тому, что в заголовке catch стоит "ex", а в теле - "e" и это не будет компилироваться.
Но, и если подумать - а неплохая идея, именно "Exception" логировать на информационном уровне, чтобы независимо от установленного фильтра видеть есть ли оно в журнале, а сами подробности - скрыты. :)

в-третьих задачу "Отфильтровать все сообщения из лога, происходящие в FoobarActivity, на уровне логирования info/warn/error/etc" ваш код как не решал, так и не решает.

Берем IDE с реализацией клиента для adb logcat, которой пользуемся, затем берем утилиту для реверс-инжиниринга, которую сами написали... Максимум двенадцать часов работы - и вот, он уже решает задачу.
Извращение ли - добавить новую нестандартную возможность в Logcat, если она полезна, или хотя бы не помешает, поскольку стандартный Logcat так "хорошо" продуман, что помешать ему сложно? Очевидно, нет.
Извращение ли - делать это с применением утилиты для реверс-инжиниринга, если у IDE есть открытый код, но при условии, что использование утилиты, выборочно изменяющей код, гораздо быстрее, чем полная сборка, еще при условии, что разработка таких утилит - гораздо перспективнее, а также при том, что писать подобные утилиты попросту гораздо интереснее, чем собирать готовый код или даже писать обычный декомпилятор, просто декомпиляторы-то уже есть, как и просто дизассемблеры, а именно таких утилит - нет? Очевидно, нет.

Логгер (не стандартный андроидовский, а вообще как сама идея абстракции для логирования событий, происходящих в приложении) позволит быстро отделить одно от другого, printStackTrace() - нет.

Простейший плагин, написанный вами для Eclipse, прекрасно справится с рефакторингом такого кода, за считанные секунды заменив все printStackTrace() на то, что душе вашей угодно. Или обратно. Или не все, а только заданные.

Answer 3

[aol-nnov]

stackoverflow.com/questions/8237080/exception-getm...

номера строк пушкин будет проставлять? ответ, вообще-то в стектрейсе на расстоянии одного мышиного клика (если ты, конечно, не в блокноте программируешь)