【发布时间】：2015-07-07 08:21:35
【问题描述】：
我的Activity 中有以下适配器。我在GroupView 上设置了一个CheckBox，当触摸它时将选中/取消选中子视图（也有一个CheckBox）。
CheckBox(true or false) 的状态存储在数据库中，最初设置为 true/checked。复选框最初设置为选中。
当我触摸GroupCheckbox 时出现一些奇怪的行为，它还会取消选中不属于已触摸GroupView 的复选框。
例如，如果我有 5 个 GroupViews 并且每个 GroupView 有 5 个孩子。
如果我取消选中第一个 GroupView CheckBox，它会取消选中它的所有孩子，但也会取消选中第三个 GroupView CheckBox 及其孩子。
有什么想法吗？
[编辑 1]
我在开始赏金时删除了以前的代码 sn-ps。由于 Ankit Kumar 帮助我处理了 getGroupView 方法中的 Viewholder 模式，因此代码已与原始代码有所不同。当仅触摸一个复选框时，它仍然取消选中多个 groupview 复选框。后备数据库将复选框的所有状态都选中为true。
视图似乎与数据库不同步。我已经注销了证明所有复选框的状态都是真实的声明。
有没有办法只取消用户触摸的复选框而不影响其他复选框？
public class ExpList extends ExpandableListActivity {
String arrGroupelements[];
String arrChildelements[][];
private static final String TAG = ExpList.class.getSimpleName();
DisplayMetrics metrics;
int width;
ExpandableListView expList;
RROnCallApplication appObj;
Cursor companies;
Button mainMenu;
ExpAdapter adapter;
int companyCount;
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
Log.e(TAG, "oncreate");
appObj = (RROnCallApplication) getApplication();
mainMenu = (Button)findViewById(R.id.buttonmainmenu);
mainMenu.setOnClickListener(new OnClickListener() {
@Override
public void onClick(View v) {
Intent i = new Intent(ExpList.this, MenuActivity2.class);
startActivity(i);
}
});
try {
companies = appObj.dbModel.queryAllFromCompanyBranch();
arrGroupelements = new String[companies.getCount()];
Log.e(TAG, "companies count = " + companies.getCount());
arrChildelements = new String[arrGroupelements.length][20];
if(companies != null && companies.getCount() > 0) {
if(companies.moveToFirst()) {
int i = 0;
do {
arrGroupelements[i] = companies.getString(companies.getColumnIndex(DBModel.C_COMPANYBRANCH_NAME));
Log.e(TAG, "arrGroupelements[" + i +"] = " + arrGroupelements[i]);
int compID = appObj.dbModel.getCompanyidFromName(arrGroupelements[i]);
Log.e(TAG, "compID = " + compID);
String[] branchesArr = appObj.dbModel.getBranchNamesfromCompanyId(compID);
Log.e(TAG, "branchesArr length = " + branchesArr.length);
for(int h = 0; h < branchesArr.length; h++) {
arrChildelements[i][h] = branchesArr[h];
}
i++;
}while(companies.moveToNext());
Log.e(TAG, "arrGroupelements size = " + arrGroupelements.length);
}//end of moveToFirst
}
}
catch(Exception e) {
Toast.makeText(this, "There was a problem downloading companies and branches", Toast.LENGTH_LONG).show();
Log.e(TAG, "********Exception = " + e.toString());
}
finally {
companyCount = companies.getCount();
companies.close();
}
expList = getExpandableListView();
metrics = new DisplayMetrics();
getWindowManager().getDefaultDisplay().getMetrics(metrics);
width = metrics.widthPixels;
//this code for adjusting the group indicator into right side of the view
expList.setIndicatorBounds(width - GetDipsFromPixel(50), width - GetDipsFromPixel(10));
expList.setAdapter(new ExpAdapter(this));
for(int h = 0; h < companyCount; h++){
expList.expandGroup(h);
}
expList.setOnGroupExpandListener(new OnGroupExpandListener() {
@Override
public void onGroupExpand(int groupPosition) {
Log.e("onGroupExpand", "OK");
Log.e("groupPosition", " " + groupPosition);
}
});
// expList.setOnGroupClickListener(new OnGroupClickListener() {
//
// @Override
// public boolean onGroupClick(ExpandableListView parent, View v,
// int groupPosition, long id) {
//
//
// Log.e(TAG, "groupPosition in onGroupClickListener = " + groupPosition);
//
// int count = 0;
//
// for (int i = 0; i < arrChildelements[groupPosition].length; i++){
//
// if(arrChildelements[groupPosition][i] != null){
//
// count += arrChildelements[groupPosition][i] != null ? 1 : 0;
//
// Log.e("TAG", "child count in onGroupClickListener = " + count);
// Log.e(TAG, "arrChildelements[groupPosition][i] = " + arrChildelements[groupPosition][i]);
//
// int branchID = appObj.dbModel.getBranchIdFromName(arrChildelements[groupPosition][i]);
//
// appObj.dbModel.updateBranchSelectedStatus(String.valueOf(branchID), "N");
//
//
//
// }//end of if
//
// }//end of for loop
//
//
//
// return false;
// }
// });
expList.setOnGroupCollapseListener(new OnGroupCollapseListener() {
@Override
public void onGroupCollapse(int groupPosition) {
Log.e("onGroupCollapse", "OK");
}
});
expList.setOnChildClickListener(new OnChildClickListener() {
@Override
public boolean onChildClick(ExpandableListView parent, View v, int groupPosition, int childPosition, long id) {
Log.e("OnChildClickListener", "OK Group = " + groupPosition + " child = " + childPosition);
TextView tvBranchName = (TextView) v.findViewById(R.id.tvPlayerName);
String branchName = tvBranchName.getText().toString();
Log.e(TAG, "branch name = " + branchName);
int branchID = appObj.dbModel.getBranchIdFromName(branchName);
Log.e(TAG, "branch ID = " + branchID);
String companyName = arrGroupelements[groupPosition];
Log.e(TAG, "**********CompanyName = " + companyName);
final CheckBox cb = ((CheckBox)v.findViewById(R.id.checkbox));
if(cb.isChecked() == true) {
Log.e(TAG, "checkBox is true but setting it to false now" );
cb.setChecked(false);
appObj.dbModel.updateBranchSelectedStatus(String.valueOf(branchID), "N");
Log.e(TAG, "just called updateBranchSelectedStatus with values " + String.valueOf(branchID) + " " + "N");
Log.e(TAG, "Branches selected are " + appObj.dbModel.getBranchList());
if(appObj.dbModel.isCompanySelected(companyName) == false){
//set companySeelcted to false
appObj.dbModel.updateCompanySelectedStatus(companyName, "N");
Log.e(TAG, "Setting company to no longer selected as no branches are selected for " + companyName);
}
}
else {
Log.e(TAG, "checkBox is false but setting it to true");
cb.setChecked(true);
appObj.dbModel.updateBranchSelectedStatus(String.valueOf(branchID), "Y");
Log.e(TAG, "just called updateBranchSelectedStatus with values " + String.valueOf(branchID) + " " + "Y");
Log.e(TAG, "Branhes selected are " + appObj.dbModel.getBranchList());
//set company to selected
appObj.dbModel.updateCompanySelectedStatus(companyName, "Y");
}
return false;
}
});
}//end of onCreate
@Override
public void onBackPressed() {
super.onBackPressed();
Intent i = new Intent(this, OnCallMenuActivity.class);
i.setFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP);
startActivity(i);
}
public int GetDipsFromPixel(float pixels) {
// Get the screen's density scale
final float scale = getResources().getDisplayMetrics().density;
// Convert the dps to pixels, based on density scale
return (int) (pixels * scale + 0.5f);
}
public class ExpAdapter extends BaseExpandableListAdapter {
private Context myContext;
class ViewHolder {
public TextView groupName;
public CheckBox groupCheckBox;
}
public ExpAdapter(Context context) {
myContext = context;
}
@Override
public Object getChild(int groupPosition, int childPosition) {
return null;
}
@Override
public long getChildId(int groupPosition, int childPosition) {
return 0;
}
@Override
public View getChildView(int groupPosition, int childPosition, boolean isLastChild, View convertView, ViewGroup parent) {
if (convertView == null) {
LayoutInflater inflater = (LayoutInflater) myContext.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
convertView = inflater.inflate(R.layout.child_row, null);
}
TextView tvPlayerName = (TextView) convertView.findViewById(R.id.tvPlayerName);
tvPlayerName.setText(arrChildelements[groupPosition][childPosition]);
CheckBox cb = (CheckBox)convertView.findViewById(R.id.checkbox);
int branchID = appObj.dbModel.getBranchIdFromName(arrChildelements[groupPosition][childPosition]);
Log.e(TAG, "inside getchildView and branchID = " + branchID);
boolean isBranchSelected = appObj.dbModel.isBranchSelected(String.valueOf(branchID));
Log.e(TAG, "isBranchSelected = " + isBranchSelected);
if(isBranchSelected == true) {
cb.setChecked(true);
Log.e(TAG, "inside getchildView and cb.setChecked(true)");
}
else {
cb.setChecked(false);
Log.e(TAG, "inside getchildView and cb.setChecked(false)");
}
return convertView;
}
@Override
public int getChildrenCount(int groupPosition) {
//return arrChildelements[groupPosition].length;
Log.e(TAG, "getChildrenCount");
int count = 0;
for (int i = 0; i < arrChildelements[groupPosition].length; i++)
count += arrChildelements[groupPosition][i] != null ? 1 : 0;
return count;
}
@Override
public Object getGroup(int groupPosition) {
return null;
}
@Override
public int getGroupCount() {
Log.e(TAG, "getGroupCount");
return arrGroupelements.length;
}
@Override
public long getGroupId(int groupPosition) {
return 0;
}
ViewHolder viewHolder;
@Override
public View getGroupView(final int groupPosition, boolean isExpanded, View convertView, ViewGroup parent) {
Log.e(TAG, "getGroupView");
viewHolder = new ViewHolder();
if (convertView == null) {
LayoutInflater inflater = (LayoutInflater) myContext.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
convertView = inflater.inflate(R.layout.group_row, null);
viewHolder.groupName = (TextView) convertView.findViewById(R.id.tvGroupName);
viewHolder.groupCheckBox = (CheckBox) convertView.findViewById(R.id.groupcheckbox);
convertView.setTag(viewHolder);
} else viewHolder = (ViewHolder) convertView.getTag();
final ViewHolder holder = viewHolder;
holder.groupName.setText(arrGroupelements[groupPosition]);
holder.groupCheckBox.setTag(groupPosition);
holder.groupCheckBox.setOnCheckedChangeListener(new OnCheckedChangeListener() {
@Override
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
int pos = (Integer) holder.groupCheckBox.getTag();
Log.e(TAG, "oncheckChanged has fired at position " + pos);
int yCount = 0;
Cursor c = appObj.dbModel.queryAllFromCompanyBranch();
c.moveToFirst();
do {
String str = c.getString(c.getColumnIndex(DBModel.C_COMPANYBRANCH_SELECTED));
if(str.equalsIgnoreCase("Y")) {
yCount++;
}
} while(c.moveToNext());
Log.e(TAG, "yCount before = " + yCount);
if(isChecked == true) {
Log.e(TAG, "checkBox true");
int count = 0;
for (int i = 0; i < getChildrenCount(pos); i++) {
if(arrChildelements[pos][i] != null) {
count += arrChildelements[pos][i] != null ? 1 : 0;
Log.e("TAG", "child count in onGroupClickListener = " + count);
Log.e(TAG, "arrChildelements[groupPosition][i] = " + arrChildelements[pos][i]);
int branchID = appObj.dbModel.getBranchIdFromName(arrChildelements[pos][i]);
appObj.dbModel.updateBranchSelectedStatus(String.valueOf(branchID), "Y");
appObj.dbModel.updateCompanySelectedStatus(arrGroupelements[pos], "Y");
}//end of if
}//end of for loop
}
else if(isChecked == false) {
Log.e(TAG, "checkBox false");
int count = 0;
for (int i = 0; i < getChildrenCount(pos); i++) {
if(arrChildelements[pos][i] != null) {
count += arrChildelements[pos][i] != null ? 1 : 0;
Log.e("TAG", "child count in onGroupClickListener = " + count);
Log.e(TAG, "arrChildelements[groupPosition][i] = " + arrChildelements[pos][i]);
int branchID = appObj.dbModel.getBranchIdFromName(arrChildelements[pos][i]);
appObj.dbModel.updateBranchSelectedStatus(String.valueOf(branchID), "N");
appObj.dbModel.updateCompanySelectedStatus(arrGroupelements[pos], "N");
Log.e(TAG, "Setting company to no longer selected as no branches are selected for " + arrGroupelements[pos]);
}//end of if
}//end of for loop
}
int yCount2 = 0;
Cursor c2 = appObj.dbModel.queryAllFromCompanyBranch();
c2.moveToFirst();
do {
String str2 = c2.getString(c2.getColumnIndex(DBModel.C_COMPANYBRANCH_SELECTED));
if(str2.equalsIgnoreCase("Y")){
yCount2++;
}
} while(c2.moveToNext());
Log.e(TAG, "yCount2 after = " + yCount2);
notifyDataSetChanged();
}
});
ExpandableListView mExpandableListView = (ExpandableListView) parent;
mExpandableListView.expandGroup(groupPosition);
return convertView;
}
@Override
public boolean hasStableIds() {
return false;
}
@Override
public boolean isChildSelectable(int groupPosition, int childPosition) {
return true;
}
}
}
【问题讨论】：
-
使用 ViewHolder 初始化和使用适配器中的视图
-
您在滚动时遇到问题吗？
-
@AnkitKumar viewHolder 通过不必查找对象来加快适配器的响应速度，但这是否解释了为什么当我触摸一个 GroupView 复选框时，另一个 groupview 也会被选中？
-
嗯，这是一堵代码墙。此外，它比它需要的复杂很多倍，因为它与一大堆与应用程序的业务逻辑相关的东西联系在一起。创建一个单独的应用程序，您可以在其中尝试重现问题，不使用数据库并且最少使用模型对象。如果您可以重现该问题，那将是在问题中使用的更好的示例代码。如果您无法重现该问题，则您有一个可行的解决方案，并且需要将您现有的代码更接近该解决方案中的内容。
-
这有很多代码和很多地方会出错。话虽如此，我对
RadioButtons（相同概念）所做的是将每个孩子的ID存储在HashMap<String, Boolean>中，以及是否检查它。我还必须每次都将它们全部设置为未选中，然后检查 id 是否在getChildView()的列表中并进行相应设置。
标签： android checkbox expandablelistview expandablelistadapter