Android Cursor Index out of Bound Exception

gibt es eine Sache falsch mit diesem code, ich möchte zur Abfrage der Daten durch die Verwendung von barcode und es mir zeigen, dass Cursor Index out of Bound exception .

public String getIdByBarcode(String ss) throws SQLException{
    String[] column = new String[]{Pro_ID,Pro_Barcode, Pro_Name,Pro_NameKhmer, Pro_Quantity, Pro_Price, Pro_Description, Pro_Date};
    Cursor c = ourDatabase.query(TABLE_NAME, column, Pro_Barcode + "= '" + ss + "' " , null, null, null, null);

    if(c != null){
        c.moveToFirst();
        String id = c.getString(0);
        Log.v(id, id + "Id" );
        return id;
    }
    return null;
}

InformationsquelleAutor user2149618 | 2013-07-13

  1. 9

    Keine Ergebnisse in der Cursor. Sie sollten überprüfen, was moveToFirst() zurück (wahrscheinlich false). Auch sollten Sie verwenden moveToNext(), nicht moveToFirst(). Auch aufpassen, dass man nicht überprüfen ss parameter. Dies könnte dazu führen, SQL-injection-Schwachstellen. Sie sollten die Verwendung von Parametern. Auch ich denke, Sie können mit einem einzigen return in der Methode.

    public String getIdByBarcode(String ss) throws SQLException {
    String[] column = new String[]{Pro_ID,Pro_Barcode, Pro_Name,Pro_NameKhmer, Pro_Quantity, Pro_Price, Pro_Description, Pro_Date};
    final String args = new String[1];
    args[0] = ss;
    Cursor c = ourDatabase.query(TABLE_NAME, column, Pro_Barcode + " = ?" , args, null, null, null);
    String ret = null;
    if(c.moveToNext()) {
        ret = c.getString(0);     
    }
    return ret;
}
    Vielen Dank für den Hinweis auf m0skit0 🙂

    InformationsquelleAutor m0skit0

  2. 1

    Literatur moveToFirst () - Methode:

    public abstract boolean moveToFirst () Bewegen Sie den cursor auf die erste Zeile. Diese Methode gibt false zurück, wenn der cursor leer ist.

    Gibt, ob die Maßnahme erfolgreich war.

    Damit Ihre moveToFirst Aufruf fehlschlägt(weil cusrsor hat 0 Elemente) und das ist der Grund für den Absturz.

    Hierzu:

    if(c != null && c.moveToFirst()) {
    String id = c.getString(0);
    Log.v(id, id + "Id" );
    return id;
}

    InformationsquelleAutor Rajeev

  3. 0

    versuchen, diese 

    String id;
if (c!= null && c.moveToNext){
   String id = c.getString(0);
   return id;   
 }
    Über eine Weile hier, nur macht das keinen Sinn. id wird immer, immer das Letzte element. Nicht zu vergessen id ist nicht zugänglich von außerhalb der Schleife...

    InformationsquelleAutor Srikanth Roopa

Schreibe einen Kommentar