CustomAdapter index refresh

khh · 11. Juni 2013

Hallo zusammen,
ich habe einen

CustomAdapterAdresse extends ArrayAdapter<Adresse> impelments android.location.LocationListener

im OnclickListener lösche ich einen Eintrag aus der DB und aus der Liste:

public View getView(final int position, View convertView, ViewGroup parent) ....
...
...
adressList.remove(position);

notifyDataSetChanged();

das funktioniert leider nur beim ersten mal, da der Index der Liste nicht aktualisiert wird.

was mache ich falsch?

Ich danke euch

Kogoro-Christopher · 11. Juni 2013

Versuch es mal mit einem überschriebenen ListIterator.
Dort definierst du dann, das wenn etwas gelöscht wird, alles dahinter einen Index nach vorne geht, bis es keinen Nachfolger hat.
Das sollte gehen

khh · 12. Juni 2013

ich danke dir.

Und einfacher gehts nicht?

Jemand noch ne idee ?

matthias · 12. Juni 2013

Hoi,

Zitat von khh

adressList.remove(position);

Von welchem Datentyp ist dein adressList denn eigentlich? Eine ArrayList rutscht ja die nachfolgenden Einträge immer nach, sobald du in der Mitte eins raus nimmst. Dann müsste die position eig. auch der Index des richtigen Elements sein ... oder raff ich grad nicht, was du meinst

Gruß,
matze

khh · 12. Juni 2013

adressList ist so definiert:
List<AdressList> adressList;

AdressList ist ne Klasse.
mit adressList.remove(position); hab ich das ja machen wollen, klappt aber nicht

khh · 12. Juni 2013

ichhab hier mal die komplette Klasse:

Java

import android.content.Context;
import android.content.Intent;
import android.location.Location;
import android.os.Bundle;
import android.util.Log;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.ArrayAdapter;
import android.widget.Button;
import android.widget.TextView;
import android.widget.Toast;


import java.util.List;
import java.util.ListIterator;


/**
 * Created on 22.05.2013
 */
public class CustomAdapterAdressfinder extends ArrayAdapter&lt;AdressFinder&gt; implements android.location.LocationListener {


  
	private Context context;
    private List&lt;AdressFinder&gt; adressfinderList;
    private AdressFinder adressfinder;
    private String parameter = "" ; 
	Context mContext;
  
	


    public CustomAdapterAdressfinder(Context context, List&lt;AdressFinder&gt; adressfinderList) {
        super(context, R.layout.cell_custom_adressfinder,adressfinderList);
        this.context = context;
        this.adressfinderList = adressfinderList;
        Log.i("SET", "SET " +this.adressfinderList.size());
        


    }


    @Override
    public View getView(final int position, View convertView, ViewGroup parent) {


        View view = convertView;
        final ViewHolder holder;
        
        Log.i("position","position " + position);
        if(view == null){
            LayoutInflater inflater = (LayoutInflater) context
                    .getSystemService(Context.LAYOUT_INFLATER_SERVICE);
            view = inflater.inflate(R.layout.cell_custom_adressfinder,parent,false);
            final ListIterator&lt;AdressFinder&gt; AdressfinderListIterator = adressfinderList.listIterator(adressfinderList.size());
            holder = new ViewHolder();
            
           
            holder.abfrageloeschbutton = (Button)view.findViewById(R.id.abfrageloeschbutton);
            holder.abfrageausfuehrenbutton = (Button)view.findViewById(R.id.abfrageausfuehrenbutton);
            holder.abfragetext = (TextView)view.findViewById(R.id.abfragetext);
           
            view.setTag(holder);
            
            
            // Clicklistener Favorit
            holder.abfrageloeschbutton.setOnClickListener( new View.OnClickListener() {  
                public void onClick(View v) {  
                	
                	  
                	   Button mybutton = (Button)v;
                       AdressFinder adressfinder = (AdressFinder)mybutton.getTag();
                    
                        int id =  adressfinder.getID();
                        Utilities.loescheAdressAbfrage(getContext(),id); 
                        String	 meldung = "Abfrage gelöscht";
                                                          	               	
                	    Toast.makeText(context, meldung, Toast.LENGTH_SHORT).show();
                	    int wert = position;
                	    
                	 
                	    // Eintrag löschen  
                	    // hier ist der  index nicht aktualisiert !!!!!!!!!!!!!!!!!!!!!!!!
                		adressfinderList.remove(position);              		               	               	     
                	    notifyDataSetChanged();
                	 
                	  
                	    
                }  
                
            });  // ende Clicklistener favorit
            
            // Clicklistener verabreden
            holder.abfrageausfuehrenbutton.setOnClickListener(new View.OnClickListener() {


                @Override
                public void onClick(View v) {
                    // TODO Auto-generated method stub
                	  Button mybutton = (Button)v;
                      AdressFinder adressfinder = (AdressFinder)mybutton.getTag();  
                      parameter = adressfinder.getBezeichnung();
                      abfrageAusfuehren(parameter);
                 	
                }
            }); // ende Clicklistener verabreden
            
            
       
            
            
        }else {
            holder = (ViewHolder) view.getTag();
        }
        
       


        adressfinder = adressfinderList.get(position);
      //  boolean ok = true;
      
        holder.abfragetext.setText(adressfinder.getBezeichnung());
     
       
         holder.abfrageloeschbutton.setTag(adressfinder);
         holder.abfrageausfuehrenbutton.setTag(adressfinder);
        
     
        
        return view;
        
        
    }  // Ende View


    private class ViewHolder{
    	Button   abfrageloeschbutton;
    	Button   abfrageausfuehrenbutton;
        TextView abfragetext;
       
    }
    
    
 
 


@Override
public void onLocationChanged(Location arg0) {
	// TODO Auto-generated method stub
	
}


@Override
public void onProviderDisabled(String arg0) {
	// TODO Auto-generated method stub
	
}


@Override
public void onProviderEnabled(String provider) {
	// TODO Auto-generated method stub
	
}


@Override
public void onStatusChanged(String provider, int status, Bundle extras) {
	// TODO Auto-generated method stub
	
}


public void abfrageAusfuehren(String parameter) {
	// führt die abfrage mit den übergebenen parametern aus 
    Intent openStartingPoint = new Intent("com.testfirma.adressapp.ADRESSABFRAGE");
      
    String suchbegriffe = parameter;
    openStartingPoint.putExtra("Suchbegriffe", suchbegriffe);
    context.startActivity(openStartingPoint);


	
}


}

Alles anzeigen

matthias · 12. Juni 2013

Hoi,

hast du die Klasse und variablen zwischenzeitlich umbenannt oder hast du den falschen Adapter erwischt? Irgendwie bin ich etwas verwirrt

Ich denk das mal "laut" durch:
Also du hast einen Adapter, da sind 5 Elemente drin. Du löschst das 2. Element also position ist 1. Rufst also die .remove auf, der wirft das 2. Element weg. Jetzt müsste eigentlich adressList.get(position) das ursprünglich 3. Element in der Liste liefern. Tut er das? Debug das mal. Änder auch mal zum Test dein List in ArrayList.

Gruß,
Matze

khh · 12. Juni 2013

ja, hab die klasse umbenannt, sorry, hätte ich sagen sollen

khh · 12. Juni 2013

alssooo

ich habe 4 Einträge in der Liste
beim Löschen des 3ten Eintrages steht richtig 2 in position

adressfinderList.remove(position);

beim nächsten löschen des (jetzt richtig angezeigten) 2 ten Eintrags steht immer noch 2 in position und der falsche, also dritte, Eintrag wird gelöscht

matthias · 12. Juni 2013

Was passiert denn, wenn du das Objekt übergibst? Also deine "adressfinder" Variable anstatt die position?

khh · 12. Juni 2013

mal langsam
ein adressfinderList.get(position);
NACH dem remove liefert richtig den nächsten eintrag, in position steht aber noch der alte index.

Heisst das, dass ich vor dem remove einfach ein get brauche?
EDIT: Quatsch da steht ja der falsche index drin?

was meinst du welche Variable soll ich übergeben?
EDIT: muss jetzt leider weg, ich schau heut abend nochmal rein.

ich danke dir

Marco Feltmann · 12. Juni 2013

Irgendwie check ich das Problem nicht.

Wo wird wann wie was warum gelöscht?
Teilst du deiner Liste auch mit, dass sich die zu Grunde liegenden Daten verändert haben?

matthias · 12. Juni 2013

Java

holder.abfrageloeschbutton.setOnClickListener( new View.OnClickListener() {  
                public void onClick(View v) {  
                                Button mybutton = (Button)v;
                                AdressFinder adressfinder = (AdressFinder) mybutton.getTag();
                    
                                int id =  adressfinder.getID();
                                Utilities.loescheAdressAbfrage(getContext(),id); 
                                String	 meldung = "Abfrage gelöscht";
                                                          	               	
                                Toast.makeText(context, meldung, Toast.LENGTH_SHORT).show();
                                
                                // Eintrag aus liste löschen
                		adressfinderList.remove(adressfinder);              		               	               	     
                	        notifyDataSetChanged();
               }  
});  // ende Clicklistener favorit

Alles anzeigen

Ich hätts mal so probiert. Du hast ja bereits das Objekt selbst und List.remove(object) müsste eigentlich genau das Objekt auch finden.

Zitat

public abstract boolean remove (Object object)

Added in API level 1
Removes the first occurrence of the specified object from this List.

Parameters
object the object to remove.
Returns
true if this List was modified by this operation, false otherwise.
Throws
UnsupportedOperationException if removing from this List is not supported.

Alles anzeigen

Aber irgendwie kommt mir der Aufbau generell etwas seltsam vor ... das ganze in der getView machen, die wird ja mehrfach aufgerufen. Ich stell mir die Frage ob er evtl. schon im nächsten getView steckt, während der OnClickListener in einem eigenen Thread läuft und verhaut deshalb die Position .... probier den Code oben mal aus, vll. fruchtet es ja.

Gruß,
matze

khh · 13. Juni 2013

Hallo Matze,

adressfinderList.remove(adressfinder); funktioniert!
Ich danke dir.

Dass dir der Aufbau seltsam vorkommt, liegt vieleicht daran, dass sich auf jedem Listeneintrag ein Löschbutton befindet.

Wie komme ich sonst da dran ausser im onClickListener der view?

matthias · 13. Juni 2013

Hoi,

schön, dass es jetzt funktioniert
Mir fällt ehrlich gesagt grad nichts ein, wie man es anders machen könnte ... irgendwie kams mir seltsam vor aber vermutlich hätt ichs auch so gemacht

Gruß,
matze

CustomAdapter index refresh

Jetzt mitmachen!

Teilen