PHP - foreach function with array

Question

Hi I have an error when I call a function.

"Warning: Illegal string offset 'id' in C:\xampp\htdocs\blog\posts.php on line 28 2"

function:

function get_short_posts() {
    $sql = "SELECT * FROM posts ORDER by id DESC LIMIT 0, 5";
    $result = mysql_query($sql);
    while($row = mysql_fetch_assoc($result)) {
        $timestamp = new DateTime($row['date']);
        return array (
            "id" => $row['id'],
            "title" => $row['title'],
            "content" => $row['content'],
            "author" => $row['author'],
            "date" => $timestamp->format('d-m-Y'),
            "time" => $timestamp->format('H:i')
        );
    }

Call:

require_once "functions.php";
    $_posts = get_short_posts();
    foreach($_posts as $_post) {
        echo $_post['id'];
    }

Try doing die(var_dump($row)); in your while loop. This will halt all other execution, and display the error on screen. — Justin E
– Justin E, Commented Jan 18, 2014 at 21:15
You know you return after the first iteration in the get_short_posts function right, so the foreach will not work as expected. — Lawrence Cherone
– Lawrence Cherone, Commented Jan 18, 2014 at 21:15
Show your table columns names, please. Maybe column "id" called "Id" or "ID"? — Victor Bocharsky
– Victor Bocharsky, Commented Jan 18, 2014 at 21:15
Also, try: $result = mysql_query($sql)or die(mysql_error()); — Justin E
– Justin E, Commented Jan 18, 2014 at 21:16

Zoe - Save the data dump · Accepted Answer · 2019-05-25 14:00:26Z

You know you return after the first iteration in the get_short_posts function right, so the foreach will not work as expected.

Try:

<?php 
function get_short_posts() {
    $sql = "SELECT * FROM posts ORDER by id DESC LIMIT 0, 5";
    $result = mysql_query($sql);
    $return = array();
    while($row = mysql_fetch_assoc($result)) {
        $timestamp = new DateTime($row['date']);
        $return[] = array (
            "id" => $row['id'],
            "title" => $row['title'],
            "content" => $row['content'],
            "author" => $row['author'],
            "date" => $timestamp->format('d-m-Y'),
            "time" => $timestamp->format('H:i')
        );
    }
    return $return;
}
?>

<?php 
require_once "functions.php";

foreach(get_short_posts() as $_post) {
    echo $_post['id'];
}
?>

Also, Don't use mysql_* functions in new code. They are no longer maintained and are officially deprecated. See the red box? Learn about prepared statements instead, and use PDO, or MySQLi - this article will help you decide which. If you choose PDO, here is a good tutorial.

Gert B. · Accepted Answer · 2014-01-18 21:18:09Z

0

function get_short_posts() {
    $sql = "SELECT * FROM posts ORDER by id DESC LIMIT 0, 5";
    $result = mysql_query($sql);
    while($row = mysql_fetch_assoc($result)) {
        $timestamp = new DateTime($row['date']);
        $data [] = array (
            "id" => $row['id'],
            "title" => $row['title'],
            "content" => $row['content'],
            "author" => $row['author'],
            "date" => $timestamp->format('d-m-Y'),
            "time" => $timestamp->format('H:i')
        );

    }
    return $data;
    }

you return data, so the loop stops, save your data in a array and return that array like abive code

answered Jan 18, 2014 at 21:18

Gert B.

2,34720 silver badges22 bronze badges

Comments

Abhik Chakraborty · Accepted Answer · 2014-01-18 21:19:02Z

Your code is wrong it should be as below, assuming the query is returning data as mentioned.

function get_short_posts() {
    $sql = "SELECT * FROM posts ORDER by id DESC LIMIT 0, 5";
    $result = mysql_query($sql);
    $rows = array();
    $return_data = array();
    while($row = mysql_fetch_assoc($result)) {
        $timestamp = new DateTime($row['date']);
        $data = array (
            "id" => $row['id'],
            "title" => $row['title'],
            "content" => $row['content'],
            "author" => $row['author'],
            "date" => $timestamp->format('d-m-Y'),
            "time" => $timestamp->format('H:i')
        );
        $return_data[] = $data;
    }
    return $return_data ;

}


require_once "functions.php";
    $posts = get_short_posts();
    foreach($posts as $key=>$val) {
        echo $val['id'];
    }

Collectives™ on Stack Overflow

PHP - foreach function with array

3 Answers 3

Try:

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Try:

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related