Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Wstepna wersja loggera do oceny
q.michal
post
Post #1





Grupa: Zarejestrowani
Postów: 111
Pomógł: 1
Dołączył: 24.12.2013

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


Hej,

W koncu znalazlem troche czasu aby dokonczyc wstepna wersje wlasnej implementacji loggera, ktora pragne sie z Wami podzielic. Jak zwykle licze na konstruktywna krytyke, a jednoczesnie chcialbym takze zapytac bardziej doswiadczonych PHP-owcow o jeden detal. Mianowicie wywolanie date('u'); zwraca same zera zamiast mikrosekund. Udalo sie moze komus to naprawic?


Kod do wgladu: http://wklej.org/hash/b0997746849/
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
destroyerr
post
Post #2





Grupa: Zarejestrowani
Postów: 879
Pomógł: 189
Dołączył: 14.06.2006
Skąd: Bytom

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


Cytat
to moim zdaniem gorzej byc nie moze

Ktoś może mieć inne zdanie, dlatego są różne poziomy i w danym systemie dla danego zdarzenia przyjmujesz określony poziom.
Cytat
Chyba, ze caly serwer padnie, ale wtedy ten emergency na nic Ci sie nie przyda.

Co jeśli mam wiele serwerów?
Cytat
Poza tym jezeli zatrzyma sie baza danych, to notyfikacja powinna przyjsc z innego zrodla.

Nie rozumiem, np. z jakiego źródła jak nie ze skryptu?

Cytat
Nie ma wiec sensu bezmyslnie podazac za wszystkimi trendami i standardami.

Za trendami to masz rację, że nie warto bezmyślnie podążać. Natomiast trend a standard to kompletnie różne rzeczy. Pewnie przestrzenie nazw to też jakiś dziwny standard, ale niekorzystanie z niego prowadzi do takich kwiatków:
  1. $class = 'LoggerProcessor' . ucfirst($processor);

Jeszcze w kwestii bezmyślnego podążania, to kopiowanie bibliotek też się w to wlicza? Twój kawałek kodu moim zdaniem zbyt mocno przypomina inny.

Stałe jak QEVENT_WARNING zdefiniowałbym w zasięgu klasy. Statyczne zależności i magiczne stałe. Abstrakcyjna klasa z pustym konstruktorem i destruktorem. Kod ogólnie trudny do rozbudowy i to chyba największa wada.
Go to the top of the page
+Quote Post
q.michal
post
Post #3





Grupa: Zarejestrowani
Postów: 111
Pomógł: 1
Dołączył: 24.12.2013

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


Cytat(destroyerr @ 18.03.2016, 19:33:05 ) *
Ktoś może mieć inne zdanie, dlatego są różne poziomy i w danym systemie dla danego zdarzenia przyjmujesz określony poziom.

Co jeśli mam wiele serwerów?

To wtedy powinienes dostac notyfikacje z klastra lub innego systemu zapewniajacego HA.
Rozroznij bledy w skrypcie, od bledow w aplikacji / serwerze.

Cytat(destroyerr @ 18.03.2016, 19:33:05 ) *
Nie rozumiem, np. z jakiego źródła jak nie ze skryptu?


Tak jak napisalem: klaster, monitoring, etc.

Cytat(destroyerr @ 18.03.2016, 19:33:05 ) *
Jeszcze w kwestii bezmyślnego podążania, to kopiowanie bibliotek też się w to wlicza? Twój kawałek kodu moim zdaniem zbyt mocno przypomina inny.


Ciekawe. Z jednej strony zarzucasz mi ze nie korzystam z przestrzeni nazw itp, a z 2 ze kopiuje biblioteki.
Go to the top of the page
+Quote Post

Posty w temacie


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

 



RSS Aktualny czas: 15.10.2025 - 12:28