[kot1401]

Mam takiego helpera:

function flight_duration($speed, $distance)

{//Obliczamy czas lotu

    if(! is_int($distance) OR ! is_int($speed)) throw new InvalidArgumentException;

    $cruise_duration = round($distance / ($speed / 3600));//Przelot na wys. przelotowej

    $departure_duration = 1200;//Start i wznoszenie zajmują dodatkowy czas

    $landing_duration = 1200;//lądowanie też...

    return $departure_duration + $cruise_duration + $landing_duration; //w sekundach

}

Chciałbym zrefaktoryzować ten spaghetti code i przenieść zmienne $departureduration i $landing_duration do jakiegoś pliku konfiguracyjnego (np. config/game.php)

Teraz pytanie, jak się odwoływać do dwóch tych zmiennych, żeby funkcja działała w izolacji podczas testów jednostkowych?

#php #laravel #webdev #programowanie

[m.....t]

@kot1401: Piekny kod.

Zrob z tego klase i wstrzykuj konfiguracje.

[MacDada]

@kot1401 Tak jak piosał @mariit, zamiast funkcji, zrób klasę. Niech wymagane zmienne konfigurujące dostaje w konstruktorze. Twórz obiekt w DIC.

[normanos]

@kot1401: jak często zmieniasz te $*duration że chcesz je dać w config?

[kot1401]

@normanos: Pewnie nigdy ich nie zmienię, ale to logika biznesowa, więc powinna być odseparowana.

[normanos]

@kot1401: how about... NO ;)

[normanos]

@kot1401: "One of the most common errors when defining application configuration is to create new options for

values that never change. BEST PRACTICE: Use constants to define configuration options that rarely change."

źródło: Symfony2 best practices. http://symfony.com/doc/download-best-practices-book/

[Jaslanin]

@kot1401:

ogólnie jeżeli czegoś nie planujesz zmieniać to nie rób z tego zmiennej konfiguracyjnej na zapas w myśl zasady że przedwczesna optymalizacja jest źródłem zła

[Jaslanin]

@kot1401: jeszcze taka uwaga do twojego kodu komentarze też pisz po angielsku raczej

nie każda linijka musi mieć komentarz + nie komentuj oczywistych rzeczy, takie coś doprowadzi do tego że nikt komentarzy nie będzie czytał bo zwykle nic ważnego tam nie będzie, i idąc za ciosem zignoruje coś ważnego.

np.$landing_duration = 1200;//lądowanie też...