- 1
reader = new CSVReader(new BufferedReader(new InputStreamReader(new FileInputStream(csvFile), "UTF-8")), Config.getCSVDelimiter());
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−3
reader = new CSVReader(new BufferedReader(new InputStreamReader(new FileInputStream(csvFile), "UTF-8")), Config.getCSVDelimiter());
Хорошо что файл ещё не зазипован...
+4
s.replaceAll("(\\/\\.\\.\\/)+", "/").replaceFirst("^((\\.)+\\/)+", "");
вот такие вот суровые паттерны
как я это увидел - мне сразу вид двух друзей переханных трактором вспомнился
String s="../../../a./../../files/file1.rar"; // надо убрать строки вида "../"
+5
public class SomeClass
{
@SuppressWarnings("unused")
private Validator validator;
...
}
Эта переменная реально не используется. Удалить бы - ан нет, автор почему-то считает, что лучше пометить её как неиспользуемую, чтобы компилятор и Эклипс не ругались.
Чем только народ думает...
+9
@ValidationExpression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
"char[] st = inn.toCharArray();\n" +
"for(int i=0; i < st.length; i++) {\n" +
"if (!Character.isDigit(st[i])) { return false; }\n" +
"}\n" +
"}" +
"return true;", message = Messages.WRONG_INN_FORMAT)
@Expression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
"char[] st = inn.toCharArray();\n" +
"for(int i=0; i < st.length; i++) {\n" +
"if (!Character.isDigit(st[i])) { return false; }\n" +
"}\n" +
"}" +
"return true;", messageKey = Messages.WRONG_INN_FORMAT)
public String getinn() {
return getPerson().getinn();
}
@Required(messageKey = Messages.REQUIRED)
@DictEntityExists(dictId = MDictionaries.DICT_CURRENCY, message = Messages.NO_DICT_ENTITY)
@Expression(value = "SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(agentStatementCurrencyId) ? " +
" (" +
" SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(currency_id) ? " +
" true : " +
" SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_XXX.equals(contract_class_id) ? " +
" false : " +
" is_currency_equivalent == 1" +
" ) : (" +
" SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_GENERIC.equals(contract_class_id) &&" +
" is_currency_equivalent == 0 &&" +
" agentStatementCurrencyId.equals(currency_id)" +
" )", messageKey = Messages.CURRENCY_NOT_EQUIVALENT_WITH_AGENT_STATEMENT)
public String getcurrency_id() {return getContract().getcurrency_id();}
нет слов
+7
private void getCurrentBranch();
public TDictBranch getCurrentBrach();
public TDictBranch2 getCurrentBrach2();
public TDictBranch2 getCurrentBrach2(Date lookupDate);
public String getCurrentBrachId();
public String getCurrentBrunchCode();
public static String getCurrentBranchCharId();
некоторые методы из класса конфигурации приложения
0
public static String getClassName(final HWND hWnd) {
final char[] cname_b = new char[1024];
User32.INSTANCE.GetClassName(hWnd, cname_b, 1000);
final char[] cname_b_truncated = ArrayUtils.toPrimitive(Iterables
.toArray(Functional.takeWhile(Arrays.asList(ArrayUtils
.toObject(cname_b)), Predicates.not(Predicates
.equalTo(Character.valueOf((char) 0)))),
Character.class));
return String.valueOf(cname_b_truncated);
}
Только что выдавил)
Интерес представляют строки с 5 по 9. Тут должно было быть всего-лишь
takeWhile (/= '\0') cname_b
но из-за боксинга/анбоксинга (примитивные типы в Java не могут быть параметрами Generic), и из-за того, что массивы в Java не Iterable - получилось ЭТО.
+6
float lan = getLan();
float lon = getLan();
if(lan == lan || lon == lon)
{
throw new RuntimeException("Bad value");
}
Из моего реалного проекта. Этот код реально делает полезную вещь
+5
for (JButton button : buttons) {
if(buttons.length>0&&button==buttons[0]){
if (canCreate()||button.getName().equals(BTN_EDIT_CONTRACT)) {
UiUtils.setEnabled(button, true, canCreate()|| button.getName().equals(BTN_EDIT_CONTRACT));
} else {
UiUtils.setEnabled(button, false, canCreate()|| button.getName().equals(BTN_EDIT_CONTRACT));
}
}
if(buttons.length>1&&button==buttons[1]){
if (canEdit()||canView()||button.getName().equals(BTN_EDIT_CONTRACT)) {
UiUtils.setEnabled(button, true, canEdit()||canView()|| button.getName().equals(BTN_EDIT_CONTRACT));
} else {
UiUtils.setEnabled(button, false, canEdit()||canView() || button.getName().equals(BTN_EDIT_CONTRACT));
}
}
if(buttons.length>2&&button==buttons[2]){
if (canDelete()||button.getName().equals(BTN_EDIT_CONTRACT)) {
UiUtils.setEnabled(button, true, canDelete()|| button.getName().equals(BTN_EDIT_CONTRACT));
} else {
UiUtils.setEnabled(button, false, canDelete() || button.getName().equals(BTN_EDIT_CONTRACT));
}
}
if(buttons.length>3&&button==buttons[3]){
if (canEdit()||canView()||button.getName().equals(BTN_EDIT_CONTRACT)) {
UiUtils.setEnabled(button, true, canEdit()||canView()|| button.getName().equals(BTN_EDIT_CONTRACT));
} else {
UiUtils.setEnabled(button, false, canEdit()||canView() || button.getName().equals(BTN_EDIT_CONTRACT));
}
}
if(buttons.length>4&&button==buttons[4]){
if (canEdit()||canView()||button.getName().equals(BTN_EDIT_CONTRACT)) {
UiUtils.setEnabled(button, true, canEdit()||canView()|| button.getName().equals(BTN_EDIT_CONTRACT));
} else {
UiUtils.setEnabled(button, false, canEdit()||canView() || button.getName().equals(BTN_EDIT_CONTRACT));
}
}
}
+11
void setEnabled(boolean disable)
−4
Link link = new Link( element );
if( !Browser.getBrowser().isElementPresent( link.getLocator() ) )
{
return false;
}
else
{
return true;
}
зачэм так?