long list show incorrect list items on custom listview with images from android universal image loader

I tried to display a custom list view that display custom list items with image and text lines. I want two type of list items (left align and right alined, just as message thread). I have used Android-Universal-Image-Loader to load web images. bt when i load the list, it load wrong list items time to time. when i scroll fast it changed the alignment of list items time to time.

here u can see the one user (a girl profile picture) reply not in a right aligned all the time

here’s my coding,

    DisplayImageOptions options = new DisplayImageOptions.Builder()
      .showImageForEmptyUri(R.drawable.social_person)
      .showStubImage(R.drawable.ic_launcher)
      .showImageOnFail(R.drawable.ic_launcher)
      .showStubImage(R.drawable.social_person)
      .cacheInMemory(true)
      .cacheOnDisc(true)
      .displayer(new RoundedBitmapDisplayer(20))
      .build();

    @Override
        public View getView(final int position, View convertView, ViewGroup parent) {

        ViewHolder vh = null;       
        threadMessageObj = threadMessagesList.get(position);

        mSenderId = threadMessageObj.getSenderID();

        if(convertView == null){    
            vh = new ViewHolder();

            if(threadMessageObj.getSenderID().equals(mLoggedUserID)){ 
                convertView = inflater.inflate(R.layout.thread_message_row_right, null);
                vh.messageTxt = (TextView) convertView.findViewById(R.id.message_body_r);
                vh.dateTimeView = (TextView) convertView.findViewById(R.id.message_datetime_r);
                vh.imageView = (ImageView) convertView.findViewById(R.id.receipient_image);
            }else{
                convertView = inflater.inflate(R.layout.thread_message_row_left, null);
                vh.messageTxt = (TextView) convertView.findViewById(R.id.message_body);
                vh.dateTimeView = (TextView) convertView.findViewById(R.id.message_datetime);
                vh.imageView = (ImageView) convertView.findViewById(R.id.sender_image);

            }

            convertView.setTag(vh);

        }else{
            vh = (ViewHolder) convertView.getTag();

        }

        String messageText = null;
        String messageTimeText = null;
        String messageDateText;
        String mSenderImage = null;


        mSenderImage = threadMessageObj.getSenderImage();
        imageUri = ConstantValues.API_IP_NEW +  threadMessageObj.getSenderImage(); // from Web

        if( threadMessageObj.getMessageTime() != null){
            messageTimeText = threadMessageObj.getMessageTime().trim();
        }

        if( threadMessageObj.getMessageBody() != null){
            messageText = threadMessageObj.getMessageBody().trim();
        }

        String DateTimeArray[] = messageTimeText.split("\\s+");
        String dateTxt = DateTimeArray[0];
        String timeTxt = DateTimeArray[1];

        String processedTime = vip.getProcessedDate(dateTxt) + " " +  vip.getProcessedTime(timeTxt);

        if(messageText != null){
            vh.messageTxt.setText(messageText);
        }else{
            vh.messageTxt.setText("");
        }
        if(messageTimeText != null){ 
            vh.dateTimeView.setText(processedTime); 
        }

        String[] imageUrls = new String[imageURIList.size()];
        imageUrls = imageURIList.toArray(imageUrls);

        if(mSenderImage != null){
            imageLoader.displayImage(imageUri, vh.imageView, options, animateFirstListener);
        }else{
            imageLoader.displayImage(null, vh.imageView);
        }

        return convertView;

    }// end getView

how to get solved this issue?

[guoyongxin]

You should check your holder here

else{
        vh = (ViewHolder) convertView.getTag();

    }

If the holder holds the view you want, you can reuse it. If not you should instantiate a new one.

Hope this could help 😃

[nostra13]

Eventually you can ask for help on StackOverFlow - http://stackoverflow.com/