PHP OOP Design for simple Models

Question

i've a little problem with the proper design for some simple database models. Lets say i have an User Object with getter/setters and an read method. Read querys the database and sets the properties.

class User extends MyDbBaseClass
{

protected $_id;
protected $_name;

public function setId($id)
{
    $this->_id = $id;
}

public function setName($name)
{
    $this->_name = $name;
}

public function getId()
{
    return (int) $this->_id;
}

public function getName()
{
    return (string) $this->_name;
}

public function read($id)
{
    // fetch ONE record from Database
    $this->_id = $this->setId($sqlResult['id');
    $this->_name = $this->setName($sqlResult['name']);
}

public function save()
{
    // do some sql stuff to save user to database
}

}

My Problem is, how to return multiple users?

public function getCollection()
{
    // fetch all user records from database
    forearch ($sqlResult as $result) {
        // ... no idea..
    }
}

Goal:

// works
$u = new User();
$u->read(1);
echo $u->getName();

// dont know the best way
$u = new User();
$uC = $u->getCollection();

foreach ($uC as $u)
{
    echo $u->getName();
}

Any best practices for this?

Robert Cabri · Accepted Answer · 2009-12-14 08:31:42Z

1

You could just return an array with users

public function getCollection()
{
  // fetch all user records from database
  $users = array();
  forearch ($sqlResult as $result) {
    // ... no idea..
    $user = new User();
    $user->_name = $result->name; // just an example
    $user[] = $users;
  }
  return $users;
}

answered Dec 14, 2009 at 8:31

Robert Cabri

4,0111 gold badge22 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

PHP OOP Design for simple Models

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related