Events of TextWatcher are being called twice

Question

On my application I put TextWatcher on EditText. When I change the text of the EditText, the events of TextWatcher are being called twice.

I am using emulator for running the app.

see here http://stackoverflow.com/questions/17535415/textwatcher-events-are-being-fired-multiple-times — Hun, Apr 06 '15 at 08:10

score 10 · Accepted Answer · edited Mar 22 '17 at 19:05

10

How does your code looks like? that is the normal behaviour of TextWatcher. Example:

myInput.addTextChangedListener(new TextWatcher() {
        boolean mToggle = false;

        public void onTextChanged(CharSequence cs, int s, int b, int c) {}

        public void afterTextChanged(Editable editable) {
            if (mToggle) { 
                Toast.makeText(getBaseContext(), "HIT KEY",Toast.LENGTH_LONG).show();
            }
            mToggle = !mToggle;
        }

        public void beforeTextChanged(CharSequence cs, int i, int j, int k) {}
    });

edited Mar 22 '17 at 19:05

TheoKanning

1,631
11
20

answered Aug 22 '12 at 14:55

nano_nano

12,351
8
55
83

yes my code is same but 'hit key' is shown twice when i press a key – Jay Gajjar Aug 23 '12 at 06:17
than move the Toast call into afterTextChanged. This method should be called only one time per hit. – nano_nano Aug 23 '12 at 06:46
Yes if the "HIT KEY" comes twice, paste your Toast in the afterTextChanged AS Beike suggested. – raman Aug 23 '12 at 07:14
1

Realy? :-) Can't believe that but I have added a quick&dirty solution in the code above. With that your code is called only one times. – nano_nano Aug 23 '12 at 16:07
okay i got it am sorry i didnt read it properly... thank you sir – Jay Gajjar Aug 25 '12 at 07:32

score 3 · Answer 2 · answered Jan 30 '17 at 22:41

My problem was I added the textWatcher twice mEditText.addTextChangedListener(mTextWatcher), which leads to calling its callbacks twice!

I had added the textWatcher once in onCreate() and once in onStart(). I should only add in onStart and remove in onStop().

score 0 · Answer 3 · edited Jun 03 '20 at 18:31

In case you call editText.setText("string") inside your TextWatcher listener (for example in afterTextChanged method), TextWatcher will detect a new text change, that might end up by calling again setText and create a loop.

An alternative to editText.setText("string") could be:

editText.getText().clear();
editText.append("string");

which will not evoke a new detection by the TextWatcher listener.

Events of TextWatcher are being called twice

3 Answers3

Linked