"PHP" / Говнокод #26390 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
public function update_balance()
	{
		$this->balance = $this->balance_at_end_of_day(time());
	}
    
       public function balance_at_end_of_day($time)
	{
		return $this->balance_at_beginning_of_day($time + 24*60*60);
	}

когда окунулся в легаси

Запостил: alucas alucas, (Updated )

Комментарии (10) RSS

    • Гипотезы:

      1. time() + 24*60*60 вернёт завтрашнее время. Оно ещё не наступило.
      2. Баланс обычно пересчитывают не в произвольное время, а в определённый час (конец смены или типа того).
      Ответить
      • баланс на начало завтра = баланс на конец сегодня
        Ответить
    • Смотря, что там в balance_at_beginning_of_day().
      Возможно, вызов update_balance() в неожиданное время может поломать данные.
      Ответить
  • Окунул тебя в прорубь на питерском вокзале, проверь.
    Ответить
  • Главное, чтобы в balance_at_beginning_of_day не было balance_at_end_of_day($time - 24*60*60)
    И вдруг високосная секунда.
    Ответить
    • Это ещё что. Вот когда они наткнутся на летнее/зимнее время — вот тогда у них будет багорище.
      Ответить

Добавить комментарий

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8