[Klasa][PHP7] Cache Opcje

[KsaR]

Źródło:
https://github.com/KsaR99/php-cache

Plik: class.Cache.php, zawiera klasę.
Plik: cache.php, to przykładowe zastosowanie.

Klas cache są miliony jednak chciałem stworzyć swoją własną, prostą.

Do oceny:

• Użyteczność
• Co można poprawić żeby sobie/innym życia nie skomplikować
• Jakość kodu (ciągle uczę się nią polepszać)

Zastanawiam się nad metodą do kasacji cache,
Jednak cache zazwyczaj się nie kasuje a podmienia.
Warto dodać? Jeśli tak to prosił bym też o przykład - po co.

Wszelkie uwagi mile widziane.

Ten post edytował KsaR 8.06.2016, 17:51:54

[com]

Dobre IDE podpowie, że typ się zmienił, a Storm używa phpDoca do podpowiadania, tylko mnie rażą te nazwy typów UpperCasem.

No ale skoro może spowalniać to poco wgl ja trzymać jako opcje?

Do tego throw Exception() przy użyciu namespace może wyrzucać błąd nieznalezienia klasy.

Nawet nie może tylko wyrzuci

komentarze to nie

[PHP] pobierz, plaintext 
/**
*
*
*/ 
[PHP] pobierz, plaintext 

tylko

[PHP] pobierz, plaintext 
/**
 *
 *
 */
[PHP] pobierz, plaintext 

No i czemu taka archaiczna konwencja nazewnictwa?
Gdzie PSR

Nazywaj jakoś sensownie te commity

Z ścieżki i innych zrobiłbym VO, bo czemu klasa cache ma za to odpowiadać, czy one są poprawne, jak to nie jej rola.

I poco ustawiać wartości domyślne, parametrom którym wymuszasz ich zmianę?

Zaraz, ja muszę stworzyć sobie plik żeby do niego pisać?

poza tym else to zło, naprawdę nie można było

[PHP] pobierz, plaintext 
if (!isset($path) || !file_exists($path)) {
      throw new \Exception('bla bla');
}
 
$this->path = $path;
[PHP] pobierz, plaintext 

Btw treść commita to nie komentarz, bo wpisujesz tam to co ma znaczenie i utrudniasz sobie i innym czytanie commitów

No i nazywaj te parametry tak żeby mówiły co robią, a nie np $compress - co sugeruje że coś jest skompresowane a nie wskazuje na flage itp

Skoro używasz dobrodziejstw php 7 to poco sprawdzać isset($compress) && $compress bool to zawsze bool

zrób sobie jakaś metodę do budowania właściwej ścieżki, bo tak ciężko to testować/refaktorwać potem

Zrobiłem Ci pull requesta, bo za dużo tego było

Ten post edytował com 8.06.2016, 20:58:01