Светит такую ошибку Parse error: syntax error, unexpected 'echo' (T_ECHO), expecting ',' or '; пересматривала все знаки вроде на месте, что не так?

Question

[DianNaA]

<?php
		$i = 0;
		while($i < count($names)) {

		 echo "<li>"
		 echo "<div class = \"avatar\">;
		 <img src=\"images/user.png\">;
		 </div>";
		 echo "<h2>" . $names[$i] . "</h2>";

		  echo "<p>Привет!</p>";
		   echo "<div class = \"time\">09:15</div>";
		echo"</li>";

		$i = $i + 1;
		}
?>

ошибку пишет на 6 строке

Comments

[Дмитрий]

echo "<li>" // тут нужно добавить ;

[Дмитрий]

Ну и на будущее.. Пишите хтмл в одинарных кавычках, не нужно будет экранировать двойные
\"

[Sergej]

Что бы замыленные глаза не мешали, используйте PHPStorm IDE или редактор по типу vscode с плагинами PHP.
Так вы эту ошибку увидите моментально.

PHPStorm


VSCode

Answer 1

[Северное Сияние]

Выброси свою книжку.
Начинай писать сразу правильно. Так лучше, согласись:

<?php $names=['петя','вася']; ?>

<?php if ($names): ?>
    <ul>
        <?php foreach ($names as $name): ?>
            <li>
                <div class = "avatar">
                    <img src="images/user.png">
                </div>
            </li>
            <h2><?=$name?></h2>
            <p>Привет!</p>
            <div class = "time">09:15</div>
        <?php endforeach; ?>
    </ul>
<?php endif; ?>

Это называется шаблоны.

Comments

[Дмитрий]

Ну ты же должен был понять по коду, что это новичок совсем

[Одиночка Айс]

Вы бы тоже выбросили всякие endforeach, дебажить заманаешься потом

[Виталий Хоменко]

Если макака дает совет, то пусть дает правильный совет. Та ссылка, по которой статья, это дрянь. Начиная от коротких тэгов в примерах, заканчивая правилами фильтрации XSS. Разве макаки не знают, что XSS надо фильтровать ДО сохранения в БД? Там еще много чего такого

[Северное Сияние]

Виталий Хоменко, давай ты не будешь писать о том, о чем не имеешь ни малейшего представления?

Начиная от коротких тэгов в примерах

Где именно ты увидел там короткие теги? Тег <?php, ровно как и тег вывода на печать <?=?> - является неотъемлемой частью языка. Тег <?= доступен всегда, вне зависимости от настройки short_open_tag начиная c версии 5.4.

Разве макаки не знают, что XSS надо фильтровать ДО сохранения в БД?

НИКОГДА ПЕРЕД СОХРАНЕНИЯ В БД не надо "ФИЛЬТРОВАТЬ" данные. Данные в базу идут в том виде, в котором они пришли от пользователя. Прогон данных через htmlspecialchars делается исключительно при выводе на html. Ибо разные форматы конечных документов могут иметь свои собственные требования к выводу тех или иных символов. И если в HTML нужно преобразовывать в мнемоники символы, использующиеся в HTML, это не значит, что это нужно делать для формата Word, PDF или чего-то еще.

Там еще много чего такого

Я весь во внимании.

[Северное Сияние]

Одиночка Айс,

Вы бы тоже выбросили всякие endforeach, дебажить заманаешься потом

теперь перепешите пример со скобками вместо альтернативного синтаксиса и вы точно заманаетесь дебажить.

[Виталий Хоменко]

Василий Берестов, где я увидел короткие тэги? В статье

НИКОГДА ПЕРЕД СОХРАНЕНИЯ В БД не надо "ФИЛЬТРОВАТЬ" данные. Данные в базу идут в том виде, в котором они пришли от пользователя.

Упоротость такая забавная вещь. Я начинаю понимать, почему вы с десятком лет опыта сосете лапу. Я даже не буду пытаться вас переубедить

[Северное Сияние]

Виталий Хоменко, а нет, коли ты на себя навешал "лычек", и считаешь себя самым умным, то давай, расскажи мне в чем я не прав.

Мы разрабатываем систему, где некий Виталий Хоменко Tech Lead и Senior PHP Developer (а не пхп-макака) решил, что перед инсертом в БД нужно "ко-ко-ко удалять плохие символы". Ок. Мы внедрили этот чудо-код, скорее всего Виталий напишет strip_tags или иную библиотеку. Все кавычки вида ' и " мы заменим на мнемоники.

Далее, нам потребуется сделать вывод данных в plain/text, например для каких-то отчетов или логов. И вот незадача - в тексте вместо амперсанда, кавычек и, возможно, знаков больше и меньше - мнемоники. Виталий унывать не будет и сделает обратный перегон данных - из мнемоник в символы. Что уже вопиющая глупость, за которую Виталия надо гнать с должности Tech Lead ссаными тряпками.

Далее, бизнес-требования изменились и мы должны принимать от пользователя (администратора) HTML. Но мы же "фильтруем" данные перед инсертом - так построена вся система. В лучшем случае в базе у нас мнемоники, в худшем работает какой-нибудь strip_tags. И опять - система требует переделки.

А все почему? Все от тотального вопиющего непонимания и излишний самоуверенности.

[Виталий Хоменко]

На лычки всем плевать. Важна не лычка, а результат.

Все от тотального вопиющего непонимания и излишний самоуверенности.

Василий Берестов, Вот именно, тут полностью соглашусь: упоротость и непонимание корень всех проблем.

[Северное Сияние]

Виталий Хоменко, ясно, слился.

[Одиночка Айс]

Василий Берестов, в нормальном IDE я ткну в скобку и увижу начало/конец блока

[Северное Сияние]

Одиночка Айс, наверно разработчики многих фреймворков идиоты, что используют для шаблонов альтернативный синтаксис.. ваше право. Я писал и так и так. При альтернативном синтаксисе мне код более понятный.

Answer 2

[Валерий]

<?php
 $i = 0;
    while($i < count($names)) {

     echo "<li>"; //Здесь не было ;
     echo "<div class = \"avatar\">
     <img src=\"images/user.png\">
     </div>";
     echo "<h2>" . $names[$i] . "</h2>";

      echo "<p>Привет!</p>";
       echo "<div class = \"time\">09:15</div>";
    echo"</li>";

    $i = $i + 1;
    }