Правильно ли написана программа из книжки «Задачник по PHP»?

Question

[Александр Черноусов]

В задачнике по PHP нашел такой код

<?php
$visit_counter = 0;
 if (isset($_COOKIE["visitCounter"]) && 
 is_numeric($_COOKIE["visitCounter"])){ 
 $visit_counter = $_COOKIE["visitCounter"]*1; }  // Приращиваем счетчик посещений 
 $visit_counter++;  //Инициализруем переменную для хранения значения последнего посещения // Если соответствующие данные передавались из cookie, сохраняем их в эту переменную 
 $last_visit = ""; 
 if (isset($_COOKIE["lastVisit"]))
 { 
 $last_visit=htmlspecialchars($_COOKIE["lastVisit"], ENT_QUOTES); $last_visit=stripslashes(trim($last_visit)); 
 
 }

 //устанавливаем cookie
 setcookie ("visitCounter", $visit_counter, 0x7FFFFFFF);
 setcookie ("lastVisit", date ("d/m/Y H:i:s"), 0x7FFFFFFF);
 //Выводим информацию о количестве посещений и дате последнего посещения 
 if($visit_counter == 1) { 
 print "<h2>Добро пожаловать!</h2>"; } 
 else { print " <h2> Вы здесь уже $visit_counter раз</h2> <p>Последнее сообщение: $last_visit</p>"; }
 echo $visit_counter;
?>

код не рабочий - понятно что переменная $visit_counter при перезагрузке обнуляется. Подскажите, а как сделать чтобы не обнулялась.

книжку можно посмотреть здесь
https://codernet.ru/books/php/zadachnik_po_php_s_r...

стр. 35
8) Задачи на Cookie Задание 1 : Инициализируйте переменную для подсчета количества посещений. Если соответствующие данные Передавались через cookie сохраняйте их в эту переменную. Нарастите счетчик посещений. Инициализируйте переменную для хранения значения последнего посещения страницы. Если соответствующие данные передавались из cookie , отфильтруйте их и сохраните в эту переменную. Установите соответствующие cookie Задание 2: Выводите информацию о количестве посещений и дате последнего посещения

Comments

[xmoonlight]

Линк на книжку - в студию.

[Александр Черноусов]

xmoonlight, вот эта книжка https://codernet.ru/books/php/zadachnik_po_php_s_r...

[edward_freedom]

alexCHE1974,

понятно что переменная $visit_counter при перезагрузке обнуляется

почему, если она в куках сохраняется?

[Александр Черноусов]

edward_freedom, да точно, но почему-то все равно не работает.

[Александр Черноусов]

всем спасибо - разобрался - код рабочий -

<?php
 if (isset($_COOKIE["visitCounter"])&& 
 is_numeric($_COOKIE["visitCounter"])){ 
 $visit_counter = $_COOKIE["visitCounter"]*1; }  // Приращиваем счетчик посещений 
 $visit_counter++;  //Инициализруем переменную для хранения значения последнего посещения // Если соответствующие данные передавались из cookie, сохраняем их в эту переменную 
 $last_visit = ""; 
 if (isset($_COOKIE["lastVisit"]))
 { 
 $last_visit=htmlspecialchars($_COOKIE["lastVisit"], ENT_QUOTES); $last_visit=stripslashes(trim($last_visit)); 
  }
 //устанавливаем cookie
setcookie ("visitCounter", $visit_counter, 0x7FFFFFFF);
 setcookie ("lastVisit", date ("d/m/Y H:i:s"), 0x7FFFFFFF);
 //Выводим информацию о количестве посещений и дате последнего посещения 
 if($visit_counter == 1) { 
 print "<h2>Добро пожаловать!</h2>"; } 
 else { print " <h2> Вы здесь уже $visit_counter раз</h2> <p>Последнее сообщение: $last_visit</p>"; }
 echo $visit_counter;
 ?>

все из за пробелов перед setcookie

setcookie() задает cookie, которое будет передано клиенту вместе с другими HTTP-заголовками. Как и любой другой заголовок, cookie должны передаваться до того как будут выведены какие-либо другие данные скрипта (это ограничение протокола). Это значит, что в скрипте вызовы этой функции должны располагаться до остального вывода, включая вывод тегов и , а также пустые строки и пробельные символы.

https://www.php.net/manual/ru/function.setcookie.php

если их просто убрать и не выводить код html все работает

[DevMan]

edward_freedom, огня ... требует жопа твоя!

Answer 1

[Araik]

В этой строчке, переменной $visit_counter присваивается значение из $_COOKIE["visitCounter"]

$visit_counter = $_COOKIE["visitCounter"]+1;  // обратите внимание, должно быть +1, а не *1

Которую вы устанавливаете в этой строчке:

setcookie ("visitCounter", $visit_counter, 0x7FFFFFFF);

т.е. по сути код рабочий.

Comments

[Александр Черноусов]

Понял, но у меня почему-то в браузере вот так

Warning: Cannot modify header information - headers already sent by (output started at C:\OSPanel\domains\myproject3.loc\index5.php:7) in C:\OSPanel\domains\myproject3.loc\index5.php on line 22

Warning: Cannot modify header information - headers already sent by (output started at C:\OSPanel\domains\myproject3.loc\index5.php:7) in C:\OSPanel\domains\myproject3.loc\index5.php on line 23
Добро пожаловать!
1

[Александр Черноусов]

вот на этот код:

//устанавливаем cookie
setcookie ("visitCounter", $visit_counter, 0x7FFFFFFF);
setcookie ("lastVisit", date ("d/m/Y H:i:s"), 0x7FFFFFFF);

[Araik]

alexCHE1974,

setcookie() задает cookie, которое будет передано клиенту вместе с другими HTTP-заголовками. Как и любой другой заголовок, cookie должны передаваться до того как будут выведены какие-либо другие данные скрипта (это ограничение протокола). Это значит, что в скрипте вызовы этой функции должны располагаться до остального вывода, включая вывод тегов и , а также пустые строки и пробельные символы.

[Александр Черноусов]

Araik, действительно так

Answer 2

[xmoonlight]

$visit_counter = $_COOKIE["visitCounter"]*1; }  // Приращиваем счетчик посещений

*1 замените на +1

Comments

[Araik]

Блин, заметил, но забыл в решении написать, отвлекся на:

код не рабочий - понятно что переменная $visit_counter при перезагрузке обнуляется.

[Александр Черноусов]

не работает все равно.

[xmoonlight]

alexCHE1974,
1. в кодировке файла скрипта, поставьте Utf-8 without BOM.
2. Ничего не выводите на экран до установки всех заголовков (headers).

[Александр Черноусов]

всем спасибо - разобрался - код рабочий -

<?php
 if (isset($_COOKIE["visitCounter"])&& 
 is_numeric($_COOKIE["visitCounter"])){ 
 $visit_counter = $_COOKIE["visitCounter"]*1; }  // Приращиваем счетчик посещений 
 $visit_counter++;  //Инициализруем переменную для хранения значения последнего посещения // Если соответствующие данные передавались из cookie, сохраняем их в эту переменную 
 $last_visit = ""; 
 if (isset($_COOKIE["lastVisit"]))
 { 
 $last_visit=htmlspecialchars($_COOKIE["lastVisit"], ENT_QUOTES); $last_visit=stripslashes(trim($last_visit)); 
  }
 //устанавливаем cookie
setcookie ("visitCounter", $visit_counter, 0x7FFFFFFF);
 setcookie ("lastVisit", date ("d/m/Y H:i:s"), 0x7FFFFFFF);
 //Выводим информацию о количестве посещений и дате последнего посещения 
 if($visit_counter == 1) { 
 print "<h2>Добро пожаловать!</h2>"; } 
 else { print " <h2> Вы здесь уже $visit_counter раз</h2> <p>Последнее сообщение: $last_visit</p>"; }
 echo $visit_counter;
 ?>

все из за пробелов перед setcookie

setcookie() задает cookie, которое будет передано клиенту вместе с другими HTTP-заголовками. Как и любой другой заголовок, cookie должны передаваться до того как будут выведены какие-либо другие данные скрипта (это ограничение протокола). Это значит, что в скрипте вызовы этой функции должны располагаться до остального вывода, включая вывод тегов и , а также пустые строки и пробельные символы.

https://www.php.net/manual/ru/function.setcookie.php

если их просто убрать и не выводить код html все работает

[xmoonlight]

alexCHE1974, ну как я и написал во 2-м пункте.

[Александр Черноусов]

да все так и есть