PHP return - beginner fault

Question

I do not know what I'm doing wrong. Perhaps someone could advise me. I'm trying to define seasons and then use the result in index.

...

File with function:

function getCurrentTheme($for_area) {

// Definition of seasons
$spring='spring theme title';
$spring_season=array('03-21',......);
$summer='summer theme title';
$summer_season=array(......);
$autumn='autumn theme title';
$autumn_season=array(......);
$winter='winter theme title';
$winter_season=array(......);

// Today
$current_date=date('m-d');

// So what season is now?
if ($for_area==='some_area') {
    if (in_array($current_date,$spring_season) {
        $theme=$spring;
    }
    else if (in_array($current_date,$summer_season) {
        $theme=$summer;
    }
    else if (in_array($current_date,$autumn_season) {
        $theme=$autumn;
    }
    else if (in_array($current_date,$winter_season) {
        $theme=$winter;
    }
    else {}
}
else if ($for_area==='other_area') {
    // ...
}
else {}

return $theme;

}

...

Index:

$area='some_area';
getCurrentTheme($area);

// And here is the fault. Hope sometimes I will stop being retarded.

echo $theme;

// What should be printed?
summer theme title

Thanks in advance and please try understand my innocence.

If someone needs to find out what season currently is psoug.org/snippet/What-Season-Is-It_233.htm — falcon
– falcon, Commented Aug 27, 2014 at 9:42

Laurence · Accepted Answer · 2014-08-26 11:15:00Z

1

You need to store the value, so that you can then echo it:

$area='some_area';
$theme = getCurrentTheme($area);    
echo $theme;

answered Aug 26, 2014 at 11:15

Laurence

60.3k22 gold badges178 silver badges215 bronze badges

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

2 Comments

nl-x Over a year ago

So what you miss is the $theme = in front of getCurrentTheme($area);

falcon Over a year ago

Thank you very much. I will accept as soon as possible.

Basit · Accepted Answer · 2014-08-26 11:20:28Z

1

You're not storing the value returned from the getCurrentTheme function. Either store it in a variable and then echo it

$theme = getCurrentTheme($area);
echo $theme;

or simply echo the call without having to use a variable.

echo getCurrentTheme($area);

Your choice!

answered Aug 26, 2014 at 11:20

Basit

1,8402 gold badges31 silver badges50 bronze badges

Collectives™ on Stack Overflow

PHP return - beginner fault

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related