[kot1401]

Ma ktoś ochotę na code review mojego kodu?

https://gist.github.com/anonymous/a3231dfe379eeb39ed5f

Mam potencjał na junior deva za 2k netto miesięcznie, czy jeszcze za wcześnie? :(

#php #programowanie #laravel

[m.....t]

@kot1401: Wrzuc gista z tego, zeby dalo sie komentowac.

[MacDada]

@kot1401: http://www.php-fig.org/psr/psr-2/

[kot1401]

@matiit: https://gist.github.com/anonymous/a3231dfe379eeb39ed5f

[DiKey]

@kot1401:

1. Komentuj w stylu phpdoc

2. Tutaj enter przed namespacem

<?php namespace Game\Services;

[kot1401]

@DiKey: Wiem, wiem :P

[6gcqG9nN58zhTK]

@kot1401: za 2 netto chcesz zyc?

[DiKey]

@kot1401: i przeczytaj to co @MacDada wkleił, tutaj masz więcej standardów: https://github.com/php-fig/fig-standards/tree/master/accepted (m. in. 4 spacje zamiast tab, spacje po if itd)

[kot1401]

@6gcqG9nN58zhTK: od czegoś trzeba zacząć

poza tym mam drugie tyle z adsennse na moich prywtanych stronach

[MacDada]

@kot1401:

1.)

Config::get('game.default_ticket_price')

– wstrzykuj obiekt konfiguracji albo od razu wartości w konstruktorze

2.) Dodaj PHPDOCi

3.) Proponuję ustalić styl kodowania na PSR1 + PSR2

4.) Używaj obiektu

\DateTime

zamiast latać po kodzie stringami czy intami:

strtotime($dowMap[$weekday].' next week +'.$hour.' hours')

5.)

is_object($best_flight)

–

instanceof

i zrób konkretne klasy (domyślam się, że tam jest

\StdClass

)

6.)

chooseBestFlight()

rzekomo zwraca najlepszy lot, a tak na prawdę zwraca

bool

– hę?

7.)

[kot1401]

@DiKey: Chodziło mi bardziej o architekturę obiektów itp. niż o standardy notacji kodu (tego można się nauczyć w tydzien :P)

[ozzie]

@kot1401:

jeszcze dodam nazewnictwo zmiennych - staraj się, aby nazwa była jak najbardziej zwięzła oraz nie miksuj camelCase z e snakecase

Komentarze w wielu miejscach zupełnie niepotrzebne na przykład:

if($pts <= $highest

score) continue; //This flight hasn't highest score, so continue

to wynika z nazw zmiennych i kodu, więc komentarz jest zbędny

Tak samo tutaj:

//Nominal price on given flight distance

$price*nominal = Config::get('game.default*ticket_price') * $flight->distance;

Lepiej opisz co funkcja

[MacDada]

@kot1401: Tak jak pisze @ozzie – zamiast robić komentarze, rozbijaj na mniejsze metody, których nazwy powiedzą co tam robisz.

Czyli zamiast komentarza

// Nominal price on given flight distance

wydziel po prostu kod do metody

calculateNominalPrice($flight)

i wtedy kod mówi sam za siebie.

[MacDada]

Wrzuc gista z tego, zeby dalo sie komentowac.

@matiit: W gistach nie da się niestety komentować konkretnych linii :/

@kot1401 Skoro Twój kod służy do nauki, może po prostu daj publiczne repo z całością?

0. Obcykasz się z GITem

1. Można wtedy komentować konkretne linii i konkretne commity

2. Można robić

Pull Request

y (czyli mogę Ci od razu podrzucić np mały refactoring)

3. Można robić

issue

(czyli zgłaszać problemy)

[m.....t]

@MacDada: Masz racje... myslalem, ze mozna jak na reszcie githuba :)

[rysi3k_]

@kot1401: potnij na kilka mniejszych, lepiej nazwanych metod i będzie cacy :)

Tak czy siak, na junior deva na pewno się nadajesz. Widywałem gorsze kody osób które już pracowały na różnych stanowiskach, nawet nie junior... Ja bym Cie przyjął jakbyś był z Wrocławia xD