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

    +114

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

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

    someone, 04 Июня 2013

    Комментарии (131)
  2. Java / Говнокод #13092

    +68

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 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)
  3. Java / Говнокод #13091

    +70

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 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)
  4. Java / Говнокод #13085

    +76

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

    wtf?!

    orion, 31 Мая 2013

    Комментарии (13)
  5. Java / Говнокод #13083

    +77

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 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)
  6. Java / Говнокод #13064

    +65

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    // вот такой вот паттерн инициализации статических переменных во всех классах проекта... 
    private static Properties globalProps = null;
    
    static {
            globalProps = new Properties();
    }

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

    aa_kovalev, 29 Мая 2013

    Комментарии (7)
  7. Java / Говнокод #13061

    +69

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

    пиу-пиу!!111

    myzone, 28 Мая 2013

    Комментарии (10)
  8. Java / Говнокод #13058

    +66

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    96. 96
    97. 97
    98. 98
    99. 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)
  9. Java / Говнокод #13054

    +115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 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)
  10. Java / Говнокод #13041

    +69

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    public class Tree {
        
        ...
    
        public static int treeDepth;
    
        ...
    
        public static void runDepthTree(Node start, String method) {
    //        System.out.println("looking at " + start);
            ++treeDepth;
            Statement statement = buildStatement(start, method);
            try {
                statement.execute();
            }
            catch (Exception e) { 
                String msg = start + " ERROR at tree depth= " + treeDepth;
                
                System.out.println(); 
            }
            Node[] nodes = start.getChildren();
            if (nodes != null) {
                int count = start.getComponentCount();
                for (int i = 0; i<count; i++)  {
                    runDepthTree((Node)nodes[i], method); // recurse
                }
            }
            --treeDepth;
        }
    
        ...
    
    }

    Я бы не точно догадался

    myzone, 23 Мая 2013

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