in string contains if-query

Question

I am searching for a solution for fixing the below code

$tableString .= "
    <tr>
    <td>img</td>
    <td>"
    if (!empty($configArray[$configCounter]['Glasart'])){
        ."<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>"
        }
    if (!empty($configArray[$configCounter]['Material'])){
        ."<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>"

I always get an error when this code runs, but I am unsure what is causing it.

You cannot use `if` construct within expression. Use ternary operator instead — hindmost, Jul 21 '14 at 14:10
What error do you get? What do you think it means? What did you try to do to solve it? What specifically are you having trouble with? — oliakaoil, Jul 21 '14 at 14:10

score 1 · Answer 1 · answered Jul 21 '14 at 14:11

Your syntax is totally messed up. You've got IF statements within concatenating stings. Rogue dots/periods within the IFs that mean absolutely nothing, missing semicolons etc.

See below:

$tableString .= "<tr><td>img</td><td>";
if (!empty($configArray[$configCounter]['Glasart'])){
    $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
}
if (!empty($configArray[$configCounter]['Material'])){
    $tableString .= ."<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
}

score 0 · Answer 2 · answered Jul 21 '14 at 14:10

Change your code into this:

$tableString .= "
    <tr>
    <td>img</td>
    <td>";
if (!empty($configArray[$configCounter]['Glasart'])){
        $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
}
if (!empty($configArray[$configCounter]['Material'])){
        $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
}

You cannot use a if statement inside an expression. So split your code in multiple steps.

score 0 · Answer 3 · answered Jul 21 '14 at 14:11

0

Try something like this:

$tableString .= "
<tr>
<td>img</td>
<td>";

if (!empty($configArray[$configCounter]['Glasart'])){
    $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
    }
if (!empty($configArray[$configCounter]['Material'])){
    $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";

answered Jul 21 '14 at 14:11

Aryess

530
2
12

2

`+=` doesn't work for concatenating strings, use `.=` – Jonathon Jul 21 '14 at 14:11
Yup, already corrected. – Aryess Jul 21 '14 at 14:12

score 0 · Answer 4 · answered Jul 21 '14 at 14:11

Try using this -

    $tableString .= "<tr>
    <td>img</td>
    <td>";
    if (!empty($configArray[$configCounter]['Glasart'])){
        $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
        }
    if (!empty($configArray[$configCounter]['Material'])){
        $tableString .= "<p> Glasart: ".$configArray[$configCounter]['Glasart']."</p>";
    }

in string contains if-query

4 Answers4