我正在使用custom list view，其中每行包含两个图像视图和一个文本视图。我在view holder的{{1}}方法中使用getView()。我已在custom adapter方法中的一个图片视图上设置了Onclicklistener()，getview()中的活动中onSelectItemListener()也设置了listview。现在，当我点击imageview时，它会影响特定行，但当我向下滚动时，我会得到受该点击影响的随机行。当我点击imageview时会发生这种情况但是当我不使用viewholder时就不会发生这种情况。我真的不知道为什么会这样。请帮我。
以下是我的getView()方法代码：
public View getView(int position, View convertView, ViewGroup parent) {
final int myposition = position;
// TODO Auto-generated method stub
if (convertView == null) {
LayoutInflater inflater = LayoutInflater.from(context);
convertView = inflater.inflate(R.layout.fileviewrowfrnview_layout, null);
ViewHolder holder = new ViewHolder();
holder.icon = (ImageView) convertView.findViewById(R.id.icon2);
holder.text = (TextView) convertView.findViewById(R.id.file_name2);
holder.download = (ImageView) convertView.findViewById(R.id.play_buttonOropen2);
convertView.setTag(holder);
}
final ViewHolder holder = (ViewHolder) convertView.getTag();
holder.download.setOnClickListener(new View.OnClickListener() {
@
Override
public void onClick(View v) {
// TODO Auto-generated method stub
if (frnshare_list.get(myposition).getFileType().trim()
.equalsIgnoreCase("file")) {
// it is file
if (frnshare_list.get(myposition).getDownloadStatus().trim()
.equalsIgnoreCase("0")) {
// is is not a downloaded file do download it
holder.icon.setVisibility(View.GONE);
holder.download.setVisibility(View.GONE);
holder.text.setTextSize(14);
Log.e("file to be download", "" + frnshare_list.get(myposition).getFileName());
if (frnshare_list.get(myposition).getFileName().length() > 15)
holder.text.setText(frnshare_list.get(myposition).getFileName().subSequence(0, 15) + " will be downloaded soon.");
else
holder.text.setText(frnshare_list.get(myposition).getFileName() + " will be downloaded soon.");
UserService us = new UserService(context);
Frnshared fdto = frnshare_list.get(myposition);
fdto.setDownloadStatus("2"); // in queue
fdto.setMessageId("0");
us.updateDowloadStatusById(fdto);
us.updatemsgIdbyId(fdto);
if (CommonUtility.isNetworkAvailable(context)) {
AsyncTask < Void, Void, Void > httpconnection = new AsyncTask < Void, Void, Void > () {@
Override
protected Void doInBackground(Void...params) {
// TODO Auto-generated method stub
NetworkCommunication nc = new NetworkCommunication(
context);
try {
UserService us = new UserService(
context);
Frnshared fdto = frnshare_list
.get(myposition);
Log.i("file to be download", "" + fdto.getFileName());
fdto.setDownloadStatus("2"); // in queue
fdto.setMessageId("0");
us.updateDowloadStatusById(fdto);
us.updatemsgIdbyId(fdto);
String response = nc
.MyHttpPostDownload(frnshare_list
.get(myposition));
ParsersAndDataInsertion.DownloadRequestResponseParser(
response, context,
frnshare_list.get(myposition));
} catch (IOException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
// http call for download
return null;
}
}.execute(null, null, null);
}
Apitable(adto);
}
} else if (frnshare_list.get(myposition)
.getDownloadStatus().equalsIgnoreCase("1")) {
Intent intent = new Intent();
intent.setAction(android.content.Intent.ACTION_VIEW);
File file = new File(frnshare_list.get(myposition)
.getFileLocation());
intent.setDataAndType(Uri.fromFile(file), "audio/*");
context.startActivity(intent);
} else if (frnshare_list.get(myposition)
.getDownloadStatus().equalsIgnoreCase("2")) {
Toast.makeText(context, "already request snet", 2000)
.show();
}
} else if (frnshare_list.get(myposition).getFileType()
.equalsIgnoreCase("folder")) {}
}
});
// view purpose
String file_name = frnshare_list.get(position).getFileName();
if (file_name.length() > 15)
holder.text.setText(file_name.substring(0, 15) + "");
else
holder.text.setText(file_name); // frn name
try {
if (frnshare_list.get(myposition).getFileType()
.equalsIgnoreCase("file")) { // it is file
holder.icon.setImageResource(R.drawable.mp3_icon);
if (frnshare_list.get(myposition).getDownloadStatus()
.equalsIgnoreCase("0")) {
Log.i("file is ", "" + frnshare_list.get(myposition).getFileName());
holder.download.setImageResource(R.drawable.download_png);
} else if (frnshare_list.get(myposition).getDownloadStatus()
.equalsIgnoreCase("1")) {
} else if (frnshare_list.get(myposition).getDownloadStatus()
.equalsIgnoreCase("2")) {
Log.i("file is in queue", "" + frnshare_list.get(myposition).getFileName());
holder.icon.setVisibility(View.GONE);
holder.download.setVisibility(View.GONE);
holder.text.setTextSize(14);
if (frnshare_list.get(myposition).getFileName().length() > 15)
holder.text.setText(frnshare_list.get(myposition).getFileName().substring(0, 1);
else
holder.text.setText(frnshare_list.get(myposition).getFileName() );
}
} else if (frnshare_list.get(myposition).getFileType()
.equalsIgnoreCase("folder")) {
holder.icon.setImageResource(R.drawable.folder);
}
} catch (Exception e) {
// TODO: handle exception
Intent i = new Intent(context, TabViewActivity.class);
i.putExtra("show", "frntab");
context.startActivity(i);
BugSenseHandler.sendException(e);
e.printStackTrace();
}
return convertView;
}
答案 0 :(得分：2)
我已经看过你的代码了，而且我自己的代码很多次，这是我的建议
if(position==1){
//here change color or hide view
holder.someView.setVisibility(View.Invisible);
holder.someView.setColor ==== green; //forgive me abt syntx
}else{
//Here in else part it is important that you must reverse above things
//so that when you scroll list, GREEN color do not show up in random rows
holder.someView.setVisibility(View.Visible);
holder.someView.setColor ==== red; //forgive me abt syntx
}
//same is for on click listener etc
答案 1 :(得分：0)
您正在从ListView执行异步任务。那是在惹麻烦。这是事物 - ListView回收行。因此，当任务完成时，它实际上可能指向一个完全不同的位置，而不是它开始时的位置。您必须非常小心，并确保异步任务根本不影响UI，它强制整个列表重绘而不是在任何一行上进行更改，或者它永远不会假定被单击的行仍然有效，而是知道单击按钮时行中的位置，并确定现在需要更新的行。
基本上，如果可以，请避免使用AsyncTasks。如果不能，则需要确保AsyncTask不会触及UI（调用notifyDataSetChanged除外），并且在任务过程中需要的任何数据（比如保存到的文件名）都会被复制到本地AsyncTask创建时的成员变量，而不是从行视图中读取。
答案 2 :(得分：0)
我认为您的代码是实现它的常规方式，但是由于在创建时回收了您的问题。
你能做的是
1.检查你在方法中直接获得的位置 - 它应该按照重新绘制的顺序
2.而不是重新使用持有者中已经设置的内容，一旦从转换视图中获得持有者，从其他数据源（如List<<>>）设置该内容 - 这样可以节省重新创建视图的时间，内容将在您的其他数据持有者中。