Android ListView Recycler吓坏了,还是我失败了?

时间:2012-02-02 15:04:55

标签: android android-listview

我创建了一个使用row.xml文件填充的自定义列表视图。每行包含两个TextView和一个ImageView。我从JSON文件中填充TextView(但这样工作正常,所以我认为你可以放心地忽略它),并且我根据SharedPreferences填充ImageView ......这就是问题所在。

基本上逻辑是这样的:有人点击listview中的一行,如果关联的活动成功完成,我通知SharedPreferences然后,一旦用户返回列表,一个开关/案例读取SharedPreferences文件并更新该特定行中的ImageView,显示绿色图标。如果用户未通过活动,则该行中的ImageView将显示一个红色图标。列表中第一个未完成的活动旁边会显示一个蓝色图标。

我是通过在视图形成循环中为SharedPreferences运行switch / case命令来完成的。

所以...我的问题:我打开列表,一切正常。该列表按预期显示一个蓝色图标。当我向下滚动时,我看到另一个蓝色图标向下......这不应该在那里。好吧,你在想“你已经完成了你的编码”。但是 - 当我向上滚动列表时,我看到蓝色图标现在已从预期的列表项“跳转”到下面的列表。我再次向下滚动,我看到蓝色图标成倍增加 - 现在有两个在两个不同的行中,之前不存在。所以我完成了一项任务。它应该出现一个绿色图标。我上下滚动了几次,突然在不同的列表行旁边有3-4个绿色图标。

我唯一的猜测是,回收商正在吓坏了。我对吗?有什么我可以做的来解决这个问题吗?我试图偷偷摸摸 - 我使用'setVisibility-GONE'用于那些不需要imageview的行...在经过一些滚动之后,它就是 - 它令人烦恼的绿色点亮了它不可能出现的行。这是我的源代码:

public class MainList extends ListActivity {
static SharedPreferences statusSettings;
String jsonString;
static JSONObject json;
static JSONArray arrayTest;
static int bob = 3;


@Override
protected void onListItemClick(ListView l, View v, int position, long id) {
    // TODO Auto-generated method stub
    super.onListItemClick(l, v, position, id);

    try {
        JSONObject e = arrayTest.getJSONObject(position);
        Intent intent = new Intent();
        intent.putExtra("clientName", e.getString("proj_name"));
        intent.putExtra("clientAddress", e.getString("c_address"));
        intent.putExtra("clientComments", e.getString("comments"));
        intent.putExtra("clientOrder", e.getString("order"));
        intent.setClass(MainList.this, ClientDetails.class);
        MainList.this.startActivity(intent);


    } catch (JSONException e) {
        // TODO Auto-generated catch block
        Log.e("JSON", "Problem creating object from array!");
        e.printStackTrace();
    }
}

private static class EfficientAdapter extends BaseAdapter {
    private LayoutInflater mInflater;


    public EfficientAdapter(Context context) {
        // Cache the LayoutInflate to avoid asking for a new one each time.
        mInflater = LayoutInflater.from(context);

    }

    public int getCount() {
        return arrayTest.length();

    }

    public Object getItem(int position) {
        return position;
    }

    public long getItemId(int position) {
        return position;
    }

    public View getView(int position, View convertView, ViewGroup parent) {
        // A ViewHolder keeps references to children views to avoid unneccessary calls
        // to findViewById() on each row.
        ViewHolder holder;

        // When convertView is not null, we can reuse it directly, there is no need
        // to reinflate it. We only inflate a new View when the convertView supplied
        // by ListView is null.
        if (convertView == null) {
            convertView = mInflater.inflate(R.layout.mainlist, null);

            // Creates a ViewHolder and store references to the two children views
            // we want to bind data to.
            holder = new ViewHolder();
            holder.icon = (ImageView) convertView.findViewById(R.id.ivMainIcon);
            holder.textName = (TextView) convertView.findViewById(R.id.tvMainName);
            holder.textAddress = (TextView) convertView.findViewById(R.id.tvMainAddress);


            convertView.setTag(holder);
        } else {
            // Get the ViewHolder back to get fast access to the TextView
            // and the ImageView.
            holder = (ViewHolder) convertView.getTag();
        }

        // Bind the data efficiently with the holder.

        JSONObject e;
        try {
            e = arrayTest.getJSONObject(position);
            holder.textName.setText(e.getString("proj_name"));
            holder.textAddress.setText(e.getString("c_address"));   

            switch (statusSettings.getInt(e.getString("order"), 0)){
                case 1:
                    holder.icon.setVisibility(View.INVISIBLE);
                    break;
                case 2:
                    if(bob == 3){
                        holder.icon.setImageResource(R.drawable.next_delivery);
                        bob = 5;
                    }
                    break;
                case 3:
                    holder.icon.setImageResource(R.drawable.delivered_icon);
                    break;
                case 4:
                    holder.icon.setImageResource(R.drawable.undelivered_icon);
                    break;
            }


        } catch (JSONException e1) {
            // TODO Auto-generated catch block
            Log.e("JSON", "Couldn't put one of JSON arrays into object");
            e1.printStackTrace();
        }

        return convertView;
    }

    static class ViewHolder {
        ImageView icon;
        TextView textName;
        TextView textAddress;

    }
}

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    jsonString = getIntent().getExtras().getString("jsonString");
    try {
        json = new JSONObject(jsonString);
        arrayTest = json.getJSONArray("client_list");
    } catch (JSONException e) {
        // TODO Auto-generated catch block
        Log.e("JSON", "Couldn't create the JSON Array");
        e.printStackTrace();
    }
    setListAdapter(new EfficientAdapter(this));

}

@Override
protected void onResume() {
    // TODO Auto-generated method stub
    super.onResume();
    bob = 3;        
    statusSettings = getSharedPreferences("status", 0);

    setListAdapter(new EfficientAdapter(this));
}


}

非常感谢任何帮助。

/ edit:如果你需要它 - 这是row.xml文件:

<?xml version="1.0" encoding="utf-8"?>

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:orientation="vertical" >

<LinearLayout
    android:id="@+id/LinearLayout02"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:layout_alignParentBottom="true" >

    <TextView
        android:id="@+id/tvMainName"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:layout_weight="1"
        android:paddingBottom="10dp"
        android:paddingTop="10dp"
        android:text="Large Text"
        android:textColor="#FFFFFF"
        android:textSize="25dp" />

    <ImageView
        android:id="@+id/ivMainIcon"
        android:layout_width="15dp"
        android:layout_height="15dp"
        android:layout_gravity="bottom"
        android:layout_marginRight="20dp"
        android:layout_marginBottom="10dp"
        android:layout_weight="0"
        android:scaleType="fitXY" />
</LinearLayout>

<TextView
    android:id="@+id/tvMainAddress"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:paddingBottom="5dp"
    android:text="TextView"
    android:textSize="15dp" />

</LinearLayout>

更新了Switch / Case(由Brigham提供)......仍然使用bob变量的半解决方案。至少现在一切正常!:

switch (statusSettings.getInt(e.getString("order"), 0)){
                case 1:
                    holder.icon.setVisibility(View.INVISIBLE);
                    break;
                case 2:
                    if(bob.toString().equalsIgnoreCase("do it") || bob.toString().equalsIgnoreCase(e.getString("proj_name"))){

                        holder.icon.setVisibility(View.VISIBLE);
                        holder.icon.setImageResource(R.drawable.next_delivery);
                        bob = e.getString("proj_name");
                    }else{
                        holder.icon.setVisibility(View.INVISIBLE);
                    }
                    break;
                case 3:
                    holder.icon.setVisibility(View.VISIBLE);
                    holder.icon.setImageResource(R.drawable.delivered_icon);
                    break;
                case 4:
                    holder.icon.setVisibility(View.VISIBLE);
                    holder.icon.setImageResource(R.drawable.undelivered_icon);
                    break;
                default:
                    holder.icon.setVisibility(View.INVISIBLE);
                    break;
            }

2 个答案:

答案 0 :(得分:1)

case 2中,您只需设置bob == 3的图像。您应该添加一个else子句,并在bob != 3时使图像不可见。您还应该确保在switch语句之前将图标设置为可见,因为大多数情况都要求它可见。

您还应该考虑在switch语句中添加默认大小写。

        holder.icon.setVisibility(View.VISIBLE);
        switch (statusSettings.getInt(e.getString("order"), 0)){
            case 1:
                holder.icon.setVisibility(View.INVISIBLE);
                break;
            case 2:
                if(bob == 3){
                    holder.icon.setImageResource(R.drawable.next_delivery);
                    bob = 5;
                } else {
                    holder.icon.setVisibility(View.INVISIBLE);
                }
                break;
            case 3:
                holder.icon.setImageResource(R.drawable.delivered_icon);
                break;
            case 4:
                holder.icon.setImageResource(R.drawable.undelivered_icon);
                break;
            default:
                holder.icon.setVisibility(View.INVISIBLE);
                break;
        }

答案 1 :(得分:0)

另一种选择是禁用您的回收商。你可能不想这样做,但你要做的就是删除convertView == null和它的else语句。这将阻止回收视图中的工件。