Проблема в потоке android

wifi.getDhcpInfo() может выбросить не только NPE. gateway volatile? ошибка в IntToStr?

А что ещё может выбросить? Там же ошибка будет, если не все catch впишу, так получается что больше ничего не надо получать.

gateway volatile?

А этот вопрос мне не понятен.

ошибка в IntToStr?

Не может быть. Там точно всё нормально, если пришло число адреса. Потом в другой функции используется gateway, и если бы gateway был ошибочен, то ничего бы не выполнилось, а так другое выполняется нормально, используя gateway как строку ip адресом.

В твоем коде не видно как объявлен gateway, ты пишешь и читаешь эту переменную в разных потоках, java не гарантирует что соседний поток увидит изменения другого потока, с volatile гарантирует.

Во-первых ты здесь канонически налажал с памятью. Ты удерживаешь сильную ссылку на активити, она в свою очередь держит ссылки на вьюшки, вьюшки на контекст и поехали... вниз по цепочке вплоть до самых недр. Во-вторых, в голую с потоками никто не работает, это очень херовая практика. Тебе надо:
1) Вынести тяжелую работу в другой поток
2) Быть уверенным в том, что все вызовы обновления UI дергаются из основного потока, иначе ты получишь CalledFromWrongThreadException.
В ведре есть AsyncTask, выглядит он, конечно, уродски, но свою работу выполняет. Поковыряй в его сторону. Можно еще заюзать Executor из Java + runOnUiThread, но это более велосипедно, кмк.

Попробую volatile, но используя gateway_label.post <- это как раз то, чтобы написать в главном потоке из другого потока.

volatile не помог.

делая .post выполнение считается потоко-безопасным.

Ни разу, вообще и никакой связи. Потокобезопасность - это обмазывание блокировками и контроль доступа к данным (чтение / запись) в многопоточной среде. Post - это сугубо ведроидная штука. У тебя есть Looper, который привязан к основному потоку, looper обрабатывает сообщения из т.н. message queue. Когда ты делаешь .post - ты просто добавляешь message в эту самую очередь, который потом будет обработан looper'ом. Но это никаких гарантий потокобезопасности тебе не даёт.

try {
    final WifiManagar wifi = (WifiManager) getActivity().getApplicationContext().getSystemService(Context.WIFI_SERVICE);
    final DhcpInfo di = wifi.getDhcpInfo();
    final int gateway = IntToStr(di.gateway);
    gateway_label.post(new Runnable() {
         @Override
         public void run() {
             gateway_label.setText("wtf.ok[" + gateway + "]");
         }
    });
} catch (Exception e) {
     gateway_label.post(new Runnable() {
         @Override
         public void run() {
             gateway_label.setText("wtf.fail[" + e.getMessage() + "]");
         }
    });
}

что лейбл так покажет? (мб не скомпилится)

https://developer.android.com/guide/components/processes-and-threads.html?hl=ru

Найди там «Рабочие потоки»

Ничего не покажет, а gateway это String. Я же говорю, что gateway работает потом нормально, то есть в других функциях он используется и выполняется правильно.

может у тебя gateway_label невидим, напиши туда что-нибудь изначально.

Так пишется, если не в потоке писать.

Вот что помогло.

                getActivity().runOnUiThread(new Runnable() {

Всем спасибо.

Учи матчасть. Из документации

public void onClick(View v) {
    new Thread(new Runnable() {
        public void run() {
            final Bitmap bitmap = loadImageFromNetwork("http://example.com/image.png");
            mImageView.post(new Runnable() {
                public void run() {
                    mImageView.setImageBitmap(bitmap);
                }
            });
        }
    }).start();
}

У тебя судя по написанию

wifi = (WifiManager) getActivity().getApplicationContext().getSystemService(Context.WIFI_SERVICE);
di = wifi.getDhcpInfo();
gateway = IntToStr(di.gateway);

Что ты подразумеваешь под сильной ссылкой?

Обычная ссылка в Java считается сильной. Это тебе тоже стоит изучить. Есть 4 типа: Strong, Weak, Soft, Phantom. Тип ссылки влияет на стратегию её сбора GC. В Android это особенно актуальная проблема.

public void onClick(View v) {
    new Thread(new Runnable() {
        public void run() {
            final Bitmap bitmap = loadImageFromNetwork("http://example.com/image.png");
            mImageView.post(new Runnable() {
                public void run() {
                    mImageView.setImageBitmap(bitmap);
                }
            });
        }
    }).start();
}

А если у меня фрагменты, то тоже будет утечка, а если статичные?

Разницы нет. К утечке приводит связка сильной ссылки и того, что Android уничтожает Activity со всеми потрохами при повороте. Можно, конечно, задать в манифесте ignore config changes, тогда при повороте уничтожения происходить не будет, но это повлечет за собой другие проблемы. Я обычно делаю конструкцию, когда обработчик асинхронного запроса удерживается как слабая ссылка (WeakReference). Результат асихронного запроса будет потрачен (но это так и так произойдет), но зато ты не протечёшь по памяти.

По подробней о слабой ссылке?

Что именно подробнее? Про механизмы работы - это тебе про GC надо читать. Я могу только чуть позже пример накидать. Если есть телега - напиши мне. @v_sulimov