Values Of Edittexts In Listview Are Duplicated Randomly

I have listview that has a an Edittext and Button in every row.When i insert the value in the Edittext in first row then i scroll down the list , I find some other EditText in the

Solution 1:

You can have a look at my answer here. Though the problem is not the same but yes this is a similar case you've got here.

So in your case you can solve the problem in many ways. I can suggest you one though. You might keep an external array which will contain all empty strings initially. The array size need to be equal to the size of the RecyclerView. Add a TextWatcher with your EditText inside getView(). Hence the idea is to store the text in the external array when you add some text in an EditText field. Place the text in the specific index of the array to keep track which item in the list is modified.

Now in your getView() method

if(externalArray[pos].equals("")) 
    txtQuantity.setText("");
else 
    txtQuantity.setText(externalArray[pos]);

And yes, declare the EditText outside the onClickListener where you initialized other views of the list item.

So from your updated code, I can suggest you some modification

// Declare an String arrayprivateString[] externalArray = newString[objects.size()];

Now your TextChangedListener should look like this

viewHolder.txtQuantity.addTextChangedListener(newTextWatcher() {
  @OverridepublicvoidonTextChanged(CharSequence s, int start, int before, int count) {
    externalArray[position] = viewHolder.txtQuantity.getText().toString();
  }

  @OverridepublicvoidbeforeTextChanged(CharSequence s, int start, int count, int after) {
    // Do nothing
  }

  @OverridepublicvoidafterTextChanged(Editable s) {
    // Do nothing
  }
});

Now set the txtQuantity like this

//if (externalArray.get(position).equals(""))//  viewHolder.txtQuantity.setText("");//else//  viewHolder.txtQuantity.setText(externalArray.get(position));// OR you can put a single line. I added the else part// to make you understand the scenario actually. 
viewHolder.txtQuantity.setText(externalArray.get(position));

Solution 2:

This line is creating trouble

EditText   txtQuantity=(EditText)((View)v.getTag()).findViewById(R.id.txtQuantity);

You need to take EditText directly from viewholder. Just use:

viewHolder.txtQuantity.getText().toString()

And you need to initialize txtQuantity along with other views of viewholder.

Solution 3:

It's all because your List items are being recycled and if convertView is not null then it's recycling the same by convertView.getTag();.

To prevent this you can just prevent recycling convertView codes by removing

if (convertView == null) {

and matching else clause as well.

But after that you may get scroll lagging issue, So that what I can suggest is try to optimize your list items that means try to reduce the number of list items because It causes the high CPU consumption.