Правильно ли я сделалл кеш?

Question

[Владимир Шикльгрубер]

Привет. За часик сделал кеш на файлах. Хочу узнать по поводу качества моего велосипедика :D
Зарание спасибо. Вот сам код
P.S отредактировал по вашим советам

class Cache {

  function fileTo($file, $time) {
    $filename = 'storage/cache/' . $file;
    if (file_exists($filename)) {
      if ((time() - $time) < filemtime($filename)) {
        return true;
      } else {
        return false;
      }
    } else {
      return false;
    }
  }

  function fileEcho($file) {
    $file = 'storage/cache/' . $file;
    return file_get_contents($file);
  }

  function fileNewCache($file, $data) {
    $file = 'storage/cache/' . $file;
    $fp = fopen($file, "w");
    flock ($fp,LOCK_EX);
    fwrite($fp, $data);
    flock ($fp,LOCK_UN);
    fclose($fp);
  }

  function fleDelete($file) {
    $file = 'storage/cache/' . $file;
    unlink($file);
  }

}

Answer 1

[Сергей]

Это не кэш, это просто чтение-запись текстовых данных. Где хотя бы простая сериализация? Мой Вам совет - почитайте исходники фреймворков, там реализован всякий разный кэш (файловый, редисковый ит.д), хотя бы Yii. Посмотрите как там сделано, переделайте свой велосипед, сделайте так, чтобы он работал и выглядел правильно, а потом никогда не используйте его в реальных проектах.

Answer 2

[index0h]

У вашего велосипеда есть узкие места:
1. Вы в принципе не думали про то, что несколько потоков могут затребовать файл, и в этот момент один из них его может удалить, или обновлять.
2. Каталог storage должен быть доступен для записи, в случае чего - нужно бросать исключение на этот счет.
3. Вот эта функция не должна существовать))

function fileEchoCache($file) {
    $file = 'storage/cache/' . $file;
    echo file_get_contents($file);
  }

Она не входит в обязанности менеджера кэша.
4. В имя файла может входить что угодно например: "some \\ // \n \tcool file\r\b\b\a HeRe"
5. Посмотрите https://github.com/php-fig/fig-standards/blob/mast...
6. Если у вас будет реально много файлов - это станет источником нагрузки на файловую систему. Вложенные каталоги решают эту проблему.

Comments

[Владимир Шикльгрубер]

сейчас всё переделаю под каталоги. Для каждого юзера генерировать буду папку. Сделаю блокировку файла сейчас.

[Владимир Шикльгрубер]

а как тогда выводить кеш?

[index0h]

> Для каждого юзера генерировать буду папку
плохая идея

> а как тогда выводить кеш?
Выводить в любом другом месте, но точно не в кэше.

Посмотрите в сторону mogilefs

[Alexander Litvinenko]

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

[index0h]

Alexander Litvinenko Смотря для какой цели.
Если это именно просто кэш для внутреннего потребления - безусловно идея бредовая. Но если для отдачи статики через например тот же X-Sendfile заголовок - почему нет?

[index0h]

Как пример: сбор большого количества less с файловым кэшом (ключи - md5 от less файла + его зависимостей) - вполне норм подход, только по времени он не должен удаляться и используется в момент диплоя.

[Alexander Litvinenko]

index0h: изначально автор сделал простой echo, потому ни о каких заголовках речь не идет, а вообще это должно быть внутри самой лесс библиотеки. Более того, лесс можно собирать и с помошью JS.

[index0h]

Alexander Litvinenko Можно и с js. Автор спросил об ошибках. Файловый кэш имеет право на существование. Если у него нагрузка будет 1.5 человека в день - в принципе все равно, что файловый, что например в redis.

Answer 3

[D']

А где обработка ошибок?
Названия функций ну вот вообще ни о чем не говорят.
А если я сделаю вот так:
$cache->fileEchoCache('../../../../config.php');
Что будет?

unlink($file); без проверки пути...

А если я захочу использовать кеш в памяти, заместо файлового? Мне по всему проекту бегать искать и заменять вызовы?
Лучше удали это.

Comments

[Владимир Шикльгрубер]

название функций довольно понятны. Первое слово тип кеша Второе действие третье красота :D

[D']

Владимир Шикльгрубер: и что мне помешает сделать $cache->fileEchoCache('../../../../../../../../etc/passwd'); ?

[D']

Владимир Шикльгрубер: Почему ничего не будет? Как мне это понять?

>название функций довольно понятны. Первое слово тип кеша Второе действие третье красота :D
Зачем такие сложности? Названия должны отражать что конкретно делает данная функция.
Почему не сделать нормальный интерфейс, с функциями get, put, exists, etc...?
>Первое слово тип кеша
Тип кеша должен быть указан в названии класса, а не в каждой функции.
И добавлять Cache к каждой функции - не имеет смысла.

Почему этот класс выводит содержимое файлов на экран? А если мне не нужно выводить, что тогда?
Может добавим в этот класс еще проверку авторизации? А что, по моему самое место.

[Владимир Шикльгрубер]

D' Normalization: как исправить эту проблему ?

[D']

Владимир Шикльгрубер: этот класс ниочем. Это просто набор каких-то функций для чтения\записи файлов.
Его нужно выкинуть, и написать нормальную реализацию, расширяемую, и с нормальной обработкой ошибок.

[Владимир Шикльгрубер]

D' Normalization: вывод исправил. А зачем авторизация то? Слово Cache убрал.

[D']

Владимир Шикльгрубер: про авторизацию - это был сарказм. Класс который занимается работой с кешем, не должен ничего выводить ни на какой экран. Это не его работа. Он должен либо положить, либо забрать данные из кеша и вернуть их.

[Владимир Шикльгрубер]

D' Normalization: вывод исправил. Теперь оно возвращает. return file_get_contents($file);

[D']

Владимир Шикльгрубер: а если файла нету? А если к нему нету доступа? А если там что угодно, но не путь к файлу?

Нет смысла дальше обсуждать этот код. Это поделка студента первого курса бухучета, который только вчера узнал что такое PHP.

Answer 4

[Дмитрий Ковальский]

Ну технически кэш - это данные которые хранятся в памяти. Поправьте если я ошибаюсь. Ваше решение завязано на имя файла(а они могут и повторяться для разных файлов(всякие image0001.jpg) . Также операции чтения/записи на диск - не бесплатны.А порой и дорогие для больших файлов. Ну и потокобезопасность может страдать от такого кода. Хотя я почти не знаком с php. Возможно это и решение.

Comments

[Владимир Шикльгрубер]

есть файловый кеш которых хранить довольно долго надо. Например сгенеророваная статья с бд

[Владимир Шикльгрубер]

для остального юзаю мем кеш

[Дмитрий Ковальский]

Владимир Шикльгрубер: Ну коллеги в других комментариях указали на узкие места более точно.