PHP foreach loop JSON

Question

I know there are like hundreds similar questions on this site, but I just can't get my code working...

I have this JSON

{
"version":"1.0.0",
"buildDate":20131029,
"buildTime":165127,

"lockPath":"..\\var\\lock",
"scriptPath":"..\\var\\lock",
"connections":
[
{
    "name":"o016561",
    "bez":"GEW-NRW",
    "type":"OVPN"
},
{
    "name":"o016482",
    "bez":"GEW-BW",
    "type":"OVPN"
},
{
    "name":"o019998",
    "bez":"GEW-SH",
    "type":"OVPN"
}
]}

how can I access the "name" values to check if there's an existing file with an equal name? I tried

$json_config_data = json_decode(file_get_contents($json_path,true));

    foreach($json_config_data->connections as $connectionName)
    {
        if($connectionName->name == $fileName)
        {
            $status = 1;
        }
        else
        {
            $status = 0;
        }
    }

but I always get $status = 0... I think there's an easy solution for this, but I'm pretty new to PHP so I'd be glad for any kind of help. Thanks in advice

h2ooooooo · Accepted Answer · 2013-11-25 10:50:19Z

5

You're resetting the value of $status for every iteration which means that the last connection HAS to be the correct one. You're probably looking for a break statement.

$json_config_data = json_decode(file_get_contents($json_path,true));

$status = 0; //Default to 0
foreach($json_config_data->connections as $connectionName)
{
    if($connectionName->name == $fileName)
    {
        $status = 1;
        break; //End the loop
    }
}

answered Nov 25, 2013 at 10:50

h2ooooooo

39.6k8 gold badges70 silver badges107 bronze badges

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

1 Comment

p4b4 Over a year ago

after reading your post, it all makes sense now. thanks a lot!

cmbuckley · Accepted Answer · 2013-11-25 10:50:14Z

2

This would only result in $status == 1 if the final name matched your requirements; otherwise you're setting $status back to 0. You should break out of the loop when you find a match:

$status = 0;

foreach ($json_config_data->connections as $connectionName) {
    if ($connectionName->name == $fileName) {
        $status = 1;
        break; // this breaks out of the foreach loop
    }
}

answered Nov 25, 2013 at 10:50

cmbuckley

43k10 gold badges83 silver badges95 bronze badges

Collectives™ on Stack Overflow

PHP foreach loop JSON

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related