'Unexpected echo' PHP error message - outputting php variable in html input 'value' field

Question

I keep on receiving this error message when running the PHP code below ::

"Parse error: syntax error, unexpected 'echo' (T_ECHO) in C:\xampp\htdocs\informationdb\item.php on line 93"

I am trying to output dynamic PHP variables in a static html table using a foreach loop. By any chance, can anyone spot where the error lies?

My code is as follows::

<table>
<form action='', method="POST"> 

                            <?php 
                            foreach($libitem["libraries"] as $library) { 
                                foreach($libitem["libraryitemids"] as $libraryitemid) { 
                                    foreach($libitem["sectionnames"] as $sectionName) { 
                                        foreach($libitem["sectionnumbers"] as $sectionNumber) {
                                            echo "<tr><td id='radiocell'><input type='radio' name='libraryitemid' id='radio' value='" . $libraryitemid . "'></td><td>" . echo $library . "</td><td>" . echo $sectionName . "</td><td>" . echo $sectionNumber . "</td><td class='available'>Available</td></tr>";                                    
                                        } 
                                    } 
                                } 
                            } 

                            ?>


                        </table>

P.s. just in case anyone is wondering, line 93 is the line that the 'echo' statement is on :) — Robert Young, Apr 20 '15 at 12:37
One echo is enough when concatenating. Also you may consider re-writing those foreaches into something less....performance loss-y. — Andrei, Apr 20 '15 at 12:37

score 3 · Accepted Answer · answered Apr 20 '15 at 12:38

When you are concatenating a string, you use dots, but no echo, also see the manual:

echo "<tr><td id='radiocell'><input type='radio' name='libraryitemid' id='radio' value='" . $libraryitemid . "'></td><td>"
    . echo $library . "</td><td>"
    . echo $sectionName . "</td><td>"
    . echo $sectionNumber . "</td><td class='available'>Available</td></tr>";

should be:

echo "<tr><td id='radiocell'><input type='radio' name='libraryitemid' id='radio' value='" . $libraryitemid . "'></td><td>"
    . $library . "</td><td>"
    . $sectionName . "</td><td>"
    . $sectionNumber . "</td><td class='available'>Available</td></tr>";

Thanks @jeroen , you're a star! All the answers here were right, btw. — Robert Young, Apr 20 '15 at 12:44
@RobertYoung I imagine, it is one of the basic things of string concatenation :-) — jeroen, Apr 20 '15 at 12:45

score 1 · Answer 2 · answered Apr 20 '15 at 12:38

1

Don't use echo here

 . echo $library . "</td><td>" . echo $sectionName . "</td><td>" . echo $sectionNumber . "

There is need only first echo. For concatenation isn't necessary.

answered Apr 20 '15 at 12:38

Kristiyan

1,655
14
17

score 0 · Answer 3 · answered Apr 20 '15 at 12:38

0

Remove echo from inside the string.

For example, " . $libraryitemid . "'></td><td>" . echo $library . "</td><td>" . echo $sectionName . shoule be " . $libraryitemid . "'></td><td>" . $library . "</td><td>" . $sectionName . "<

answered Apr 20 '15 at 12:38

Grim...

16,518
7
45
61

score 0 · Answer 4 · answered Apr 20 '15 at 12:38

Try this:

echo "<tr><td id='radiocell'><input type='radio' name='libraryitemid' id='radio' value='" . $libraryitemid . "'></td><td>" . $library . "</td><td>" . $sectionName . "</td><td>" . $sectionNumber . "</td><td class='available'>Available</td></tr>";

score 0 · Answer 5 · answered Apr 20 '15 at 12:42

you are using echo in the echo which is an error . remove this line .

echo "<tr><td id='radiocell'><input type='radio' name='libraryitemid' id='radio' value='" . $libraryitemid . "'></td><td>" . echo $library . "</td><td>" . echo $sectionName . "</td><td>" . echo $sectionNumber . "</td><td class='available'>Available</td></tr>";

you should use this

<table>
<form action='', method="POST"> 

                            <?php 
                            foreach($libitem["libraries"] as $library) { 
                                foreach($libitem["libraryitemids"] as $libraryitemid) { 
                                    foreach($libitem["sectionnames"] as $sectionName) { 
                                        foreach($libitem["sectionnumbers"] as $sectionNumber) {
                                            echo "<tr><td id='radiocell'><input type='radio' name='libraryitemid' id='radio' value='" . $libraryitemid . "'></td><td>" .  $library . "</td><td>" .  $sectionName . "</td><td>" . $sectionNumber . "</td><td class='available'>Available</td></tr>";                                    
                                        } 
                                    } 
                                } 
                            } 

                            ?>


                        </table>

'Unexpected echo' PHP error message - outputting php variable in html input 'value' field

5 Answers5