Wstepna wersja loggera do oceny Opcje

[q.michal]

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/

[q.michal]

Jak juz napisalem, jest to *WSTEPNA* wersja.
Co do PSR - owszem, mozna wszystko do tego sprowadzac, co nie znaczy ze jest idealny pod kazdym wzgledem.
A przykladu daleko szukac nie trzeba:

[PHP] pobierz, plaintext 
<?php
    /**
     * System is unusable.
     *
     * @param string $message
     * @param array $context
     * @return null
     */
    public function emergency($message, array $context = array());
 
    /**
     * Action must be taken immediately.
     *
     * Example: Entire website down, database unavailable, etc. This should
     * trigger the SMS alerts and wake you up.
     *
     * @param string $message
     * @param array $context
     * @return null
     */
    public function alert($message, array $context = array());
[PHP] pobierz, plaintext 

Skoro alert wymaga obudzenia kogos kto naprawi problem i opisany jest jako database unavailable czy entire website down, to moim zdaniem gorzej byc nie moze. Chyba, ze caly serwer padnie, ale wtedy ten emergency na nic Ci sie nie przyda. Poza tym jezeli zatrzyma sie baza danych, to notyfikacja powinna przyjsc z innego zrodla. Inna sprawa, ze jak cala strona bedzie nieosiagalna, to z duzym prawdopodobienstwem tej notyfikacji nie dostaniesz bo skrypt moze sie w ogole nie wykonac.
Od tego masz Icinge, Zabbixa i inne profesjonalne narzedzia. Nie ma wiec sensu bezmyslnie podazac za wszystkimi trendami i standardami.

Odnosnie daty - w konstruktorze przekazujesz date z mikrosekundami, a ja chce poznac aktualny czas.

Ten post edytował q.michal 18.03.2016, 18:33:02