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

    • мрак... по хорошему бы в модель бы эту фигню....
      или уж имя метода адекватное на худой конец...
      Ответить

    Добавить комментарий