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

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

+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

public class DataRetriever
{
    public static Object deserializeData(String fileName)
    {
        Object returnValue = null;
        try
        {
            File inputFile = new File(fileName);
            if (inputFile.exists() && inputFile.isFile())
            {
                try (ObjectInputStream readIn = new ObjectInputStream(new FileInputStream(fileName)))
                {
                    returnValue = readIn.readObject();
                }
            }
            else
            {
                throw new RuntimeException(new FileNotFoundException(fileName + " not found"));
            }
        }
        catch (ClassNotFoundException | IOException exc)
        {
            throw new RuntimeException(exc);
        }
        return returnValue;
    }

    private DataRetriever() { throw new AssertionError(); }
}

Паранойя неконтролируемых исключений

dwinner, 17 Января 2012

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

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

+65

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

for (...) {
  Comparator<Date> date_comparator = new Comparator<Date>() {
    @Override
    public int compare(Date s1, Date s2) {
      long n1 = s1.getTime();
      long n2 = s2.getTime();
      if (n1 < n2)
        return -1;
      else if (n1 > n2)
        return 1;
      else
        return 0;
    }
  };
  Date beforeSaveDate = (Date) beforeSaveParam.getValue();
  beforeSaveDate.setSeconds(0);
  Date toSaveDate = (Date) toSaveParam.getValue();
  comparedValue = date_comparator.compare(beforeSaveDate, toSaveDate);
}

Задача была сравнить две даты, игнорируя при этом секунды.

LexeY4eg, 13 Января 2012

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

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

+81

1
2
3
4
5
6
7
8

public void getApplicationContext(ApplicationContext applicationContext ) {
    applicationContext.setUserId(currentUserService.getCurrentUser().getName());
    applicationContext.setTransactionType(transactionType);
    applicationContext.setApplication(applicationName);
    applicationContext.setSubSystem(subSystemName);
    applicationContext.setAddress(ipAddress);
    applicationContext.setCorrelationId(correlationIDGenerator.getRandomGUID());
}

*facepalm*

roman-kashitsyn, 12 Января 2012

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

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

+78

1
2
3
4
5
6

switch (blds [i])
        {
          default:
            points [0] [i] = InputGameMessage.readString ();
            break;
        }

замечательная индусятина

alexmelyon, 10 Января 2012

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

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

+88

1

LinkedHashMap<Long, Map<String, Map<Long, List<Long>>>> propertyTemplates;

bozho, 09 Января 2012

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

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

+76

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

public class FormForThemeView extends LayoutContainer {

    private double innerRightPanelHeight = 0.7;
    private LayoutContainer leftContentPanel;
    private LayoutContainer rightContentPanel;

    private ContentPanel innerLeftDataPanel;
    private ContentPanel innerRightDataPanel;
    private ContentPanel upperRightDataPanel;
    private ContentPanel bottomRightDataPanel;
    private ContentPanel bottomMiddleDataPanel;

    public FormForThemeView() {
        leftContentPanel = new LayoutContainer(new RowLayout(Style.Orientation.HORIZONTAL));
        rightContentPanel = new LayoutContainer(new RowLayout(Style.Orientation.VERTICAL));

        innerLeftDataPanel = new ContentPanel(new RowLayout(Style.Orientation.VERTICAL));
        innerRightDataPanel = new ContentPanel(new RowLayout(Style.Orientation.VERTICAL));
        bottomMiddleDataPanel = new ContentPanel(new RowLayout(Style.Orientation.VERTICAL));

        upperRightDataPanel = new ContentPanel(new RowLayout(Style.Orientation.VERTICAL));
        bottomRightDataPanel = new ContentPanel(new RowLayout(Style.Orientation.VERTICAL));
    }

    public void render() {
        innerLeftDataPanel.setBorders(false);
        innerLeftDataPanel.setBodyBorder(false);
        innerLeftDataPanel.setHeaderVisible(false);
        innerLeftDataPanel.setBodyStyle("backgroundColor: transparent");
        innerLeftDataPanel.add(bottomMiddleDataPanel, new RowData(-1, -1));

        innerRightDataPanel.setBorders(false);
        innerRightDataPanel.setBodyBorder(false);
        innerRightDataPanel.setHeaderVisible(false);
        innerRightDataPanel.setBodyStyle("backgroundColor: transparent");

        bottomMiddleDataPanel.setBorders(false);
        bottomMiddleDataPanel.setBodyBorder(false);
        bottomMiddleDataPanel.setHeaderVisible(false);
        bottomMiddleDataPanel.setBodyStyle("backgroundColor: transparent");

        leftContentPanel.setBorders(false);
        leftContentPanel.add(innerRightDataPanel, new RowData(156, 1));
        leftContentPanel.add(innerLeftDataPanel, new RowData(1, 1));


        rightContentPanel.setStyleAttribute("borderLeft", "2px dotted #99BBE8");
        rightContentPanel.add(upperRightDataPanel, new RowData(1, innerRightPanelHeight));
        rightContentPanel.add(bottomRightDataPanel, new RowData(1, 1 - innerRightPanelHeight));

        setLayout(new RowLayout(Style.Orientation.HORIZONTAL));
        add(leftContentPanel, new RowData(1, 1));
        add(rightContentPanel, new RowData(156, 1));
    }
}

Попытка сделать панель на GXT. Попутаны стороны right находится слева, тк смотрим с другой стороны. По названиям переменных ничего не понятно. inner, bottom, left, rigth - найдите лишнее.

johnny1987, 06 Января 2012

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

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

+78

1
2
3
4
5
6

while (entityIterator.hasNext()) {
    Object[] results = entityIterator.next();
    Long policyId = ((PolicyEntity) results[0]).getId();
    PolicyEntity policy = (PolicyEntity) policyDao.findById(policyId);
    processPolicy(policy);
}

Спецально не рефакторил этот шедевр, ждал, пока ГК поднимется.

roman-kashitsyn, 05 Января 2012

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

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

+83

01
02
03
04
05
06
07
08
09
10

public String getNext() {
    try {
        return items[pointer];
    } finally {
        pointer++;
        if(pointer >= items.length) {
            pointer = 0;
        }
    }
}

huitka, 04 Января 2012

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

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

+75

1
2

StringBuilder append = new StringBuilder();
...

redenemy, 03 Января 2012

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

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

+89

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

/*
     * helper function that will complete the entire compile
     * process, but allow both filestreams and regular input
     * streams to be compiled.
     */
    static boolean doCompile(InputStream in,
        String pathspec,
        String scriptname,
        String filename,
        String encoding,
		String swf_options,
		String avmplus_exe,
        ObjectList<IncludeInfo> includes,
    ObjectList<String> import_filespecs,
    ObjectList<String> use_namespaces,
	String language,
    ObjectList<ConfigVar> configs,
    ObjectList<CompilerPlug> plugs,
    CompilerHandler handler,
        boolean emit_doc_info /*false*/,
			boolean emit_debug_info /*=false*/,
                boolean show_instructions /*=false*/,
                    boolean show_machinecode /*=false*/,
                        boolean show_linenums /*=false*/,
                            boolean show_parsetrees /*=false*/,
                                boolean show_bytes /*=false*/,
                                    boolean show_flow /*=false*/,
                                        boolean lint_mode /*=false*/,
                                            boolean use_static_semantics /*=false*/,
                                                boolean emit_metadata,
                                                    boolean save_comment_nodes/*=false*/,
                                                        int dialect /*=0*/,
                                                            int target,
                                                                boolean optimize,
							 ObjectList<ConfigVar> optimizer_configs,
							 int api_version)
    { ... }

От меня что-то ускользает, или это так нормально?
http://opensource.adobe.com/svn/opensource/flex/sdk/trunk/modules/asc/src/java/macromedia/asc/embedding/Compiler.java

wvxvw, 18 Декабря 2011

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

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