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

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

+73

1

if (fooTextField.getText().equalsIgnoreCase("")) {

Не то, чтобы очень говно...
Просто порадовало :)

myzone, 04 Июня 2013

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

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

+114

1

switch (Strings.nullToEmpty(value)) { ... }

Жабовский string switch не умеет в null. Приходится так.

someone, 04 Июня 2013

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

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

+68

1
2
3
4
5
6
7
8

private void setFooMode() {
    if (foo.isSelected()) { // is currently showing
        App.appProps.put(Strings.BAR_MODE, Strings.TRUE);
    }
    else {
        App.appProps.put(Strings.BAR_MODE, Strings.FALSE);
    }
}

myzone, 03 Июня 2013

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

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

+70

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

package <hostname>.sbrf.taxaccounting.service.impl;

import org.junit.Test;

import <hostname>.sbrf.taxaccounting.service.FormDataService;

public class FormDataServiceTest {
	private FormDataService formDataService;
	
	public void tearUp() {
		 formDataService = new FormDataServiceImpl();
	}
	
	@Test
	public void testCheckMandatoryColumns() {
		
	}
}

Зато он используются юнит тесты
и посмотрите внимательно кому это делается)))
вот такие вот у нас тесты :)

smpl, 03 Июня 2013

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

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

+76

1

https://github.com/mongodb/mongo-java-driver/blob/master/src/main/com/mongodb/ConnectionStatus.java#L213

wtf?!

orion, 31 Мая 2013

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

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

+77

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

public CommandResult update() {
            CommandResult res = null;
            try {
                long start = System.nanoTime();
                res = _port.runCommand(_mongo.getDB("admin"), isMasterCmd);
                long end = System.nanoTime();
                float newPingMS = (end - start) / 1000000F;
                if (!successfullyContacted)
                    _pingTimeMS = newPingMS;
                else
                    _pingTimeMS = _pingTimeMS + ((newPingMS - _pingTimeMS) / latencySmoothFactor);

                getLogger().log(Level.FINE, "Latency to " + _addr + " actual=" + newPingMS + " smoothed=" + _pingTimeMS);

                successfullyContacted = true;

                if (res == null) {
                    throw new MongoInternalException("Invalid null value returned from isMaster");
                }

                if (!_ok) {
                    getLogger().log(Level.INFO, "Server seen up: " + _addr);
                }
                _ok = true;

                // max size was added in 1.8
                if (res.containsField("maxBsonObjectSize")) {
                    _maxBsonObjectSize = (Integer) res.get("maxBsonObjectSize");
                } else {
                    _maxBsonObjectSize = Bytes.MAX_OBJECT_SIZE;
                }
            } catch (Exception e) {
                if (!((_ok) ? true : (Math.random() > 0.1))) {
                    return res;
                }

                final StringBuilder logError = (new StringBuilder("Server seen down: ")).append(_addr);

                if (e instanceof IOException) {

                    logError.append(" - ").append(IOException.class.getName());

                    if (e.getMessage() != null) {
                        logError.append(" - message: ").append(e.getMessage());
                    }

                    getLogger().log(Level.WARNING, logError.toString());

                } else {
                    getLogger().log(Level.WARNING, logError.toString(), e);
                }
                _ok = false;
            }

            return res;
        }

https://github.com/mongodb/mongo-java-driver/blob/master/src/main/com/mongodb/ConnectionStatus.java

Незаметен.

serpinski, 31 Мая 2013

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

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

+65

1
2
3
4
5
6

// вот такой вот паттерн инициализации статических переменных во всех классах проекта... 
private static Properties globalProps = null;

static {
        globalProps = new Properties();
}

вот такой вот паттерн инициализации статических переменных во всех классах проекта...ин-лайн инициализацию автору делать почему то не хотелось...и ведь вроде не индус писал, а белый человек...

aa_kovalev, 29 Мая 2013

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

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

+69

1
2
3

void superFire(ActionEvent evt) {
    super.fireActionPerformed(evt);
}

пиу-пиу!!111

myzone, 28 Мая 2013

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

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

+66

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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99

package islands.general.playerlist;

import islands.general.player.Player;
import java.io.Externalizable;
import java.io.IOException;
import java.io.ObjectInput;
import java.io.ObjectOutput;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

/**
 * Список игроков.
 *
 * @author Григорий
 */
public class PlayerList implements Externalizable, Iterable<Player> {

    private static final long serialVersionUID = -8237639142683577971L;
    private static final Player[] emptyPlayerArray = new Player[0];
    private List<Player> list;
    private int maxIndex;

    public PlayerList() {
        this(emptyPlayerArray);
    }

    public PlayerList(Player... players) {
        list = new ArrayList<Player>();
        maxIndex = 0;
        for (int i = 0; i < players.length; i++) {
            Player player = players[i];
            add(player);
        }
    }

    /**
     * Добавить нового игрока в конец списка.
     *
     * @param player новый игрок
     * @return индекс нового игрока
     */
    public final int add(Player player) {
        list.add(maxIndex, player);
        return maxIndex++;
    }

    /**
     * Установить нового игрока по указанному индексу.
     *
     * @param index индекс для установки
     * @param player новый игрок
     */
    public void set(int index, Player player) {
        list.set(index, player);
    }

    /**
     * Возвратить количество игроков.
     *
     * @return количество игроков
     */
    public int getLength() {
        return maxIndex;
    }

    /**
     * Возвратить игрока по указанному индексу.
     *
     * @param index индекс игрока
     * @return игрок
     */
    public Player get(int index) {
        return list.get(index);
    }

    @Override
    public void writeExternal(ObjectOutput out) throws IOException {
        out.writeInt(maxIndex);
        for (int i = 0; i < maxIndex; i++) {
            list.get(i).writeExternal(out);
        }
    }

    @Override
    public void readExternal(ObjectInput in) throws IOException, ClassNotFoundException {
        maxIndex = in.readInt();
        for (int i = 0; i < maxIndex; i++) {
            Player player = new Player();
            player.readExternal(in);
            list.add(player);
        }
    }

    @Override
    public Iterator<Player> iterator() {
        return list.iterator();
    }
}

Dougrinch, 27 Мая 2013

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

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

+115

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

/**
 * Converts the Accpac fields to names that do not
 * require Sherlock Holmes to decipher.
 * 
 * @param arcus Accpac customer object
 */
public Store(final ARCus arcus) {
	name = trim(arcus.getIdcust());
	description = trim(arcus.getNamecust());
	addressLine1 = trim(arcus.getTextstre1());
	addressLine2 = trim(arcus.getTextstre2());
	addressLine3 = trim(arcus.getTextstre3());
	addressLine4 = trim(arcus.getTextstre4());
	suburb = trim(arcus.getNamecity());
	state = trim(arcus.getCodestte());
	postalCode = trim(arcus.getCodepstl());
	country = trim(arcus.getCodectry());
	contactName = trim(arcus.getNamectac());
	phone1 = trim(arcus.getTextphon1());
	phone2 = trim(arcus.getTextphon2());
	email = trim(arcus.getEmail2());
	department = arcus.getAudtorg();
}

someone, 27 Мая 2013

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

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