Данные вставляются дважды в ключевое событие пресса edittext в Android

Question

Я вызываю метод помощника db, когда пользователь нажимает кнопку done/next/enter, но этот код выполняется дважды, а данные дважды вводятся в таблицу.

Вот мой ключ пресс событие редактирования текста:

editTextLitre.setOnKeyListener(new OnKeyListener() { 

       @Override 
       public boolean onKey(View v, int keyCode, KeyEvent event) { 
        // TODO Auto-generated method stub 
        switch(keyCode){ 
        case KeyEvent.KEYCODE_ENTER: 
         TextView txtlitres = (TextView)findViewById(R.id.edit_text_litre); 
         TextView totalamount = (TextView)findViewById(R.id.amount); 

         RadioButton rbBuffalo = (RadioButton) findViewById(R.id.radioBuffalo); 

         RadioButton rbCow = (RadioButton) findViewById(R.id.radioCow); 
         rbBuffalo.setText(Constants.Buffalo.toString()); 
         rbCow.setText(Constants.Cow.toString()); 
         radioanimal= (RadioGroup)findViewById(R.id.radioCategory); 
         int selectedid = radioanimal.getCheckedRadioButtonId(); 

         rb = (RadioButton)findViewById(selectedid); 


         DBHelper dh=new DBHelper(getApplicationContext()); 
         // TODO Auto-generated method stub 
         boolean save = dh.insertMilkDetails(mandali,rb.getText().toString(),totalamount.getText().toString(),txtlitres.getText().toString(),dt.toString(),session.toString()); 

         if(save == true) 
         { 
          Toast.makeText(getApplicationContext(), "Details Saved Successfully", Toast.LENGTH_SHORT).show(); 

         } 

        return false; 
       } 
      }); 

Я не могу понять, почему данные вставляются дважды в таблице базы данных.

Answer 1

Изменить код этого:

editTextLitre.setOnKeyListener(new OnKeyListener() { 

     @Override 
     public boolean onKey(View v, int keyCode, KeyEvent event) { 
      // TODO Auto-generated method stub 
      if (event.getAction() == KeyEvent.ACTION_DOWN) { 
       switch (keyCode) { 
        case KeyEvent.KEYCODE_ENTER: 
         TextView txtlitres = (TextView) findViewById(R.id.edit_text_litre); 
         TextView totalamount = (TextView) findViewById(R.id.amount); 

         RadioButton rbBuffalo = (RadioButton) findViewById(R.id.radioBuffalo); 

         RadioButton rbCow = (RadioButton) findViewById(R.id.radioCow); 
         rbBuffalo.setText(Constants.Buffalo.toString()); 
         rbCow.setText(Constants.Cow.toString()); 
         radioanimal = (RadioGroup) findViewById(R.id.radioCategory); 
         int selectedid = radioanimal.getCheckedRadioButtonId(); 

         rb = (RadioButton) findViewById(selectedid); 


         DBHelper dh = new DBHelper(getApplicationContext()); 
         // TODO Auto-generated method stub 
         boolean save = dh.insertMilkDetails(mandali, rb.getText().toString(), totalamount.getText().toString(), txtlitres.getText().toString(), dt.toString(), session.toString()); 

         if (save == true) { 
          Toast.makeText(getApplicationContext(), "Details Saved Successfully", Toast.LENGTH_SHORT).show(); 

         } 

         return false; 
       } 
      } 
     }); 

Что изменилось: if (event.getAction() == KeyEvent.ACTION_DOWN) { ...

Поскольку onKey событие происходит дважды при нажатии клавиши: После того, как при нажатии кнопки, один раз, когда вы отпустите кнопку.

Answer 2

Это довольно просто, ключевые события генерируются для каждого действия во время нажатия клавиши (например, клавиши вниз и клавиши вверх). Ваш код проверяет тип кода ключа, но не подтверждает, было ли событие ACTION_DOWN или ACTION_UP.

Пройденный объект KeyEvent предоставил вам эту информацию (see docs); вы, скорее всего, захотите сделать вкладку вверх.

Другой, возможно, лучший вариант, поскольку вы используете EditText - это OnEditorActionListener (docs), который не требует от вас проверки таких вещей.