Есть ли недочеты в коде?

Question

[ortsuev33]

Есть ли не до четы в моем коде, суть кода в том что я сравниваю две даты и пишу если обе даты сегодня то , сегодня если вчера , то вчера в остальных случаях просто день месяц и год(ну это я позже сделаю а так про сегодня вчера мне главное знать там то я просто выведу дату), все ли я учел?

static function dateConvert($date){

 

	$current_date=new DateTime();
  $cur_date= getdate($current_date->getTimestamp());
$prev_date= getdate($date);
$difday=$cur_date['yday']-$prev_date['yday'];
$difyear=$cur_date['year']-$prev_date['year'];
 if($cur_date['year']==$prev_date['year'] and $cur_date['mon']==$prev_date['mon'] and $cur_date['yday']==$prev_date['yday']){
		  	return 'Сегодня';
 }elseif ($cur_date['year']==$prev_date['year'] and $cur_date['mon']==$prev_date['mon'] and $difday==1) {
		  	return 'Вчера';
}elseif ($cur_date['year']==$prev_date['year'] and   $difday==1) {
		  	return 'Вчера месяц был другой';
 }elseif (  $difyear==1 and $cur_date['mon']==1 and $prev_date['yday']==364 or $prev_date['yday']==365  ) {
		  	return 'Вчера год был другой';
		  }
return ;

	}

Comments

[ortsuev33]

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

Answer 1

[Сергей Соколов]

Код плохо отформатирован и его трудно читать.
Логика не вполне удачна. Лучше использовать разницу между двух дат DateTime::diff() или date_diff() – из двух дат получите объект с числом дней, месяцев, лет между ними. Только сбросьте на 0 время у обеих дат, чтобы работать только с датами. Такой вариант пригодится для более сложного случая, когда вы захотите ещё "Два дня назад", "Послезавтра" и "через неделю"

Comments

[ortsuev33]

Такой вариант нормальный?

static function dateConvert($date){


	$current_date=new DateTime();
	$prev_date=new DateTime();
  $current_date->getTimestamp();
  $prev_date->setTimestamp($date);
//       тут время я не стал устанавливать все равно работает нормально вроде
	// $current_date->setTime(0,0);
	// $prev_date->setTime(0,0);
	 $interval = $current_date->diff($prev_date);
		$date= $interval->format('%a');
	if($date==0){
		return 'Сегодня '.$prev_date->format(' H:i') ;
	}elseif($date==1){
		return 'Вчера '.$prev_date->format('H:i') ;
	}else{
		return $prev_date->format('d-m-Y H:i') ;
	}
 

	}

[Сергей Соколов]

ortsuev33, не поставили время, и попадете, когда у события время 23:55, прошло 10 минут и сейчас 00:05 следующего дня.

[ortsuev33]

Сергей Соколов, я пробовал такую ситуация симулировать вроде нормально, завтра еще проверю, спасибо большое!

Answer 2

[PrAw]

Есть всего 3 значимых исхода:
разница в днях = 0 - сегодня
разница в днях = 1 - вчера
разница в днях > 1 - писать дату.
Зачем какие-то монстроидальные строки писать?

Дальше - работаем с Unixtimestamp?
Разница в датах: (int) ($date2 - $date1) / 86400
без всяких лишних преобразований

Comments

[ortsuev33]

ваше решение для 1 день назад и тд а не для вчера если время только не приводить к нулю как говорил Сергей Соколов или я что то упустил

[ortsuev33]

ааа все был не прав int забыл что до точки уберет все лишнее после точки)а не как люди ))