Java — Говнокод.ру

Java / Говнокод #16193

+73

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48

// Current Month Days
            for (int i = 1; i <= _nDaysInMonth; i++) {
                if (_myPrefs.getIfConfigured()) {
                    if (_regionWithHolidays != null && _regionWithHolidays.length > 0) {

                        String holidayDateStringFormat = String.valueOf(i) + "-" + getMonthAsString(nCurrentMonth) + "-" + _strCurrentSelectedYear;
                        for (int index = 0; index < _regionWithHolidays.length; index++) {
                            if (_regionWithHolidays[index].getHolidayDate().equals(holidayDateStringFormat)) {
                                list.add(String.valueOf(i) + "-RED" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                            }
                        }
                    }
                }

                if (i == getCurrentDayOfMonth()) {
                    list.add(String.valueOf(i) + "-BLUE" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                } else {
                    list.add(String.valueOf(i) + "-WHITE" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                    if (_myPrefs.getIfConfigured()) {
                        if (_regionWithHolidays != null && _regionWithHolidays.length != 0) {
                            String otherDates = String.valueOf(i) + "-" + getMonthAsString(nCurrentMonth) + "-" + _strCurrentSelectedYear;
                            for (int index = 0; index < _regionWithHolidays.length; index++) {
                                if (_regionWithHolidays[index].getHolidayDate().equals(otherDates)) {
                                    list.remove(String.valueOf(i) + "-WHITE" + "-" + getMonthAsString(nCurrentMonth) + "-" + yy);
                                }
                            }
                        }
                    }
                }
            }

            // some code

            for (int i = 1; i <= _nDaysInMonth; i++) {
                String[] day_color = list.get(i).split("-");
                if (day_color[1].equals("WHITE")) {
                    //active dates in current month
                    dayNumberView.setTextColor(getResources().getColor(R.color.darkGrey_font));
                } else if (day_color[1].equals("BLUE")) {
                    //current date in current month
                    holder.tvTime.setTextColor(getResources().getColor(R.color.whitetranslucent));
                    dayNumberView.setTextColor(getResources().getColor(R.color.whitetranslucent));
                    cell.setBackgroundResource(R.color.blue_font);
                } else if (day_color[1].equals("RED")) {
                    //active dates in current month
                    dayNumberView.setTextColor(getResources().getColor(R.color.red));
                }
            }

Это приложение с десятками тысяч пользователей. Мне выпала честь править в нем баги. На сколько я могу судить задача этого куска была отобразить календарь на текущий месяц на экране и подсветить WHITE - обычные дни, BLUE - текущий, RED - выходные и праздничные. Итак в чем соль:
1) сама соль метода - в одном цикле создается список строк вида НОМЕР-ЦВЕТ-МЕСЯЦ-ГОД, чуть ниже в аналогичном цикле эти строки разбиваются по "-" и сравнивается по строковому значению цвета. Кроме того день может быть или текущим или праздничным, но никак не одновременно.
2) два практически одинаковых куска кода по 10 строк - строки 4-13 и 19-28, первый при определенных условия добавляет ПРАЗДНИЧНЫЙ день в календарь, потом день в 16-18 строках день всегда добавляется еще раз этот день, либо обычный(WHITE) либо текущий(BLUE), и выполняется второй кусок и если проходят те же условия т.е. фактические если был добавлен праздничный день(RED) то удаляется добавленный ОБЫЧНЫЙ день. Баг был в том что если текущий день был еще и праздничным то они задваивались. Сделать по нормальному - если день уже добавлен, не добавлять еще раз или сделать continue главному циклу. Не говоря уже о том чтобы добавить break после 9 строки, видно автор не знал про эти операторы.

TAX, 19 Июня 2014

Комментарии (5)

Java / Говнокод #16189

+87

1
2
3
4
5
6
7
8

public BarrierAssessment(int id, Client client, int homephone, int mobile, int email, int transport, int licence, boolean notInWorkForce, boolean unpaidWork, boolean cdeProgram, boolean notWorkingBut, int paidWork, 
String employerOcc, String reason, int highlevel, String further, int stableAccomodation, int loneParent, int livingWith, int policeProblems, int courtPending, int preventFrom, int medical, int drugTest, int currentBusy, 
String scurrentBusy, String reasonForNotGettingJob, int startTomorrow, String jobGoals, int currentResume, int canvasLetter, int referees, int interviewClothing, String otherBarriers, int ec, LastModified lastModified, 
Date datestamp, Date bupdated, String disabilityBarriers, Date licenceDueToExpire, Date licenceDueBack, String currentSkills, String possibleSkills) {
 
...
 
}

Мать всех конструкторов!

Meegoo, 18 Июня 2014

Комментарии (25)

Java / Говнокод #16188

+74

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78

import java.io.*;
import java.util.*;

public class Solution
{
    public static void main(String[] args) throws Exception
      { Scanner scanner = new Scanner(System.in);
        int a = scanner.nextInt();
        int b = scanner.nextInt();
        int c = scanner.nextInt();
        int z = scanner.nextInt();
        if (a < b && a < c && a < z)
        {  if (b < c && b < z)
            {if (c < z)
                {System.out.println(z + " " + c + " " + b + " " + a);}
                else
                {System.out.println(c + " " + z + " " + b + " " + a);} }
            else if (c < b && c < z)
            { if (b < z)
                {System.out.println(z + " " + b + " " + c + " " + a);}
                else
                {System.out.println(b + " " + z + " " + c + " " + a);} }
            else if (z < c && z < b)
            { if (c < b)
                {System.out.println(b + " " + c + " " + z + " " + a);}
                else
                {System.out.println(c + " " + b + " " + z + " " + a);} } }
        else if (b < a && b < c && b < z)
        { if (a < c && a < z)
            { if (c < z)
                {System.out.println(z + " " + c + " " + a + " " + b);}
                else
                {System.out.println(c + " " + z + " " + a + " " + b);} }
            else if (c < a && c < z)
            { if (a < z)
                {System.out.println(z + " " + a + " " + c + " " + b);}
                else
                {System.out.println(a + " " + z + " " + c + " " + b);} }
            else if (z < a && z < c)
            { if (a < c)
                {System.out.println(c + " " + a + " " + z + " " + b);}
                else
                {System.out.println(a + " " + c + " " + z + " " + b);} } }
        else if (c < a && c < b && c < z)
        { if (a < b && a < z)
            { if (b < z)
                {System.out.println(z + " " + b + " " + a + " " + c);}
                else
                {System.out.println(b + " " + z + " " + a + " " + c);} }
            else if (b < a && b < z)
            { if (a < z)
                {System.out.println(z + " " + a + " " + b + " " + c);}
                else
                {System.out.println(a + " " + z + " " + b + " " + c);} }
            else if (z < a && z < b)
            { if (a < b)
                {System.out.println(b + " " + a + " " + z + " " + c);}
                else
                {System.out.println(a + " " + b + " " + z + " " + c);} } }
        else if (z < a && z < b && z < c)
        { if (a < c && a < b)
            { if (c < b)
                {System.out.println(b + " " + c + " " + a + " " + z);}
                else
                {System.out.println(c + " " + b + " " + a + " " + z);} }
            else if (c < a && c < b)
            { if (a < b)
                {System.out.println(b + " " + a + " " + c + " " + z);}
                else
                {System.out.println(a + " " + b + " " + c + " " + z);} }
            else if (b < a && b < c)
            {
                if (a < c)
                {System.out.println(c + " " + a + " " + b + " " + z);}
                else
                {System.out.println(a + " " + c + " " + b + " " + z);} } }
}
}

Ня!

dagonlaf, 18 Июня 2014

Комментарии (4)

Java / Говнокод #16186

+79

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18

public Long getRecordCount() {
        long result = 0;
        if(pfrFile != null){
            if(pfrFile.getПачкаВходящихДокументов().getРАСЧЕТ_ПО_СТРАХОВЫМ_ВЗНОСАМ_НА_ОПС_И_ОМС_ПЛАТЕЛЬЩИКАМИ_ПРОИЗВОДЯЩИМИ_ВЫПЛАТЫ_ФЛ() != null){
                result++;
            }
            if(pfrFile.getПачкаВходящихДокументов().getРСВ1() != null){
                result++;
            }
            if(pfrFile.getПачкаВходящихДокументов().getРАСЧЕТ_ПО_СТРАХОВЫМ_ВЗНОСАМ_НА_ОПС_И_ОМС_ПЛАТЕЛЬЩИКАМИ_СВ_ПРОИЗВОДЯЩИМИ_ВЫПЛАТЫ_ФЛ_НАЧИНАЯ_С_2012_ГОДА() != null){
                result++;
            }
            if(pfrFile.getПачкаВходящихДокументов().getРАСЧЕТ_ПО_СТРАХОВЫМ_ВЗНОСАМ_НА_ОПС_И_ОМС_ПЛАТЕЛЬЩИКАМИ_СВ_ПРОИЗВОДЯЩИМИ_ВЫПЛАТЫ_ФЛ_НАЧИНАЯ_С_2013_ГОДА() != null){
                result++;
            }
...
        return result;
}

:D

skobets, 18 Июня 2014

Комментарии (5)

Java / Говнокод #16185

+74

1
2
3
4
5

/**
 * Some UI hacks for JRuby Facet
 */
public class NiiChAVOUtil { 
}

Слова лишнии

vistall, 18 Июня 2014

Комментарии (7)

Java / Говнокод #16179

+83

1
2
3
4
5
6
7

try {
    progress.dismiss();
    progress = null;
    progress.dismiss();
} catch (Exception e) {
    Log.e("error", e.getMessage() + " " + e.getCause());
}

andrew91, 17 Июня 2014

Комментарии (4)

Java / Говнокод #16160

+72

01
02
03
04
05
06
07
08
09
10
11
12

Date convertedDate = new Date();
		SimpleDateFormat dateFormatter = new SimpleDateFormat("yyyy:MM:dd",
				Locale.getDefault());

		try {
			convertedDate = dateFormatter.parse("" + year + ":" + month + ":"
					+ day);
		} catch (ParseException e) {
			e.printStackTrace();
		}
		SimpleDateFormat fmtOut = new SimpleDateFormat("MM/dd/y");
		String curDate = fmtOut.format(convertedDate).toString();

andrew91, 13 Июня 2014

Комментарии (9)

Java / Говнокод #16153

+68

1
2
3
4
5
6
7

int k = 50;
while (true){
    i = i + i;
    System.out.println(i);
    k--;
    if (k<0) break;
}

http://stackoverflow.com/questions/24173463/why-does-i-i-i-give-me-0

evg_ever, 12 Июня 2014

Комментарии (23)

Java / Говнокод #16150

+174

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22

/*Caused by: java.lang.StackOverflowError
    at com.sun.org.apache.xalan.internal.xsltc.dom.SimpleResultTreeImpl.characters(SimpleResultTreeImpl.java)
    at com.sun.org.apache.xalan.internal.xsltc.dom.SimpleResultTreeImpl.copy(SimpleResultTreeImpl.java:438)
    at com.sun.org.apache.xalan.internal.xsltc.runtime.BasisLibrary.copy(BasisLibrary.java:1317)
    at GregorSamsa.replace()
    at GregorSamsa.replace()
    at GregorSamsa.replace()
    at GregorSamsa.replace()
    at GregorSamsa.replace()
  начали разбираться
*/
    /** 
     * As Gregor Samsa awoke one morning from uneasy dreams he found himself
     * transformed in his bed into a gigantic insect. He was lying on his hard,
     * as it were armour plated, back, and if he lifted his head a little he
     * could see his big, brown belly divided into stiff, arched segments, on
     * top of which the bed quilt could hardly keep in position and was about
     * to slide off completely. His numerous legs, which were pitifully thin
     * compared to the rest of his bulk, waved helplessly before his eyes.
     * "What has happened to me?", he thought. It was no dream....
     */
    protected final static String DEFAULT_TRANSLET_NAME = "GregorSamsa";

http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/com/sun/org/apache/xalan/internal/xsltc/trax/TransformerFactoryImpl.java
transofm, factory, bug - какие начитанные разработчки, а.

3.14159265, 12 Июня 2014

Комментарии (3)

Java / Говнокод #16147

+71

1
2
3
4
5

List<String> list = ...;
for (String s : someStringList)
    list.add(s);
list.set(SOME_CONST, someString);
list.add(0, secondSomeString);

Поначалу никак не мог понять, почему list.get(SOME_CONST) != someString. Ну и копирование через цикл тоже норм

evg_ever, 11 Июня 2014

Комментарии (29)

Говнокод: по колено в коде.