1. 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)

  2. Java / Говнокод #2520

    +78.6

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

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

    Loord, 01 Февраля 2010

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

  3. 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)

  4. 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)

  5. Java / Говнокод #2503

    +87.6

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public class CheckHouseNumber implements HouseNumberChecker {

//код

}

    chiffa, 30 Января 2010

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

  6. Java / Говнокод #2499

    +64.9

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

    nxx, 29 Января 2010

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

  7. 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)

  8. 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)

  9. 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)

  10. Java / Говнокод #2349

    +78.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    public void pause(){
        Object lock=new Object();
        synchronized(lock){
            try{
                lock.wait(1000);
            }catch(Exception e){
                System.out.println(e.getMessage());
            }
        }
    }

    _jk_, 29 Декабря 2009

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