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

    +77.2

    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
    /**
     * todo не самый лучший способ обработки завершения диалогов
     * 
     * @author roman
     */
    public final class MyBoolean
    {
        private boolean value = false;
        public boolean getValue()
        {
            return value;
        }
        public void setValue(
            boolean value )
        {
            this.value = value;
        }
    }

    ctepx, 09 Февраля 2010

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

    +62

    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
    public class SendProtocol
    {
            public static String SESSIONCLOSED = "SESSIONCLOSED";
            public static String MSG = "MSG";
            public static String LOGIN = "LOGIN";
            public static String LOGINRESULT = "LOGINRESULT";
            public static String JOINROOM = "JOINROOM";
            public static String JOINROOMRESULT = "JOINROOMRESULT";
            public static String LEFTROOM = "LEFTROOM";     
            public static String JOINTABLERESULT = "JOINTABLERESULT";
            public static String JOINTABLE = "JOINTABLE";
            public static String LEFTTABLE = "LEFTABLE";
            public static String INVITE = "INVITE";
            public static String INVITERESULT = "INVITERESULT";
            public static String REPLYINVITEPLAY = "REPLYINVITEPLAY";
            public static String DEALCARD = "DEALCARD";
            public static String GETCARD = "GETCARD";
            public static String TAKECARD = "TAKECARD";     
            public static String DROPPHOM = "DROPPHOM";
            public static String PLAYCARD = "PLAYCARD";
            // Biến tạo phòng chơi
            public static String CREATEROOM = "CREATEROOM";
            public static String CREATETABLE = "CREATETABLE";
            public static String ADDPLAYERTOPLAYTABLE = "ADDPLAYERTOPLAYTABLE";
            //
            public static String CHANGEVIEW = "CHANGEVIEW";
            public static String CREATEPLAYER = "CREATEPLAYER";
            public static final String TABLESTATUS = "TABLESTATUS";
    }

    http://code.google.com/p/phom-javagame/source/browse/trunk/src/com/bitjsc/phom/server/SendProtocol.java

    danilissimus, 04 Февраля 2010

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

    +78.6

    1. 1
    2. 2
    if(data==null)
           throw new ServerException("L0050",data.number);

    Видимо невнимательность автора

    Loord, 01 Февраля 2010

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

    +65.5

    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
    private static String getUTF8String(byte[] b, int off, int len) {
    	// First, count the number of characters in the sequence
    	int count = 0;
    	int max = off + len;
    	int i = off;
    	while (i < max) {
    	    int c = b[i++] & 0xff;
    	    switch (c >> 4) {
    	    case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7:
    		// 0xxxxxxx
    		count++;
    		break;
    	    case 12: case 13:
    		// 110xxxxx 10xxxxxx
    		if ((int)(b[i++] & 0xc0) != 0x80) {
    		    throw new IllegalArgumentException();
    		}
    		count++;
    		break;
    	    case 14:
    		// 1110xxxx 10xxxxxx 10xxxxxx
    		if (((int)(b[i++] & 0xc0) != 0x80) ||
    		    ((int)(b[i++] & 0xc0) != 0x80)) {
    		    throw new IllegalArgumentException();
    		}
    		count++;
    		break;
    	    default:
    		// 10xxxxxx, 1111xxxx
    		throw new IllegalArgumentException();
    	    }
    	}
    	if (i != max) {
    	    throw new IllegalArgumentException();
    ....

    В либе работы с зипом

    Cdf-EaSy, 01 Февраля 2010

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

    +81.7

    1. 1
    2. 2
    int page = ServletRequestUtils.getIntParameter(request, "page2", 0);
    int page2 = ServletRequestUtils.getIntParameter(request, "page", 0);

    striker, 01 Февраля 2010

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

    +87.6

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public class CheckHouseNumber implements HouseNumberChecker {
    
    //код
    
    }

    chiffa, 30 Января 2010

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

    +64.9

    1. 1
    2. 2
    3. 3
    // ...
    private Blob lClob; //файл
    // ...

    nxx, 29 Января 2010

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

    +77.3

    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
    protected static Map<Long, List<Bank.BankMoney>> getWinnersBanksMap(Map<Long, List<Bank.BankMoney>> sharedBank) {
            Map<Long,List<Bank.BankMoney>> winnersWithBanks = new HashMap<Long, List<Bank.BankMoney>>();
    
            for (Map.Entry<Long, List<Bank.BankMoney>> longListEntry : sharedBank.entrySet()) {
                    Long playerId = longListEntry.getKey();
                    List<Bank.BankMoney> banksWonOnThisIteration = longListEntry.getValue();
    
                    List<Bank.BankMoney> banks1 = winnersWithBanks.get(playerId);
                    if (banks1 == null) {
                        banks1 = new ArrayList<Bank.BankMoney>();
                        winnersWithBanks.put(playerId, banks1);
                    }
                    banks1.addAll(banksWonOnThisIteration);
                }
            return winnersWithBanks;
        }

    В результате серии рефакторингов чужого кода обнаружил следующий метод, над которым долго ломал голову - что он делает... Оказалось, ничего. Возвращает копию исходного объекта.

    dvb, 21 Января 2010

    Комментарии (1)
  9. Java / Говнокод #2444

    +76.4

    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
    public String getServiceClient(final Service service) {
            SqlDao dao = (SqlDao) getBean("sqlDao");
            return (String) dao.getTemplate().query(SQL_GET_CLIENT, new Long[]{service.getClient().getId()}, new ResultSetExtractor() {
                public Object extractData(ResultSet rs) throws SQLException, DataAccessException {
                    String result;
                    SqlDao dao = (SqlDao) getBean("sqlDao");
                    rs.next();
                    if (rs.getInt("IS_ORG") == 1) {
                        result = (String) dao.getTemplate().query(SQL_GET_LEGAL_INFO, new Long[]{service.getClient().getId()}, new ResultSetExtractor() {
                            public Object extractData(ResultSet rsi) throws SQLException, DataAccessException {
                                if (rsi.next()) {
    			        return rsi.getString("C_INN") + ", " + rsi.getString("C_NAME");
                                } else {
                                    log.debug("====== No legal client found for service: " + service.getRegistrationNumber() + " (" + service.getId() + ")");
                                    return null;
                                }
                            }
                        });
                     } else {
                         result = (String) dao.getTemplate().query(SQL_GET_PERSONINFO, new Long[]{service.getClient().getId()}, new ResultSetExtractor() {
                             public Object extractData(ResultSet rsi) throws SQLException, DataAccessException {
                                 if (rsi.next()) {
                                     return rsi.getString("LASTNAME") + " " + rsi.getString("FIRSTNAME") + " " + rsi.getString("MIDDLENAME");
                                 } else {
                                     log.debug("====== No phisical client found for service: " + service.getRegistrationNumber() + " (" + service.getId() + ")");
                                     return null;
                                 }
                            }
                        });
                    }
                    return result;
                }
          });
    }

    Я конечно все понимаю, deadline, все эти вещи... Но return на 20 с чем-то строк это сильно.

    nagakhl, 19 Января 2010

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

    +68.2

    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
    public Vector<SystemUser> listAllSystemUsers()
            {
    
                Vector <SystemUser> v = new Vector<SystemUser>();
    
                Query q = manager.createNativeQuery("SELECT * FROM SystemUser;");
                List list  = (List)q.getResultList();
                for (int i=0; i<list.size(); i++)
                {
                try {
                    Vector a = (Vector) list.iterator().next();
                    SystemUser SU = new SystemUser();
                    System.out.print("99*1 > "+SU.getClass().getMethods()[1].getName());
                    SU.setId(new Integer (a.get(0).toString()));
                    SU.setLogin(
                                new String (
                                    new BASE64Decoder().decodeBuffer(a.get(1).toString())
                                    ));
                    System.out.print("506*1*1+0 > " + SU);
                } catch (SecurityException ex) {
                    Logger.getLogger(UserManagement.class.getName()).log(Level.SEVERE, null, ex);
                }catch (IOException e){
                    
                }
                }
                return v;
            }

    Однако O_o

    AlDjabad, 12 Января 2010

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