Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

3 Stron V  < 1 2 3  
Reply to this topicStart new topic
> [Klasa][PHP7] Cache
com
post 10.06.2016, 11:55:45
Post #41





Grupa: Zarejestrowani
Postów: 3 032
Pomógł: 366
Dołączył: 24.05.2012

Ostrzeżenie: (0%)
-----


Jeśli już bardzo chcesz to typ jako stała, nigdy 0,1 wink.gif
Go to the top of the page
+Quote Post
KsaR
post 10.06.2016, 22:25:14
Post #42





Grupa: Zarejestrowani
Postów: 520
Pomógł: 102
Dołączył: 15.07.2014
Skąd: NULL

Ostrzeżenie: (0%)
-----


Cytat(com @ 10.06.2016, 12:55:45 ) *
Jeśli już bardzo chcesz to typ jako stała, nigdy 0,1 wink.gif

Pozostałość po testach biggrin.gif,
Już poprawiłem.

PPs. Co do "VO"
Czyli mam zrobić oddzielną klasę do validacji oraz zwracania ścieżki i tam zrobić coś jak w Cache "fullPath()"?


--------------------
Go to the top of the page
+Quote Post
com
post 11.06.2016, 16:42:31
Post #43





Grupa: Zarejestrowani
Postów: 3 032
Pomógł: 366
Dołączył: 24.05.2012

Ostrzeżenie: (0%)
-----


Ale jako parametr ma być stała przekazywana również i no ostatecznie mogło by tak też być wink.gif
Go to the top of the page
+Quote Post
LowiczakPL
post 11.06.2016, 17:52:30
Post #44





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

Ostrzeżenie: (0%)
-----


Czy czasami nazwy stałych nie powinny być napisane drukowanymi literami?

http://php.net/manual/en/language.oop5.constants.php


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
com
post 11.06.2016, 17:56:53
Post #45





Grupa: Zarejestrowani
Postów: 3 032
Pomógł: 366
Dołączył: 24.05.2012

Ostrzeżenie: (0%)
-----


Powinny smile.gif

Zgodnie z PSR, bo php dopuszcza, bo to tylko konwencja smile.gif
Go to the top of the page
+Quote Post
LowiczakPL
post 11.06.2016, 18:13:03
Post #46





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

Ostrzeżenie: (0%)
-----


Dodatkowo dałbym strategię dla typów kesza, co zlikwidowało by IF-y,

zamiast liczbowej nazwy typu przekazywał bym nazwę strategii.


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
com
post 11.06.2016, 18:31:33
Post #47





Grupa: Zarejestrowani
Postów: 3 032
Pomógł: 366
Dołączył: 24.05.2012

Ostrzeżenie: (0%)
-----


Owszem można by zrobić fabrykę nawet wink.gif

I nie żadnego elseif

  1. if (!isset($path)) {
  2. throw new \InvalidArgumentException('Please specify path for cache.');
  3. } elseif (!file_exists($path)) {
  4. throw new \DomainException('Path:'.PHP_EOL."„{$path}”".PHP_EOL.'Does not exists.');
  5. }


Dlaczego jesteś taki uparty, skoro Pyton_000 napisał tak
  1. if(!isset($path)) {
  2. throw new \Exception('Please specify path for cache.');
  3. }
  4.  
  5. if (!file_exists($path)) {
  6. throw new \Exception('Path:'.PHP_EOL.$path.PHP_EOL.'Doesn\'t exists.');
  7. }
  8.  
  9. $this->path = $path;


I co to wgl za „ ” są ohmy.gif

pomijając to że nie stosujesz się do tego co pisał nospor apropo poprawności komunikatu wink.gif

a sam path jako byt, to nie Domain wink.gif
Go to the top of the page
+Quote Post
KsaR
post 11.06.2016, 18:41:33
Post #48





Grupa: Zarejestrowani
Postów: 520
Pomógł: 102
Dołączył: 15.07.2014
Skąd: NULL

Ostrzeżenie: (0%)
-----


Cytat(com @ 11.06.2016, 19:31:33 ) *
Owszem można by zrobić fabrykę nawet wink.gif

I nie żadnego elseif

(...)

I co to wgl za „ ” są ohmy.gif

pomijając to że nie stosujesz się do tego co pisał nospor apropo poprawności komunikatu wink.gif

a sam path jako byt, to nie Domain wink.gif

* Co do strategy, factory - nie widzę takiej potrzeby dla takiej mikro-klasy.

* Poprawie.

* Cudzysłowia. biggrin.gif

* Jak mam się stosować do czegoś czego nie rozumiem ?

* Zmienie na Exception. ;d


--------------------
Go to the top of the page
+Quote Post
com
post 11.06.2016, 18:49:38
Post #49





Grupa: Zarejestrowani
Postów: 3 032
Pomógł: 366
Dołączył: 24.05.2012

Ostrzeżenie: (0%)
-----


chodzi o to, że tak jest poprawnie:
throw new \Exception("Path '$path' doesn't exist.");

No albo się pisze czysty kod albo ma byle polska nie zginęła tongue.gif

Nie stosował bym egzotycznych znaczków bo rożnie mogą to przeglądarki interpretować, poza tym
to tylko błąd, poco jakieś eksperymenty bo można smile.gif

Co do stałych jeszcze bym się zastanowił czy ten zwrot jako jest nam potrzebny smile.gif
Go to the top of the page
+Quote Post

3 Stron V  < 1 2 3
Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 28.03.2024 - 22:00