[eloar]

} catch(Exception e) {
 throw new Exception(e->getMessage());
}

#itgraduate #nieprogramowanie

[niepoprawnyhumanista]

@eloar: mialoby sens gdybys chcial z 'e' wyciagnac pelny message z wszystkim inner exceptionami. no ale to by wymagalo i tak troche innego kodu :P

[Szab]

@eloar: Może zrobił to specjalnie żeby zgubić stack trace czy coś. To przecież popularna sytuacja ( ͡° ͜ʖ ͡°)

[n.....a]

@eloar: miałoby też sens gdybyś chciał wrzucić wyjątek do loga i dalej w catchu zrobić throw e;

[eloar]

@niepoprawnyhumanista: @Szab: @nochybaja: ten człowiek robi to bardzo często, praktycznie nigdy nic nie wpisuje do logu, na dodatek często na najwyższym poziomie już wyjątku nie przechwytuje... Ten geniusz siedzi na szczęśćie na tyle daleko, że nie chce mi się wstawać za każdym razem gdy coś takiego napotykam. No i jest powodem dla którego wprowadziliśmy wreszcie pełnoprawne code review w projekcie. (⌐ ͡■ ͜ʖ

[n.....a]

@eloar: Code review powinno być omawiane publicznie np. na rzutniku - tak aby obnażyć wszystko przed wszystkimi, a zarazem w miarę przyjemne dla właściciela kodu (w sensie ze śmieszkowaniem) żeby potem nie pieprzył się godzinami nad paroma linijkami. Takie jest moje zdanie.

[eloar]

@nochybaja: wprowadzamy 'shame wall'. Będą zdjęcia z największymi głupotami jakie człowiek wykonał na maszynie deweloperskiej. Autor kodu z początku będzie miał zdjęcie z kartką:

sudo rm -rf app/logs /

[n.....a]

wprowadzamy 'shame wall'.

@eloar: (⌐ ͡■ ͜ʖ ͡■) dobre.
sudo rm -rf app/logs / zauważyłem tę przykrą spację ( ͡° ͜ʖ ͡°) Podobno taka właśnie spacja przyczyniła się do wyczyszczenia spamu i nie tylko w takiej firmie.. Jak ona się nazywała? AdWeb?

[plushy]

Code review powinno być omawiane publicznie np. na rzutniku - tak aby obnażyć wszystko przed wszystkimi, a zarazem w miarę przyjemne dla właściciela kodu (w sensie ze śmieszkowaniem) żeby potem nie pieprzył się godzinami nad paroma linijkami. Takie jest moje zdanie.

@nochybaja: Z tym że mało kto potrafi coś takiego przeprowadzić i mało kto taką krytykę przyjmie. W CR nie chodzi o to by się publicznie popisywać.