Android游标索引超出绑定异常

时间:2013-07-13 18:35:20

标签: android sqlite cursor

这段代码有什么问题,我想用条形码查询数据,它会向我显示光标索引超出绑定异常。

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;
}

3 个答案:

答案 0 :(得分:9)

Cursor没有结果。您应该检查moveToFirst()返回的内容(最有可能false)。您还应该使用moveToNext(),而不是moveToFirst()。另请注意,您没有检查ss参数。这可能会导致SQL注入漏洞。你应该使用参数。另外我认为你可以在你的方法中使用一个回报。

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;
}

答案 1 :(得分:1)

moveToFirst()方法的文献:

  

public abstract boolean moveToFirst()将光标移动到第一行。如果光标为空,则此方法将返回false。

     

返回移动是否成功。

所以你的moveToFirst调用失败了(因为cusrsor有0个元素),这就是崩溃的原因。

这样做:

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

答案 2 :(得分:0)

试试这个

String id;
if (c!= null && c.moveToNext){
   String id = c.getString(0);
   return id;   
 }