RecyclerViewの中のEditText、IME_ACTION_NEXTでsetOnEditorActionListener、setOnFocusChangeListener 両方動いてしまう

Question

前提

Android＋kotlinでRecyclerViewの中にEditTextがあるアプリを作っています。
よくある、Excelのような表形式のイメージで、EditTextに値を入力し、DB(Room)を更新するようなアプリです。

DBの更新のきっかけとしてはキーパットのIME_ACTION_NEXTで次の行(EditText)に移る場合、
EditTextからフォカスが外れる場合（タップで他の行に移動する場合）
があります。

どちらも、EditTextに入力した数値をDB(Room)に更新しに行きます。

ここでキーパットのIME_ACTION_NEXTの時に、EditTextのイベントハンドラ、
setOnEditorActionListener
setOnFocusChangeListener
の両方が動いてしまいます。

EditTextからフォカスが外れる場合（タップで他の行に移動する場合）は
setOnFocusChangeListener
しか動きません。

今回はどっちしても、同じ値（EditTextの入力値）でDBを更新しにいくので、キーパットのIME_ACTION_NEXTの時は、同じSQLのUPDATE文を2回実行しているだけなので、まー、多少は無駄ですが、動作には大きな影響はありません。
が、しかし、これが+1カウントアップするような処理だと困ってしまいます。キーパットのIME_ACTION_NEXTの時は2回、カウントアップされてしまいます。

実現したいこと

キーパットのIME_ACTION_NEXTの時に
setOnEditorActionListenerだけ
setOnFocusChangeListeneは呼ばないようにする方法はないのでしょうか。

一般的に、RecyclerViewの中にEditTextがあるExcelの表形式のようなものを実現したいとしたら、どのようにすればよいのでしょうか？
EditTextからフォカスが外れる（タップで他の行に移動できる）というのが、処理をややこしくしているのですが、これをできないようにする方策はなんかあるのでしょうか？

補足情報（FW/ツールのバージョンなど）

AndroidStudio Dolphine 2021.3.1 patch1
Kotlin 1.7.20

Comments

[jimbe]

一応分かるのでちょっとした書き方の問題ですが、
>EditTextのイベントハンドラ、
>setOnEditorActionListener
>setOnFocusChangeListener

set～ はイベントハンドラではありません。イベントハンドラを登録するメソッドです。
イベントハンドラはそのパラメータになっている方ですので、 OnEditorActionListener とか OnFocusChangeListener といった書き方になります。

また、
>キーパットのIME_ACTION_NEXTの時に
>setOnEditorActionListenerだけ
>setOnFocusChangeListeneは呼ばないようにする

イベントは発生した時に呼ばれるのは、イベントハンドラのメソッドですので、そのイベントハンドラのメソッドの方を書かれたほうが良いです。

[退会済みユーザー]

ご指摘の通りですkotlinだと、lambdaで、
XXX.setOnFocusChangeListener { ・・・
}

XXX.setOnEditorActionListener{ 。。。
}

という書き方をするので、そのまま書いてしまいましたが、意図しているのはご指摘の通りです

[jimbe]

問題を再現できる実際に動作するコード(DBの更新部分はログ表示等で構いません)はご提示頂くことは可能でしょうか。

Answer 1

なるほど、View#getTag()、View#setTag(Object)を使って比較しますか・・・

それもそうなんですが、自分で気が付いてしまいました。
ここでキーパットのIME_ACTION_NEXTの時に、EditTextのイベントハンドラ、
OnEditorActionListener
OnFocusChangeListener
の両方が呼ばれる。

EditTextからフォカスが外れる場合（タップで他の行に移動する場合）は
OnFocusChangeListener
が呼ばれる。

両方のハンドラからDB更新しにくからIME_ACTION_NEXTの時に2回更新されるので、どっちの道、OnFocusChangeListenerは必ず呼ばれる。
IME_ACTION_NEXTの時に次の行のEditTextにフォーカスを移すという制御やっているので、その時、OnFocusChangeListenerは呼ばれる。

なのでDB更新はOnFocusChangeListenerだけでやればうまくいくということがわかりました。

Comments

[jimbe]

私の回答で setTag/getTag を用いているのは、 ViewHolder でリスナを登録しているからです。onBindViewHolder で登録するようにすれば item が使えますので setTag/getTag は必要ありません。

Answer 2

EditText の更新を DB に反映するということだけから見れば、更新のトリガが発生したら実際に更新されていようがいまいが更新をかけるのでは無く、 EditText の値が実際に更新されていた場合にのみ DB を更新するようにすれば、そのチェックのトリガは連続して何度あったとしても更新は一回で済むのではないでしょうか。
もし更新回数をカウントアップしているということであれば、これも実際に更新した時だけカウントアップすれば、問題無いように思います。

以下 java ですが、アクション、フォーカスが発生しても、更新されている場合のみ UPDATE が出ると思います。

MainActivity.java

import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.recyclerview.widget.RecyclerView;

import android.os.Bundle;
import android.util.Log;
import android.view.*;
import android.widget.EditText;

import java.util.*;

public class MainActivity extends AppCompatActivity {

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    RecyclerView recyclerView = findViewById(R.id.recyclerView);
    recyclerView.setAdapter(new Adapter());
  }

  private static class Item {
    final long id;
    final String text;
    Item(long id, String text) {
      this.id = id;
      this.text = text;
    }
    @Override
    public String toString() {
      return super.toString() + "[id=" + id + ", text='" + text + "']";
    }
  }

  private static class Adapter extends RecyclerView.Adapter<Adapter.ViewHolder> {
    class ViewHolder extends RecyclerView.ViewHolder {
      final EditText editText;
      public ViewHolder(@NonNull View itemView) {
        super(itemView);
        editText = itemView.findViewById(R.id.editText);
        editText.setOnEditorActionListener((v, actionId, event) -> {
          if(actionId != EditorInfo.IME_ACTION_NEXT) return false;
          Item item = (Item)editText.getTag();
          String text = editText.getText().toString();
          Log.d("Adapter **", "OnEditorActionListener. orig=" + item + ", new text=" + text);
          update(item, text);
          return true;
        });
        editText.setOnFocusChangeListener((v, hasFocus) -> {
          if(hasFocus) return;
          Item item = (Item)editText.getTag();
          String text = editText.getText().toString();
          Log.d("Adapter **", "OnFocusChangeListener. orig=" + item + ", new text=" + text);
          update(item, text);
        });
      }
    }

    private List<Item> itemList = new ArrayList<>();

    Adapter() {
      //テストデータ
      itemList.add(new Item(1, "ABC"));
      itemList.add(new Item(2, "DEF"));
      itemList.add(new Item(3, "GHI"));
      itemList.add(new Item(4, "JKL"));
    }

    private void update(Item item, String text) {
      if(item.text.equals(text)) return;

      Log.d("Adapter **", "UPDATE table (text) VALUES ('" + text + "') WHERE id=" + item.id);

      int index = itemList.indexOf(item);
      itemList.set(index, new Item(item.id, text));
      notifyItemChanged(index);
    }

    @NonNull
    @Override
    public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
      return new ViewHolder(LayoutInflater.from(parent.getContext()).inflate(R.layout.item, parent, false));
    }

    @Override
    public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
      Item item = itemList.get(position);
      holder.editText.setTag(item);
      holder.editText.setText(item.text);
    }

    @Override
    public int getItemCount() {
      return itemList.size();
    }
  }
}

res/layout/activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.recyclerview.widget.RecyclerView
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    tools:context=".MainActivity"
    android:id="@+id/recyclerView"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    app:layoutManager="androidx.recyclerview.widget.LinearLayoutManager" />

res/layout/item.xml

<?xml version="1.0" encoding="utf-8"?>
<EditText
    xmlns:android="http://schemas.android.com/apk/res/android"
    android:id="@+id/editText"
    android:layout_width="match_parent"
    android:layout_height="wrap_content" />