- 1
- 2
- 3
- 4
- 5
- 6
public MarshalInfo getMarshalInfo(IType type) {
if (type == null) {
return null;
}
...
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−47
public MarshalInfo getMarshalInfo(IType type) {
if (type == null) {
return null;
}
...
}
И вот такое в куче методов по всему коду. Система начиналась в 2001 году, видимо в то время писали по принципу "В любой непонятной ситуации возвращай null". Так оно и работает - если что-то идет не плану, все падает с NPE, и сука попробуй разберись потом, откуда там неожиданный null прилетел. Самое ужасное, что до сих пор куча разработчиков, многим из которых за 60, продолжают писать в том же духе по сей день.
1024-- 23.10.2016 07:03 # 0
shadow_of_stertor 23.10.2016 17:34 # 0