Question

我正在学习android，我的代码工作正常。它从网站获取数据并将其显示在列表中。但是当subcat==1时，它有9个要获取的项目。它显示7个不同的项目并重复两个项目。首先它只显示了四个是NullPointerException第五个图像没有名字所以它重复4个图像直到9次，但现在一切都很好仍然只显示7并在此之后重复2。这是我的代码：

public class JSONAsyncTask extends AsyncTask<String, Void, Void> {
    ProgressDialog pd;
    Context context;


    JSONAsyncTask(Context context) {
        this.context = context;
    }

    @Override
    protected void onPreExecute() {

        super.onPreExecute();
        pd = new ProgressDialog(context);
        pd.setIndeterminate(true);
        pd.setCancelable(false);
        pd.setCanceledOnTouchOutside(false);
        pd.setMessage("Please wait..\nLoading data");
        pd.show();
    }

    @Override
    protected Void doInBackground(String... strings) {
        ArrayList<item> jsonArrayList = new ArrayList<item>();
        try {
            HttpClient client = new DefaultHttpClient();
            HttpGet httpget = new HttpGet("http://avdeal.in/get_all_products.php");
            HttpResponse response = client.execute(httpget);
            HttpEntity entity = response.getEntity();
            String data = EntityUtils.toString(entity);
            JSONObject jsonObject = new JSONObject(data);
            if (jsonObject != null) {
                String posts = jsonObject.getString("products");
                JSONArray postsArray = new JSONArray(posts);
                if (postsArray != null) {
                    if (postsArray.length() > 0) {
                        for (int i = 0; i < postsArray.length(); i++) {
                            JSONObject postsObject = postsArray
                                    .getJSONObject(i);
                            int subcat = postsObject.getInt("subcat_id");

                            if (subcats == subcat) {
                                int id=postsObject.getInt("id");
                                String title = postsObject.getString("product_title");
                                String price = postsObject.getString("product_price");
                                String url = "http://avdeal.in/seller/upload_images/" + postsObject.getString("product_url");


                                list1.add(new item(id, title, price, url));
                            }

                        }
                    }


                }
            }
        } catch (Exception e) {
            e.printStackTrace();
            return null;
        }
        return null;
    }

    @Override
    protected void onPostExecute(Void result) {

        super.onPostExecute(result);
        if (list1 != null) {
            pd.dismiss();
            adp = new actorAdapter(context, R.layout.listview, list1);
            lv.setAdapter(adp);
            adp.notifyDataSetChanged();
        }
        else {
            pd.setMessage("No internet access");
        }
        lv.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> adapterView, View view, int i, long l) {
                Toast.makeText(itemView.this, "Please Wait " + i + 1, Toast.LENGTH_SHORT).show();
                int id1 = list1.get(i).getId();
                String name = list1.get(i).getName();
                String price = list1.get(i).getPrice();
                Bitmap bitmap = list1.get(i).getImage();

                if (bitmap == null) {
                    Intent intent = new Intent(context, productDetail.class);
                    intent.putExtra("name", name);
                    intent.putExtra("price", price);
                    intent.putExtra("id", id1);
                    intent.putExtra("imagena", 1);
                    startActivity(intent);
                } else {
                    ByteArrayOutputStream stream = new ByteArrayOutputStream();
                    bitmap.compress(Bitmap.CompressFormat.PNG, 100, stream);
                    byte[] byteArray = stream.toByteArray();
                    Intent intent = new Intent(context, productDetail.class);
                    intent.putExtra("name", name);
                    intent.putExtra("price", price);
                    intent.putExtra("image", byteArray);
                    intent.putExtra("id", id1);
                    startActivity(intent);
                }
            }
        });
    }
}

private class ImageLoadTask extends AsyncTask<String, String, Bitmap>{
            Context context;
        ImageLoadTask(Context context) {
            this.context = context;
        }
        @Override
        protected Bitmap doInBackground(String... strings) {
            for(int i = 0 ; i < list1.size() ; i++)
            {
                if (list1.get(i).getImage()==null);
                {
                    String url=list1.get(i).getUrl();
                    InputStream is = null;
                    try {
                        is = (InputStream) new URL(url).getContent();
                    } catch (IOException e) {
                        e.printStackTrace();
                    }
                    Bitmap b = BitmapFactory.decodeStream(is);
                    list1.set(i, new item(list1.get(i).getId(), list1.get(i).getName(), list1.get(i).getPrice(), list1.get(i).getUrl() ,b));
                }
            }
                return null;

    }

        @Override
        protected void onProgressUpdate(String... values) {
            super.onProgressUpdate(values);

        }
        protected void onPostExecute(Bitmap result){
                super.onPostExecute(result);
                adp = new actorAdapter(context, R.layout.listview, list1);
                lv.setAdapter(adp);
                adp.notifyDataSetChanged();
        }
    }

我可以对延迟加载图片做什么吗？我尝试创建另一个AsyncTask，但它会在加载所有图像时显示图像。

我可以在下载一张图片后立即更新我的ListView吗？

请帮忙。我不能自己搞清楚。这两个问题阻碍了我学习更多。

public View getView(int position, View convertView, ViewGroup parent){
    inflater= (LayoutInflater) context.getSystemService(context.LAYOUT_INFLATER_SERVICE);
    View v =convertView;
    if (v == null) {

        holder=new ViewHolder();
        v=inflater.inflate(Resource,null);
        holder.name= (TextView) v.findViewById(R.id.titles);
        holder.price= (TextView) v.findViewById(R.id.price);
        holder.img= (ImageView) v.findViewById(R.id.itemimage);
        v.setTag(holder);
        holder.name.setText(list1.get(position).getName());
        holder.price.setText((CharSequence) list1.get(position).getPrice());
        holder.img.setImageBitmap(list1.get(position).getImage());



    }
    else {
        holder = (ViewHolder) v.getTag();
    }


    return v;
}

Answer 1

对于延迟图片加载，您需要为每张图片创建一个AsyncTask而不是AsyncTask所有图片（在分开的AsyncTasks中逐个加载它们）。

我建议你使用其中一个库：

这是4个库（来自here）之间基于意见的比较表，它可能有助于您选择：

修改

对于重复的项目，请更改您的getView方法：

public View getView(int position, View convertView, ViewGroup parent){ inflater= (LayoutInflater) context.getSystemService(context.LAYOUT_INFLATER_SERVICE); View v =convertView; if (v == null) { holder=new ViewHolder(); v=inflater.inflate(Resource,null); holder.name= (TextView) v.findViewById(R.id.titles); holder.price= (TextView) v.findViewById(R.id.price); holder.img= (ImageView) v.findViewById(R.id.itemimage); v.setTag(holder); } else { holder = (ViewHolder) v.getTag(); } holder.name.setText(list1.get(position).getName()); holder.price.setText((CharSequence) list1.get(position).getPrice()); holder.img.setImageBitmap(list1.get(position).getImage()); return v; }

使用循环视图时，忘记更新项目数据（v！= null）。

Answer 2

是的，它等待，因为您的图像加载器异步任务遍历所有位图并下载它们然后通知列表视图绘制，您可以使异步任务一次只加载一个图像。但我建议您使用像Picasso或Glide这样的图像加载器。

如果您不使用，也可以在适配器中使用视图设计模式。

Answer 3

即使在创建列表后在后台加载图片并在加载图片时刷新图片，但在用户体验方面更好，这不是您面临的问题。

我认为问题在于您的自定义适配器中的 getView 方法，如果您提供它，我们就能确定。问题是您可能检查某个位置的对象的图像是否有图片，如果是，请设置它。可能是这样的：

public View getView (int position, View convertView, ViewGroup parent)
{
    //some code to create the views and view holder..

    YourItem currentItem = mItems.get(position);
    if (currentItem.getBitmap() != null)
    {

       imageView.setImageBitmap(currentItem.getBitmap());
    }

    //rest of code
}

你应该做的是添加一个else子句，因为上一个创建的视图中的图片仍然填充了ImageView。要么改变ImageView的可见性，要么改变它的位图。

修改
看起来应该是这样的：

public View getView (int position, View convertView, ViewGroup parent) { //some code to create the views and view holder.. YourItem currentItem = mItems.get(position); if (currentItem.getBitmap() != null) { imageView.setVisibility(View.VISIBLE); imageView.setImageBitmap(currentItem.getBitmap()); } else { imageView.setVisibility(View.INVISIBLE); } //rest of code }

listview在customadapter中重复项目

3 个答案: