1. PHP / Говнокод #8794

    +159

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    // Strip out the embedded timestamp
if ( ! preg_match("/(\d+TS--->)/", $cache, $match))
{
  return FALSE;
}

// Has the file expired? If so we'll delete it.
if (time() >= trim(str_replace('TS--->', '', $match['1'])))
{
...

    CodeIgniter → System → Core → Output.php
    Это чтение временной метки в кэш-файле. Почему бы в регулярке в "карман" не взять только \d+?

    Запостил: Andretti, 10 Декабря 2011

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

    • ava ZX_Spectrum 10.12.2011 19:09 # +2

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

      • ava Sulik78 10.12.2011 20:46 # −1

        json? нет не слышал
        Ответить

        • ava ZX_Spectrum 10.12.2011 21:09 # +1

          Городить json для этих целей, на мой взгляд, тоже не лучший вариант, если там будет храниться фактически 2 поля - временная метка (размер - 10 байт) и контент (100500 байт).
          Лучше хранить отдельно контент и отдельно правила кэширования.
          Ответить

          • ava nikita2206 10.12.2011 22:03 # 0

            да господи, просто читаем первые 10 байт из файла. и записываем соот-но тоже туда же
            Ответить

            • ava istem 11.12.2011 12:51 # 0

              Там что, время в контент файла сохраняется?
              там нет проверки по дате изменения файла?
              Ответить

              • ava nikita2206 11.12.2011 12:52 # 0

                видимо нет
                Ответить

              • ava ZX_Spectrum 11.12.2011 16:12 # 0

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

    • ava Lure Of Chaos 11.12.2011 15:16 # 0

      filemtime, блеать
      Ответить

    • ava anton_slim 14.12.2011 00:55 # +1

      .
      Ответить

      • ava anton_slim 14.12.2011 00:57 # 0

        а вообще странный код кеша :) я даже оправдание найти не могу этому, типа вначале каждого файла есть метка TS---> ?
        Ответить

        • ava Andretti 14.12.2011 10:44 # 0

          Да, именно так. Но меня больше восхитило как автор вытягивает эту метку из файла. Еще раз хочу отметь, что если бы в "карман" в регулярке бросили только число, то во втором условии отрезать маркер, да еще и trim применять (чтоб наверняка), не было обходимости...
          Ответить

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