[vilay]

Właśnie robię code review hindusa z 15 letnim stażem. Do tej pory myślałem że to tylko memy, ale jednak nie :D

#it #pracbaza #programowanie #programista15k #hindusdrivendevelopment #korposwiat

[RRybak]

@vilay: czepiasz się - może i nie działa, ale przynajmniej się kompiluje!

[groman43]

@vilay: Pokaż proszę co tam jest po prawej. Jeśli to jest przypisanie na zasadzie item = JSonInDb == true ? true : false, to faktycznie słabo. Natomiast w samym JsonInDb == true nie koniecznie widzę coś złego.

[vilay]

@groman43: mam nadzieje że troll tylko, jaki jest sens porównywana bool do true? Przecież sam bool daje Ci wartość tego porównania

[rapujaca_jaszczurka]

@vilay: A co jeśli tam jest wartość nullable i pajeet sprawdza czy jest dokładnie true?

[groman43]

@vilay: 5 minut googlowanie - https://stackoverflow.com/questions/15393935/boolean-in-an-if-statement
Być może problem nie jest z hinduskim kodem, ale pomiędzy krzesłem a klawiaturą.

[zwei]

@rapujaca_jaszczurka: to czemu pajeet użył == a nie ===?

[Ingvarr100th]

@zwei: bo to C# a nie JavaScript. Jako że C# jest silnie typowany nie potrzebuje takich #!$%@? operatorów ( ͡° ͜ʖ ͡°)

[vilay]

@rapujaca_jaszczurka: nie jest nullable, mogłem dodać w poście sry

[zwei]

@Ingvarr100th: Jezu, nie zauważyłem xd js rzucił mi się na mózg. Chyba potrzebuję lekarza

[yupitr]

@vilay: pracowałem z hindusami - oni faktycznie nawet JavaScript piszą ze swoim "akcentem" ( ͡° ͜ʖ ͡°) podobnie było z ukraińcami ( ͡° ͜ʖ ͡°)

[Ingvarr100th]

@groman43: no właśnie teoretycznie sama wartość boolowa w ifie, (nawet z negacją) w większości przypadków powinna starczyć jednakże w projekcie w którym teraz pracuje wymagane jest jawne porównanie boola z oczekiwaną wartością

[zibizz1]

@Ingvarr100th: @vilay:
dokładnie, czasem są w projekcie takie wymagania które mają zminimalizować błędu typu:
- zawsze używaj bloku czyli nawiasów {} nawet jeśli to tylko jedna linijka
- i tak samo niektórzy wymagają żeby pisać d == false zamiast !d bo wykrzyknik nie rzuca się w oczy

Jeśli np używasz == true to nie musisz potem tego pooprawiać jak ktoś zamieni bool w "bool?"

Czasem IDE podpowiada że coś da

[groman43]

@Ingvarr100th: Obawiam się, że jedyna dobra odpowiedź tutaj to słynne "to zależy".
Jeśli projekt jest duży i ma swoją historię, to faktycznie porównanie wartości boolej z true może mieć sens, żeby uniknąć przykrych niespodzianek. Poza tym, nie wiem dlaczego, takie porównanie jakoś lepiej mi się czyta.

W końcu, to naprawdę nie jest wielki problem i tak naprawdę nie ma praktycznie żadnego znaczenia. Jeśli wpływa na performance to znaczy, że coś poszło

[zibizz1]

@vilay: a i jeszcze jedno, wiem ze hindusi maja opinie jaka maja ,ale ja np mam jednego ogarniętego na 6 w zespole. Wiec nie myślmy stereotypowo bo to krzywdzące dla tego jednego

[y.....l]

@vilay: Macie coding standard? Może on zawsze tak robił, ja nie widzę problemu tutaj. Jak mam jakiś język skryptowy np. ruby czy python to tak robię zawsze jak ten Hindus. Nie widzę tutaj niczego złego. Pewnie jakby Polak tak napisał ten kod to byś olał, ale skoro Hindus to trzeba się pochwalić. ;)

[Hulleck123]

@groman43: bo problem leży w 'hype driven development' gdzie wyjdzie Ci jakiś kolo, powie tak jest fajnie, a tak nie fajnie i ludzie to potem bezrefleksyjnie łykają

[K.....n]

@vilay: śmiechu warte, informatyk 15k a zrzutu ekranu nie umie zrobić XD

[groman43]

@Hulleck123: Eh, w IT to chyba jest największy problem ( ͡° ͜ʖ ͡°) Sam, kiedy byłem juniorem bezmyślnie wierzyłem takim oświeconym prorokom. Zdrowy rozsądek przyszedł znacznie później.

[Hulleck123]

@groman43: u mnie również, taki jakiś romantyzm XD