Jag tycker att det är värt att ändra - men förmodligen inte värt att göra en markering innan insättning.
Jag uppdaterar bara de fält som har ändrats, det är en del av driften av min DbEntity-klass som följer ett activerecord-mönster. Det kostar lite extra att göra detta eftersom jag har den aktuella posten och originalposterna - helt enkelt kopierar när en post laddas.
Skälen är korthet - inte riktigt prestanda. Du kan också kontrollera om det finns samtidiga ändringar genom att lägga till en where-klausul på det gamla värdet för de uppdaterade fälten och skicka rätt fel.
I skriv/uppdateringsmetoden:
$s1 = "";
foreach ($this->record as $key => $value)
{
// only update fields that have been changed
if ($value != $this->orig_record[$key])
{
$s1 .= $comma."`$key`='".mysql_real_escape_string($value)."'";
$comma = ", ";
}
}
$query = "UPDATE ".$this->table." SET $s1 where {$this->id_field}='".$this->get_keyfield()."'";
$query .= $this->extra_sql_update;
mysql_query($query);
$ar = mysql_affected_rows();
//
// the number of affected rows is actually those changed by the update operation, which will
// either be zero, or 1. If the query affects more than one row then we have a problem.
if ($ar < 0 || $ar > 1)
{
cbf_error("cbf_dbentity: {$this->table} :: only one row (not $ar) must be affected by an insert operation. $query",
E_USER_ERROR);
}
else
{
$new_id = $this->get_keyfield();
GlobalEventBus::notify_all(new AuditLogSQL($this->table, "update", $query));
}
$this->orig_record = Array();
foreach ($this->record as $key => $value)
$this->orig_record[$key] = $value;
//
// sanity check - ensure that what we have just written is actually there.
$this->load($new_id);
foreach ($this->orig_record as $key => $value)
if (trim($this->record[$key]) != trim($value)
&& (!$this->record[$key] == "0" && $value=""))
cbf_error("cbf_dbentity: {$this->table} :: record differs during write after reload: field $key was \"$value\", after write it is now \"".
$this->record[$key]."\"",E_USER_ERROR);
I laddningsmetoden
$this->orig_record = Array();
foreach ($this->record as $key => $value)
$this->orig_record[$key] = $value;