dekemvrios / phpexpressive Goto Github PK

A implementação atual da biblioteca tem performance prejudicada quando realizado inserção de registros com relacionamento hasMany devido as operações do active record. Viabilizar a inserção em grupo desses registros ao invés de um a um pelo procedimento padrão do active record.

O controle de transaction automático do expressive é muito bom.
(Quando ele abre e commita a transaction automaticamente).

Porém, ele faz o programador esquecer que a operação em si passa por uma transaction, visando simplificar, ou até mesmo facilitar a curva de aprendizado, será que seria legal exigir a abertura de transaction pelo programador? Quando chamar um método do expressive caso não tenha sido aberta ainda ele poderia gerar uma exception...

O que acha? Ou poderiamos melhorar a doc, um arquivo markdown exclusivo para explicar as transactions...

A primeira opção é mais de imposição, mas ensina até mesmo devs que não olharem a doc!

Eu fiz um teste com o expresssive, onde removi 95% dos blocos de:

try {
} catch ($e) {
}

Isso facilitou muito o controle de erros do aplicativo dentro de um micro serviço, porque esse controle de vazamentos de erro, normalmente fica instalado no sistema de rotas, que converte automaticamente erros em json de "response" através da sua mensagem e código!

Deixando os erros do expressive subirem, quem instalar a biblioteca fica livre pra fazer como preferir seu próprio controle de erros, e a biblioteca fica menos opinativa nesse assunto!

ATUALIZADO
Olhando o código atualizado da lib, já está assim, vou conferir detalhadamente mais tarde, porém considero fechada a issue.

Obs.: Ainda tem alguns try ... catch, sem eles a lib perderia a trigger para rodar o rollback da transaction! Mesmo assim o bloco de controle sobe a Exception em seguida.

Ex.:

    public function create(ExpressiveContract $model)
    {
        $record = $this->beforeInsertVerifyDuplicity($model);
        if ($record) {
            return $record;
        }
        $table = $model::$schema->getRepository();
        Database::beginTransaction($model);
        try {
            $model = Actions::doThingWhenDatabaseAction($model, 'whenInsert', 'Before');
            $model = $this->hasOneDependency($model);
            Capsule::table($table)->insert($this->getInsertFields($model));
        } catch (\PDOException $exception) {
            Database::rollbackActiveTransaction($model);
            // aqui a exception sobe de novo
            throw new Exception($exception->getMessage(), 400);
        }