OnItemClickListenerの実装は、MainActivityでは、行えないでしょうか?

Question

このようにエラーがでます。
I/System.out: 予期しない例外が発生しました。:java.lang.IllegalStateException: System services not available to Activities before onCreate()

java.lang.IllegalStateException：onCreate（）の前にアクティビティーが利用できないシステム・サービす
というエラーが onItemClickした時に、出るのですが、これは、
LocationActivityのなかで、OnItemClickListenerを実装するコードを書いているからなのでしょうか？

下のOnItemClick()のように、MainActivityとは、別のクラスで、これらを行わないといけないのでしょうか？

字数制限のため、全体の流れを書き込みたく、省略されている行が多いです。

public class LocationActivity extends AppCompatActivity implements
        GoogleApiClient.ConnectionCallbacks,
        GoogleApiClient.OnConnectionFailedListener,
        LocationListener,OnMapReadyCallback,GoogleMap.OnMapLongClickListener,
        ListView.OnItemClickListener,ListView.OnItemLongClickListener {




    final ArrayList<String> listItems = new ArrayList<>();

    static String username = null;

    //---------------------------------------------------------onCreate()------------------------------------------------------------------//
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        favorite();
    }

    //------------------------------------------------------------ログイン--------------------------------------------------------//
    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        int id = item.getItemId();

        if (id == R.id.action_setlogin) {

            // カスタムビューを設定
            LayoutInflater inflater = (LayoutInflater) this.getSystemService(
                    LAYOUT_INFLATER_SERVICE);
            final View layout = inflater.inflate(R.layout.dialog_contact_us, (ViewGroup) findViewById(R.id.layout_root));


            // アラーとダイアログ を生成
            AlertDialog.Builder builder = new AlertDialog.Builder(this);
            builder.setView(layout);

            
            builder.setNegativeButton("ログイン", new DialogInterface.OnClickListener() {
                public void onClick(DialogInterface dialog, int which) {
                    
                    String sql = "select username from user where username = '" + getusername + "' and password = '" + password + "';";

                      Cursor c = db.rawQuery(sql, null);


                    c.moveToFirst();
                    username = checkusername;
                  
                    favorite();

                }
            });
        }

        builder.create().show();


//----------------------------------------------------------onStart()-------------------------------------------------------------------//
        @Override
        protected void onStart() {
            super.onStart();


            listView = (ListView) findViewById(R.id.list_view);

//--------------------------------------------お気に入りの選択---------------------------------------------------------------------------//
            if (item.equals("おきにいり")) {

            String sql = "select latitude,longitude from favorite where placename = '" + item + "' " +
                    "and username = '" + username + "';";

            Cursor c = db.rawQuery(sql, null);

            c.moveToFirst();

        }
    }
        //リスナーの終わり、
        ArrayAdapter<String> adapter = new ArrayAdapter<String>(this, android.R.layout.simple_spinner_item, spinnerItems);

        //ここにcreateFromResourceがないことだけ
        adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
        spinner.setAdapter(adapter);

        // 初回起動時の対応
        spinner.setFocusable(false);


//-----------------------------------------------ここからはコメント送信ボタン、spinner----------------------------------------------------//
        Button buttonTransmission = (Button) findViewById(R.id.buttonTransmission);
        buttonTransmission.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View v){

            sql = "insert into neardb(data,username,comment,latitude,longitude)" +
                    " values ('" + data + "','" + username + "','" + comment + "'," + latitude + "," + longitude + ");";
         db.execSQL(sql);

        }
//----------------------------------------------------------onClick--------------------------------------------------------------------//

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

try{
            String getdesdata = item.substring(0, item.indexOf(":"));

            data = dataand.substring(0, dataand.indexOf(" "));
            comment = usernameand.substring(usernameand.indexOf(" ") + 1);

            LayoutInflater inflater = (LayoutInflater) LocationActivity.this.getSystemService(LAYOUT_INFLATER_SERVICE);
            final View layout = inflater.inflate(R.layout.dialog_edit_comment, (ViewGroup) findViewById(R.id.layout_edit));
            // アラーとダイアログ を生成
            AlertDialog.Builder builder = new AlertDialog.Builder(LocationActivity.this);
            builder.setView(layout);
            //---------dailogの削除ボタンを追加、それが押されたらデータベースから削除する--------//
            builder.setPositiveButton("更新", new DialogInterface.OnClickListener() {
                public void onClick(DialogInterface dialog, int which) {
                }
            });

            builder.create().show();
        }
 } catch (NullPointerException e) {
        System.out.println("NullPointerExceptionが発生しました。");
        } catch (Throwable e) {
        System.out.println("予期しない例外が発生しました。:"+e);
        //I/System.out: 予期しない例外が発生しました。
//:java.lang.IllegalStateException: System services not available to Activities before onCreate() このようにエラーがでました。
        }

//--------------------------------------------------------favorite---------------------------------------------------------------------//
//これをログインした時に usernameを使って、と、newPlaceをinsertしたあとに呼び出す、
public void favorite() {
        MyOpenHelper helper = new MyOpenHelper(LocationActivity.this);
        SQLiteDatabase db = helper.getReadableDatabase();

        if (username != null) {
            String sql = "select placename from favorite where username = '" + username + "';";


            Cursor c = db.rawQuery(sql, null);
          
            boolean mov = c.moveToFirst();
            int mam = 0;
            while (mov) {
                String in = c.getString(0);
                spinnerItems[mam + 2] = in;
                mov = c.moveToNext();
                mam = mam + 1;

            }

            onStart();
            c.close();
            db.close();
        }

//----------------------------------------------------------startFusedLocation()----------------------------------------------------//
//startFusedLocation()

    private void startFusedLocation() {

        if (!mResolvingError) {

            mGoogleApiClient = new GoogleApiClient.Builder(this)
                    .addApi(LocationServices.API)
                    .addConnectionCallbacks(this)
                    .addOnConnectionFailedListener(this) //failedListenerがある
                    .build();

            if (!(mGoogleApiClient.isConnected())){
                mGoogleApiClient.connect();
            }
        }

    }

//-------------------------------------------------------select()---------------------------------------------------------------------//

    public void select(double latitude, double longitude) {

        listItems.clear();

        String sql = "select data,username,comment from neardb ;";


        Cursor c = db.rawQuery(sql, null);

        c.moveToFirst();

        for (int i = 0; i < c.getCount(); i++) {
            //SQL文の結果から、必要な値を取り出す

            in += date + " " + name + "  " + comment + "\n";
            listItems.add(in);
        }

//-------------------------------------------------------ArrayAdapterにlistItemsをset----------------------------------------------------//

        listView = (ListView) findViewById(R.id.list_view);
        ArrayAdapter<String> adapterlist = new ArrayAdapter<String>(LocationActivity.this, android.R.layout.simple_list_item_1, listItems);
        adapterlist.setDropDownViewResource(android.R.layout.simple_list_item_1);
        listView.setAdapter(adapterlist);

//--------------------------------------------setOnItemClickListener(new LocationActivity());------------------------------------------//
        listView.setOnItemClickListener(new LocationActivity());

        //ArrayAdapterをspinnerで使ってた
        c.moveToNext();

    }
}

//このコードのように別途クラスを作成しなくてはいけないのでしょうか？

public class ListView01 extends Activity {
  /** Called when the activity is first created. */
          @Override
  public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    ListView list = (ListView)findViewById(R.id.list_view);
    String[] items = {"Item1","Item2","Item3"};
    ArrayAdapter<String> adapter = new ArrayAdapter<String>(this, R.layout.list, items);
    list.setAdapter(adapter);
    list.setOnItemClickListener(new AdapterView.OnItemClickListener(){
      // 項目がクリックされた時のハンドラ
              @Override
      public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
        // クリックされた時の処理を記述
        String msg = "ItemClick : Item" + (position + 1);
        Log.v("OnItemClick", msg);
      }
    });
  }
}

Comments

[yona]

エラーメッセージは省略しないでください。

[edoooooo]

はい、修正します。申し訳ありません

[yona]

省略しないとは全ての行を記載することですよ。1行なわけありません。少なくとも10数行はあるはずです。

[edoooooo]

エラーメッセージではなく、 } catch (Throwable e) { System.out.println("予期しない例外が発生しました。:"+e);で取得したメッセージのことでした。間違えました。すみません

[yona]

何でもかんでもcatchするとエラーの原因がわからなくなります。やめましょう。

[edoooooo]

catchしないことで、長文のエラーが発生するのですね。そのことを考えたことがありませんでした。ありがとうございます。

Answer 1

listView.setOnItemClickListener(new LocationActivity());

Activityを自分でnewしてはいけません。正しくはこうです。

listView.setOnItemClickListener(this);

Comments

[edoooooo]

ありがとうございます。
修正して実行してみます。

[edoooooo]

AlartDialogを表示することができました。
swordoneさんに、教えていただいた、一箇所を修正しただけで、解決しました。
ありがとうございました。