Как более профессионально написать код

[pompiduskus]

Есть 2 решения одной и той же задачи , Подскажите как будет более правильно его реализовать ? И почему ?
................
Первый вариант

PHP код:

if(!isset($_COOKIE['user_data'])){

    $user_data['date'] = date("h:i.s");
    $user_data['cnt'] = '1';

setcookie("user_data", serialize($user_data), time()+20);
}else {
$user_data = unserialize($_COOKIE['user_data']);
    $last_visit = $user_data['date'];
    
    $user_data['date'] = date("h:i.s");
    $user_data['cnt']++;    
    
setcookie("user_data", serialize($user_data), time()+20);
} 


...............
Второй вариант

PHP код:

$visit_counter = 0;
if(isset($_COOKIE['visitCounter']) && is_numeric($_COOKIE['visitCounter'])){
    $visit_counter = $_COOKIE['visitCounter'] * 1;
}
$visit_counter++;

$last_visit = '';
if(isset($_COOKIE['lastVisit'])){
    $last_visit = (int)$_COOKIE['lastVisit'];
}
setcookie('visitCounter', $visit_counter, 0x7FFFFFFF);
setcookie('lastVisit', date('d/m/Y H:i:s'), 0x7FFFFFFF); 


[Gorychev]

Первый вариант больше нравится, чисто с эстетической точки зрения, хотя по сути они одинаковы. Только почему кук на 20 сек ставите? А во втором варианте на хрен знает сколько? Ставьте на месяц, этого хватит.

[pompiduskus]

Это просто пример, не важны сами данные .

[Andkorol]

1-й вариант:
- присутствует ненужный копипаст кода:

PHP код:

$user_data['date'] = date("h:i.s");
//...
setcookie("user_data", serialize($user_data), time()+20); 


- нет проверки, что unserialize вернула массив, и что в нем есть нужный нам элемент:

PHP код:

$user_data = unserialize($_COOKIE['user_data']);
$last_visit = $user_data['date']; 


2-й вариант:
- проверка isset - бесполезна в данном контексте, ибо допускает нулевое значение - а это ничем не отличается от $visit_counter = 0;
- $last_visit - задается строка с датой и временем, но потом зачем-то выполняется приведение типа к (int):

PHP код:

if(isset($_COOKIE['lastVisit'])){
    $last_visit = (int)$_COOKIE['lastVisit'];
}
//...
setcookie('lastVisit', date('d/m/Y H:i:s'), 0x7FFFFFFF); 


Более компактно:

PHP код:

$last_visit = (!empty($_COOKIE['lastVisit'])) ? $_COOKIE['lastVisit'] : ''; 
$visitCounter = (!empty($_COOKIE['visitCounter'])) ? ++$_COOKIE['visitCounter'] : 1; 
setcookie("lastVisit", date('Y-m-d H:i:s'), time()+20); 
setcookie("visitCounter", $visitCounter, time()+20); 


Ну и в функцию всё это вынести, конечно же.

[pompiduskus]

Понятно, Спасибо .