[adrninistrator]

#programowanie #python

Czy prosicie o usunięcie pustej linii przed return w funkcji gdy robicie CR? Ja tak, daje na request changes I to ma zostać usunięte. Sam też nigdy tak nie robię

[Saly]

@adrninistrator: jak nie możesz tego wymusić linterem to oznacza to, że
* nikt nie daje o to j*****a
* problem jest dyskusyjny i obie sytuacje są ok według gustu danej osoby

i nie powinno się zawracać głowy takimi pierdołami

[sokytsinolop]

@adrninistrator: to niezle code review robizz. Pewnie koledzy mówia na ciebie linter

[adrninistrator]

@sokytsinolop: @Saly przecież każda pustą linie można zastąpić komentarzem. No i nie powinno się dawać pustych linii przed pojedynczym statementem lul

[Arogancky]

@adrninistrator: lintery puszczają? To czepiający niech s-------a albo naprawi lintery

[walokid]

@adrninistrator: żeby lepiej widzieć gdzie są "exit pointy" w funkcji

[adrninistrator]

@walokid: a co ty kret jesteś ze ci kolorek nie wystarczy?

[p.....t]

@adrninistrator: ja moich kodziarzy zmuszam wszystko w jednej linijce pisać żeby nie marnować firmowych bitów

[eU2MO]

@adrninistrator: Puste linie mają sens, gdy oddzielają części funkcji, które mają różne funkcjonalności. Np. ta funkcja z add() z przykładu robi coś z parametrami, linia odstępu i potem linia typu deus ex: koniec działania - zwrot wartości. Albo inny przykład: funkcja służy do pobierania zawartości HTML strony o podanym stringowym adresie URL. Składa się z trzech części: walidacji parametru URL, pobierania strony do stringa i zwracania tego stringa. Wtedy warto

[n0c0Mpr3h3nD]

@adrninistrator: dobry bejt, 8/10

[adrninistrator]

@eU2MO: ale dlaczego zamiast nowej linii nie można napisać komentarza?

[eU2MO]

@adrninistrator: Przykładowo, w mojej ostatniej firmie pisałem takie komentarze rozdzielające, które się dezaktualizowały po paru latach. Ktoś zmieniał mój kod (cała firma tak robiła) i nie patrzył wcale na komentarz, który często nie miał wtedy sensu. Więc dawałem pustą linię i było git.

[adrninistrator]

@eU2MO: Masz rację, nie warto pisać komentarzy w kodzie ^^

[eU2MO]

@adrninistrator: Zgadza się, jak spojrzymy na opis TDD, to jest: testy są wykonywalną specyfikacją. A w innym miejscu: komentarze są zawsze nieaktualne.