ORM i zapisywanie danych do bazy Opcje

[Apo]

Witam
Zacząłem ostatnio pisać ORM i natrafiłem na problem przy zapisywaniu danych. Narazie rozwiązałem to tak:

[PHP] pobierz, plaintext 
<?php
class ORM {
 
  public function Save(OrmDataRecord $obj)
  {
  $keys = '';
  $data = '';
	if(!array_key_exists($this->unique_field, $obj->fields))
	{
	  foreach($obj->fields as $key => $value)
	  {
		$keys .= !empty($keys) ? ', ' : null;
		$keys .= '`'.$key.'`';
		$data .= !empty($data) ? ', ' : null;
		$data .= '`'.$key.'` = "'.$value.'"';
	  }
	return db::query('INSERT INTO '.$this->table.'('.$keys.') VALUES('.$data.')');
	}
   // tu bedzie UPDATE
  }
}
?>
[PHP] pobierz, plaintext 

$this->unique_field - to string który przechowuje nazwe pola głównego (czyli głównie id)
$obj->fields - to tablica asocjacyjna gdzie klicze są nazwami kolumn a wartość, wartościami tych kluczy które zostana zapisane do bazy.
$obj - to obiekt innej klasy (DataObject).

Wydaje mi sie że trochę --[zmoderowane]-- sprawe z tymi przecinkami, ale chodzi mi głównie na wydajności. No i mam pytanie co by było lepsze czy zrobienie tak jak to zrobiłem w pętli foreach z tymi instrukcjami warunkowymi czy bawienie się tablicami (impode itd) aby prowidłowo wstwić przecinki.

Pozdrawiam

---
Zagwiazdkowane wulgaryzmy to nadal wulgaryzmy. Jak sie wysłowić bez tego nie potrafisz to nie wypowiadaj się publicznie.
Jeszcze raz i bedzie ostrzeżenie.
~mike_mech

[bigZbig]

Ja tam uzywam implode. Tak naprawde implode to tez pentla tyle ze wyspecjalizowana i napisana w C a wiec wydajniejsza. Foreach jest bardziej elastyczne i uniwersalne, ale jesli mozna zastosowac bezposrednio gotowa funkcje php to moim zdaniem nalezy to zrobic.

[PHP] pobierz, plaintext 
<?php
//...
  public static function getQueryInsert($aFieldsAndValues, $sTable){
	$aFields = array_keys($aFieldsAndValues);
	$aValues = self::_addQuotationMarks($aFieldsAndValues);
	$sQuery = 'INSERT INTO '.$sTable.' ('.implode(",", $aFields).')';
	$sQuery .= ' VALUES ('.implode(",", $aValues).')';
	return $sQuery;
  }
//...
?>
[PHP] pobierz, plaintext