
Kevin Major - 2010-03-31 17:22:13
Well, this class isn't really the Ultimate DB Access Wrapper, as it only wraps MySQL. Since PHP already has MySQLi, to say nothing of PDO, this class is redundant. Quite simply, this class isn't necessary, or even very useful, as there are already two superior alternatives tied to the language itself.
There's also the matter of a complete lack of encapsulation. Some methods, like connect(), should be private, as should all of your data members. There's no point in having an object whose guts can be changed on a whim. Public methods should be used to enforce behavior - that is, they're used to force others to use the object in a certain way.
Encapsulation is a cornerstone of OOP.