Question

所有时间都没有回复否。

-(BOOL)updateProductTable:(NSString *)productid column_shop_Product_quantity:(NSString *) productquantity{
    NSLog(@"%@",productid);
    NSLog(@"%@",productquantity);

    const char *dbpath = [databasepath UTF8String];
    if (sqlite3_open(dbpath, &database) == SQLITE_OK)
    {
        NSString *updateSQL = [NSString stringWithFormat:@"UPDATE ShopProduct set column_shop_Product_quantity=%@ WHERE column_shop_Product_id=%@",productquantity,productid];
        NSLog(@"%@",updateSQL);

        const char *update_stmt = [updateSQL UTF8String];
        if (sqlite3_prepare_v2(database, update_stmt, -1, &statement, nil) == SQLITE_OK)
        {
            if (sqlite3_step(statement) == SQLITE_DONE)
            {
                sqlite3_reset(statement);

                return YES;
            }   
            else {
                return NO;
            }

            sqlite3_finalize(statement);
        }
    }

    sqlite3_close(database);
    return NO;
}

Answer 1

您的代码存在以下问题：

永远不要使用stringWithFormat:将值绑定到查询。
在大多数情况下，您不会关闭数据库。
在大多数情况下，您没有最终确定准备好的陈述。
您应该添加更多错误记录以确定任何问题的原因。
为什么产品数量是以字符串而不是数字形式传递的？
您应该使用sqlite3_open_v2。
对数据库和预准备语句使用局部变量。

以下是针对所有这些问题更新的代码：

-(BOOL)updateProductTable:(NSString *)productid column_shop_Product_quantity:(NSString *)productquantity {
    NSLog(@"%@",productid);
    NSLog(@"%@",productquantity);

    BOOL res = NO;

    sqlite3 *database = NULL;
    const char *dbpath = [databasepath UTF8String];
    if (sqlite3_open_v2(dbpath, &database, SQLITE_OPEN_READWRITE, NULL) == SQLITE_OK)
    {
        const char *updateSQL = "UPDATE ShopProduct set column_shop_Product_quantity=? WHERE column_shop_Product_id=?";
        sqlite3_stmt *statement = NULL;
        if (sqlite3_prepare_v2(database, updateSQL, -1, &statement, NULL) == SQLITE_OK)
        {
            sqlite3_bind_int(statement, 0, [productquantity intValue]);
            sqlite3_bind_text(statement, 1, [productid UTF8String], -1, SQLITE_TRANSIENT);
            if (sqlite3_step(statement) == SQLITE_DONE) {
                res = YES;
            } else {
                NSLog(@"Unable to update data: %s", sqlite3_errmsg(database));
            }

            sqlite3_finalize(statement);
        } else {
            NSLog(@"Unable to prepare statement: %s", sqlite3_errmsg(database));
        }

        sqlite3_close(database);
    } else {
        NSLog(@"Unable to open database at %@: %s", databasepath, sqlite3_errmsg(database));
    }

    return res;
}

sqlite3_step（statement）== SQLITE_DONE语句不返回yes

1 个答案: