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

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

+171

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

public abstract class BaseDateTime
        extends AbstractDateTime
        implements ReadableDateTime, Serializable {

    /** The millis from 1970-01-01T00:00:00Z */
    private volatile long iMillis;
    /** The chronology to use */
    private volatile Chronology iChronology;

/////////////////////////////////////////////////////////////////	
/*	
 * DateTime is thread-safe and immutable, provided that the Chronology is as well.
 * All standard Chronology classes supplied are thread-safe and immutable.
 *
 * @see MutableDateTime
 */
public final class DateTime
        extends BaseDateTime

Любителям joda-time.
Cмущает меня этот volatile, который приходит в немутабельный класс от родителя.

3.14159265, 14 Октября 2013

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

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

+74

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

public class Encoder {
	public static void encode(final OutputStream out, Node node) throws IOException {
		node.accept(new NodeVisitor() {
			@Override
			public void string(StringNode node) {
				byte[] value = node.toByteArray();
				out.write(Integer.toString(value.length).getBytes(Constants.CHARSET));
				out.write(':');
				out.write(value);
			}
                        // ... другие методы для других типов нод ...
                }
        }
}

Решил поменять в паре-тройке модулей пачки ифов на паттерн visitor... И получил пинка от жабы ;(

write() кидает IOException, а значит и метод string() в анонимном классе тоже должен кидать, и метод string() в интерфейсе NodeVisitor тоже... Но ведь соседним посетителям этот IOException нахер не сдался...

Checked exceptions - зло.

bormand, 12 Октября 2013

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

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

+71

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

app_preferences = PreferenceManager.getDefaultSharedPreferences(context);
			
	        mApplication = (MyApplication)context;
if(mApplication.nomgor == 0 ){
              mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID", "0"));
		      mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID","");//в работе
              }
            if(mApplication.nomgor == 1 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID1", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID1","");//в работе
              }
            if(mApplication.nomgor == 2 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID2", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID2","");//в работе
               }
            if(mApplication.nomgor == 3 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID3", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID3","");//в работе
               }
            if(mApplication.nomgor == 4 ){
              mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID4", "0"));
              mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID4","");//в работе
              }
            if(mApplication.nomgor == 5 ){
              mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID5", "0"));
              mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID5","");//в работе
              }
            if(mApplication.nomgor == 6 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID6", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID6","");//в работе
               }
            if(mApplication.nomgor == 7 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID7", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID7","");//в работе
               }
            if(mApplication.nomgor == 8 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID8", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID8","");//в работе
               }
            if(mApplication.nomgor == 9 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID9", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID9","");//в работе
               }
            if(mApplication.nomgor == 10 ){
              mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID10", "0"));
              mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID10","");//в работе
              }
            if(mApplication.nomgor == 11 ){
              mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID11", "0"));
              mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID11","");//в работе
              }
            if(mApplication.nomgor == 12 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID12", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID12","");//в работе
               }
            if(mApplication.nomgor == 13 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID13", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID13","");//в работе
              }
            if(mApplication.nomgor == 14 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID14", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID14","");//в работе
               }
            if(mApplication.nomgor == 15 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID15", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID15","");//в работе
               }
            if(mApplication.nomgor == 16 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID16", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID16","");//в работе
               }
            if(mApplication.nomgor == 17 ){
               mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID17", "0"));
               mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID17","");//в работе
               }

кто-то с детства привык копипастить

grampe, 09 Октября 2013

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

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

+70

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

// there is class PlayerExt, which extends class Player...
// min >= 0
// max <= players.size()

    List<PlayerExt> players = playerManager.getPlayers(contestId);
    Player[] response = new Player[players.size()];

    for (int i = min; i < max; i++) {
        response[i] = players.get(i);
        if (!players.get(i).isQualified()) {
            response[i].setChipStack(BigDecimal.valueOf(-1));
        }
        response[i].setPosition(i + 1);
        response[i].setCustomerId(players.get(i).getCustomerId());
    }

Для таких начальных условий, как обозначено в комментарии в начале кода, формируем список игроков.
Особенно вдохновляет самая последняя инструкция в теле цикла.

wissenstein, 09 Октября 2013

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

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

+69

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

package first;

import java.util.Scanner;

public class First {

	public static void main(String[] args) {

		String line = "Empty directories can be moved. If the directory is not empty,";

		System.out.println("Enter simbol to delete:");
		Scanner s = new Scanner(System.in);
		String del = s.nextLine();
		s.close();

		StringBuilder sb = new StringBuilder(line);
		for (int i = 0; i < sb.length(); i++) {
			if (sb.charAt(i) == del.charAt(0)) {
				sb.deleteCharAt(i);
				
			}
		}

		System.out.println(sb);
	}

}

Это из раннего. Давно что-то не выкладывал, вот руки зачесались.

spivti, 06 Октября 2013

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

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

+64

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

public final class z extends t {
   public final String toString() {
      return "" + this.a + "!=" + this.b;
   }
   public final void a(bw var1) {
      this.a.a(var1);
      Object var2 = var1.b;
      this.b.a(var1);
      Object var3 = var1.b;
      if(var2 instanceof gh) {
         if(var3 instanceof gh) {
            var1.b = new gm(((gh)var2).a != ((gh)var3).a);
            return;
         }
         if(var3 instanceof ex) {
            var1.b = new gm(((gh)var2).a != ((ex)var3).a);
            return;
         }
         if(var3 instanceof ha) {
            var1.b = new gm(((gh)var2).a != ((ha)var3).a);
            return;
         }
         if(var3 instanceof hh) {
            var1.b = new gm((long)((gh)var2).a != ((hh)var3).a);
            return;
         }
         if(var3 instanceof a) {
            var1.b = new gm((float)((gh)var2).a != ((a)var3).a);
            return;
         }
      } else if(var2 instanceof ex) {
         if(var3 instanceof ex) {
            var1.b = new gm(((ex)var2).a != ((ex)var3).a);
            return;
         }
         if(var3 instanceof gh) {
            var1.b = new gm(((ex)var2).a != ((gh)var3).a);
            return;
         }
         if(var3 instanceof ha) {
            var1.b = new gm(((ex)var2).a != ((ha)var3).a);
            return;
         }
         if(var3 instanceof hh) {
            var1.b = new gm((long)((ex)var2).a != ((hh)var3).a);
            return;
         }
         if(var3 instanceof a) {
            var1.b = new gm((float)((ex)var2).a != ((a)var3).a);
            return;
         }
      } else if(var2 instanceof hh) {
         if(var3 instanceof hh) {
            var1.b = new gm(((hh)var2).a != ((hh)var3).a);
            return;
         }
         if(var3 instanceof ex) {
            var1.b = new gm(((hh)var2).a != (long)((ex)var3).a);
            return;
         }
         if(var3 instanceof ha) {
            var1.b = new gm(((hh)var2).a != (long)((ha)var3).a);
            return;
         }
         if(var3 instanceof gh) {
            var1.b = new gm(((hh)var2).a != (long)((gh)var3).a);
            return;
         }
         if(var3 instanceof a) {
            var1.b = new gm((float)((hh)var2).a != ((a)var3).a);
            return;
         }
      } else if(var2 instanceof ha) {
         if(var3 instanceof ha) {
            var1.b = new gm(((ha)var2).a != ((ha)var3).a);
            return;
         }
         if(var3 instanceof ex) {
            var1.b = new gm(((ha)var2).a != ((ex)var3).a);
            return;
         }
         if(var3 instanceof gh) {
            var1.b = new gm(((ha)var2).a != ((gh)var3).a);
            return;
         }
         if(var3 instanceof hh) {
            var1.b = new gm((long)((ha)var2).a != ((hh)var3).a);
            return;
         }
         if(var3 instanceof a) {
            var1.b = new gm((float)((ha)var2).a != ((a)var3).a);
            return;
         }
      } else if(var2 instanceof a) {
         if(var3 instanceof a) {
            var1.b = new gm(((a)var2).a != ((a)var3).a);
            return;
         }
         if(var3 instanceof ex) {
            var1.b = new gm(((a)var2).a != (float)((ex)var3).a);

полиморфизм? - не, не слышал.
даже не смотря на "защиту" от декомпиляции кода слегка доставил))
взято из декомпиленых кодов одной комэрчэской игры.
http://twwk.ru/ - она самая

lowercase, 04 Октября 2013

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

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

+80

01
02
03
04
05
06
07
08
09
10
11

...
SomeObject o = null;
try {
    o = someGetOperation();
} catch (Exception e) {}

try {
    o.doSomeMethod();
    ...
} catch (Exception e) {}
...

1. второй try/catch это просто защита от NPE, который возникнет, если не прошел первый try
2. catch-секции приведены как они есть в проекте - пустые и ловится всегда Exception

в 95% проекта (>1000 классов) используются такие конструкции

tir, 02 Октября 2013

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

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

+80

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

/**
   * Форматирует double, c arround кол-вом знаков после запятой
   * @param d double
   * @param arround int
   * @return String
   */
  public static String formatDouble(double d, int arround) {
    if (arround > 6) {
      throwIllegalFormat();
    }
    StringBuffer sb = new StringBuffer(10);
    boolean minus = d < 0;
    if (minus) {
      d = -d;
    }
    for (int i = 0; i < arround; i++) {
      d *= 10;
    }
    long l = (long) (d + 0.000000001);
    do {
      char digit = (char) ('0' + (l % 10));
      l = l / 10;
      sb.append(digit);
      if (--arround == 0) {
        sb.append('.');
      }
    } while (l != 0 || arround >= 0);
    if (minus) {
      sb.append('-');
    }
    return sb.reverse().toString();
  }

glprizes, 02 Октября 2013

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

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

+114

1

Ranges.closedOpen(0, list.size()).asSet(DiscreteDomains.integers())

Самый простой известный мне способ сделать аналог питоновского range(len(list)). Даже с Guava.

someone, 18 Сентября 2013

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

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

+77

1

float a = Float.parseFloat(new_size + "");

Приведение дабла ко флоту.

Vladiator, 11 Сентября 2013