php output constructor array

Question

I tried executing the code below to output each value from an array, end up result show:

Notice: Array to string conversion in C:\xampp\htdocs\test\tutor.php on line 22
_data : Array

PHP

<?php
class CateData 
{
    private $_data = array();

    public function __construct($data){

        $this->_data = $data;
    }
}

$data = array(
            'a'=>'Cate1',
            'b'=>'Cate2',
            'c'=>'Cate3',
            'd'=>'Cate4'
        );

$cate = new CateData($data);

foreach($cate as $key => $val){
    echo $key." : ". $val;
}
?>

How can I solve this?

Please read up on the basics of OOP.

Script47
– Script47

2018-06-08 03:39:30 +00:00
Commented Jun 8, 2018 at 3:39 — Script47
– Script47, Commented Jun 8, 2018 at 3:39

Karlo Kokkak · Accepted Answer · 2018-06-08 03:45:08Z

4

You're looping on the class object and not on the actual data.

In your class add:

public function getData(){
   return $this->_data;
}

Then change:

foreach($cate as $key => $val){
    echo $key." : ". $val;
}

To:

foreach($cate->getData() as $key => $val){
    echo $key." : ". $val;
}

edited Jun 8, 2018 at 3:45

answered Jun 8, 2018 at 3:34

Karlo Kokkak

3,7344 gold badges21 silver badges35 bronze badges

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

Comments

score 0 · Accepted Answer · 2018-06-08 04:05:40Z

0

While the accepted answer is better, an alternative is to change the scope so you you dont need to add a getData() method. But can then access the class variables directly.

//change the variable to public, and loop $cate->_data

class CateData 
{
    public $_data = array();

    public function __construct($data){

        $this->_data = $data;
    }
}

$data = array(
            'a'=>'Cate1',
            'b'=>'Cate2',
            'c'=>'Cate3',
            'd'=>'Cate4'
        );

$cate = new CateData($data);


foreach($cate->_data as $key => $val){
    echo $key." : ". $val;
}

edited Jun 8, 2018 at 4:05

answered Jun 8, 2018 at 3:49

user9487972

1 Comment

Script47 Over a year ago

No, it would be better practice you use a getter function as suggested by the accepted answer. No need to mess with the visibility.

JustBaron · Accepted Answer · 2018-06-08 11:10:04Z

0

First you should clear the differences between public protect and private.

In your code, you should change your _data to public because you want to visit outside your class.

Then change:

foreach($cate as $key => $val){
    echo $key." : ". $val;
}

to:

foreach($cate->_data as $key => $val){
    echo $key." : ". $val;
}

edited Jun 8, 2018 at 11:10

JustBaron

2,3457 gold badges25 silver badges37 bronze badges

answered Jun 8, 2018 at 3:42

clearT

1

1 Comment

Script47 Over a year ago

Please format your code properly and why should they change the visibility of the property? A getter would be better as suggested by the accepted answer.

Collectives™ on Stack Overflow

php output constructor array

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related