Как правильно спроектировать тарифные планы?

Question

[lexstile]

Ребята, подскажите разумное решение.
Вопрос: в какой момент отключать тариф? (active = 0)

1 решение (предлагает мой ментор):
Дату valid_till делать всегда на конец дня - 23:59:59 (today()->addDays(30)->endOfDay()).
В 00:00:00 - по крону проходимся по БД и отключаем просроченные тарифы.

2 решение (предлагаю я):
Дату valid_till брать текущее время в UTC (now()->addDays(30)).
Деактивируем тариф только при переходе на другой, либо при продлении текущего (т. е. только после действия пользователя).

Comments

[mayton2019]

В топике мало информации. Как-же мы определим правильные тарифные планы? Нужен какой-то критерий. Выгода. Надежность. Экономия вычислительных ресурсов.

[maksam07]

Дату valid_till делать в то же время, что и у пользователя

А если у пользователя 2999 год на компьютере?

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

Первое правило программиста: не верить данным, пришедшим от пользователя...

[lexstile]

maksam07, я неправильно написал - время в UTC

[lexstile]

maksam07, время в UTC сервера, от пользователя таких данных не берем (исправил).

[maksam07]

lexstile, если это время берется на сервере, то проблем не вижу. Как сказал Одиночка Айс - не верить данным, пришедшим от пользователя...

На счет первого и второго варианта - существенной разницы не вижу. Или крон создается на одно время (1 вариант), или на ежеминутной основе (2 вариант). ут наверное больше удобство для пользователя:
1. Когда активировал - деакцивация в полночь, через месяц
2. Когда активировал, в то же время деактивация, через месяц

[maksam07]

lexstile, на вашем месте я бы поинтересовался мотивами ментора. Он, как более опытный человек, может сказать, почему его вариант удобнее

[lexstile]

maksam07, в первом варианте пользователь может остаться без тарифа на какое-то время (пока не отработает крон).
То есть, особой разницы нет, как вы сказали? - Выбирать наиболее удобное для себя?

[maksam07]

lexstile,

в первом варианте пользователь может остаться без тарифа на какое-то время (пока не отработает крон)

так у вас по крону должен ОТКЛючаться тариф, а не ВКЛючаться

[lexstile]

maksam07, он топит за то, что за активность тарифа должно отвечать одно поле в таблице - active.
В моем же кейсе (второй вариант) - это будут 2 поля как минимум - valid_till и active.
То есть, пока я не переведу тариф в active = 0 - он считается действующим, хоть и просроченным в определенный момент (просрочку проверять отдельным методом просто).

[lexstile]

maksam07, все верно
в 23:59:59 текущего дня тариф отключился (точнее дата valid_till истекла).
а в 00:00:00 следующего дня крон пошел отключать, но не факт, что он очень быстро отработает.

И этот промежуток как раз и будет интервалом без тарифа.

[maksam07]

lexstile, я вообще запутался.
1. Отключать тариф по крону в 00 00 00 у всех, у кого valid_till просрочился
2. Отключать тариф только при: переходе на другой или продление нового
?
У меня вопрос по второму варианту. То есть, если пользователь НЕ переходит на другой и НЕ продлевает, то текущий тариф работает бесконечно долго?

[lexstile]

maksam07, он не совсем работает.
в первом варианте (от ментора), достаточно проверить active - это будет говорить о том, что тариф и просрочен в том числе или, наоборот, не просрочен.
Но есть микро-проблема, о которой я писал выше.
Плюс, я не могу вернуть на фронт текущий тариф (уже просроченный), не на что опереться.
Приходится извращаться и возвращать отдельным полем, если не подключен тариф, то возвращаем последний отключенный:

public function lastTariff() {
        return $this->hasOne(ProjectTariff::class)->NotActive()->orderBy('id', 'desc');
    }

А в моем случае проверяется два поля - active и valid_till.
Например, у нас есть админка и публичная часть.
В админке я владельцу возвращаю его просроченный тариф, т. к. active = 1 (в случае 1 варианта, мне уже не вернуть последний тариф без дополнительных манипуляций - а-ля - получить последний активный тариф).
А в публичной части для всех пользователей закрываю доступ, т. к. valid_till просрочена.

public function scopeActiveNow($query) {
        return $query
            ->where('active', 1)
            ->where('valid_from', '<=', now())
            ->where('valid_till', '>=', now());
    }

[lexstile]

maksam07,

1. Отключать тариф по крону в 00 00 00 у всех, у кого valid_till просрочился
2. Отключать тариф только при: переходе на другой или продление нового

Под отключением я подразумеваю active = 0.
А если active = 1, а valid_till просрочена, тариф в этом случае носит только информативный характер (для админки).
Такого кейса не может быть в том, что предлагает мой ментор.
Но чтобы отобразить на фронте, что тариф закончился, приходится возвращать последний активный тариф.

[maksam07]

lexstile, что-то очень сложно и запутано. Для чего в вашем варианте where('active', 1), если у вас есть valid_till и можно по нему проверять, превышает ли время окончания тарифа, текущее время на сервере. Если да, то тариф активе, нет - просрочен

[lexstile]

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

И есть кейс, например, идёт переход на один тариф, он действует до конца года, но через месяц я решаю перейти на другой, первый я отключаю - active = 0, перехожу на другой.
Но в течение этого года я могу вернуться на первый тариф и он будет действовать до конца года.