Как правильно использовать AsyncTask на Android?

Question

Мне нужна помощь в проверке того, подключен ли мой телефон к Интернету. Я начал с pinging IP для проверки соединения. Выглядело это так:

protected Boolean checkInternetConnection(Void... params) { 
     try { 
      InetAddress ipAddr = InetAddress.getByName("https://www.google.com"); 

      if (!ipAddr.isReachable(10000)) { 
       return false; 
      } else { 
       return true; 
      } 

     } catch (Exception e) { 
      Log.e("exception", e.toString()); 
      return false; 
     } 
    } 
} 

Однако ВСЕГДА бросил NetworkOnMainThreadException, так что я использовал AsyncTask:

private class CheckConnectionTask extends AsyncTask<Void, Void, Boolean> { 
    @Override 
    protected Boolean doInBackground(Void... params) { 
     try { 
      InetAddress ipAddr = InetAddress.getByName("https://www.google.com"); 

      if (!ipAddr.isReachable(10000)) { 
       return false; 
      } else { 
       return true; 
      } 

     } catch (Exception e) { 
      Log.e("exception", e.toString()); 
      return false; 
     } 
    } 
} 

Является ли этот код правильно? Потому что я не знаю, как это назвать. Я пробовал:

new CheckConnectionTask().execute(); 

Что-нибудь не хватает там? Потому что это не работает. Также обратите внимание, что я видел много предыдущих вопросов, но я не нашел ответа на эту проблему, поэтому я задал свой вопрос. Если вы видели ответ на вопрос, который может решить мою проблему, вы можете связать его здесь, но имейте в виду, что я не знаком с Android или Java, так что это может быть непонятно мне. Я бы предпочел правильный код в качестве ответа и короткое объяснение, почему мое не сработало. Кроме того, мне нужно знать, подключен ли телефон к ИНТЕРНЕТ, а не СЕТЬ, поэтому ConnectivityManager не будет беспокоить меня.

EDIT - благодарю вас всех за ответы. Прежде всего, у меня есть все необходимые разрешения. Кроме того, я не могу получить каких-либо результатов в настоящее время, как и Android-студия выделяет следующий код:

Boolean internetConnection; 
internetConnection = new  CheckConnectionTask().execute(); 

Как неправильно и он просто не позволит мне вызвать функцию. Что с этим не так? Не хватает ли каких-либо параметров? Потому что я определил params как Void, так что кажется нелогичным.

Edit 2 - Я использовал onPostExecute, как это было предложено @vandaics, и это выглядит, как это сейчас:

private class CheckConnectionTask extends AsyncTask<Void, Void, Boolean> { 
    @Override 
    protected Boolean doInBackground(Void... params) { 
     try { 
      InetAddress ipAddr = InetAddress.getByName("google.com"); 

      if (!ipAddr.isReachable(10000)) { 
       return false; 
      } else { 
       return true; 
      } 

     } catch (Exception e) { 
      Log.e("exception", e.toString()); 
      return false; 
     } 
    } 
    @Override 
    protected void onPostExecute(Boolean result) { 
     super.onPostExecute(result); 
     internetConnection = result; 
    } 
} 

Он работает, и я называю это при вызове метода OnCreate:

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_main); 
    new CheckConnectionTask().execute(); 
} 

Работает, мои приложения проверяют, подключен ли Интернет и реагирует должным образом. Если вы считаете, что что-то может не работать, или есть более простой способ сделать это, дайте мне знать. Большое спасибо вам, ребята, особенно @vandaics и @Sunil. Кроме того, мне нужно использовать суперкласс, как здесь ?:

@Override 
    protected void onPostExecute(Boolean result) { 
     super.onPostExecute(result); 
     internetConnection = result; 
    } 

Что он делает и это необходимо?

Answer 1

, чтобы проверить, если подключение к Интернету доступен, вы можете сделать это:

private boolean isNetworkAvailable() { 
    ConnectivityManager connectivityManager = (ConnectivityManager) getSystemService(Context.CONNECTIVITY_SERVICE); 
    NetworkInfo activeNetworkInfo = connectivityManager.getActiveNetworkInfo(); 
    return activeNetworkInfo != null && activeNetworkInfo.isConnectedOrConnecting(); 
    //return activeNetworkInfo != null && activeNetworkInfo.isConnected(); 
} 

Вы не можете позвонить

Boolean internetConnection; 
internetConnection = new CheckConnectionTask().execute(); 

возвращаемое значение excute() не то, что вы хотите. Если вы хотите использовать asyntask вы можете установить логическую переменную в onPostExecute(...):

private boolean isConnect = false; 
private class CheckConnectionTask extends AsyncTask<Void, Void, Boolean> { 
@Override 
protected Boolean doInBackground(Void... params) { 
    try { 
     InetAddress ipAddr = InetAddress.getByName("https://www.google.com"); 

     if (!ipAddr.isReachable(10000)) { 
      return false; 
     } else { 
      return true; 
     } 

    } catch (Exception e) { 
     Log.e("exception", e.toString()); 
     return false; 
    } 
} 
@Override 
protected void onPostExecute(Boolean result) { 
    super.onPostExecute(aVoid); 
    isConnect = result; 
} 
} 

Но, я никогда не использовать этот подход, чтобы получить статус подключения к Интернету, обусловленно я должен запустить этот asyntask, прежде всего, чтобы получить точный статус.

Answer 2

Если вы собираетесь использовать AsyncTaks вы пропустили onPostExecute метод:

protected void onPostExecute(Boolean result) { 
    //your code here 
} 

Вы также можете добавить дополнительный onProgressUpdate метод:

protected void onProgressUpdate(Integer... progress) { 
    //do something on update 
} 

Тогда для выполнения вы делаете:

новый DownloadFilesTask(). Execute();

Вот хороший пример, который вы можете использовать: http://developer.android.com/reference/android/os/AsyncTask.html

Answer 3

Вы в настоящее время не видят ничего, потому что у вас нет никаких заявлений в журнале или вы действительно не проверяя ни за что.

Вызов

новый CheckConnectionTask(). Выполнить()

работает в AsyncTask. Просто для вывода задачи нет вывода.

вы можете проверить выход и посмотреть, действительно ли это то, что вы хотите.

private class CheckConnectionTask extends AsyncTask<Void, Void, Boolean> { 
    @Override 
    protected Boolean doInBackground(Void... params) { 
     try { 
      InetAddress ipAddr = InetAddress.getByName("https://www.google.com"); 

      if (!ipAddr.isReachable(10000)) { 
       return false; 
      } else { 
       return true; 
      } 

     } catch (Exception e) { 
      Log.e("exception", e.toString()); 
      return false; 
     } 
    } 

    @Override 
    public void onPostexecute(Boolean result) { 
     // TO DO on the UI thread 
     Log.i("AsyncTask", "Result = " + result"); 
    } 
} 

EDIT: Вызов:

новый CheckConnectionTask() выполнить()

возвращает экземпляр AsyncTask, что он выполняется и не логическое значение (хотя. вы указываете, что Boolean должен быть выходным). Следовательно, здесь вы видите ошибку компиляции. AsyncTask был разработан как автономный - он делает все, что должен делать, а затем завершается. Если вам нужно изменить переменную экземпляра уровня класса (из класса, содержащего AsyncTask, как внутреннюю задачу), которая может быть выполнена, но не предлагается.