"Parse error: syntax error, unexpected T_IF in ..." when I use the return statement

Question

This is part of a bigger code, I keep getting the parse error ::: if I create the if function outside of the $return & reference it with a session it works but this is not "good" coding :::how can I resolve this nagging issue or better construct my return value? ::: Any help greatly appreciated :::

$return = '';
    $return .='<div id="viewport">'. "\n";
    $return .= $ic . "\n";
    $return .='<div id="' . $wallID . '"></div>'. "\n"; 
    $return .='</div>'. "\n";                                  
    $return .= if( isset($coda) )'<div id="coda' . $wallID . '"></div>';. "\n";
    $return .='</div>'. "\n";                                   
    $return .='<div class="clearfix"></div>'. "\n";

 return $return;

Which line is the error in? Please always specify the line of the error message — Pekka
– Pekka, Commented Nov 6, 2011 at 13:39
Ah, I see now. You can't use if like that - put it before the $return.= assignment — Pekka
– Pekka, Commented Nov 6, 2011 at 13:40

Mat · Accepted Answer · 2011-11-06 13:46:19Z

3

Just use a proper if:

...
if( isset($coda) ) {
  $return .= '<div id="coda' . $wallID . '"></div>'. "\n";
}
$return .= ...
...

edited Nov 6, 2011 at 13:46

answered Nov 6, 2011 at 13:41

Mat

208k41 gold badges407 silver badges423 bronze badges

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

Comments

genesis · Accepted Answer · 2011-11-06 13:41:08Z

1

This one is probably problematic

$return .= if( isset($coda) )'<div id="coda' . $wallID . '"></div>';. "\n";

it should be

$return .= ((isset($coda)) ? '<div id="coda' . $wallID . '"></div>' : '')."\n";

To make it all more clean, I'd suggest you to use HEREDOC

answered Nov 6, 2011 at 13:41

genesis

51.1k20 gold badges99 silver badges127 bronze badges

1 Comment

John Montgomery Over a year ago

THX for the heads up on HEREDOC ::: Read some documentation & implemented it in my code ... looks much cleaner

catalin.costache · Accepted Answer · 2011-11-06 13:49:51Z

0

Use a ternary operator instead of the misused if block:

$return .= isset($coda) ? "<div id='coda{$wallID}'></div>\n" : '';

or a more readable if block as @Mat sugested

answered Nov 6, 2011 at 13:49

catalin.costache

3,2131 gold badge27 silver badges16 bronze badges

Collectives™ on Stack Overflow

"Parse error: syntax error, unexpected T_IF in ..." when I use the return statement

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related