無名クラスを原始的なクラスの方法がうまくいかない

Question

こんにちわ、

無名クラスの表記だと素人だとわかりづらいので、原始的なクラスにして理解を深めようとしましたが
ビルドがうまくいきません。アドバイスお願いします。

エラー: 不適合な型: MainActivity.MyListenerをOnItemClickListenerに変換できません:
エラー: MainActivity.MyListenerはabstractでなく、OnClickListener内のabstractメソッドonClick(View)をオーバーライドしません

public class MainActivity extends AppCompatActivity {

    ArrayAdapter<String> adapter;

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

        final ArrayList<String> data = new ArrayList<>();
        data.add("胡椒");
        data.add("ターメリック");
        data.add("コリアンダー");
        data.add("生姜");
        data.add("ニンニク");
        data.add("サフラン");

        adapter = new ArrayAdapter<>(
                this, android.R.layout.simple_list_item_1, data);
        ListView list = (ListView) findViewById(R.id.list);
        list.setAdapter(adapter);
        
        ////////ここ無名クラスを///start////////////////////////////
        list.setOnItemClickListener(
            new AdapterView.OnItemClickListener() {
                public void onItemClick(AdapterView<?> av,
                                        View view, int position, long id) {
                   adapter.remove((String) ((TextView) view).getText());
                   // adapter.add("ナツメグ");

                }
            }
        );
        //////////////////////end////////////////////////////
  }
}

/*原始的なクラスにしたがうまくいかない*/

public class MainActivity extends AppCompatActivity {

    ArrayAdapter<String> adapter;

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

        final ArrayList<String> data = new ArrayList<>();
        data.add("胡椒");
        data.add("ターメリック");
        data.add("コリアンダー");
        data.add("生姜");
        data.add("ニンニク");
        data.add("サフラン");

        adapter = new ArrayAdapter<>(this, android.R.layout.simple_list_item_1, data);
        ListView list = (ListView) findViewById(R.id.list);
        list.setAdapter(adapter);

        list.setOnItemClickListener(new MyListener());

    }
    
    /////原始的なクラスに変換/////////start///////////////////////
    class MyListener implements AdapterView.OnClickListener{

        public void onItemClick(AdapterView<?> av, View view, int position, long id) {

                adapter.remove((String) ((TextView) view).getText());
                // adapter.add("ナツメグ");

        }
    }
    //////////////////////////////end///////////////////////////////

}

Answer 1

implementsするのを間違えていませんか？AdapterView.OnClickListenerではなくAdapterView.OnItemClickListenerではないでしょうか。

---

（BA後の余談）
本件のような「匿名クラス（無名クラス）を内部クラスに変えたい」というときには、匿名クラスの範囲内にカーソルを置いて、メニューバーから "Refactor" → "Convert Anonymous to Inner..." と選べば、内部クラス名を設定するダイアログを経て一発変換してくれる機能があります。これも試してみるといいのではないかと思います（本件のような間違いをせずに済みますし）。

Comments

[eisaku123]

あ　ここ間違ってました。ありがとうございます助かりました。