Web视图onReceivedError已处理，但仍显示网页不可用

Question

I was trying to show a web page on my web view. and in case if an error occurs or page loading fails I want to show a sad smiley and a text showing that your page is not loaded or anything like that. and for that I tried to put my code inside onReceivedError() but somehow it does not work except toasts and logs.

My code is as follows:

webView.setWebViewClient(new WebViewClient(){

            @Override
            public void onPageStarted(WebView view, String url, Bitmap favicon) {
                super.onPageStarted(view, url, favicon);
                actionBar.setTitle(Constants.TITLE_VERIFYING);
                hideError();
                showProgress();
                Toast.makeText(WebViewActivity.this, "start loading", Toast.LENGTH_SHORT).show();
            }

            @Override
            public void onPageFinished(WebView view, String url) {
                super.onPageFinished(view, url);
                actionBar.setTitle(Constants.TITLE_VERIFIED);
                hideError();
                hideProgress();
                Toast.makeText(WebViewActivity.this, "Web view jas loaded", Toast.LENGTH_SHORT).show();
            }

            @Override
            public void onReceivedError(WebView view, int errorCode, String description, String failingUrl) {


                hideProgress();
                showError();
                Toast.makeText(WebViewActivity.this, "Could not load your page", Toast.LENGTH_SHORT).show();
                super.onReceivedError(view,errorCode,description,failingUrl);
                Toast.makeText(WebViewActivity.this, "error", Toast.LENGTH_SHORT).show();
            }
        });

Other methods are:

private void showProgress() {

    avLoadingIndicatorView.setVisibility(View.VISIBLE);
        loadingText.setVisibility(View.VISIBLE);
        avLoadingIndicatorView.show();
        webView.setAlpha(Constants.ALPHA_LAYOUT);
    }

    private void hideProgress() {
        avLoadingIndicatorView.hide();
        webView.setAlpha(1f);
        loadingText.setVisibility(View.GONE);
        avLoadingIndicatorView.setVisibility(View.GONE);
    }

    private void showError(){
        webView.setVisibility(View.GONE);
        sadSmiley.setVisibility(View.VISIBLE);
        errorText.setVisibility(View.VISIBLE);
    }

    private void hideError(){
        webView.setVisibility(View.VISIBLE);
        sadSmiley.setVisibility(View.GONE);
        errorText.setVisibility(View.GONE);
    }

Layout code:

<android.support.constraint.ConstraintLayout 

xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".activity.WebViewActivity">

    <android.support.v7.widget.Toolbar
        android:id="@+id/toolbar"
        android:layout_width="match_parent"
        android:layout_height="?attr/actionBarSize"
        android:background="@color/primary"
        app:titleTextColor="@color/white"
        android:minHeight="?attr/actionBarSize" />

    <WebView
        android:id="@+id/web_view"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:layout_marginTop="?attr/actionBarSize"
        />

    <com.wang.avi.AVLoadingIndicatorView
        android:id="@+id/loading"
        style="AVLoadingIndicatorView.Small"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_margin="10dp"
        android:padding="10dp"
        android:elevation="10dp"
        app:indicatorColor="@color/progress"
        app:indicatorName="BallScaleIndicator"
        android:visibility="gone"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"/>

    <TextView
        android:id="@+id/loading_text"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        style="@style/tvDarkLargeStyle"
        android:text="Verifying Your Document"
        android:visibility="gone"
        android:textColor="@color/primary"
        app:layout_constraintTop_toBottomOf="@+id/loading"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"/>

    <ImageView
        android:id="@+id/img_sad_smiley"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:src="@drawable/sad_smiley_primary_64"
        android:layout_margin="10dp"
        android:padding="10dp"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        android:visibility="gone"/>

    <TextView
        android:id="@+id/error_text"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        style="@style/tvDarkLargeStyle"
        android:text="Sorry! we could not load your page!"
        android:textColor="@color/primary"
        app:layout_constraintTop_toBottomOf="@+id/img_sad_smiley"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        android:textAlignment="center"
        android:layout_margin="10dp"
        android:visibility="gone"
        />
</android.support.constraint.ConstraintLayout>

Please help me. Thanks.

Answer 1

Because onReceivedError called before onPageFinished , so your sadSmiley and errorText gone.

Try below code,

boolean errorOccurred = false; // Global variable

webView.setWebViewClient(new WebViewClient() {
    @Override
    public void onPageStarted(WebView view, String url, Bitmap favicon) {
        super.onPageStarted(view, url, favicon);
        hideError();
        showProgress();
        Toast.makeText(Test.this, "start loading", Toast.LENGTH_SHORT).show();
        errorOccurred=false;
    }
    @Override
    public void onPageFinished(WebView view, String url) {
        super.onPageFinished(view, url);
        if (!errorOccurred) {
            hideError();
        }
        hideProgress();
        Toast.makeText(Test.this, "Web view was loaded", Toast.LENGTH_SHORT).show();
    }
    @Override
    public void onReceivedError(WebView view, int errorCode, String description, String failingUrl) {
        errorOccurred = true;
        hideProgress();
        showError();
        Toast.makeText(Test.this, "Could not load your page", Toast.LENGTH_SHORT).show();
        super.onReceivedError(view, errorCode, description, failingUrl);
        Toast.makeText(Test.this, "error", Toast.LENGTH_SHORT).show();
    }
});

Answer 2

Move the line super.onReceivedError(view,errorCode,description,failingUrl); above hideProgress(); in onReceivedError() method.