NavigationDrawerSpinnerItem - Listener reagiert zu häufig

prototype0815 · 2. März 2015

Hey folks, ich habe derzeit in meiner App einen NavigationDrawer eingebaut, dieser kann auch Spinner Items fassen. Diese Spinner Items haben natürlich einen OnItemSelectedListener, und die Methode onItemSelected springt andauernd an, auch ohne das ich auf eins der Items klicke. Zum prüfen mache ich bei sowas immer einen Toast hinein, diesen erkennt man am schnellsten.

Hier mein NavigationDrawer und ganz oben ein SpinnerItem...
[Blockierte Grafik: http://preview.slick.pics/FT0ay9z45.png]

der Listener springt zum Beispiel an sobald die App gestartet wird, ich vermute sobald der NavigationDrawer erstellt wird. Desweiteren springt er an wenn ich im NavigationDrawer nach unten un wieder hoch scrolle ( ich habe sehr viele Einträge ), also sobald das SpinnerItem nach oben hinaus geschoben wird, und wieder herein kommt.

Das alles ist natürlich ziemlich nervig, da diese Items ja irgendwann Activities starten oder Fragmente tauschen.

Hier mein Code...

Java

if (dItem.isSpinner()) {
  	drawerHolder.headerLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.itemLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.spinnerLayout.setVisibility(LinearLayout.VISIBLE);


  	List&lt;CustomSpinnerItem&gt; userList = new ArrayList&lt;CustomSpinnerItem&gt;();


//  	for (int i = 0; i &lt; dItem.entries.length; i++) {
//    	userList.add(new CustomSpinnerItem(i % 2 == 0 ? R.drawable.user1 : R.drawable.user2, dItem.entries[i],
//            	"[email protected]"));
//  	}


  	userList.add(new CustomSpinnerItem(R.drawable.user1, "Ahamed Ishak",
          	"ishakgmail.com"));


  	userList.add(new CustomSpinnerItem(R.drawable.user2, "Brain Jekob",
          	"brain.jgmail.com"));


  	CustomSpinnerAdapter adapter = new CustomSpinnerAdapter(context,
          	R.layout.custom_spinner_item, userList);


  	drawerHolder.spinner.setAdapter(adapter);


  	drawerHolder.spinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
            	@Override
            	public void onItemSelected(AdapterView&lt;?&gt; arg0, View arg1, int arg2, long arg3) {
              	Toast.makeText(context, "User Changed", Toast.LENGTH_SHORT).show();
            	}
            	@Override
            	public void onNothingSelected(AdapterView&lt;?&gt; arg0) {
              	Toast.makeText(context, "Nothing selected", Toast.LENGTH_SHORT).show();
            	}
          	});


	}

Alles anzeigen

Ich habe schon versucht einen OnItemClickListener einzubauen, der Compiler bringt keine Fehler aber sobald ich dann meine App starte, crasht diese mit dem Fehler das man mit einem Spinner kein OnItemClickListener verwenden kann.

Wie kann ich es also erreichen das meine Einträge im Spinner nur bei einem Click darauf das machen was sie sollen?

LG proto

Marco Feltmann · 3. März 2015

Ich fürchte, den wichtigen Codeteil enthältst Du uns vor.

Irgendwo sagst Du dem Spinner sicherlich, was er vorausgewählt haben soll. Vermutlich via 'setSelectedItem'.
Und das ruft dann den OnSelectedItemListener auf. Also alles so wie es sein soll.

Eine (gute) Möglichkeit wäre es, 'setSelectedItem' des SpinnerAdapters aufzurufen bevor Du den Listener registrierst.
Meines Wissens wird die Selektion nicht nachträglich aufgerufen.

prototype0815 · 3. März 2015

Hier meine ganze Klasse des Adapters... Ab Zeile 69 wird ein Item als Spinner erkannt und eingebunden.

Java

package com.example.paul_io.custom;


import android.app.Activity;
import android.content.Context;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.ImageView;
import android.widget.LinearLayout;
import android.widget.Spinner;
import android.widget.TextView;
import android.widget.Toast;


import com.example.paul_io.R;


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


/**
 * Created by SW on 02.03.2015.
 */
public class CustomDrawerAdapter extends ArrayAdapter&lt;CustomDrawerItem&gt; {


  Context context;
  List&lt;CustomDrawerItem&gt; customDrawerItemList;
  int layoutResID;


  public CustomDrawerAdapter(Context context, int layoutResourceID, List&lt;CustomDrawerItem&gt; listItems) {
	super(context, layoutResourceID, listItems);
	this.context = context;
	this.customDrawerItemList = listItems;
	this.layoutResID = layoutResourceID;
  }


  @Override
  public View getView(int position, View convertView, ViewGroup parent) {
	// TODO Auto-generated method stub


	DrawerItemHolder drawerHolder;
	View view = convertView;


	if (view == null) {
  	LayoutInflater inflater = ((Activity) context).getLayoutInflater();
  	drawerHolder = new DrawerItemHolder();


  	view = inflater.inflate(layoutResID, parent, false);
  	drawerHolder.ItemName = (TextView) view.findViewById(R.id.drawer_itemName);
  	drawerHolder.icon = (ImageView) view.findViewById(R.id.drawer_icon);


  	drawerHolder.spinner = (Spinner) view.findViewById(R.id.drawerSpinner);


  	drawerHolder.title = (TextView) view.findViewById(R.id.drawerTitle);


  	drawerHolder.headerLayout = (LinearLayout) view.findViewById(R.id.headerLayout);
  	drawerHolder.itemLayout = (LinearLayout) view.findViewById(R.id.itemLayout);
  	drawerHolder.spinnerLayout = (LinearLayout) view.findViewById(R.id.spinnerLayout);


  	view.setTag(drawerHolder);


	} else {
  	drawerHolder = (DrawerItemHolder) view.getTag();


	}


	CustomDrawerItem dItem = (CustomDrawerItem) this.customDrawerItemList.get(position);


	if (dItem.isSpinner()) {
  	drawerHolder.headerLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.itemLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.spinnerLayout.setVisibility(LinearLayout.VISIBLE);


  	List&lt;CustomSpinnerItem&gt; userList = new ArrayList&lt;CustomSpinnerItem&gt;();


  	for (int i = 0; i &lt; dItem.entries.length; i++) {
    	userList.add(new CustomSpinnerItem(i % 2 == 0 ? R.drawable.user1 : R.drawable.user2, dItem.entries[i],
            	"[email protected]"));
  	}


//  	userList.add(new CustomSpinnerItem(R.drawable.user1, "Ahamed Ishak",
//          	"ishakgmail.com"));
//
//  	userList.add(new CustomSpinnerItem(R.drawable.user2, "Brain Jekob",
//          	"brain.jgmail.com"));


  	CustomSpinnerAdapter adapter = new CustomSpinnerAdapter(context,
          	R.layout.custom_spinner_item, userList);


  	drawerHolder.spinner.setAdapter(adapter);


  	drawerHolder.spinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {


    	@Override
    	public void onItemSelected(AdapterView&lt;?&gt; arg0, View arg1, int arg2, long arg3) {
      	Toast.makeText(context, "User Changed", Toast.LENGTH_SHORT).show();
    	}


    	@Override
    	public void onNothingSelected(AdapterView&lt;?&gt; arg0) {
      	Toast.makeText(context, "Nothing selected", Toast.LENGTH_SHORT).show();
    	}
  	});


	} else if (dItem.getTitle() != null) {
  	drawerHolder.headerLayout.setVisibility(LinearLayout.VISIBLE);
  	drawerHolder.itemLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.spinnerLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.title.setText(dItem.getTitle());


	} else {


  	drawerHolder.headerLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.spinnerLayout.setVisibility(LinearLayout.INVISIBLE);
  	drawerHolder.itemLayout.setVisibility(LinearLayout.VISIBLE);


  	drawerHolder.icon.setImageDrawable(view.getResources().getDrawable(
          	dItem.getImgResID()));
  	drawerHolder.ItemName.setText(dItem.getItemName());


	}
	return view;
  }


  private static class DrawerItemHolder {
	TextView ItemName, title;
	ImageView icon;
	LinearLayout headerLayout, itemLayout, spinnerLayout;
	Spinner spinner;
  }
}

Alles anzeigen

Ich habe kein Fall gefunden wo ich dem Spinner sage welches Item er setzen soll. Das aber jedes mal der Listener wieder anspringt wenn ich scrolle finde ich eben so seltsam...

Hier das Anlegen der verschiedenen Elemente für den Navigation Drawer...

Java

dataList = new ArrayList&lt;CustomDrawerItem&gt;();
	dataList.add(new CustomDrawerItem(true, new String[]{"Ein- / Ausgänge", "Baugruppen", "FIFO", "Systemsteuerung"})); // adding a spinner to the list
	dataList.add(new CustomDrawerItem("My Favorites")); // adding a header to the list
	dataList.add(new CustomDrawerItem("Message", R.drawable.ic_action_email));
.
.

Hier die Konstruktoren des CustomDrawerItem...

Java

package com.example.paul_io.custom;


/**
 * Created by SW on 02.03.2015.
 */
public class CustomDrawerItem {


  String ItemName;
  int imgResID;
  String title;
  boolean isSpinner;
  String[] entries;


  public CustomDrawerItem(boolean isSpinner, String[] entry) {
	this(null, 0);
	this.isSpinner = isSpinner;
	this.entries = entry;
  }


  public CustomDrawerItem(String title) {
	this(null, 0);
	this.title = title;
  }


  public CustomDrawerItem(String itemName, int imgResID) {
	super();
	ItemName = itemName;
	this.imgResID = imgResID;
  }


  public String getItemName() {
	return ItemName;
  }


  public void setItemName(String itemName) {
	ItemName = itemName;
  }


  public int getImgResID() {
	return imgResID;
  }


  public void setImgResID(int imgResID) {
	this.imgResID = imgResID;
  }


  public String getTitle() {
	return title;
  }


  public void setTitle(String title) {
	this.title = title;
  }


  public boolean isSpinner() {
	return isSpinner;
  }
}

Alles anzeigen

Hier wird die dataList verwendet...

Java

mDrawerList = (ListView) findViewById(R.id.left_drawer);
	adapter = new CustomDrawerAdapter(this, R.layout.custom_drawer_item, dataList);
	mDrawerList.setAdapter(adapter);
.
.

Dieser komische Listener bringt mich noch zur Weißglut

prototype0815 · 3. März 2015

der Listener spingt ebenfalls an wenn ich auf ein anderes Item klicke, zum Beispiel "Likes" oder "Games".
hoffentlich finde ich nicht noch mehr solcher auslösenden Ereignisse

Marco Feltmann · 3. März 2015

Also in dem Fall muss irgendwas mit Deiner DrawHolder.Spinner Verdrahtung kaputt sein.

Lass Dir mal ausgeben, was genau DrawHolder.Spinner für ein Objekt zur Laufzeit ist.

prototype0815 · 4. März 2015

meinst du so ?
[Blockierte Grafik: http://i.slick.pics/FT3GvnrtE.png]

Marco Feltmann · 5. März 2015

Das sieht sehr gut aus.
Was es insgesamt noch etwas schwieriger macht.

Die nächste logische Idee wäre, dass Du Dich im betreffenden onItemSelected() mit einem Breakpoint reinhängst und jeweils schaust, welchen Typ und Wert arg0 hat.
Eigentlich kann es nur noch sein, dass sich Spinner und Drawer einen Adapter teilen, aber das wäre unrealistisch.

prototype0815 · 6. März 2015

könnte es an meinem CustomSpinnerAdapter liegen ???

Java

public class CustomSpinnerAdapter extends ArrayAdapter&lt;CustomSpinnerItem&gt; {


  public CustomSpinnerAdapter(Context context, int layoutResourceID,
                          	int textViewResourceId, List&lt;CustomSpinnerItem&gt; spinnerDataList) {
	super(context, layoutResourceID, textViewResourceId, spinnerDataList);


	this.context=context;
	this.layoutResID=layoutResourceID;
	this.spinnerData=spinnerDataList;


  }


  Context context;
  int layoutResID;
  List&lt;CustomSpinnerItem&gt; spinnerData;


  public CustomSpinnerAdapter(Context context, int layoutResourceID,
                          	List&lt;CustomSpinnerItem&gt; spinnerDataList) {
	super(context, layoutResourceID, spinnerDataList);


	this.context=context;
	this.layoutResID=layoutResourceID;
	this.spinnerData=spinnerDataList;


  }


  @Override
  public View getDropDownView(int position, View convertView, ViewGroup parent) {
	// TODO Auto-generated method stub
	return getCustomView(position, convertView, parent);
  }


  @Override
  public View getView(int position, View convertView, ViewGroup parent) {
	// TODO Auto-generated method stub
	return getCustomView(position, convertView, parent);
  }


  public View getCustomView(int position, View convertView, ViewGroup parent) {


	View row=convertView;
	SpinnerHolder holder;


	if(row==null)
	{
  	LayoutInflater inflater=((Activity)context).getLayoutInflater();


  	row=inflater.inflate(layoutResID, parent, false);
  	holder=new SpinnerHolder();


  	holder.userImage=(ImageView)row.findViewById(R.id.left_pic);
  	holder.name=(TextView)row.findViewById(R.id.text_main_name);
//  	holder.email=(TextView)row.findViewById(R.id.sub_text_email);


  	row.setTag(holder);
	}
	else
	{
  	holder=(SpinnerHolder)row.getTag();


	}


	CustomSpinnerItem CustomSpinnerItem=spinnerData.get(position);


	holder.userImage.setImageDrawable(row.getResources().getDrawable(CustomSpinnerItem.getDrawableResID()));
	holder.name.setText(CustomSpinnerItem.getName());
//	holder.email.setText(CustomSpinnerItem.getEmail());


	return row;


  }


  private static class SpinnerHolder
  {
	ImageView userImage;
	TextView  name,email;
  }


}

Alles anzeigen

prototype0815 · 6. März 2015

Irgendwie feuert die Klasse AdapterView den onItemSelected ab wenn die View erstellt wird.

Java

drawerHolder.spinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
.
.
.
  	});

[Blockierte Grafik: http://i.slick.pics/FT5fGcmdQ.png]

Marco Feltmann · 9. März 2015

Wie gesagt, bei der Erstellung muss ja bei einigen Views (Spinner) eine Auswahl getroffen werden, damit überhaupt was angezeigt wird.

Das hängt aber sicherlich nicht mit dem Auswählen aus der Liste zusammen.

NavigationDrawerSpinnerItem - Listener reagiert zu häufig

Jetzt mitmachen!

Teilen