[2010–02–11] Cache a ukládání „callbacků“ kvůli dramatickému zrychlení

wow tak to je mazec! musím kouknout jak je to implementováno porotože to je hodně cool hračička…

Rozhodování, zda jde o data nebo callback, může být trochu zvláštní.

• „Do keše lze ukládat jakékoliv struktury, nemusí to být jen řetězec.“ ( https://doc.nette.org/cs/caching )
• Do callbacků může patřit i array(‚Foo‘, ‚bar‘) a další.

Aha, to je ale celkem důležité zmínit, protože jinak je to celkem WTF chování.

Obalovat closuru callbackem mi přijde poněkud zbytečný…

Teď mně tak napadá, toto asi nebude fungovat s DummyStorage? Nebo se mýlím?

Hlásí se někdo dobrovolně o vytvoření ArrayDummyStorage? Na druhou stranu, přes takovýto storage už nezjistíš, co se stane, pokud aplikaci někdo pod rukama smaže tempy. Ale pravda, je to dost málo pravděpodobné.

ke zprovoznění postačí i tohle (proti verzi 0.9.4 pro PHP 5.2):

diff --git a/Caching/DummyStorage.php b/Caching/DummyStorage.php
index 9f4d0b5..6cbb885 100644
--- a/Caching/DummyStorage.php
+++ b/Caching/DummyStorage.php
@@ -21,6 +21,11 @@
 class DummyStorage extends Object implements ICacheStorage
 {

+	/** @var array */
+	private $data = array();
+
+
+
 	/**
 	 * Read from cache.
 	 * @param  string key
@@ -28,6 +33,8 @@ class DummyStorage extends Object implements ICacheStorage
 	 */
 	public function read($key)
 	{
+		if (isset($this->data[$key])) return $this->data[$key];
+
 		return NULL;
 	}

@@ -42,6 +49,16 @@ class DummyStorage extends Object implements ICacheStorage
 	 */
 	public function write($key, $data, array $dp)
 	{
+		if (!empty($dp[Cache::TAGS]) || isset($dp[Cache::PRIORITY]) || !empty($dp[Cache::ITEMS])
+			|| !empty($dp[Cache::EXPIRE]) || !empty($dp[Cache::SLIDING])) {
+			throw new NotSupportedException('Tags, priority, dependent items, expiration and sliding expiration are not supported by DummyStorage.');
+		}
+
+		if ($data instanceof Callback || $data instanceof Closure) {
+			$data = $data->__invoke();
+		}
+
+		$this->data[$key] = $data;
 	}


@@ -53,6 +70,7 @@ class DummyStorage extends Object implements ICacheStorage
 	 */
 	public function remove($key)
 	{
+		unset($this->data[$key]);
 	}


@@ -64,6 +82,7 @@ class DummyStorage extends Object implements ICacheStorage
 	 */
 	public function clean(array $conds)
 	{
+		$this->data = array();
 	}

 }

stejně se mi to ale celé nelíbí. je to hack a porušení SRP. úložiště se teď musí starat nejen o ukládání dat, ale i o jejich vytváření

Aktualizoval jsem první příspěvek.

Zavolání callbacku se přesunulo mimo Storage, takže je v režii třídy Cache. Zároveň metoda save() vrací uložená data, tak se k nim dá dostat i bez nutnosti z cache číst.

Je to pecka. Díky.

Dovolím si dotaz ohledně callbacku – většinou mám metody modelu v podobném stylu:

class ArticleModel
{
	/** @var Cache */
	protected $cache;

	private $table = 'clanky';



	// inicializace cache, apod.



	public function getArticle($id)
	{
		$key = 'article-' . $id;

		if (!isset($this->cache[$key])) {
			$article = $this->connection
					->select('*')
					->from($this->table)
						->where("[id] = %i", $id)
					->fetch();

			if ($article) {
				$this->cache->save($key, $article, array(
					Cache::EXPIRE => time() + Tools::WEEK,
					Cache::SLIDING => TRUE,
				));
			}

		} else {
			$article = $this->cache[$key];
		}

		return $article;
	}

	// ...
}

Tzn., že kdybych chtěl tento způsob nahradit novým, výše doporučovaným, musel bych nějakým způsobem callbacku předat parametr $id, na kterém metoda lpí… Existuje nějaké řešení (nevím jak pro closures, ale osobně vyvíjím na PHP 5.2)?

ten patch se zatim nejak neobjevil nikde, ne? (stahl jsem si 0.9.5 i 1.0 pro PHP3 a kde nic tu nic… ani v api dokumentaci)

David Grudl napsal(a):
Tobě to s DummyStorage nefunguje?

abych pravdu rekl, nezda se mi, ze to neco dela… pouzivam to s callbacky a mam takovy pocit, ze se callback ani nespusti, cimz nedojde ani k vytvoreni nejakych dat… takze:

1. nic se mi necachuje – coz je u DummyStorage cilem
2. bohuzel se mi data z callbacku neposlou do aplikace – protoze se asi ani zadne nevyrobi

… ale musim se priznat, ze jsem to do hloubky nestudoval, mel sem na spech, tak sem DummyStorage simuloval neustalym restartovanim memcached