cursorindexoutofboundsexception

UweApps · 15. März 2012

Wenn du gerade vorher eine leere ArrayList angelegt hast, dann ist da wahrscheinlich auch nichts drin, oder?
Du solltest schon eine Liste mit Inhalten nehmen, sonst wird das auch nix...

Probiere doch vorher auch noch mal PlayerList.size() - das gibt dir die Anzahl der Elemente, ich denke mal, das ist bei dir 0, oder??

M3doXX · 15. März 2012

Ja das stimmt wohl..

Also Ausgangslage ist ja folgende:

Ich habe eine listView in der eine Tabelle voller Player angezeigt wird. Wenn ich nun auf einen Player klicke, dann öffnet sich eine neue Activity, in der ich nur den Datensatz von dem Player brauche, auf den ich vorher geklickt habe.
Um die gesamte Liste zu holen benutze ich ja folgende Methode:

Java

public void ListTeam (View view) {
		
		setContentView(R.layout.team);
		
		PlayerList.clear();
		try {
			
			playerdatasource.open();
			PlayerList = playerdatasource.getAllEntriesPlayer();
			playerdatasource.close();
			
			
		}
		catch (Exception ex) {
			Toast.makeText(this, ex.toString(), Toast.LENGTH_SHORT).show();
		}


		ArrayAdapter&lt;Entry&gt; adapterVerlauf = new ArrayAdapter&lt;Entry&gt;(Team.this, android.R.layout.simple_list_item_1, PlayerList);
		
		ListView lPlayer = (ListView) findViewById(R.id.listViewPlayer);
		lPlayer.setAdapter(adapterVerlauf);

Alles anzeigen

In diese Methode müsste ich ja dann irgendwie das mit dem Iterator einbauen, aber irgendwie stehe ich auf dem Schlauch gerade und hab ehrlichgesagt keinen Plan wie ich das anstellen soll.
Könntest du mir da vielleicht nochmal kurz helfen?

UweApps · 16. März 2012

Ich hatte ja schon geschrieben, dass du dir mal das Tutorial genauer anschauen sollst - dort im Adapter ist ein onClick-Listener eingebaut, hier kannst du die Activity für den Player aufrufen und die position als Parameter übergeben - oder auch das Player-Objekt (PlayerList.get(position)), wenn deine Player implements Serializable haben...

In der Player-Activity kannst du dann aus dem getIntent den Datensatz mit getSerializableExtra wieder rausziehen.

Du brauchst eigentlich nur den ArrayAdapter ersetzen durch einen eigenen Adapter, der die PlayerList als Datenliste bekommt.

Und dann noch mit lPlayer.setOnItemClickListener(myAdapter) die Click-Behandlung an den Adapter übertragen.

M3doXX · 16. März 2012

Also ich glaube im Großen und Ganzen habe ich es verstanden, aber es klappt trotzdem noch nicht ganz.
Ich habe jetzt folgende Activitys:

Team.java (Hier werden alle Player in einer listView angezeigt)

Java

package manager.scg;


import java.io.Serializable;
import java.util.ArrayList;
import java.util.List;


import android.app.Activity;
import android.content.Intent;
import android.os.Bundle;
import android.view.View;
import android.widget.AdapterView;
import android.widget.AdapterView.OnItemClickListener;
import android.widget.Button;
import android.widget.ArrayAdapter;
import android.widget.ListView;
import android.widget.Toast;








public class Team extends Activity implements Serializable
{
	
	public void onCreate(Bundle savedInstanceState) 
	{
    	super.onCreate(savedInstanceState);
    	setContentView(R.layout.team);
    	playerdatasource = new PlayerDataSource(this);
    	Team.this.ListTeam(null); 	
    	
    	
    	Button prev = (Button) findViewById(R.id.prev_team);
    	prev.setOnClickListener(new View.OnClickListener() 
    	{
        	public void onClick(View view) 
        	{
            	Intent intent = new Intent();
            	setResult(RESULT_OK, intent);
            	finish();
        	}
    	});
    	
    	Button addplayer = (Button) findViewById(R.id.addplayer);
    	addplayer.setOnClickListener(new View.OnClickListener() {
        	public void onClick(View view) {
        		 Intent myIntent = new Intent(view.getContext(), Add_Player.class);
             	startActivityForResult(myIntent, 0);
        	}


    	});
    	
    	Button adddata = (Button) findViewById(R.id.adddatateam);
    	adddata.setOnClickListener(new View.OnClickListener() {
        	public void onClick(View view) {
        		playerdatasource.open();
        		playerdatasource.InsertDataPlayer(); 	 
        		playerdatasource.close();
        	}


    	});
    	


    	
  	}
	
   
	
	private PlayerDataSource playerdatasource;
	List&lt;Entry&gt; PlayerList = new ArrayList&lt;Entry&gt;();  
	
	public void ListTeam (View view) {
		
		setContentView(R.layout.team);
		
		PlayerList.clear();
		try {
			
			playerdatasource.open();
			PlayerList = playerdatasource.getAllEntriesPlayer();
			playerdatasource.close();
			
			
		}
		catch (Exception ex) {
			Toast.makeText(this, ex.toString(), Toast.LENGTH_SHORT).show();
		}


		ArrayAdapter&lt;Entry&gt; adapterVerlauf = new ArrayAdapter&lt;Entry&gt;(Team.this, android.R.layout.simple_list_item_1, PlayerList);
		
		ListView lPlayer = (ListView) findViewById(R.id.listViewPlayer);
		lPlayer.setAdapter(adapterVerlauf);
		
		lPlayer.setTextFilterEnabled(true);
		lPlayer.setOnItemClickListener(new OnItemClickListener() {
			   
		public void onItemClick(AdapterView&lt;?&gt; parent, View view,
			    	int position, long id) {
			
			Intent myIntent = new Intent(getApplicationContext(), Player.class);
				
			 	 	myIntent.putExtra("PlayerList", PlayerList.get(position));


             	startActivityForResult(myIntent, 1);
				
				
			}		
		
		});
		
	}
	}

Alles anzeigen

und Player.java:

Java

package manager.scg;


//import java.util.ArrayList;
//import java.util.List;


import java.io.Serializable;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import manager.scg.Entry;
import manager.scg.R.string;
import android.app.Activity;
import android.content.Intent;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
//import android.widget.AdapterView;
//import android.widget.AdapterView.OnItemClickListener;
//import android.widget.ArrayAdapter;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.ListView;
import android.widget.TextView;
import android.widget.Toast;
//import android.widget.ListView;
//import android.widget.Toast;
//import android.widget.ArrayAdapter;






//import android.widget.AdapterView.OnItemClickListener;


public class Player extends Activity implements Serializable {
		
	public void onCreate(Bundle savedInstanceState) 
	{
    	super.onCreate(savedInstanceState);
    	setContentView(R.layout.player);
    	
    	List&lt;Entry&gt; PlayerList = getIntent().getSerializableExtra("PlayerList");
    		
    	Button prev = (Button) findViewById(R.id.prev_player);
    	prev.setOnClickListener(new View.OnClickListener() 
    	{
        	public void onClick(View view) 
        	{
            	Intent intent = new Intent();
            	setResult(RESULT_OK, intent);
            	finish();
        	}
    	});


    	
    	Button test = (Button) findViewById(R.id.test);
    	test.setOnClickListener(new View.OnClickListener() 
    	{
        	public void onClick(View view) 
        	{
            	
        		try{
        			Player.this.readPlayer();
        		} catch (NumberFormatException ex) {
        			
            	}
        		
        		
        	}
    	});
    	
    	
    	}
	
	
	
	public void readPlayer(){
		Iterator&lt;Entry&gt; myIterator = PlayerList.iterator();
		while (myIterator.hasNext()) {
			Entry player = myIterator.next();
			String name = player.getName();
			Toast.makeText(this, player.getName(), Toast.LENGTH_LONG).show();


		}
		
	}
  	
}

Alles anzeigen

In der Team.java übergebe ich nun in der

Java

public void onItemClick

mit

Java

myIntent.putExtra("PlayerList", PlayerList.get(position));

die PlayerList an die Player.java. Ich hoffe das das soweit richtig ist?!
Und in der Player.java nehme ich die Liste mit

Java

List&lt;Entry&gt; PlayerList = getIntent().getSerializableExtra("PlayerList");

entgegen.
Und dann lasse ich den Iterator in der

Java

public void readPlayer()

drüberlaufen um die einzelnen Einträge "rauszufiltern".
Allerdings sagt er mir beim getIntent: "Type mismatch: cannot convert from Serializable to List<Entry>"
Also ganz richtig kann es noch nicht sein
Und was genau muss ich implements Serializable setzten?

UweApps · 16. März 2012

Das hatten wir schon mal, oder?

Nimm bitte überall ArrayList<T> statt List<T> - die List ist nicht serializable, ArrayList aber sehr wohl...

Java

ArrayList&lt;Entry&gt; PlayerList = getIntent().getSerializableExtra("PlayerList");

M3doXX · 16. März 2012

Geht trotzdem nicht...

Die Methode sieht jetzt so aus:

Java

public void readPlayer(){
		ArrayList&lt;Entry&gt; PlayerList = getIntent().getSerializableExtra("PlayerList");
		Iterator&lt;Entry&gt; myIterator = PlayerList.iterator();
		while (myIterator.hasNext()) {
			Entry player = myIterator.next();
			String name = player.getName();
			Toast.makeText(this, player.getName(), Toast.LENGTH_LONG).show();


		}

Jetzt markiert er mir das "getIntent().getSerializableExtra("PlayerList");" und sagt:"Type mismatch: cannot convert from Serializable to ArrayList<Entry>"

Ist denn das "myIntent.putExtra("PlayerList", PlayerList.get(position));" in der ersten Activity richtig?

UweApps · 16. März 2012

Ich vermute, du hast den Fehler schon richtig benannt.

Du musst die ganze ArrayList übergeben und nicht einen einzelnen Eintrag:

Java

myIntent.putExtra("PlayerList", PlayerList);

Ich hoffe, das hilft. *däumchendrück*

M3doXX · 16. März 2012

Java

public void onItemClick(AdapterView&lt;?&gt; parent, View view,
			    	int position, long id) {
			
			Intent myIntent = new Intent(getApplicationContext(), Player.class);
				
			 	 	myIntent.putExtra("PlayerList", PlayerList);


             	startActivityForResult(myIntent, 1);
				
				
			}		
		
		});

Alles anzeigen

Dann gibt er mir aber für ".putExtra()" einen "The method putExtra(String, boolean) in the type Intent is not applicable for the arguments (String, List<Entry>)" Fehler aus.
Außerdem muss ich doch irgendwie die ID mitgeben, damit ich den Datensatz von dem Spieler kriege auf den ich geklickt habe oder?
Weil so übergebe ich ja die gesamte PlayerList...?!

UweApps · 16. März 2012

Ich weiß nicht ob das bei dir war oder wo anders: List ist nicht serialisable, ArrayList ist es schon.

Also: " is not applicable for the arguments (String, List<Entry>)" bedeutet, dass du ihm eine List und keine ArrayList gegeben hast - schreib doch bitte überall statt List lieber ArrayList, dann gehts auch.

Vor allem an dieser Stelle muß vorne ArrayList stehen:

Java

ArrayList&lt;Entry&gt; PlayerList = new ArrayList&lt;Entry&gt;();

Du könntest auch schreiben:

Java

myIntent.putExtra("PlayerList", (ArrayList) PlayerList);

aber das ist m.E. nicht die gute Variante, da du ja sowieso nur die ArrayList verwendest und keine anderen Listentypen in der Variable PlayerList unterbringst.

M3doXX · 16. März 2012

OK - Wieder ein Teilerfolg.

Ich übergebe die PlayerList mit

Java

myIntent.putExtra("PlayerList", PlayerList);

Java

public void readPlayer(){
	 	
		ArrayList&lt;Entry&gt; PlayerList = (ArrayList&lt;Entry&gt;) getIntent().getSerializableExtra("PlayerList");
		Iterator&lt;Entry&gt; myIterator = PlayerList.iterator();
				
		while (myIterator.hasNext()) {
			Entry player = myIterator.next();
			String name = player.getName();
			Toast.makeText(this, name, Toast.LENGTH_LONG).show();
		}
	}

Alles anzeigen

(Ohne den Cast geht es nicht, obwohl ich alles von List auf ArrayList geändert habe. Ist aber erstmal egal.)

Mit dieser Methode hole ich die PlayerList ab und will dann einen Toast mit dem Namen des Spielers ausgeben, auf den ich vorher geklickt habe.
Allerdings gibt er mir so für jeden Spielernamen einen Toast nacheinander aus.
Warum der das macht ist mir klar - die while-Schleife verstehe ich.
Aber wie mache ich das jetzt das er mir nur den Namen des Spielers ausgibt auf den ich vorher geklickt habe?
Muss ich dann nicht im putExtra() auch die ID übergeben oder so ähnlich?

UweApps · 16. März 2012

ah - entschuldige - da bin ich dann auch etwas durcheinander gekommen, weil ich dachte, du willst die ganze Liste übergeben...

Wenn du nur einen Player übergibst, dann heißt das natürlich (die Zeile steht im onItemClick vom Adapter, oder?):

Java

myIntent.putExtra("Player", PlayerList.get(position));

und

Java

public void readPlayer(){
	 	
	Entry player = (Entry) getIntent().getSerializableExtra("Player");


	String name = player.getName();
	Toast.makeText(this, name, Toast.LENGTH_LONG).show();
}

M3doXX · 17. März 2012

PERFEKT!!!

Es klappt!

Aller Besten Dank!!!

UweApps · 17. März 2012

Da bin ich aber froh, dass es endlich klappt...

*stirnabwisch*

Ich hoffe, du fühlst dich in Java jetzt besser zu Hause und hast weiterhin Erfolg mit Android!!!

LG
Uwe

cursorindexoutofboundsexception

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema