java.lang.NullPointerException : Tentativo di richiamare il metodo virtuale ' android.view.View .MainActivity.findViewById ( int) ' su un riferimento oggetto null

? Kaos @ | Original: StackOverFlow
---

Ho una classe chiamata MainActivity.java che chiamano una classe AsyncTask . L' ultima classe hanno un findViewById() che in esecuzione restituisce questo errore :

java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.View <mypackage>.MainActivity.findViewById(int)' on a null object reference

Non capisco come posso modificare un ImageView posizionato R.layout.activity_main, dopo che un AsyncTask finish al lavoro .

MainActivity.java

public class MainActivity extends Activity {

    public MainActivity() {}

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

}

Connection.java

public class Connection extends AsyncTask<String, Void, String> {
    public String result;

    //I know, this isn't correct, how can i do?
    public MainActivity MainActivity;

    @Override
    protected String doInBackground(String... arg0) {
        // TODO Auto-generated method stub
                //...

        return "a string";
    }

    protected void onPostExecute(String result) {
        super.onPostExecute(result);
                    //...

            // Where the error is generated
            ImageView image = (ImageView) MainActivity.findViewById(R.id.image);

            //...

    }
}
---

Top 5 Risposta

1Lucian Novac @

mettere ImageView come una variabile della classe

immagine ImageView privato;

sul tuo initialize onCreate

image = (ImageView) findViewById(R.id.image);

Collegamento public class estende AsyncTask {     risultato String pubblico ;

//I know, this isn't correct, how can i do?
public MainActivity MainActivity;

@Override
protected String doInBackground(String... arg0) {
    // TODO Auto-generated method stub
            //...

    return "a string";
}

protected void onPostExecute(String result) {
    super.onPostExecute(result);
                //...

        // Where the error is generated
       //do other stuff with your imageview

        //...

}

}

2Devill @

Move this line

ImageView image = (ImageView) MainActivity.findViewById(R.id.image);

a onCreate () con

 ImageView image = (ImageView)findViewById(R.id.image);
3Doctoror Drive @

L'errore è che

public MainActivity MainActivity;

non viene mai inizializzato, indicando in tal modo a null . Per rendere il vostro lavoro codice il passo minimo viene in MainActivity

new Connection(this).execute();

In Connection

public class Connection extends AsyncTask<String, Void, String> {

    public MainActivity MainActivity;

    public Connection(MainActivity activity) {
        MainActivity = activity;
    }

Ma la creazione di un'attività in onCreate e passando una attività non è comunque l'idea migliore . Inoltre, i nomi dei campi devono sempre iniziare con una lettera minuscola .

Il modo migliore passa un ImageView al AsyncTask . Non iniziare un compito fino all'avvio della attività e anche, non dimenticare di annullare l' operazione quando l'attività viene arrestata .

public final class MainActivity extends Activity {

    public MainActivity() {}

    private Connection connection;
    private ImageView imageView;

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

        imageView = (ImageView) findViewById(R.id.image);
    }

    @Override
    protected void onStart() {
        super.onStart();
        if (connection == null || connection.getStatus() != AsyncTask.Status.RUNNING) {
            connection = new Connection(imageView);
            connection.execute();
        }
    }

    @Override
    protected void onStop() {
        super.onStop();
        if (connection != null && connection.getStatus() == AsyncTask.Status.RUNNING) {
            connection.cancel(true);
        }
    }

}

In Connection.java, memorizzare un ImageView come WeakReference per evitare perdite .

public final class Connection extends AsyncTask<String, Void, String> {

    private final WeakReference<ImageView> imageViewRef;

    public Connection(ImageView view) {
        imageViewRef = new WeakReference<ImageView>(view);
    }

    @Override
    protected String doInBackground(String... arg0) {
        // TODO Auto-generated method stub
                //...

        return "a string";
    }

    @Override
    protected void onPostExecute(String result) {
        super.onPostExecute(result);
                //...

        final ImageView imageView = imageViewRef.get();
        // if the Activity is still alive, the ImageView will not be null
        if (imageView != null) {
            // set an image or whatever you need
            image.setImageResource(666);
        }

    }