tak já se obávám, že __get se bude volat po každé, protože data budou asi uložena jinde než v $this->name, většina ORM, alespoň napsaných v PHP je má uloženy v nějakém poli, tedy typicky $this->_data[‚name‘], hlavně proto, aby poznali při přístupu k hodnotě, že se jedná o hodnotu uloženou v mysql

„Jinak jako nástin dobrý, ale doufám, že v této podobě to nikdo nebude používat.“

co je na podobě špatné?

no asi rozumím jak to myslíš jen mi něco uniká, že nevidíš ten problém (třeba tam není), ale já když provádím save() tak přece vždycky budu k těm hodnotám přistupovat!!! a to je to co nechci! já chci změnit cenu výrobku, aniž bych načítal ten řádek z databáze!

Chci udělat tohle

1. $c=Customer::get­ByPrimary(1);
2. $c->name=„František Omáčka“;
3. $c->save();

aniž bych provedl jakýkoli SELECT do databáze

to si opět nerozumíme,

1. právě proto, aby si orm pamatovalo co se změnilo, je nutné využívat magické metody a proto nemůže být nikdy nastavena vlastnost/hodnota právě v takto $this->name, tedy public $name;, ale právě v nějakém jiném poli _data[$name] to už sem psal

2. problém je právě v defaultních hodnotách, následující postup

2a. new Customer – v této chvíli mi to nastaví defaultní hodnoty, tedy $this->payerOfVat=0;

tedy ORM si myslí, že payerOfVat byl nastaven, což není úplně pravda

protože ve chvíli, kdy provedu update

1. $c=Customer::get­ByPrimary(1);
2. $c->password=„Franta“;
3. $c->save();

mi to změní i hodnotu payerOfVat což jsem vůbec změnit nechtěl, já chtěl změnit jen jeho heslo!

samozřejmě je možnost, že metoda getByPrimary, by defaultní hodnoty nenastavovala, ale opět nevím jestli je to OK

proč by to nešlo, já právě nechci pokaždé načítat záznam, představte si že děláte tohle

for($i=0;$i<=­40000;$i++) {
$p=Product::get­ByPrimary($i);
$p->salesPrice= „slozity vypocet zalozeny na nekolika funkcich a okolnostech, ktere nedovoluji volat hromadne zpracovani, nebo je hromadne zpracovani z mnoha duvodu problematicke, nebo dokonce chci vratit pouze dotaz, ktery volam pozdeji hromadne“;

$p->save();

alternativně
$update[]=$p->getUpdateSql(); }

alternativně $sql->multi_query(im­plode(„;“, $update));

protože multi_query je tisíckrát rychlejší a z několika minut zpracování dělá několik sekund! ale to už je jiný článek a opravdu je to tak, věřte tomu prosím, je to tak!

citace z článku: „Vytvářet novou instanci new Customer by mohlo být matoucí, co takhle tedy, jak navrhuje Jakub (getByPrimary), to ale také trochu zavání tím, že se to načítá z databáze (programátor by to tak mohl chápat?) a mohlo by to být matoucí.“

řekněme tedy jiná metoda než konstruktor, tedy new Customer, máš nápad??

k věci, ať z toho zase není bezvýznamný flame, neodpověděl si na otázku

zkusím na to udělat příklad snad to bude jasnější

přidal jsem ukázku na git, článek byl updatován a jsou tam odkazy na zdrojáky

co se týče defaultních hodnot budou nestěžejnější metody

<?php
public function & __get($name) {
        return $this->getValue($name);
}
public function __set($name, $value) {
        $this->setValue($name, $value);
}
public function __isset($name) {
        if (isset($this->data[$name])) return true;
        if (isset($this->$name)) return true;
        return false;
}

public function __unset($name) {
        if (isset($this->data[$name])) {
                unset($this->data[$name]);
                unset($this->modified[$name]);
        }
        else unset($this->$name);
}
public function setValue($name, $value) {
        $columns=self::config()->getColumns();
        if (isset($columns[$name])) { // vlastnost se uklada do db
                if (!is_null($value)) { // typova kontrola
                        $type=$s[$string]['type'];
                        if ($type=="%i") $value=(int)$value;
                        elseif ($type=="%f") $value=(float)$value;
                        elseif ($type=="%s" || $type=="%t") $mixed=(string)$value;
                        else throw new DbUndefinedDataTypeException();
                }
                        $this->data[$name]=$value;
                        $this->modified[$name]=1; // byla hodnota zmenena? phpactiverecord to oznacuje jako dirty
                } else { // nejedná se o vlastnost ukladanou do db
                        $this->$name=$value;
                }
        }

public function & getValue($name) {
        if (array_key_exists($name, $this->data)) {
                return $this->data[$name];
                } elseif (array_key_exists($name, $this->defaults)) {
                return $this->defaults[$name];
                } else {
                return $this->$name;
        }
}

?>

1. se domnívám, že chceme zajistit při každém přístupu k vlastnosti objektu volání metody __get a __set
2. metoda __set vždy provádí typovou kontrolu, je to vhodné?
3. je na zvážení jak by se měli chovat hodnoty isset a unset, předpokládám, že defaultní hodnota neznamená že isset bude vracet true,
4. unset předpokládám, že zruší i informaci o tom, že záznam byl někdy modifikován?

nějaký další názor na kód?