 Wstepna wersja loggera do oceny |
| Wstepna wersja loggera do oceny |
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/ |
 |
 
|
 |
|
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 3 034 Pomógł: 366 Dołączył: 24.05.2012 Ostrzeżenie: (0%)
|
Zanim kolejny raz zaczniesz zgrywać mądrego polecam przeczytać całość, tam wyraźnie zostało napisane dlaczego mamy 8 poziomów logowania.
Cytat The LoggerInterface exposes eight methods to write logs to the eight RFC 5424 levels (debug, info, notice, warning, error, critical, alert, emergency). Całość oparta jest na standardzie opisany w RFC. To co mówisz owszem, te mechanizmy powinny tam zadziałać, ale jest standard i interfejs musi być z nim zgodny. Pomijając już sam fakt iż piszesz na serwerze z php7, a nawet przestrzeni nazw z 5.3 nie stosujesz, to powinieneś stosować się do standardu, po to abym jak zechcę mógł Twoją implementację zamienić na np monolog, jeśli stwierdzę, że akurat jego potrzebuję. System nie powinien zależeć od konkretnej implementacji... A wymyślanie koła na nowo mija się z celem. Cytat Jak juz napisalem, jest to *WSTEPNA* wersja. Jakie to ma znaczenie? Jeśli jesteś dobrym programistą a nie wyglądasz na pierwszego lepszego gimbusa, powinieneś już to wiedzieć iż są programiści i klepacze kodu. Rozróżnić ich można po jakości kodu. Jasne można pisać kod, który nadaje się tylko do refaktoringu, ale to nie jest produktywne. Cytat Odnosnie daty - w konstruktorze przekazujesz date z mikrosekundami, a ja chce poznac aktualny czas. Przeczytałeś cytat z dokumentacji? Ten post edytował com 18.03.2016, 23:23:53 |
|
|
|
q.michal Wstepna wersja loggera do oceny 18.03.2016, 00:04:27 
KsaR Skasuj "? true : false" bo to bez sensu,... 18.03.2016, 00:23:35 q.michal Racja, poprawilem.
Co do problemu z data, to mam r... 18.03.2016, 00:40:19 
KsaR Cytat(q.michal @ 18.03.2016, 00:40:19... 18.03.2016, 00:43:43 Pyton_000 getHandlersList() i getProcessorsList() - 98% kodu... 18.03.2016, 14:12:12 q.michal Cytat(Pyton_000 @ 18.03.2016, 14:12:1... 18.03.2016, 16:36:45 com skoro jesteś tego świadomy to czemu od razu piszes... 18.03.2016, 17:39:08 q.michal Jak juz napisalem, jest to *WSTEPNA* wersja.
Co do... 18.03.2016, 18:31:27 destroyerr Cytatto moim zdaniem gorzej byc nie moze
Ktoś może... 18.03.2016, 19:33:05 q.michal Cytat(destroyerr @ 18.03.2016, 19:33... 18.03.2016, 20:24:17 q.michal Cytat(destroyerr @ 18.03.2016, 19:33... 19.03.2016, 19:39:04 hind Nie myślałeś o implementacji PSR-3? 18.03.2016, 20:53:55 q.michal Myslalem, ale implementacja 20 poziomow logowanai ... 18.03.2016, 21:13:33 destroyerr CytatTo wtedy powinienes dostac notyfikacje z klas... 19.03.2016, 14:07:43  |
|
Aktualny czas: 17.10.2025 - 20:49 |
