3

这段代码有什么问题吗,我想使用条形码查询数据,它告诉我光标索引超出范围异常。

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

3 回答 3

11

中没有结果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;
}
于 2013-07-13T18:41:43.250 回答
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;
}
于 2013-07-13T18:44:38.010 回答
0

尝试这个

String id;
if (c!= null && c.moveToNext){
   String id = c.getString(0);
   return id;   
 }
于 2013-07-13T18:41:30.107 回答