1

「メモ」という名前の列を持つデータベーステーブルがあります。この列のデータは、特定の行が私のお気に入りであるかどうかに応じて、「はい」または「いいえ」のみを取得します (お気に入りに追加)。

この列のデータを次のコードで更新しようとしています:

public void updateFavorite(long id) 
   {
       String txt;
       System.out.println("first");
       Cursor c = database.query("recipes", null, "_id=" + id, null, null, null, null);
       c.moveToFirst();
       System.out.println("second");
       int notesIndex = c.getColumnIndex("notes");
       System.out.println("third");
       String fav=c.getString(notesIndex);
       System.out.println("fourth");

       System.out.println(fav);

       if (fav=="yes") {txt="no";}
       else {txt="yes";}

       ContentValues editEsoda = new ContentValues();

       editEsoda.put("notes", txt);

       open(); // open the database
       database.update("recipes", editEsoda, "_id=" + id, null);
       close(); // close the database
       c.close();
   }

カーソル c を取得しようとすると、エラーが発生します。

Cursor c = database.query("recipes", null, "_id=" + id, null, null, null, null);

この行と列の文字列を読み取りたい。「はい」の場合は、「いいえ」に変更します。それ以外の場合は、「いいえ」の場合は「はい」に変更します。

そこでエラーが発生するのはなぜですか? 前もって感謝します

これは私のLogCatです：

11-26 19:00:25.404: D/dalvikvm(30378): GC_FOR_MALLOC freed 4678 objects / 265896 bytes in 79ms
11-26 19:00:25.444: I/System.out(30378): first
11-26 19:00:25.444: D/AndroidRuntime(30378): Shutting down VM
11-26 19:00:25.444: W/dalvikvm(30378): threadid=1: thread exiting with uncaught exception (group=0x4001d800)
11-26 19:00:25.454: E/AndroidRuntime(30378): FATAL EXCEPTION: main
11-26 19:00:25.454: E/AndroidRuntime(30378): java.lang.IllegalStateException: Could not execute method of the activity
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.view.View$1.onClick(View.java:2072)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.view.View.performClick(View.java:2408)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.view.View$PerformClick.run(View.java:8816)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.os.Handler.handleCallback(Handler.java:587)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.os.Handler.dispatchMessage(Handler.java:92)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.os.Looper.loop(Looper.java:123)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.app.ActivityThread.main(ActivityThread.java:4627)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at java.lang.reflect.Method.invokeNative(Native Method)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at java.lang.reflect.Method.invoke(Method.java:521)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:868)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:626)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at dalvik.system.NativeStart.main(Native Method)
11-26 19:00:25.454: E/AndroidRuntime(30378): Caused by: java.lang.reflect.InvocationTargetException
11-26 19:00:25.454: E/AndroidRuntime(30378):    at development.nk.cretanrecipes.ViewRecipes.favorite(ViewRecipes.java:235)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at java.lang.reflect.Method.invokeNative(Native Method)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at java.lang.reflect.Method.invoke(Method.java:521)
11-26 19:00:25.454: E/AndroidRuntime(30378):    at android.view.View$1.onClick(View.java:2067)
11-26 19:00:25.454: E/AndroidRuntime(30378):    ... 11 more
11-26 19:00:25.454: E/AndroidRuntime(30378): Caused by: java.lang.NullPointerException
11-26 19:00:25.454: E/AndroidRuntime(30378):    at development.nk.cretanrecipes.DatabaseConnector.updateFavorite(DatabaseConnector.java:93)
11-26 19:00:25.454: E/AndroidRuntime(30378):    ... 15 more
11-26 19:00:29.094: I/Process(30378): Sending signal. PID: 30378 SIG: 9
4

2 に答える 2

1
  • まず、fav=="yes"正しくありません。以下を使用する必要があります。"yes".equals(fav)
  • 次に、可能な値が 2 つしかない場合、値をブール値としてデータベースに保存してみませんか? 収納スペースを節約できます。
  • 第三に、どのエラーが発生していますか??!!
于 2012-11-26T16:49:51.163 に答える
1

取得しようとするとエラーが発生しますCursor c
Cursor c = database.query("recipes", null, "_id=" + id, null, null, null, null);

つまり、databaseisを意味します。この行の前に使用して、変数に置き換えるnull必要があります。私はこれが何をしていると推測しています。database = helper.getWritableDatabase()helperSQLiteOpenHelperopen()

他の人が言ったことに加えて、このメソッドを少し速くすることができます:

public void updateFavorite(long id) 
{
    open(); // open the database here!
    ContentValues editEsoda = new ContentValues();

    // Only fetch the columns that you are going to work with
    Cursor c = database.query("recipes", new String[] {"notes"}, "_id=" + id, null, null, null, null);

    // If a row exists with the given id
    if(c.moveToFirst()) {
        // Toggle the value of "notes", but this should be a boolean
        if(c.getString(0).equals("yes")
            editEsoda.put("notes", "no");
        else
            editEsoda.put("notes", "yes");

        c.close();
        database.update("recipes", editEsoda, "_id=" + id, null);
    }
    else {
        // No data has this id
    }
    close(); // close the database
}
于 2012-11-26T17:21:13.357 に答える