Plánuje se Exploreru přidat podporu datových typů, jako to mají formuláře a latte?

Samozřejmě je tím myšlena i podpora poslat naplněný objekt do insertu/updatu, aby se nemuselo pracovat s obecnými poli, které je nutné explicitně konvertovat na objekty, nebo ověřovat integritu dat.

Editoval Bulldog (11. 11. 2021 22:02)

Felix napsal(a):

Udelal jsem si misto active row vlastni “active row per tabulka”.

Tak se to právě aktuálně snažím řešit taky. ale přijde mi to takové krkolomné a radši bych to měl jako integrovanou podporu.

Editoval Polki (11. 11. 2021 23:44)

Felix napsal(a):

Udelal jsem si misto active row vlastni “active row per tabulka”.

Nebyla by prosím malá ukázka jak na to? Vůbec si to nedovedu představit..
Já si primitivně posílám ActiveRow do konstruktoru objektu, kde foreachem projdu data a nasypu to do public properties, takže když potřebuju něco upravit při výpisu, tak to dělám v getteru..

No já to dělám tak, že si vytvořím třídu konkrétní entity, například User, která v konstruktoru bere ActiveRow, které může být nullable a má privátní proměnnou typu array s názvem třeba $data

Pak tato entita zprostředkovává všechny properties, co potřebuju.
Setter uloží novou hodnotu do $data pod klíčem z db tabulky a getter se podívá, jestli tato hodnota existuje v poli $data a pokud ano, tak ji vrátí. Pokud ne, tak se podívá jestli je nastavený ActiveRow a pokud ano, tak vrátí hodnotu z něj a pokud ne, tak vrátí null, nebo vyhodí error neinicializovaná proměnná.

Tím je to editovatelná entita, která umí pracovat s ActiveRow.

No a dále moje entita User se chová jako takový osekaný PROXY nad ActiveRow, takže má třeba metody ref, related, update, delete, které jen forwarduje na ten vnitřní ActiveRow.
Například metoda update udělá to, že zavolá vnitřně: $this->activeRow->update($this->data) a následně: $this->data = []
Samozřejmě tam probíhá nějaká kontrola, jestli je ActiveRow nastaven, případně když je pole $data prázdné, tak to dotaz vůbec nedělá atp., ale princip fungování je podobný.

Akorát mi to přijde takové složité, takže bych uvítal něco podobného přímo v Exploreru, nebo jako doplněk, jak psal @Felix

Editoval Bulldog (12. 11. 2021 12:18)

Ja to udelal takto. Rad bych to probral s @DavidGrudl a nebo pripadne hodil do contributte/database.

1. Mrknul jsem, kde se tvori ActiveRow (new ActiveRow). A to je trida Selection (https://github.com/…election.php#L556). Metoda se jmenuje createRow a je protected. Lze pretizit.
2. Kouknul jsem se, kde se tvori trida Selection. Jedna se o Explorer (https://github.com/…Explorer.php#…). Tak jsem si ho podedil. Tedka je moznost predelat metodu table a nebo si udelat svoji vlastni tableXyz().
3. Kdyz mame vlastni tridy, ktere umozni zmenu ActiveRow na vlastni tridu, tak lze poresit jeste automatickou detekci podle tabulky, napriklad.

   // Misto ActiveRow se pouzije nas objekt
   $context->setRowClass(CustomActiveRow::class);
   
   // Misto ActiveRow se pouzije nas objekt podle tabulky
   $context->setRowMapping([
       'user' => UserRow::class,
       'address' => AddressRow::class,
   ]);
   

4. Posledni skladacka je prepsani defaultniho Exploreru (ve starsich verzich se jmenuje Context). To se najde tak, ze je potreba dohledat registraci tridy Explorer v DatabaseExtension (https://github.com/…xtension.php#L122). Extension se stara o to, aby se do Nette DI containeru zaregistrovali vsechny potrebne sluzby a clovek si je nemusel v souboru config.neon registrovat sam.

   # 1. Registrace nove tridy. Ktera se pak pouzije jako zavislost ve vasich tridach.
   services:
       - App\CustomExplorer(@database.default.connection, @database.default.structure, @database.default.conventions)
   
   
   # 2. Prepis originalnich exploreru z nette/database. V pripade vice connection zaregistrovat prepsat vice exploreru/contextu.
   services:
       database.default.context: App\CustomExplorer(@database.default.connection, @database.default.structure, @database.default.conventions)
   

Trida CustomContext. Resi, jestli se pouzije vlastni trida pro ActiveRow a nebo se pouzije trida podle nazvu tabulky.

use Nette\Caching\IStorage;
use Nette\Database\Connection;
use Nette\Database\Context;
use Nette\Database\Conventions\StaticConventions;
use Nette\Database\IConventions;
use Nette\Database\IStructure;

final class CustomContext extends Context
{

    /** @var string */
    private $rowClass;

    /** @var string[] */
    private $rowMapping = [];

    public function setRowClass(string $class): void
    {
        $this->rowClass = $class;
    }

    /**
     * @param string[] $mapping
     */
    public function setRowMapping(array $mapping): void
    {
        $this->rowMapping = $mapping;
    }

    /**
     * @param string $table
     */
    public function table($table): CustomSelection
    {
        $selection = new CustomSelection($this, $this->conventions, $table, $this->cacheStorage);

        if ($this->rowClass) {
            $selection->setRowClass($this->rowClass);
        }

        if ($this->rowMapping) {
            $selection->setRowMapping($this->rowMapping);
        }

        return $selection;
    }

}

Trida CustomSelection. Resi jaka trida se pouzije misto ActiveRow.

use Nette\Database\Table\ActiveRow;
use Nette\Database\Table\Selection;

final class CustomSelection extends Selection
{

    /** @var string */
    private $rowClass;

    /** @var string[] */
    private $rowMapping = [];

    public function setRowClass(string $rowClass): void
    {
        $this->rowClass = $rowClass;
    }

    /**
     * @param string[] $mapping
     */
    public function setRowMapping(array $mapping): void
    {
        $this->rowMapping = $mapping;
    }

    protected function createRow(array $row)
    {
        $className = $this->resolveClass();
        return new $className($row, $this);
    }

    private function resolveClass(): string
    {
        if ($this->rowClass) {
            return $this->rowClass;
        }

        if (isset($this->rowMapping[$this->getName()])) {
            return $this->rowMapping[$this->getName()];
        }

        return ActiveRow::class;
    }

}

Ukazkovy ActiveRow per tabulka.

/**
 * @property int $id
 * @property string $name
 * @property string $surname
 */
class User extends ActiveRow
{
}

@Felix
Tak jsem to zkoušel taky, ale zasekl jsem se na tom, že nevím, jak zaregistrovat můj poděděný Explorer do DI, aby ho bralo klasické nastavení připojení k databázi.
Můžu to udělat normální registrací jako službu, ale rád bych používal i nadále výchozí nastavení připojení, což není moc kompatibilní ne?

Dobrá práce hoši.
Možná by to chtělo ještě nějak vymyslet, aby ty třídy byly volitelně editovatelné co?

Když budu mít třídu Article, kterou budu cpát do formuláře, tak nechci mít druhou třídu Article, kterou budu cpát do databáze a mezi nima muset ručně přelévat data.

Představoval bych si to nějak takto:

public function articleFormSucceeded(Article $data): void
{
	$this->articleRepository->save($data);
}

ArticleRepo:

public function save(Article $data): Article
{
	if ($data->id) {
		$data->update();
		// Nebo
		$this->db->table('article')->get($data->id)->update($data);
		// Nebo
		$this->db->table('article')->where('id', $data->id)->update($data);
		// Nebooooooooooo jelikož $data obsahují už ID, tak
		$this->db->table('article')->update($data);
	} else {
		$data = $this->db->table('article')->insert($data);
	}
	return $data;
}

Samozřejmě třída Article by se mohla vnitřně starat o validaci dat
Jo a taky místo $this->db->table('article') by byla nějaká metoda, která by nastavovala typ té třídy, případně by se dalo možná zapsat něco jako $this->db->update($data) a $this->db->insert($data), přičemž to, do jaké tabulky se to má uložit by se četlo podle typu třídy, co se tam pošle, jelikož třída je svým způsobem identifikátor tabulky?

Editoval Polki (12. 11. 2021 13:29)

Bulldog napsal(a):
Takže je to zapouzdřené i napovídá.

Ano, tohle už ano. Já jsem vycházel z popisu „která v konstruktoru bere ActiveRow, které může být nullable a má privátní proměnnou typu array s názvem třeba $data“, protože zaslaná ukázka tomuto popisu neodpovídá.

Humorné je, Polki, jak Tvé ego dosud nerozdýchalo, jak Ti nikdo nedal za pravdu ve vláknu o db Exploreru a CRUD operacích a od té doby mínusuješ cokoliv, co napíšu :) Ale pokud to Tvé psychice pomůže, rád tuto úlohu na sobě ponechám.

Polki napsal(a):

Já si myslím, že mu popis odpovídá docela dobře.

Já myslím, že ne, a tak jsem se raději zeptal. Rozdíl mezi námi je, že já si za druhé nedomýšlím a raději se konkrétně ptám, aby si někdo nezvolil cestu antipatternu.

Z toho je podle mě jednoznačně jasný, že ta entita má v sobě properties, přes které se přistupuje k obsahu proměnné data, nebo toho ActiveRow, takže napovídání, integrita dat apod. je zajištěná ne?

Ne nutně, dokud jsme neviděli ukázku.

$entita->data->column

poskytuje všechny (pseudo)properties, ale nenapovídá.

O zajištění integrity nemůžeme vůbec hovořit, dokud jsme kód neviděli. Integritu nezajišťuje jen a pouze přítomnost setteru/getteru, ale jejich konkrétní podoba…

Setter uloží novou hodnotu do $data pod klíčem z db tabulky a getter se podívá, jestli tato hodnota existuje v poli $data a pokud ano, tak ji vrátí. Pokud ne, tak se podívá jestli je nastavený ActiveRow a pokud ano, tak vrátí hodnotu z něj a pokud ne, tak vrátí null, nebo vyhodí error neinicializovaná proměnná.

Tady je to dokonce detailněji popsaný, díky čemuž už musí být tutově jasný, že ta třída vystavuje jako rozhraní gettery a settery

Popisuje právě jeden setter a getter na data, z toho tutově nevyplývá, že ta classa má setterů / getterů více…

A tímhle dokonce říká, že se ta třída chová jako PROXY nad ActiveRow a doufám, že všichni, co se naučili s Nette umí i návrhové vzory

Pouč mne prosím, myslel jsem, že všechny návrhové vzory znám, ale „osekaná proxy“ mne minula.

Je to opět zbytečná diskuze, dal jsem dotaz, tazatel to upřesnil ukázkou. Tvá intervence je dost zbytečná :)

@MarekBartoš no jo nekdy musis proste vyjadrit co myslis presne a kvuli tomu to je pak dlouhe. Proto treba nektere matematicke dukazy, ktere popisuji uplnou banalitu jsou nekolikastrankove.
Skoda je, jen to, ze i s detailnim popisem jsou lidi, kteri to i tak nepochopi a nevidi souvislosti. To je pak zbytecne psat to no 😄

Bulldog napsal(a):

že stored procedures vznikly jen proto, že programátoři neumí psát efektivní SQL dotazy

Nebo chtěli snížit traffic, nebo chtěli cachovat prováděcí plán, nebo chtěli grantovat oprávnění jen na část věty.
Každopádně to vlákno není jen o SP, je o tom, že DB má spoustu „non-crud“ dotazů, kterých se musíš zříct jen proto, aby sis mohl říct, že používáš jen Explorer a paranoidně se nebál poslat query ad-hoc.

programátorům zprostředkovávali jen ‚API‘ s těmito procedurami, což je podle tvého popisu v tom vlákně i tvůj případ.

Což vůbec není můj případ, popisoval jsem API 3. stran, kdy prostě nemáš jinou možnost. Nebo ano, můžeš klienta odmítnout, ale stále nechápu proč.

Ahojte

Nasiel by sa teda nejaky konkretny/finalny/funkcny priklad ako to spravit?

vdaka