Android ListView在向下滚动时显示不正确的内容,然后在向上滚动时显示正确的内容

时间:2018-08-05 00:18:34

标签: android listview scroll recycle

作为Android Studio的新手,我目前对相同代码的不同行为感到困惑。如果代码相同,那么上下滚动结果可能会有什么不同?这可能是一些Android View Recycling问题吗?

从Web上的JSON文件解析数据后,自定义适配器读取并设置ListView中每一行的对应数据。每行有四段信息:1)标题,2)描述,3)图像URL(通过毕加索显示)和4)分配给Button的setOnClickListener的外部URL,可通过以下方式打开浏览器意图。

虽然标题,描述和图像在每一行上都可以正常工作,但是对于分配给Button的setOnClickListener的URL,碰巧有一种非常怪异,奇怪,到目前为止(对我而言)无法解释的行为:

1)首次显示ListView时,单击“按钮”以在浏览器上打开URL时,系统将打开的不是该项目的正确URL,而是对应于Array中NEXT项的URL。 / p>

2)如果单击某个按钮(之前导致不正确的URL)时,先向下滚动然后再次向上滚动,则确实可以打开正确的URL。

在相同的代码上,在相同的运行中,该按钮在向下滚动时将显示错误的URL,在向上滚动时将显示正确的URL。

到目前为止,我的提示是考虑到这可能与Android的Recycling Views功能有关,但我仍然不知道如何解决此问题。

我确实在CustomAdapter.java中包含以下代码行,希望防止回收:

    @Override
public int getViewTypeCount() {
    return getCount();
}
@Override
public int getItemViewType(int position) {
    return position;
}

尽管如此,包括或排除上述代码并没有实际区别。

我的CustomAdapter.java的完整代码如下:

   package ca.costari.apps.ge3000;

import android.annotation.SuppressLint;
import android.content.Context;
import android.content.Intent;
import android.net.Uri;
import android.os.Build;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.BaseAdapter;
import android.widget.Button;
import android.widget.ImageView;
import android.widget.TextView;
import com.squareup.picasso.Picasso;
import java.util.ArrayList;
import java.util.Objects;

public class CustomAdapter extends BaseAdapter {

    private Context context;
    private ArrayList<PlayersModel> playersModelArrayList;
    private String clickurl;

    public CustomAdapter(Context context, ArrayList<PlayersModel> playersModelArrayList) {

        this.context = context;
        this.playersModelArrayList = playersModelArrayList;
    }

    @Override
    public int getViewTypeCount() {
        return getCount();
    }
    @Override
    public int getItemViewType(int position) {
        return position;
    }

    @Override
    public int getCount() {
        return playersModelArrayList.size();
    }

    @Override
    public Object getItem(int position) {
        return playersModelArrayList.get(position);
    }

    @Override
    public long getItemId(int position) {
        return 0;
    }

    @SuppressLint("InflateParams")
    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        ViewHolder holder;

        if (convertView == null) {
            holder = new ViewHolder();
            LayoutInflater inflater = (LayoutInflater) context

                    .getSystemService(Context.LAYOUT_INFLATER_SERVICE);
            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
                convertView = Objects.requireNonNull(inflater).inflate(R.layout.lv_item, null, true);
            }

            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) {
                holder.procesa_titulo = Objects.requireNonNull(convertView).findViewById(R.id.id_titulo);
            }
            holder.procesa_mensajepromo = convertView.findViewById(R.id.id_mensajepromo);
            holder.procesa_imagenurl = convertView.findViewById(R.id.imageViewPromo);

            convertView.setTag(holder);
        }else {
            // the getTag returns the viewHolder object set as a tag to the view
            holder = (ViewHolder)convertView.getTag();
        }

        clickurl = playersModelArrayList.get(position).getClickURL();
        holder.procesa_clickurl = convertView.findViewById(R.id.id_clickurl);
        holder.procesa_clickurl.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {
                AsignaBotonUrl(v, clickurl);
            }
        });
        holder.procesa_titulo.setText(playersModelArrayList.get(position).getTitulo());
        holder.procesa_mensajepromo.setText(playersModelArrayList.get(position).getMensajePromo());

        // Picasso to display image from URL
        Picasso.with(holder.procesa_imagenurl.getContext())
                .load(playersModelArrayList.get(position).getImageURL())
                .placeholder(R.drawable.echale_un_ojo_te_interesa)
                .into(holder.procesa_imagenurl);

        return convertView;
    }

    private void AsignaBotonUrl(View view, String url) {
        Intent browserIntent = new Intent(
                Intent.ACTION_VIEW,
                Uri.parse(url));
        view.getContext().startActivity(browserIntent);
    }

    private class ViewHolder {
        protected TextView procesa_titulo, procesa_mensajepromo;
        protected ImageView  procesa_imagenurl;
        protected Button procesa_clickurl;
    }

}

上面的代码将值分配给以下ListView xml中包含的项目:

    <?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:padding="10dp"
    android:orientation="vertical">

    <LinearLayout
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:background="#fdfdfd"
        android:orientation="vertical"
        android:layout_marginBottom="30dp"
        tools:ignore="UselessParent">

        <TextView
            android:id="@+id/id_titulo"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:textColor="#2f2f2f"
            android:layout_marginTop="10dp"
            android:layout_marginBottom="20dp"
            android:gravity="center_vertical"
            android:textSize="20sp"
            android:paddingLeft="10dp" />

        <ImageView
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:id="@+id/imageViewPromo"
            android:layout_marginBottom="20dp"
            android:contentDescription="@string/echale_un_ojo" />

        <TextView
            android:id="@+id/id_mensajepromo"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:textColor="#2f2f2f"
            android:gravity="center_vertical"
            android:textAppearance="?android:attr/textAppearanceMedium"
            android:paddingLeft="10dp"
            android:layout_marginBottom="10dp" />

        <Button
            android:id="@+id/id_clickurl"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:gravity="center"
            android:textSize="24sp"
            android:text="@string/conocer_mas"
            android:drawableRight="@drawable/open_web"
            android:layout_marginBottom="30dp" />

    </LinearLayout>

</LinearLayout>

这是我在StackOverflow上的第一篇文章,因此,请您耐心等待,因为我可能未正确提出问题,请告知我是否可能需要在此处包括更多代码或提供更多信息信息。

非常感谢!

1 个答案:

答案 0 :(得分:0)

clickurl更改为最终的局部变量,因此将其删除

private String clickurl

,只需在getView中使用它,以便每个侦听器都有一个唯一的URL。

final String clickurl = playersModelArrayList.get(position).getClickURL();

按原样,您的onClickListener使用单个clickurl实例(保存在Adapter类中的那个实例),因此它将始终使用在上一次调用getView中设置的内容。