Как можно улучшить код отображения даты выпуска новости?

Question

[yapaofficial]

<?php 
function time_ago($datetime) {
    $interval = date_create('now')->diff($datetime);

    if ($interval->y >= 5) {return $interval->y.' лет назад';}
    if ($interval->y >= 2) {return $interval->y.' года назад';}
    if ($interval->y >= 1) {return $interval->y.' год назад';}

    if ($interval->m >= 5) {return $interval->m.' месяцев назад';}
    if ($interval->m >= 2) {return $interval->m.' месяца назад';}
    if ($interval->m >= 1) {return $interval->m.' месяц назад';}

    if ($interval->d >= 5) {return $interval->d.' дней назад';}
    if ($interval->d >= 2) {return $interval->d.' дня назад';}
    if ($interval->d >= 1) {return $interval->d.' день назад';}

    if ($interval->h >= 5) {return $interval->h.' часов назад';}
    if ($interval->h >= 2) {return $interval->h.' часа назад';}
    if ($interval->h >= 1) {return $interval->h.' час назад';}

    if ($interval->i >= 5) {return $interval->i.' минут назад';}
    if ($interval->i >= 2) {return $interval->i.' минуы назад';}
    if ($interval->i >= 1) {return $interval->i.' минуту назад';}
 
    return $interval->s.' секунд';
}

$as1 = time_ago(new DateTime('2023-01-07 18:29:50')); // 29 лет

echo "$as1";

 ?>

Comments

[Vfreelance]

при ->h = 21 неправильный код

[ThunderCat]

Vfreelance, как и при 22 и 23. Ну и с десятками минут/дней/лет тоже. Остаток от деления нацело в помощь.

Answer 1

[ThunderCat]

ИМХО не надо насиловать мозг когда задача решена "не изящно, но практично". Приведенное решение вполне аккуратное и рабочее, из мелких косяков - не учтено что окончания меняются через десяток, что легко поправляется делением с остатком на 10 (%). Будет желание повозиться с более красивым решением - на досуге поколупаете, полезное упражнение...

Comments

[Дмитрий]

18 if'ов подряд - это аккуратное решение?
А вы точно на PHP программируете, а не просто пишете хрень о нём на Тостере? ;)

[ThunderCat]

Дмитрий,

18 if'ов подряд - это аккуратное решение?

В чем беда? В том что вы слышали что много ифов в общем случае плохо? Да, это малоприемлемо для объектного стиля, но для хелперов вполне нормально. Возможно вы привыкли что писать код это только "взял библиотеку, вызвал метод, передал параметры, готово", но в общем случае это не так. Берем тот же карбон, открываем сырцы и видим кроме длиннющих свитчкейсов еще десяток висячих садов из иф елсиф... Тоже выкинуть? Или зарефакторить и разбить на методы?

Можно написать и через массив и формат, через цикл, может еще чего можно придумать, но в целом это читабельный код, простая реализация "в лоб", вполне рабочая и имеющая задатки для оптимизации / упрощения. Это не мой код, и не мой вопрос, это мое мнение о коде новичка.

[ThunderCat]

Дмитрий, кстати, нормальное же решение предложили, нафига были эти пляски вокруг библиотек?

Answer 2

[Дмитрий]

Если уж речь идёт только о том, чтобы выдавать слово в нужном склонении в зависимости от входного числа, можно воспользоваться примерно такой функцией:

function getCase ($number, $params) : string {
    $cases = [2, 0, 1, 1, 1, 2];
    return $number . ' ' . $params[($number % 100 > 4 && $number % 100 < 20) ? 2 : $cases[min($number % 10, 5)]];
}

echo getCase(1, ["год", "года", "лет"]); // "1 год"
echo getCase(4, ["месяц", "месяца", "месяцев"]); // "4 месяца"
echo getCase(25, ["день", "дня", "дней"]); // "25 дней"

Answer 3

[Dark_Dante]

Тебе нужен Carbon https://carbon.nesbot.com/docs/#api-localization

Comments

[ThunderCat]

поколение потребления...

[Sanes]

ThunderCat, чем тебя не устраивает использование библиотек?

[ThunderCat]

Sanes, 20 строчек кода и 10 минут неспешного набора, если делать это в первый раз. Зачем здесь библиотека? Понимаю если бы с датовременем было действительно много работы, но тут же реально фигня вопрос...

[Sanes]

ThunderCat, написал бы, зачем умничать.

[ThunderCat]

Sanes, так а че писать то, код нормальный, пару косяков поправить и в продакшн. Для озвученной задачи то что надо.

[Дмитрий]

ThunderCat, а завтра "озвученная задача" расширится и дополнится новыми условиями и требованиями. И послезавтра, и ещё N раз. В итоге твой велосипед с вероятностью, близкой к 100%, превратится в уродливое неподдерживаемое спагетти. В то время как библиотека сто тысяч раз протестирована и содержит всё необходимое доля комфортной работы.

[Dark_Dante]

Дмитрий, Более того, если будет другой разраб код смотерть, он не будет сидеть с недоуменным видом "А что это, тут собственно происхоит", а столкнувшись с тем же карбоном на других проеках ранее, сходу вкурит - да тут все просто, чел дату локализует. И сможет это быстро поправить, расширить, улучшить, углубить и тд.

[Дмитрий]

Dark_Dante, да это просто прописные истины)
Я понимаю, когда разработка ведётся сугубо в образовательных, дидактических целях. Тогда да, полезно пописать велосипеды. Но если это коммерческий продакшн, такие подходы в нем категорически неприемлемы.

[shurshur]

ThunderCat, а когда захочется тем же кодом поддерживать узбеков, финнов и каталанцев, то сразу же станет понятно, насколько неудобны рукодельные реализации для решения такиз задач.

[ThunderCat]

Дмитрий, Когда завтра окажется что надо еще что-то делать, тогда есть смысл поискать другое решение, возможно что-то готовое. Если оно не будет заключаться в еще 1 строке кода. Культ библиотек имеет свою обратную сторону - вы тянете 50 репозиториев с зависимостями ради 2 строчек кода. Естественно нужно понимать, это разовая задача и больше ничего не надо, или с подобной задачей будет связано еще 30 мест в проекте, и стоит подключить надцать библиотек.

shurshur, Да, мультияз удобен в готовом виде, именно в таком формате вопроса я рекомендовал бы библиотеку. Спорить о том нужна ли библиотека или нет можно только зная проект и его нюансы. На поставленный вопрос ИМХО более адекватно написать 1 хелпер.