NullPointerException beim schnellen scrollen

SvenF · 9. Januar 2013

Hallo,
ich habe ein Problem das mich schon seit einigen Tagen ärgert und für das ich keine richtige Lösung finde:
Ich habe ein kleines Programm gebastelt, welches irgendwann mal zu einer Universalfernbedienung heranwachsen soll. Soweit habe ich eine Liste mit einem eigenen Listenadapter erstellt, welche verschiedene Kategorien und für jedes Element noch einen Löschen Button beinhalten soll. Die Liste zeigt verschiedene Geräte an, die mit der Fernbedienung bedient werden können. Die Daten welche angezeigt werden sollen sind in einer internen Datenbank gespeichert. Mit dem letzten Eintrag der Liste gelangt man zu einer neuen activity wo man aus einer online Datenbank neue Geräte in die Datenbank des Handys laden kann.
Soweit so gut. Es funktioniert ansich auch alles wie es soll, bis man anfängt etwas schneller zu scrollen. Dann stürzt das Programm nämlich ab und ich kann absolut nicht entdecken woran das liegen kann:

Java

package com.mamlambo.tutorial.tutlist;


import java.util.ArrayList;
import java.util.TreeSet;


import android.os.AsyncTask;
import android.os.Bundle;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.view.View.OnClickListener;
import android.app.Activity;
import android.content.Context;
import android.content.Intent;
import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.widget.BaseAdapter;
import android.widget.ListView;
import android.widget.TextView;
import android.widget.ImageView;
import android.graphics.Bitmap;
import android.graphics.BitmapFactory;


public class TutListActivity extends Activity {
	
	private DatabaseManager mHelper = new DatabaseManager(this);
	private SQLiteDatabase mDatenbank;
	private ArrayList&lt;String&gt; devicesList;
	private TreeSet&lt;Integer&gt; mSeparatorsSet = new TreeSet&lt;Integer&gt;();
	ListView listView;
	EfficientAdapter objectAdapter;
	
	private class EfficientAdapter extends BaseAdapter {
    	private LayoutInflater mInflater;
    	private Bitmap mIcon1;
    	private Bitmap mIcon2;
            	
    	private static final int TYPE_ITEM = 0;
    	private static final int TYPE_SEPARATOR = 1;
    	private static final int TYPE_MAX_COUNT = TYPE_SEPARATOR + 1;


    	public EfficientAdapter(Context context) {
        	mInflater = LayoutInflater.from(context);


        	mIcon1 = BitmapFactory.decodeResource(context.getResources(), R.drawable.icon);
        	mIcon2 = BitmapFactory.decodeResource(context.getResources(), R.drawable.icon);
    	}
	
    	public int getCount() {
        	return devicesList.size();
    	}


    	public Object getItem(int position) {
        	return position;
    	}


    	public long getItemId(int position) {
        	return position;
    	}
    	
    	@Override
    	public void notifyDataSetChanged() // Create this function in your adapter class
    	{
        	super.notifyDataSetChanged();
    	}
    	
    	@Override
    	public int getItemViewType(int position) {
        	return mSeparatorsSet.contains(position) ? TYPE_SEPARATOR : TYPE_ITEM;
    	}
    	
    	public View getView(int position, View convertView, ViewGroup parent) {
        	ViewHolder holder;
        	
        	int type = getItemViewType(position);
        	
        	if (convertView == null) {
            	
            	holder = new ViewHolder();
            	
            	switch (type) {
	            	case TYPE_ITEM:
	            		convertView = mInflater.inflate(R.layout.list_item, null);
	                	holder.text = (TextView) convertView.findViewById(R.id.Title_List_Item);
	                	holder.icon = (ImageView) convertView.findViewById(R.id.Delete_List_Item);
	                	break;
	            	case TYPE_SEPARATOR:
	                	convertView = mInflater.inflate(R.layout.seperator_item, null);
	                	holder.text = (TextView)convertView.findViewById(R.id.Text_Seperator_Item);
	                	break;
            	}
                                         	
            	convertView.setTag(holder);
            	
        	} else {
            	holder = (ViewHolder) convertView.getTag();
        	}
        	


        	System.out.println(convertView);
        	System.out.println(devicesList);
        	System.out.println(position);


        	
        	switch (type) {
        		case TYPE_ITEM:
		        	convertView.findViewById(R.id.Title_List_Item).setTag(devicesList.get(position));
		        	convertView.findViewById(R.id.Delete_List_Item).setTag(devicesList.get(position));


                	holder.text.setText(devicesList.get(position));
                	holder.icon.setImageBitmap((position & 1) == 1 ? mIcon1 : mIcon2);
                	
		        	holder.text.setOnClickListener(mOnTitleClickListener);
		        	holder.icon.setOnClickListener(mOnIconClickListener);
        			break;
        		 case TYPE_SEPARATOR:
                 	holder.text.setText(devicesList.get(position));
        			 break;
        	}
        	
        	return convertView;
    	}




    	private OnClickListener mOnTitleClickListener = new OnClickListener() {
        	public void onClick(View v) {
	            	if(v.getTag().toString() == "ADD DEVICE"){
	                	Intent addDevice = new Intent(getApplicationContext(), addDevice.class);
	                	startActivity(addDevice);
	            	}else{
	                	Intent showRemote = new Intent(TutListActivity.this, Remote.class);
	                	showRemote.putExtra("selectedDevice", v.getTag().toString());
	                	startActivity(showRemote);
	            	}
        	}
    	};


    	private OnClickListener mOnIconClickListener = new OnClickListener() {
        	public void onClick(View v) {
        		String NameOfDevice[] = new String[2];
        		NameOfDevice = v.getTag().toString().split(",");
        		mDatenbank.execSQL("DELETE FROM addedDevices WHERE name = '"+NameOfDevice[0]+"'");
        		
        		ladeDevices();
        		objectAdapter.notifyDataSetChanged();
        	}
    	};
    	
    	class ViewHolder {
        	TextView text;
        	ImageView icon;
    	}
	}
	
	
	public void onCreate(Bundle savedInstanceState) {
    	super.onCreate(savedInstanceState);
    	
    	setContentView(R.layout.mainlayout);
		    			
		listView = (ListView) findViewById(R.id.list);
		objectAdapter = new EfficientAdapter(this);


		new LoadDevices().execute();
		
	}
	
   	
	private class LoadDevices extends AsyncTask&lt;String, Integer, String&gt; {
		
		@Override
		protected void onPreExecute() {
        	mDatenbank = mHelper.getReadableDatabase();
		}
		
		@Override
		protected String doInBackground(String... params) {
			devicesList = new ArrayList&lt;String&gt;();
			Cursor devicesCursor = mDatenbank.query("addedDevices", new String[] {"name", "hersteller"}, null, null, null, null, null);
			startManagingCursor(devicesCursor);
			devicesCursor.moveToFirst();


			for (int i=0;i&lt;devicesCursor.getCount();i++){ 
				String addingElement = new String(devicesCursor.getString(0)+", "+devicesCursor.getString(1));
    			devicesList.add(addingElement);
				devicesCursor.moveToNext();
            	if (i % 4 == 0) {
            		devicesList.add("separator " + i);
                	mSeparatorsSet.add(devicesList.size() - 1);
            	}
			} 
			devicesList.add("ADD DEVICE");
			
			return "All Done!";
		}
		
		@Override
		protected void onProgressUpdate(Integer... values) {
		}
		
		@Override
		protected void onPostExecute(String result) {
			listView.setAdapter(objectAdapter);
			mDatenbank.close();
		}
	}
	
	
	
	private void ladeDevices() {
		}
	
}

Alles anzeigen

Ich habe die Datenbank abfrage schon in einen anderen thread ausgelagert, aber die Fehlermeldung bleibt die Gleiche:

Code

01-09 16:27:42.415: D/AndroidRuntime(24083): Shutting down VM
01-09 16:27:42.415: W/dalvikvm(24083): threadid=1: thread exiting with uncaught exception (group=0x4001d5a0)
01-09 16:27:42.536: E/AndroidRuntime(24083): FATAL EXCEPTION: main
01-09 16:27:42.536: E/AndroidRuntime(24083): java.lang.NullPointerException
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at com.mamlambo.tutorial.tutlist.TutListActivity$EfficientAdapter.getView(TutListActivity.java:137)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.AbsListView.obtainView(AbsListView.java:1428)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.ListView.makeAndAddView(ListView.java:1801)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.ListView.fillDown(ListView.java:671)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.ListView.fillGap(ListView.java:642)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.AbsListView.trackMotionScroll(AbsListView.java:3452)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.AbsListView.onTouchEvent(AbsListView.java:2256)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.widget.ListView.onTouchEvent(ListView.java:3502)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.View.dispatchTouchEvent(View.java:3933)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:955)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1015)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1015)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1015)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:1015)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at com.android.internal.policy.impl.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:1877)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at com.android.internal.policy.impl.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1211)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.app.Activity.dispatchTouchEvent(Activity.java:2228)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at com.android.internal.policy.impl.PhoneWindow$DecorView.dispatchTouchEvent(PhoneWindow.java:1852)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewRoot.deliverPointerEvent(ViewRoot.java:2382)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.view.ViewRoot.handleMessage(ViewRoot.java:2010)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.os.Handler.dispatchMessage(Handler.java:99)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.os.Looper.loop(Looper.java:150)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at android.app.ActivityThread.main(ActivityThread.java:4385)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at java.lang.reflect.Method.invokeNative(Native Method)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at java.lang.reflect.Method.invoke(Method.java:507)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:849)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:607)
01-09 16:27:42.536: E/AndroidRuntime(24083): 	at dalvik.system.NativeStart.main(Native Method)

Alles anzeigen

Line 137 ist folgende:

Code

convertView.findViewById(R.id.Title_List_Item).setTag(devicesList.get(position));

Allerdings wird weder convertView noch position oder devicesList zu null. Zudem bleibt der Fehler der gleiche, wenn ich die Zeile auskommentiere. Er zeigt dann nur auf die nächste Zeile.
Ich hoffe mir kann jemand helfen. Ich bin fast am verzweifeln...

Ben · 9. Januar 2013

Wenn du das Problem auskommentierst entfernst du nicht den Ursprung.

wenn ich da was von "aus dem internet laden" höre vermute ich ein Threadproblem :o

und den holder solltest du lieber nicht als Methodenvariable angeben, das ist irgendwie sinnfrei

Marco Feltmann · 10. Januar 2013

Sieht so aus als lieferte convertView.findViewById() nix, also ein null zurück - und zwar nur beim zu schnellen Scrollen.

Mit dem ViewHolder pattern sollst du ja zu viel Rumgesuche via findViewById() vermeiden, hier machst du es aber permanent. Das halte ich für keine gute Lösung und offenbar bekommst du dadurch auch die Crashes.

Leg dir diese Views lieber als Instanzvariable an oder pack sie dir mit als View in den ViewHolder.
Dann musst du halt nur beim Zuweisen der Tags erst einmal prüfen, ob die Views auch ja nicht null sind.

Ich würde übrigens auch das iconview deines ViewHolders beim Zuweisen des Separators nullen - die ConvertViews werden gern wiederverwertet und es kann durchaus passieren, dass dein Separator dann noch das vorherige Icon hat. Muss nicht zu Problemen führen, kann aber. Zumindest bei der Darstellung.

titus · 10. Januar 2013

Hi SvenF,

ich kann meinen beiden Vorredner in ihrem Verdacht nur zu stimmen.

Hier wird nochmal dargestellt wie du es machen solltest:
http://www.vogella.com/article…#adapterperformance_hoder

Gibt es eigentlich einen Grund warum du deine Icons in Bitmaps speicherst um sie dann der ImageView zu zu weisen?

mfg Titus

Ben · 14. Januar 2013

da hat titus recht, warum packst du die bilder nicht einfach in einen der drawable-* ordner und weißt diese dann per setImageRessource(R.drawable.bildname) zu?

NullPointerException beim schnellen scrollen

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema