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

    • ava nicolay-punin 21.01.2010 15:11 # 0

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

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