- 1
- 2
- 3
- 4
- 5
- 6
String filterDescription = "";
try {
filterDescription = wdContext.currentContextElement().getAttributeValue("table" + Level + "Description" + "filter").toString();
} catch (Exception e) {
// TODO: handle exception
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 7
+73
String filterDescription = "";
try {
filterDescription = wdContext.currentContextElement().getAttributeValue("table" + Level + "Description" + "filter").toString();
} catch (Exception e) {
// TODO: handle exception
}
wdContext.currentContextElement().getAtt ributeValue("table" + Level + "Description" + "filter") возвращает значение атрибуты из контекста с типом Object. try в данном случае тут добавлен, чтобы не писать лишних проверок, если вернется null, а filterDescription так и остался пустой строкой.
+69
public void openPopupWindow( com.sap.tc.webdynpro.services.session.api.IWDWindow window )
{
IWindowStackElement newWindow = wdContext.nodeWindowStack().createWindowStackElement();
newWindow.setWindow(window);
wdContext.nodeWindowStack().addElement(newWindow);
}
Вот так вот вроде называешь всё своими именами, а потом на выходе получаешь вот такую тавтологию как в 4ой строке.
+146
if ( city_name(5) = 'город' or city_name(5) = 'ГОРОД' or
city_name(5) = 'Город' or city_name(5) = 'ГоРоД' or
city_name(5) = 'гороД' or city_name(5) = 'гОрОд' or
city_name(6) = 'город.' or city_name(6) = 'ГОРОД.'
or city_name(6) = 'Город.' or city_name(6) = 'ГоРоД.'
or city_name(6) = 'гороД.' or city_name(6) = 'гОрОд.' )
and city_name <> 'Городец'.
shift city_name by 5 places.
endif.
Чистим лишнее из имени города.
+71
if (wdContext.nodeTable().size() <= 5)
wdContext.currentContextElement().setRowsIn(-1);
else
wdContext.currentContextElement().setRowsIn(-1);
+76
public static java.sql.Date StringToDate(String sDate) {
java.sql.Date Date = null;
if (!sDate.startsWith("00000000")) {
DateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd");
try {
Date rdate = dateFormat.parse(sDate);
Date = new java.sql.Date(rdate.getTime());
} catch (Exception e) {
}
}
return Date;
}
+80
try {
...
} catch (Exception ex) {
wdComponentAPI.getMessageManager().reportException("ERROR", true);
}
Информативно так
+84
search = 0;
try {
search = search + wdContext.currentSearchElement().getLifnr().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getBahns().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getINN().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getKPP().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getKtokk().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getLand1().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getBanks().length();
} catch (Exception e) {
// TODO: handle exception
}
try {
search = search + wdContext.currentSearchElement().getBankl().length();
} catch (Exception ex) {
// TODO: handle exception
}
try {
search = search + wdContext.currentSearchElement().getBankn().length();
} catch (Exception ex) {
// TODO: handle exception
}
try {
search = search + wdContext.currentSearchElement().getLocco().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
// search = search + wdContext.currentSearchElement().getLoevm().length();
search = search + wdContext.currentSearchElement().getName().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getOKPO().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getOrt01().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
try {
search = search + wdContext.currentSearchElement().getSortl().length();
} catch (Exception ex) {
// ex.printStackTrace();
}
if (search == 0)
wdComponentAPI.getMessageManager().reportException("Введите критерии поиска", true);
else {
//Далее идет поиск
}
Вот такой вот была забабахана проверка входных параметров перед вызовом функции поиска (мною добавлен только последний комментарий, все остальное оригинальное)