NullPointerExceptionが出ます。アドバイスをいただけないでしょうか？

Question

java.lang.NullPointerExceptionが SQLiteDatabase db=memoDBHelper.getWritableDatabase();に出るのですが、なぜなのでしょうか？
アドバイスをいただけないでしょうか？
どうぞよろしくお願いします。
MemoDBHelper

package com.example.android.sample.memo;

import android.content.Context;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;

/**
 * Created by endoutaichi on 2017/04/06.
 */

public class MemoDBHelper extends SQLiteOpenHelper {

    public static final String DB_NAME="myapp.db";
    public static final int DB_VERSION=1;
    public static final String CREATE_TABLE=
            "create table "  + UserContract.Users.TABLE_NAME + "(" +
                    UserContract.Users._ID+ " integer primary key autoincrement, " +
                    UserContract.Users.COL_TITLE + " title," +
                    UserContract.Users.COL_CONTENT + " content, +" +
                    UserContract.Users.COL_DATA + "data" +
                    ")";



    /**"create table"+ UserContract.Users.TABLE_NAME+"("+
     UserContract.Users._ID+"integer primary key autoincrement,"+
     UserContract.Users.COL_NAME+"text,"+
     UserContract.Users.COL_SCORE+"integer)";
     **/
    public static final String INIT_TABLE=
            "insert into users (title,content,data)values" +
                    "('仕事','出張','5月3日'),"+
                    "('遊び','海','8月4日'),"+
                    "('買い物','ららぽーと','3月3日')";

    public static final String DROP_TABLE=
            "drop table if exists users";

    public MemoDBHelper(Context c){
        super(c,DB_NAME,null,DB_VERSION);
    }


    @Override
    public void onCreate(SQLiteDatabase sqLiteDatabase) {

        //create table
        sqLiteDatabase.execSQL(CREATE_TABLE);
        //init table
        sqLiteDatabase.execSQL(INIT_TABLE);

    }

    @Override
    public void onUpgrade(SQLiteDatabase sqLiteDatabase, int i, int i1) {

        //drop table
        sqLiteDatabase.execSQL(DROP_TABLE);
        //onCreate
        onCreate(sqLiteDatabase);
    }
}

MainActivity.java

public class MainActivity extends AppCompatActivity {
    // ListView 用アダプタ
    SimpleAdapter mAdapter = null;
    // ListView に設定するデーた
    List<Map<String, String>> mList = null;

    //open helper
    //open db   memoDBHelperに変更
    MemoDBHelper memoDBHelper=new MemoDBHelper(this);
    SQLiteDatabase db=memoDBHelper.getWritableDatabase();


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






        ListView list = (ListView) findViewById(R.id.listView);
        mList = new ArrayList<Map<String, String>>();


        mAdapter = new SimpleAdapter(this,
                mList,
                //独自で定義したlayoutを適応する
                android.R.layout.simple_list_item_2,
                new String[]{"title", "content", "data"}, //Map側のkey
                new int[]{android.R.id.text1, android.R.id.text2, R.id.eTxtData} //layout側のTextViewId 表示させたいMapのkey順に合わせる
        );

        // ListView にアダプターをセット
        list.setAdapter(mAdapter);


        // ListView のアイテム選択イベント
        list.setOnItemClickListener(new AdapterView.OnItemClickListener() {

            @Override
            public void onItemClick(
                    AdapterView<?> parent, View view, int pos, long id) {
                // 編集画面に渡すデータをセットし、表示
               //この下は、ファイルとは関係ないonResume()からやる
                Intent intent = new Intent(MainActivity.this, EditActivity.class);
                intent.putExtra("NAME", mList.get(pos).get("filename"));
                intent.putExtra("TITLE", mList.get(pos).get("title"));
                intent.putExtra("CONTENT", mList.get(pos).get("content"));
                intent.putExtra("DATA", mList.get(pos).get("data"));

                startActivity(intent);
            }
        });

        // ListView をコンテキストメニューに登録
        registerForContextMenu(list);
    }





    @Override
    protected void onResume() {
        super.onResume();


        // ListView 用アダプタのデータをクリア
        mList.clear();

        /**
        for (int i=0; i<files.length; i++) {
            String fileName = files[i].getName();
            if (files[i].isFile() && fileName.endsWith(".txt")) {

         データベースのデータを全取得してそれをリストに入れるのにこのようなfor分は、必要ではないのか？
         多分whileでやってるから大丈夫
       **/

        Integer id=null;
        String title = null;
        String content = null;
        String data=null;

        //ここからコピペ

        //これはselect
        Cursor c=null;
        c=db.query(
                UserContract.Users.TABLE_NAME,
                null,//fields
                null,//where
                null,//where arg ここでは、多分何もせずに、全件取得して、リストに入れる
                null,//groupBy
                null,//having
                null,//order by
                null
        );

        Log.v("DB_TEST","Count:"+c.getCount());
        while(c.moveToNext()){
             id=c.getInt(c.getColumnIndex(UserContract.Users._ID));
             title=c.getString(c.getColumnIndex(UserContract.Users.COL_TITLE));
             content=c.getString(c.getColumnIndex(UserContract.Users.COL_CONTENT));
             data=c.getString(c.getColumnIndex(UserContract.Users.COL_DATA));
            Log.v("DB_TEST"," id: "+id+ "title: "+title+ "content: "+content+" data: "+data);
            //この上のLog不安
            //Logだけじゃなくて、リストに入れる これ以降は、ファイルと共有できると思う
        }

        c.close();
        //close db
        db.close();
        //この上こぴぺ
                //このセットでlistを作っっている?
                //ここここは、つかう。Mapにセットする
                // ListView用のアダプタにデータをセット
                Map<String, String> map = new HashMap<String, String>();
                map.put("title", title);
                map.put("content", content);
                map.put("data",data);
                mList.add(map);

        // ListView のデータ変更を表示に反映
        mAdapter.notifyDataSetChanged();
    }

04-07 10:48:28.281 24469-24469/com.example.android.sample.memo E/AndroidRuntime: FATAL EXCEPTION: main
                                                                                 java.lang.RuntimeException: Unable to instantiate activity ComponentInfo{com.example.android.sample.memo/com.example.android.sample.memo.MainActivity}: java.lang.NullPointerException
                                                                                     at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1994)
                                                                                     at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2095)
                                                                                     at android.app.ActivityThread.access$600(ActivityThread.java:137)
                                                                                     at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1206)

Comments

[masaya_ohashi]

DBに接続がうまくできていないことが原因だと思います。MemoDBHelperというコードの方も見せていただかないと答えられません。

[edoooooo]

承知いたしました。コードをすぐ載せます。

Answer 1

onCreateが実行されるまではActivityはContextを持っていません。なので、クラス初期化時にMemoDBHelperにthisを渡してはいけません。

// 初期化時に作ってはいけない
MemoDBHelper memoDBHelper=new MemoDBHelper(this);
SQLiteDatabase db=memoDBHelper.getWritableDatabase();

// メンバ変数宣言だけする
MemoDBHelper memoDBHelper;
SQLiteDatabase db;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    // この辺にもってくる
    memoDBHelper = new MemoDBHelper(this);
    db = memoDBHelper.getWritableDatabase();

Comments

[edoooooo]

色々なクラスでそれに関するdb変数を使っていたので、全て、onCreate()などの中に入れたところ無事動きました。ありがとうございます

[masaya_ohashi]

解決したならベストアンサーを選んで解決済みにしてください。ずっと未解決問題として残ってしまいます。

[edoooooo]

遅くなりました。申し訳ありません。