How to update the same EditText using TextWatcher?

JibW picture JibW · Feb 29, 2012 · Viewed 21k times · Source

In my Android application I need to implement a TextWatcher interface to implement onTextChanged. The problem I have is, I want to update the same EditText With some extra string. When I try to do this the program terminates.

 final EditText ET = (EditText) findViewById(R.id.editText1);
 ET.addTextChangedListener(new TextWatcher() {

        @Override
        public void onTextChanged(CharSequence s, int start, int before, int count)
        {
            try
            {
                 ET.setText("***"+ s.toString());
                 ET.setSelection(s.length());
            }
            catch(Exception e)
            {
                Log.v("State", e.getMessage());
            }
        }

        @Override
        public void beforeTextChanged(CharSequence s, int start, int count, int after)
        {

        }

        @Override
        public void afterTextChanged(Editable s)
        {               
        }
    });

My program terminates and even I try to catch the exception like in my code still it terminates. Does anyone have any idea why this happens and how I can achieve this? Thanks.

Answer

Orkun Ozen picture Orkun Ozen · Feb 29, 2012

The content of the TextView is uneditable on the onTextChanged event.

Instead, you need to handle the afterTextChanged event to be able to make changes to the text.

For more thorough explanation see: Android TextWatcher.afterTextChanged vs TextWatcher.onTextChanged


Note: Error onTextChanged

Obvioulsy, you are causing an endless loop by continuously changing the text on afterTextChanged event.

From the ref:

public abstract void afterTextChanged (Editable s)
This method is called to notify you that, somewhere within s, the text has been changed. It is legitimate to make further changes to s from this callback, but be careful not to get yourself into an infinite loop, because any changes you make will cause this method to be called again recursively. ...

  • Suggestion 1: if you can, check if the s is already what you want when the event is triggered.

    @Override
    public void afterTextChanged(Editable s)
    {    
        if( !s.equalsIngoreCase("smth defined previously"))
             s = "smth defined previously";              
    }
    
  • Suggestion 2: if you need to do more complex stuff (formatting, validation) you can maybe use a synchronized method like in this post.

Note 2 : Formatting the input as partially hidden with n stars till the last 4 chars ( ****four)

You can use something like this in suggestion 1:

    @Override
    public void afterTextChanged(Editable s)
    {    
       String sText = ET.getText().toString()

        if( !isFormatted(sText))
             s = format(sText);              
    }
    bool isFormatted(String s)
    {
     //check if s is already formatted
    }

    string format(String s)
    {
      //format s & return
    }