if statement inside of output

Question

   $output10 .= '            <div class="panel panel-profile list-view">
          <div class="panel-heading">
            <div class="media">
              <div class="media-left">
                <a href="#">
                  <img class="media-object img-circle" src="../images/photos/user1.png" alt="">
                </a>
              </div>
              <div class="media-body">
                <h4 class="media-heading">'. $row['username'] . '</h4>
               'if($pAdminLevel == 0) : '<p class="media-usermeta"><i class="glyphicon glyphicon-briefcase"></i>' . $pAdminLevel . '</p>' endif; '
              </div>
            </div><!-- media -->

I want to make a if statement, but it shows this error. The code I posted is not the full version, that's why closing brackets are missing, I just posted what I don't understand, thank you.

syntax error, unexpected 'if' (T_IF) in

score 1 · Answer 1 · answered Nov 01 '18 at 19:27

You cannot use if within concatenated string.

There are basically two options here:

Ternary operator:

$output = "abc" . ($pAdminLevel == 0 ? "text when condition is true" : "text when it is false") . " other text";

Separate logic for code:

$output = "abc";
if ($pAdminLevel == 0) {
    $output .= "cde";
} else {
    $output .= "xyz";
}

score 0 · Answer 2 · answered Nov 01 '18 at 19:24

You can try something like this:

   $output10 .= '            <div class="panel panel-profile list-view">
          <div class="panel-heading">
            <div class="media">
              <div class="media-left">
                <a href="#">
                  <img class="media-object img-circle" src="../images/photos/user1.png" alt="">
                </a>
              </div>
              <div class="media-body">
                <h4 class="media-heading">'. $row['username'] . '</h4>' .
               ( $pAdminLevel == 0 ? '<p class="media-usermeta"><i class="glyphicon glyphicon-briefcase"></i>' . $pAdminLevel . '</p>' : '' ) .
              '</div>
            </div><!-- media -->';

The syntax is ( boolean ? true : false ).

There is no other way to make inline conditionals. You could always just split up your string of course.

score 0 · Answer 3 · answered Nov 01 '18 at 19:30

You stopped the string literal but didnt start it again.

$output10 .= '<div class="panel panel-profile list-view">
      <div class="panel-heading">
        <div class="media">
          <div class="media-left">
            <a href="#">
              <img class="media-object img-circle" src="../images/photos/user1.png" alt="">
            </a>
          </div>
          <div class="media-body">
            <h4 class="media-heading">'. $row['username'] . '</h4>';
if($pAdminLevel == 0) : 
    $output10 .= '<p class="media-usermeta"><i class="glyphicon glyphicon-briefcase"></i>' . $pAdminLevel . '</p>' ;
endif; 

$output10 .= '</div>
        </div><!-- media -->';

score 0 · Answer 4 · answered Nov 01 '18 at 19:31

I'm a fan of the following pattern when managing html output with php:

$output10 .= '<div class="panel panel-profile list-view">';
$output10 .= '<div class="panel-heading">';
  $output10 .= '<div class="media">';
    $output10 .= '<div class="media-left">';
      $output10 .= '<a href="#">';
        $output10 .= '<img class="media-object img-circle" src="../images/photos/user1.png" alt="">';
      $output10 .= '</a>';
    $output10 .= '</div>';
    $output10 .= '<div class="media-body">';
      $output10 .= '<h4 class="media-heading">'. $row['username'] . '</h4>';
      if($pAdminLevel == 0) {
        $output10 .= '<p class="media-usermeta"><i class="glyphicon glyphicon-briefcase"></i>' . $pAdminLevel . '</p>';
      }
    $output10 .= '</div>';
  $output10 .= '</div><!-- media -->';

It's not the prettiest, but it avoids use of ternary conditions which can be bad for maintainability & avoids multiline strings (which have historically been problematic in PHP)

if statement inside of output

4 Answers4