Menu
Community /Pin to ProfileBookmark
@NogDogNov 10.2021 — #I would say just one
@NogDogNov 11.2021 — #I would probably (could be argued out of it, maybe) create a separate class for groups of Person objects, probably implementing either the [Traversable](https://www.php.net/manual/en/class.traversable.php) or [Iterator](https://www.php.net/manual/en/class.iterator.php) interface. Then you'd basically do what you're saying, maybe something like:
No guarantees on any of that code, just throwing a quick-and-dirty mock-up together. The iterator stuff would then mean that you could do foreach() loops on the Person objects it contains, something like:
@NogDogNov 12.2021 — #Hmm...I _think_ maybe you could just leverage that
Output:
So, if
Object Logic
Good Morning,
I only have basic skills in procedural coding in PHP but am trying to get into Object Oriented.
I have a question, If I had a class called person (to keep it generic). I can create a person object from a form submission, such as $person = new person($_POST[]…);
Can I have a method in my class called getPersonFromID() which creates the person within the class?
$personObject = new Person;
$person = $personObject->getPersonFromID(44);
Does this work or should I have another class to retrieve and create people from places like a database?
Thank you and I am learning.
Sign in
to post a comment6 Comments(s) ↴
0
@NogDogNov 10.2021 — #I would say just one PersonPerson::create()Person::get()create()get()[code=php]
class Person {
private $id;
private $pdo;
public function __construct(PDO $pdo, $id = null) {
$this->pdo = $pdo; // I like to pass DB objects in, not create within a class
if(!empty($id)) {
$this->get($id);
}
}
public function get($id) {
// stuff here to query database and populate relevant class properties
}
public function create(Array $data) {
// stuff here to create new person in DB and populate any relevant class properties
}
}
[/code]0
@NogDogNov 11.2021 — #I would probably (could be argued out of it, maybe) create a separate class for groups of Person objects, probably implementing either the [code=php]
class PersonGroup implements Iterator {
private $people = [];
public function __construct($group, $id){
$this->getPeople($group, $id);
}
private function getPeople($group, $id) {
// search stuff, then with results:
while($row = $result->fetch_assoc()) {
$this->people[] = new Person($id);
}
}
// The following are Iterator interface methods:
public function current() {
return current($this->people);
}
public function key() {
return key($this->people);
}
public function next() {
return next($this->people);
}
public function rewind() {
return reset($this->people);
}
public function valid() {
return isset($this->people[current($this->people)]);
}
}
[/code]No guarantees on any of that code, just throwing a quick-and-dirty mock-up together. The iterator stuff would then mean that you could do foreach() loops on the Person objects it contains, something like:
[code=php]
$group = new PersonGroup($group, $id);
foreach($group as $person) {
// do stuff with $person, which will be a Person object
}
[/code]0
@NogDogNov 12.2021 — #Hmm...I _think_ maybe you could just leverage that valid()[code=php]
<?php
class Person {
private $id;
public function __construct($id) {
$this->id = (int) $id;
}
}
Class PersonGroup implements Iterator {
private $people = [];
public function add(Person $person) {
$this->people[] = $person;
}
// Iterator methods:
public function current() {
return current($this->people);
}
public function key() {
return key($this->people);
}
public function next() {
return next($this->people);
}
public function rewind() {
return reset($this->people);
}
public function valid() {
return is_int($this->key());
}
}
$test = new PersonGroup();
$result = $test->valid();
echo "Unpopulated valid() check: ".var_export($result, 1)."n";
$test->add(new Person(123));
$test->add(new Person(456));
$result = $test->valid();
echo "Populated valid() check: ".var_export($result, 1)."n";
foreach($test as $person) {
var_export($person);
echo "n";
}
[/code]Output:
<i>
</i>$ php test.php
Unpopulated valid() check: false
Populated valid() check: true
Person::__set_state(array(
'id' => 123,
))
Person::__set_state(array(
'id' => 456,
))
So, if
valid()