- 1
- 2
- 3
- 4
- 5
- 6
Set<String> allDocs = new HashSet<String>();
allDocs.addAll(getQuery(app).execute(query.toString()));
ArrayList<String> result = new ArrayList<String>();
if (allDocs !=null){
result.addAll(allDocs);
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+77
Set<String> allDocs = new HashSet<String>();
allDocs.addAll(getQuery(app).execute(query.toString()));
ArrayList<String> result = new ArrayList<String>();
if (allDocs !=null){
result.addAll(allDocs);
}
Самое хорошее в строках 1 и 4, потому что хз, че там у этой джавы на уме. Лишняя проверка никогда не лишняя.
+80
public static java.sql.Date currentSQLDate() {
java.sql.Date result = null;
Date date = new Date();
return result;
}
ох, ёптеть...
+76
public static final String RANDOM_VALUE_QUERY = "select to_number(to_char(dbms_random.value(100,999),'999'),'999') from dual";
Прогрессивный способ генерации случайных чисел=.
+75
public Double toDouble(String str){
if ((str==null)||(str.equals(""))) str = "0.0";
if ((str.contains(","))&&(!str.contains("."))) str = str.replace(",", ".");
return new Double(str);
}
Лишняя защита никогда не бывает лишней
+114
private List<PalletePath> killDupes(List<PalletePath> pathesNew) {
List<PalletePath> noDupes = new ArrayList<PalletePath>();
for (PalletePath tp : pathesNew) {
if (!noDupes.contains(tp)) {
noDupes.add(tp);
}
}
return noDupes;
}
Set? Не, не слышали.
+74
private boolean userInOneRegistrationNode() throws DfException {
String uname = OrganizationStaffStructureHelper.getCurrentUserName();
int i = 0;
IDfCollection NodesCol = DQLHelper.getCollection(DQL_GET_REGISTRATOR_DIV, new String[]{uname});
while (NodesCol.next()) {
if (!(NodesCol == null)) {
String group_name = NodesCol.getString(GROUP_NAME);
i = i + 1;
}
}
if (i == 1) {
return true;
}
return false;
}
Заменяется 2мя строками один - select count(*), вторая - полученный результат Integer.ValueOf(...).
+71
String currentDate = new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").format(new Date());
accountNumber.append(String.valueOf(1900 + new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getYear()));
accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMonth()));
accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getDate()));
accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getHours()));
accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getMinutes()));
accountNumber.append(String.valueOf(new SimpleDateFormat("MM/dd/yyyy HH:mm:ss").parse(currentDate).getSeconds()));
ну а чо, зато внушительно выглядит
+81
public String isThisScreenWithHdpiDensity() {
DisplayMetrics metrics = new DisplayMetrics();
getWindowManager().getDefaultDisplay().getMetrics(metrics);
switch (metrics.densityDpi) {
case DisplayMetrics.DENSITY_LOW:
return "ldpi";
case DisplayMetrics.DENSITY_MEDIUM:
return "mdpi";
case DisplayMetrics.DENSITY_HIGH:
return "hdpi";
}
return "xhdpi";
}
...
if (isThisScreenWithHdpiDensity().equals("hdpi")) {
lpTabPane.height = 65;
}
if (isThisScreenWithHdpiDensity().equals("mdpi")) {
lpUpperPane.height = 70;
}
if (isThisScreenWithHdpiDensity().equals("ldpi")) {
lpTabPane.height = 32;
}
if (isThisScreenWithHdpiDensity().equals("xhdpi")) {
}
Перегоним константу в стрингу, а потом проверим ее equals. Отлично!
+113
public static void assertAlmostEquals(final double actual, final double expected) {
Assert.assertTrue(Math.abs(actual - expected) < 0.000001);
}
+142
if(!true) // это - не говнокод. это - заглушка.
return false;
Оправдывается ещё )