search

PatrickLubbers / educom-webshop-oop-1706014168

0 stars 0 forks source link

De `runSelectQuery` en de `readManyRows` lijken erg op elkaar, dit is niet D.R.Y. #17

Closed JeroenHeemskerk closed 7 months ago

JeroenHeemskerk commented 7 months ago

Onderstaande functies https://github.com/PatrickLubbers/educom-webshop-oop-1706014168/blob/6f4691bc77d255c68a4ec51feabd91015e0d258a/models/Crud.php#L28-L37 https://github.com/PatrickLubbers/educom-webshop-oop-1706014168/blob/6f4691bc77d255c68a4ec51feabd91015e0d258a/models/Crud.php#L72-L84 lijken erg op elkaar,

Dit is niet D.R.Y. Don't Repeat Yourself

Kan je in de CrudShop niet de readManyRows gebruiken?

PatrickLubbers commented 7 months ago

Ga ik doen ja, scheelt ook weer code!