Rows not updating sqlite3 - ios

Question

The following code is suppose to update 2 columns (HoursWorked and TotalEarned) in the row where Date(Primary Key) = todays date which is stored in the string dbDate. I think my update string might be written wrong. The alert view is being triggered.
sqlite3_stmt *newstatement;
sql =[NSString stringWithFormat:#"UPDATE HourLog SET HoursWorked = '%#', TotalEarned = '%#' WHERE Date ='%#'", HoursWorked, TotalEarned, dbDate];
if(sqlite3_prepare_v2(db, [sql UTF8String], -1, &newstatement, nil)==SQLITE_OK){
NSLog(#"details updated");
UIAlertView *alertDialog;
alertDialog = [[UIAlertView alloc]
initWithTitle:#"Title"
message:#"Details Updated!"
delegate:nil
cancelButtonTitle:#"Close"
otherButtonTitles: nil];
[alertDialog show];
sqlite3_step(newstatement);
}
sqlite3_finalize(newstatement);
sqlite3_close(db);

Answer 1

A couple of issues:
The UPDATE syntax is not correct. The individual fields being updated must be separated by a comma, not the word AND.
You probably shouldn't report that the update succeeded until you confirmed the result of sqlite3_step.
In case you're not getting a hit on your WHERE clause, I might suggest determining how many rows were updated, so you can confirm that the WHERE clause succeeded.
And, as always, report sqlite3_errmsg if your SQLite calls fail or else you're flying blind.
Thus:
sqlite3_stmt *statement;
int originalTotalCount = sqlite3_total_changes(db);
NSString *sql = [NSString stringWithFormat:#"UPDATE HourLog SET HoursWorked = %# , TotalEarned = %# WHERE Date='%#'", HoursWorked, TotalEarned, dbDate];
if (sqlite3_prepare_v2(db, [sql UTF8String], -1, &statement, nil) != SQLITE_OK) {
NSLog(#"%s: prepare failed: %s", __FUNCTION__, sqlite3_errmsg(db));
} else {
if (sqlite3_step(statement) != SQLITE_DONE) {
NSLog(#"%s: step failed: %s", __FUNCTION__, sqlite3_errmsg(db));
} else {
int rowsUpdated = sqlite3_total_changes(db) - originalTotalCount;
NSString *message;
if (rowsUpdated == 1)
message = #"Updated one row";
else if (rowsUpdated == 0)
message = #"No rows updated";
else
message = [NSString stringWithFormat:#"Updated %d rows", rowsUpdated]; // should never happen
NSLog(#"%#", message);
UIAlertView *alertDialog = [[UIAlertView alloc] initWithTitle:nil
message:message
delegate:nil
cancelButtonTitle:#"Close"
otherButtonTitles:nil];
[alertDialog show];
}
sqlite3_finalize(statement);
}
I personally wouldn't use the single quotes around the numeric values for HoursWorked and TotalEarned. And you might consider storing the date in one of the established SQLite date formats.