Question

I am not sure if this is totally the wrong thing to do, so I am looking for a bit of advice.

I have set up a database class with the constructor establishing a PDO connection to a MySQL database.

I’ve been looking at singletons and global variables, but there always seems to be someone who recommends against either/or.

I’m experimenting with a user class which extends the database class, so I can call upon the PDO functions/methods but maintain separate user class code. Is this a stupid thing to do?

Answer 1

You should generally pass a connection into your user, so your user class would take a database type object into its constructor and then use that database object to execute queries against the database. That way your data access logic remains separate from your business logic. This is called composition, as opposed to what you’re talking about, which is inhertance.

If you really wanted to be technical, it would be best to have a user object with nothing but public variables, and then you would use a ‘service’ to implement your business logic.

class UserService implements IUserService
{
    private $_db;
    function __construct(IDb $db) {
        $this->_db = db;
    }

    function GetAllUsers() {
        $users = Array();

        $result = $this->_db->Query("select * from user")

        foreach($result as $user) {

            //Would resolve this into your user domain object here
            users[] = $user;
        }
        return users;
    }
}