Ndtyjky

I'm wondering about this weird thing:

public function getAllCustomers()

{

    $customers = $this->redis->keys("customer:*");

    foreach ($customers as $value) {

        return new Customer($this->redis->hget($value,"name"),$this->redis->hget($value,"id"),$this->redis->hget($value,"email"));

    }

}

This method returns all customers from my database.

But if I try to loop through all of these customers:

foreach ($customerController->getAllCustomers() as $customer) {

    var_dump($customer);

}

The getName() method is not found. var_dump returns:

NULL

NULL

NULL

Customer class:

class Customer {

    var $name;

    var $id;

    var $email;



    function __construct($name, $id,$email) {

        $this->name = $name;

        $this->id = $id;

        $this->email = $email;

    }



     /**

     * @return mixed

     */

    public function getName()

    {

        return $this->name;

    }



    /**

     * @return mixed

     */

    public function getId()

    {

        return $this->id;

    }



    /**

     * @return mixed

    */

    public function getEmail()

    {

        return $this->email;

    }



    public function __toString()

    {

       return "";

    }

}

I'm pretty new to PHP and don't understand why I can't access the Customer object's field.

Your Problem: you do not return array of customer but only one. You getting null because your function return only 1 object -> and in PHP, when using foreach loop on object you get his fields -> and the fields do not have the getName function.

Solution: Init customer array, populate it and return from the function.

 public function getAllCustomers()

{

    $customers = $this->redis->keys("customer:*");

    $customersObjs = array();

    foreach ($customers as $value) {

        $customersObjs = new Customer($this->redis->hget($value,"name"),$this->redis->hget($value,"id"),$this->redis->hget($value,"email")));

    }

    return $customersObjs;

}

Now you have array of the customersObjs you can loop on with:

foreach ($customerController->getAllCustomers() as $customer) {

    echo $customer->getName();

}

Solution :

public function getAllCustomers()

{

    $customers = $this->redis->keys("customer:*");

    $custumersArray = array();

    foreach ($customers as $value) {

        $custumersArray = Customer($this->redis->hget($value,"name"),$this->redis->hget($value,"email"),$this->redis->hget($value,"id"));

    }

    return $custumersArray;

}

the problem was that you are returning a single array but not a array list.