CheckBox im ExpandableListView zum Ein- und Ausblenden einzelner Spalten funktioniert nicht immer

Moin zusammen,

ich versuch nun schon seit zwei Tagen mir einen ExpandableListView zusammen zu basteln in welchem ich auf Konopdruck einen Eintrag rausnehmen kann. Es handelt sich hierbei um eine CheckBox mit der Beschriftung "Liefer- und Rechnungsadresse identisch". Wenn diese CheckBox aktiviert ist, soll die Vie für die Rechnungsadresse nicht angezeigt werden. Das ganze funktioniert nun auch wunderbar bis auf ein paar kleinere Fehler, nämlich das die oberste View in welcher die CheckBox ist (die für die Lieferadresse) manchmal einige Klicks nicht entgegen nimmt. Allerdings nur bei CheckBoxen und RadioButtons, bei Textfeldern funktioniert alles immer tadellos.

Ich poste euch mal ein bisschen Quellcode dazu und hoffe das mir hierbei jemand helfen kann, ich weiß nämlich nicht mehr woran dasl iegen könnte. =/


Das hier ist die CheckBox welche dafür zuständig sein wird das die Rechnungsadressen-View ausgeblendet wird

Hier überschreibe ich die Methode getGroupCount(), da wenn ich eine Gruppe ausblende (die für die Rechnungsadresse), gibt es ja logischerweise eine Gruppe weniger

Hier überschreibe ich nun die Methode getGroupView() damit ich auch für die einzelnen Gruppen trotzdem noch die richtige Überschrift erhalte, auch wenn mitten drin eine View nun fehlt

Und hier überschreibe ich letzendlich die Methode getChildView(), da ja auch je nach dem ob beide Adressen gleich sind oder nicht auch eine Childiew weniger angezeigt wird

Und hier zeichne ich die View neu wenn die CheckBox verändert worden ist

Wie gesagt, es gibt nur Probleme in der View für die Lieferadresse, mit der Rechnungsadresse klappt alles einwandfrei (soweit sie natürlich auch eingeblendet ist). Lieferadresse und Rechnungsadresse sind bis auf die CheckBox komplett gleich (beide greifen auf das selbe XML-Layout zurück) und damit haben auch beide RadioButtons zum wählen der Anrede (Herr, Frau, Firma). Diese können überall wunderbar angewählt werden, nur bei der View mit der Lieferadresse wo die CheckBox integriert ist funktioniert es irgendwann nicht mehr. Nachdem man dann ein wenig in den anderen Views des ExpandableListViews umher geklickt hat, funktioniert es dann eigentlich auch immer =/

Komplexes Problem und ich find den Fehler einfach nicht, hoffe ihr wisst da mehr als ich :'(

Okay ich habe den Fehler ein wenig eingegrenzt.
Die Klicks die ich auf die CheckBox ausübe, scheinen sich ab einem bestimmten Moment (welchen ich noch nicht gefunden habe) zu sammeln. Denn sobald ich zum Beispiel 10 mal drauf klicke passiert nicht, aber wenn ich dann aufmal auf eine andere View klicke, wird erst mal 10 mal die onClick Methode auf die CheckBox ausgeübt.

Ich habe allen Views eine Textausgabe mit Log.d(String, String) gegeben um zu sehen wann wo welcher klick ankommt und dies ist nun das Verhalten welches ich beobachten konnte. Hat hierzu vielleicht noch jemand eine Idee? Ich weiß einfach nicht wo der Fehler liegen könnte =/

Ich poste einfach mal meinen neuen Versuch welcher ein bisschen weniger Quellcode enthält.

[code]
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
initializeExpandableListView();
}

private List<HashMap<String, String>> createGroupList(Object array[], String key) {
if(array == null || key == null)
return null;

ArrayList<HashMap<String, String>> result = new ArrayList<HashMap<String, String>>();
for( int i = 0 ; i < array.length ; ++i ) {
HashMap<String, String> m = new HashMap<String, String>();
m.put( key, array[i].toString() );
result.add(m);
}
return (List<HashMap<String, String>>)result;
}


private List<ArrayList<HashMap<String, String>>> createChildList(String array[], String key) {
if(array == null || key == null)
return null;

ArrayList<ArrayList<HashMap<String, String>>> result = new ArrayList<ArrayList<HashMap<String, String>>>();
for( int i = 0 ; i < array.length ; ++i ) {

ArrayList<HashMap<String, String>> secList = new ArrayList<HashMap<String, String>>();
HashMap<String, String> child = new HashMap<String, String>();
child.put( key, array[i] );
secList.add(child);
result.add(secList);
}
return result;
}



private void initializeExpandableListView() {
ExpandableListView exListView = (ExpandableListView) findViewById(R.id.exListView);
exListView.setAdapter(new SimpleExpandableListAdapter(this,
createGroupList(new String[] {"Test1", "Test2", "Test3"}, "adressDatenAngabe"),
android.R.layout.simple_expandable_list_item_1,
new String[] { "adressDatenAngabe"},
new int[] { android.R.id.text1 },
createChildList(new String[] { "Child1", "Child2", "Child3"}, "profil"),
android.R.layout.expandable_list_content,
new String[] {"profil"},
new int[] { }) {

private boolean isSame = false;
private LinearLayout checkBoxLayout = null;
private CheckBox sameAdressCheckBox = null;

private boolean isFirstExpanded = false;
private boolean isSecondExpanded = false;
private boolean isThirdExpanded = false;

private LinearLayout view1 = null;
private LinearLayout view2 = null;
private LinearLayout view3 = null;

@Override
public int getGroupCount() {
if(isSame)
return super.getGroupCount() - 1;
else
return super.getGroupCount();
}

@Override
public View getGroupView(int groupPosition, boolean isExpanded, View convertView, ViewGroup parent) {

View v = null;

switch (groupPosition) {
case 0:
isFirstExpanded = isExpanded;
v = super.getGroupView(groupPosition, isExpanded, convertView, parent);
break;
case 1:
if(isSame) {
isThirdExpanded = isExpanded;
v = super.getGroupView(groupPosition+1, isThirdExpanded, convertView, parent);
} else {
isSecondExpanded = isExpanded;
v = super.getGroupView(groupPosition, isSecondExpanded, convertView, parent);
}
return v;
case 2:
isThirdExpanded = isExpanded;
v = super.getGroupView(groupPosition, isExpanded, convertView, parent);
break;
}
return v;
}


@Override
public View getChildView(int groupPosition,
int childPosition, boolean isLastChild,
View convertView, ViewGroup parent) {


switch(groupPosition) {
case 0:
if(view1 == null) {
view1 = new LinearLayout(parent.getContext());
((LinearLayout)view1).setOrientation(LinearLayout.VERTICAL);
TextView tv1 = new TextView(parent.getContext());
tv1.setText("TextView1");
((LinearLayout)view1).addView(tv1);
((LinearLayout)view1).addView(getSameAdressCheckbox(parent));
}
convertView = view1;
break;
case 1:
if(!isSame) {
if(view2 == null) {
view2 = new LinearLayout(parent.getContext());
TextView tv2 = new TextView(parent.getContext());
tv2.setText("TextView2");
CheckBox cb2 = new CheckBox(parent.getContext());
cb2.setText("Testtext");
((LinearLayout)view2).addView(cb2);
((LinearLayout)view2).addView(tv2);
convertView = view2;

break;
}

case 2:
if(view3 == null) {
view3 = new LinearLayout(parent.getContext());
TextView tv3 = new TextView(parent.getContext());
tv3.setText("TextView3");
((LinearLayout)view3).addView(tv3);
}
convertView = view3;
break;
}
return convertView;
}


private LinearLayout getSameAdressCheckbox(ViewGroup parent) {
if(sameAdressCheckBox == null) {
checkBoxLayout = new LinearLayout(parent.getContext(), null);
sameAdressCheckBox = new CheckBox(parent.getContext());
sameAdressCheckBox.setText("Liefer- und Rechnungsadresse sind identisch");
sameAdressCheckBox.setOnCheckedChangeListener(new OnCheckedChangeListener() {

@Override
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
sameAdressCheckBox.setChecked(isChecked);
isSame = isChecked;
notifyDataSetChanged();
}
});
checkBoxLayout.addView(sameAdressCheckBox);
}
return checkBoxLayout;
}

});
}
[/code]

Hier ist die XML in welcher das Layout eingebettet ist.



Die ganzen onClick Methoden zur Überprüfung habe ich aufgrund der besseren Lesbarbeit heraus genommen. Bei dem geposteten Quellcode handelt es sich um den kompletten Adapter für den ExpandableListView.

— geändert am 19.01.2012, 09:16:40

?
Klar geht das!
Mache ein Projekt mit einer View, welche nur einen ExpandableListView enthält, mach auf den ExpandableListView einfach setAdapter, füge einen SimpleExpandableListAdapter hinzu und überschreibe die Methoden die ich hier alle gepostet habe.
Und diese Methoden sind allein dafür verantwortlich das ich ein einzelnes Element ein- sowie ausblenden kann.

Ich weiß nicht was du mit zu viel bzw zu wenig meinst.

Okay, jetzt weiß ich auch was du meinst :-)

Ich habe meinen Quellcode dementsprechend oben ergänzt und es sollte so nun auch compilierbar sein.
Und 'tschuldige für die Unannehmlichkeiten und danke das du dich mit meinem Problem beschäftigst.

Anbei vielleicht hilft es ja zur Fehlersuche bei, ich hatte gestern noch irgendwo gelesen das man mit dem ViewHolder Pattern das Problem lösen könnte. Allerdings habe ich hierzu positive sowie negative Rezesionen zu gelesen. Versuch jetzt seit gestern Abend mich in dieses Pattern rein zu arbeiten und habe hier auch schon einige kleinere Erfolge gehabt, so dass ich es heute für mein Problem umzuschreiben versuchen werde. Vielleicht hast du ja Erfahrung in diesem Pattern und weißt ob es mein Problem lösen könnte oder nicht? Ich werde es ansonsten weiter versuchen es für mich anzupassen.

— geändert am 19.01.2012, 09:22:24

Da hast du wohl recht, bei dem ganzen zusammen biegen und brechen brauch man sich nicht wundern das mal solch ein Fehler entsteht.
Nichts desto trotz habe ich es jetzt nach 2 1/2 Tagen endlich geschafft und der Fehler tritt bei mir nun nicht mehr auf. Gelöst habe ich das ganze in der Tat mit dem ViewHolder Pattern, wobei ich bisher nicht genaz verstanden habe warum es auf diese Art und Weise funktioniert. Der Nachteil des ganzen ist hier nun natürlich das ich die Daten mir selbst merken muss für den fall das die View neu gezeichnet wird und damit die alte überschrieben wird. =/

Ich poste einfach mal meine Lösung als Quellcode in der Hoffnung das es vielleicht anderen leuten weiter hilft und bedanke mich trotzdem ganz herzlich für deine Mühe Felix. (:

Anbei möchte ich mich auch gleich dafür entschuldigen das der Code-Tag dieses Forums nur sehr spärlich funktioniert und damit auf meine Unschuld hinweisen das die Formatierung des Quellcodes für 'n Arsch ist (:

[code]
public class ExpandableListViewTestActivity extends Activity {

@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
initializeExpandableListView();
}


private List<HashMap<String, String>> createGroupList(Object array[], String key) {
if(array == null || key == null)
return null;

ArrayList<HashMap<String, String>> result = new ArrayList<HashMap<String, String>>();
for( int i = 0 ; i < array.length ; ++i ) {
HashMap<String, String> m = new HashMap<String, String>();
m.put( key, array[i].toString() );
result.add(m);
}
return (List<HashMap<String, String>>)result;
}


private List<ArrayList<HashMap<String, String>>> createChildList(String array[], String key) {
if(array == null || key == null)
return null;

ArrayList<ArrayList<HashMap<String, String>>> result = new ArrayList<ArrayList<HashMap<String, String>>>();
for( int i = 0 ; i < array.length ; ++i ) {

ArrayList<HashMap<String, String>> secList = new ArrayList<HashMap<String, String>>();
HashMap<String, String> child = new HashMap<String, String>();
child.put( key, array[i] );
secList.add(child);
result.add(secList);
}
return result;
}

private void initializeExpandableListView() {
ExpandableListView exListView = (ExpandableListView) findViewById(R.id.exListView);

exListView.setAdapter(new SimpleExpandableListAdapter(this,
createGroupList(new String[] {"Test1", "Test2", "Test3"}, "adressDatenAngabe"),
android.R.layout.simple_expandable_list_item_1,
new String[] { "adressDatenAngabe"},
new int[] { android.R.id.text1 },
createChildList(new String[] { "Child1", "Child2", "Child3"}, "profil"),
android.R.layout.expandable_list_content,
new String[] {"profil"},
new int[] { }) {
// ================================ ADAPTER =====================================

private Map<Integer, Model> Data = new HashMap<Integer, Model>();
private boolean areSame = false;



/* (non-Javadoc)
* @see android.widget.SimpleExpandableListAdapter#getGroupCount()
*/
@Override
public int getGroupCount() {
if(areSame)
return super.getGroupCount()-1;
else
return super.getGroupCount();
}

/* (non-Javadoc)
* @see android.widget.SimpleExpandableListAdapter#getGroupView(int, boolean, android.view.View, android.view.ViewGroup)
*/
@Override
public View getGroupView(int groupPosition,
boolean isExpanded, View convertView,
ViewGroup parent) {
View v = convertView;

switch (groupPosition) {
case 1:
if(areSame) {
v = super.getGroupView(groupPosition+1, isExpanded, convertView, parent);
} else {
v = super.getGroupView(groupPosition, isExpanded, convertView, parent);
}
v.setFocusable(false);
v.setFocusableInTouchMode(false);
return v;
}

v = super.getGroupView(groupPosition, isExpanded, convertView, parent);
return v;
}


private int getItemViewType(View position) {
ViewHolder holder = (ViewHolder) position.getTag();
return holder.viewType;
}

@Override
public View getChildView(int groupPosition,
int childPosition, boolean isLastChild,
View convertView, ViewGroup parent) {

int viewType = 0;

if(convertView != null)
viewType = this.getItemViewType(convertView);

if(areSame && groupPosition == 1)
groupPosition += 1;

if(!Data.containsKey(groupPosition))
Data.put(groupPosition, new Model());

ViewHolder holder;

View v = convertView;

switch(groupPosition) {

case 0:
if (v == null || viewType != 1) {
v = new LinearLayout(parent.getContext());
CheckBox cb = new CheckBox(parent.getContext());
TextView tv = new TextView(parent.getContext());
TextView tv2 = new TextView(parent.getContext());


holder = new ViewHolder();
holder.cb = cb;
holder.cb.setTag(groupPosition);
holder.tv = tv;
holder.tv2 = tv2;
holder.viewType = 1;

((LinearLayout)v).addView(holder.cb);
((LinearLayout)v).addView(holder.tv);
((LinearLayout)v).addView(holder.tv2);
v.setTag(holder);
} else {
holder = (ViewHolder)v.getTag();
holder.cb.setTag(groupPosition);
}

holder.cb.setOnCheckedChangeListener(new OnCheckedChangeListener() {

@Override
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
int position = (Integer) buttonView.getTag();
Model model = Data.get(position);

buttonView.setChecked(isChecked);
model.setSelected(isChecked);
areSame = isChecked;
notifyDataSetChanged();
}
});

holder.cb.setText("Hallo Child 1");
holder.cb.setTextColor(0xFFFFFFFF);
holder.cb.setChecked(Data.get(groupPosition).isSelected());
holder.tv.setText("Test Child 1");
holder.tv.setTextColor(0xFFFF00CC);
holder.tv2.setTextColor(0xFFFF00CC);
holder.tv2.setText(" Ich bin oben!");
break;
case 1:
if(!areSame) {
if (v == null || viewType != 0) {
v = new LinearLayout(parent.getContext());
CheckBox cb = new CheckBox(parent.getContext());
TextView tv = new TextView(parent.getContext());



holder = new ViewHolder();
holder.cb = cb;
holder.cb.setTag(groupPosition);
holder.tv = tv;
holder.cb.setTag(groupPosition);
holder.viewType = 0;

((LinearLayout)v).addView(holder.cb);
((LinearLayout)v).addView(holder.tv);
v.setTag(holder);
} else {
holder = (ViewHolder)v.getTag();
holder.cb.setTag(groupPosition);
}

holder.cb.setOnCheckedChangeListener(new OnCheckedChangeListener() {

@Override
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
int position = (Integer) buttonView.getTag();
Model model = Data.get(position);

buttonView.setChecked(isChecked);
model.setSelected(isChecked);

}
});

holder.cb.setText("Hallo Child 2");
holder.cb.setTextColor(0xFFFFFFFF);
holder.cb.setChecked(Data.get(groupPosition).isSelected());
holder.tv.setText("Test Child 2");
holder.tv.setTextColor(0xFFFF00CC);

break;
}
case 2:
if (v == null || viewType != 0) {
v = new LinearLayout(parent.getContext());
CheckBox cb = new CheckBox(parent.getContext());
TextView tv = new TextView(parent.getContext());



holder = new ViewHolder();
holder.cb = cb;
holder.cb.setTag(groupPosition);
holder.tv = tv;
holder.cb.setTag(groupPosition);
holder.viewType = 0;

((LinearLayout)v).addView(holder.cb);
((LinearLayout)v).addView(holder.tv);
v.setTag(holder);
} else {
holder = (ViewHolder)v.getTag();
holder.cb.setTag(groupPosition);
}

holder.cb.setOnCheckedChangeListener(new OnCheckedChangeListener() {

@Override
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
int position = (Integer) buttonView.getTag();
Model model = Data.get(position);

buttonView.setChecked(isChecked);
model.setSelected(isChecked);


}
});



holder.cb.setText("Hallo Child 3");
holder.cb.setTextColor(0xFFFFFFFF);
holder.cb.setChecked(Data.get(groupPosition).isSelected());
holder.tv.setText("Test Child 3");
holder.tv.setTextColor(0xFFFF00CC);

break;
default:
if (v == null || viewType != 0) {
v = new LinearLayout(parent.getContext());
CheckBox cb = new CheckBox(parent.getContext());
TextView tv = new TextView(parent.getContext());

((LinearLayout)v).addView(cb);
((LinearLayout)v).addView(tv);



holder = new ViewHolder();
holder.viewType = 0;
holder.cb = cb;
holder.tv = tv;
v.setTag(holder);
} else {
holder = (ViewHolder)v.getTag();
}

holder.cb.setText("Default");
holder.cb.setTextColor(0xFFFFFFFF);
holder.tv.setText("Default");
holder.tv.setTextColor(0xFFFF00CC);
}

return v;
}

});
// =============================== ADAPTER END ====================================
}

public static class Model {

private boolean selected;

public Model() {
selected = false;
}

public boolean isSelected() {
return selected;
}

public void setSelected(boolean selected) {
this.selected = selected;
}

}

public static class ViewHolder {
protected CheckBox cb;
protected TextView tv;
protected TextView tv2;

protected int viewType;
}
}
[/code]