El problema con tu forma de validar la conexion a internet es que solo esta validando la existencia de una interfaz de red (wi fi o datos). Hay formas de validar la existencia de conexion de una interfaz de red, como la siguiente.
public boolean activeNetwork () {
ConnectivityManager cm =
(ConnectivityManager)getSystemService(Context.CONNECTIVITY_SERVICE);
NetworkInfo activeNetwork = cm.getActiveNetworkInfo();
boolean isConnected = activeNetwork != null &&
activeNetwork.isConnected();
return isConnected;
}
Pero de nuevo, si el dispositivo tiene activados los datos, pero no tiene saldo por decirlo de alguna manera, este metodo seguira retornando true incluso sin poder acceder a internet. La mejor forma de hacer esto es la siguiente.
public class CheckInternetAsyncTask extends AsyncTask<Void, Integer, Boolean> {
private Context context;
public CheckInternetAsyncTask(Context context) {
this.context = context;
}
@Override
protected Boolean doInBackground(Void... params) {
ConnectivityManager cm =
(ConnectivityManager)context.getSystemService(Context.CONNECTIVITY_SERVICE);
assert cm != null;
NetworkInfo activeNetwork = cm.getActiveNetworkInfo();
boolean isConnected = activeNetwork != null &&
activeNetwork.isConnected();
if (isConnected) {
try {
HttpURLConnection urlc = (HttpURLConnection)
(new URL("http://clients3.google.com/generate_204")
.openConnection());
urlc.setRequestProperty("User-Agent", "Android");
urlc.setRequestProperty("Connection", "close");
urlc.setConnectTimeout(1500);
urlc.connect();
if (urlc.getResponseCode() == 204 &&
urlc.getContentLength() == 0)
return true;
} catch (IOException e) {
Log.e("TAG", "Error checking internet connection", e);
return false;
}
} else {
Log.d("TAG", "No network available!");
return false;
}
return null;
}
@Override
protected void onPostExecute(Boolean result) {
super.onPostExecute(result);
Log.d("TAG", "result" + result);
if(result){
// do ur code
}
}
}
No olvides los permisos
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
<uses-permission android:name="android.permission.INTERNET" />
y se usa de la siguiente manera
new CheckInternetAsyncTask(getApplicationContext()).execute();
Este metodo lo he usado antes y me ha funcionado muy bien y lo he sacado de este post
https://stackoverflow.com/questions/1560788/how-to-check-internet-access-on-android-inetaddress-never-times-out
De la segunda respuesta, asi que si mi respuesta te sirve te solicito que visites el post y tambien le des un punto al autor original.
La explicacion del uso de este ultimo metodo es el siguiente. Se hace uso de una peticion asincrona a una api la cual resuelve una respuesta con un body vacio, por lo cual es bastante rapido y no consume muchos datos moviles. De esta manera puedes validar si tienes o no conexion a internet de forma verdadera.
Por ultimo cabe resaltar que si vas a realizar una peticion a una api remota no hay necesidad de validar si hay conexion a internet o no. Realiza tu peticion y en caso de el codio te arroje una expecion puedes validarla y con base en ella deducir cual fue el error y comunicarlo al usuario. Una de estas excepciones es por supuesto no tener acceso a internet.