Android SQLite CursorIndexOutOfBounds

Question

何らかの理由で、入力したユーザー名が見つからない場合、アプリケーションがクラッシュします。しかし、ユーザー名が見つかったら、完璧に動作するようです。私は返されたカーソル== nullかどうかを調べるためのチェックをします。相続人は、コード

public boolean isUserAuthenticated(String username, String password) { 
    // TODO Auto-generated method stub 
    boolean authenticated = false; 
    String[] columns = new String[] {LOGIN_USERNAME, LOGIN_PASSWORD}; 
    String sqlUsername = "\"" + username + "\""; 
    Cursor c = ourDatabase.query(LOGIN_TABLE, columns, LOGIN_USERNAME + "="+ sqlUsername, null, null, null, null); 
    if (c != null) { 
     c.moveToFirst(); 
     String passwordAttachedToUsername = c.getString(1); 
     if(passwordAttachedToUsername.equals(password)) { 
      authenticated = true; 
     } 
    } 

    return authenticated; 
} 

Answer 1

カーソルオブジェクトがnullではないかもしれないが、その結果セットのサイズは、の代わりに0です：

if (c != null) { 
    ... 
} 

試してみてください。

if (c.getCount() > 0) { 
    ... 
} 

また、など@muがあまりにも短いと言えば、あなたの条件にc.moveToFirst（）の戻り値を使うことができます：

if (c.moveToFirst()) { 
    String passwordAttachedToUsername = c.getString(1); 
    if (passwordAttachedToUsername.equals(password)) { 
     authenticated = true; 
    } 
} 

Answer 2

queryコマンドは常にカーソルを返しますので、nullのテストは常に失敗します。あなたはカーソルがcursor.getCount()

Answer 3

ファーストを使用して含まれているカウントを確認する必要があり、条件は次のようになります。

if (c != null && c.getCount() > 0) 

第二に、あなたはこの代わりに

String passwordAttachedToUsername = c.getString(1); 
if(passwordAttachedToUsername.equals(password)) { 
    authenticated = true; 
    } 

をリファクタリングすることができます

authenticated = password.equals(c.getString(1)); 

Answer 4

変更：

012 http：/

if (c != null) { 
    c.moveToFirst(); 
    ... 
} 

c != nullは、カーソルの大きさが0より大きい

Answer 5

if (cursor != null && cursor.getCount() > 0) { 
if (c.moveToFirst()) { 
      String passwordAttachedToUsername = c.getString(1); 
      if(passwordAttachedToUsername.equals(password)) { 
      authenticated = true; 
    }} 
        // your logic goes here 
       } else { 
        Toast.makeText(getApplicationContext(), "No Record Found", 1000).show(); 
       }