PHPMT / PseudoORM

App de persistência para fins didáticos
GNU General Public License v2.0
12 stars 7 forks source link

Refatorado classe GenericDAO para tornar mais flexível #14

Closed EvandroMohr closed 7 years ago

EvandroMohr commented 7 years ago

Acho bastante interessante. Fiz desse modo apenas para validar a GenericDAO para garantir que ela estava lendo atributos com visibilidade private. ter um bom construtor é algo legal (named constructors ainda melhores)

EvandroMohr commented 7 years ago

@enieber inclusive já tem uma issue pra isso #12

enieber commented 7 years ago

agora entendi o pq da issue #12 ...

jackmakiyama commented 7 years ago

@EvandroMohr e @enieber vamos aceitar a PR? E voltar a trampar a partir daí?

enieber commented 7 years ago

eu ia dizer isso, pq ja ta ficando muito grande kk

EvandroMohr commented 7 years ago

hehehe, quase nem fiz commits =P

EvandroMohr commented 7 years ago

Percebi que precisamos urgente de testes, tá ruim refatorar e ficar testando o código =/