Kosciany
Kosciany
133
  • Najlepsze
  • Wszystkie komentarze
feber
feber
307
@Kosciany: żebym tylko wiedział o co chodzi to może bym się zaśmiał ( ͡° ʖ̯ ͡°)
HetmanPolnyKoronny
@feber: nie dość że niepotrzebne, to jeszcze zwraca wartość odwrotnie (fałsz gdy są takie same)
patatier
@feber ktoś zamiast przejrzyście napisać w jednej linijce porównanie wartości true/false to nasrał do tego oddzielną metodę z bugiem.
groman43
@Kosciany: Już kilka razy widziałem podobne kwiatki w kilku różnych formach. Proponuję
- git blame (czy co tam masz pod ręką), żeby sprawdzić, komu pogratulować.
- sprawdzić inne commity tego samego autora, może znajdziesz coś zabawnego
- sprawdzić, kto robił code review, ta osoba również zasłużyła na szczere gratulacje
Kosciany
Kosciany Autor
8
@feber: @admiral_stulejjo: operatory logiczne.

To jakby zamiast policzyć 5x6

Zrobił program axb w pętli po a dodający za
Faynem
@Kosciany: czy jeśli rozumiem o co chodzi, to znaczy że mogę zostać #programista15k ?
s.....j
s.....j
@Faynem: nie ale będziesz na dobrej drodze
V.....e
V.....e
@feber: śpieszę z wyjaśnieniem:

public static bool CompareBooleans(bool orig, bool val) {
return orig == val
}
groman43
groman43
  • adult
3
Treść przeznaczona dla osób powyżej 18 roku życia...
W.....8
@Kosciany: Dotnetowcy zawsze byli tymi ciut gorszymi ( ͡° ͜ʖ ͡°)
ZohanTSW
ZohanTSW
@Kosciany: @HetmanPolnyKoronny: miałem w pracy podobną sytuację przy sprawdzaniu CRC. Robiły to 2 komponenty, jeden zwracał True gdy CRC się zgadzało, a drugi False ( ͡° ͜ʖ ͡°) potem trzeba było sprawdzić czy oba CRC są poprawne i była do tego podobnie głupio wyglądająca funkcja ( ͡° ͜ʖ ͡°)
szpongiel
@Kosciany: gdy HRy opanują diversity, inclusion, equity xD
henk
henk
2
@Kosciany: ale tak bez interfejsu taki ficzer implementować?
DOgi
DOgi
1
Już kilka razy widziałem podobne kwiatki w kilku różnych formach. Proponuję


@groman43: i jeszcze:
-sprawdzić wymagania

Być może to kod związany z portem z innego języka. Być może to adaptacja legacy. Bo kiedyś metoda konwertowała stringi "true" na boolean true itp.
patatier
@noelo_cohelo racja. szkoda, że generica do tego nie wyrzygał
derekperek122
@HetmanPolnyKoronny: kiedyś na studiach tez mi coś nie działało jak trzeba, podobnym sposobem to załatwiłem XD
admiral_stulejjo
@Kosciany: to śmieszne bo nie wiem o co chodzi
heater
@groman43: brzmi jak wstep stygmatyzacji xD. nie znam sie, ale czy code reviewer nie ponosi tutaj aby wiecej winy?
Izanagi013
@Kosciany: podobny kod kazali mi pisać, by nowi juniorzy byli w stanie zrozumieć bez znajomości języka. Praca została zmieniona.
RobieInteres
@Kosciany: Eh a ja się zastanawiam, czy z moją wiedzą podołam na rynku pracy, a są ludzie którzy bez żadnego stresu czy wstydu piszą w pracy takie smaczki xD