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

    +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
    public class TradingAccounts {
        private static HashMap<String, TradingAccounts> hashMap = new HashMap<String, TradingAccounts>();
        ....
        public static void clear() throws Exception {
            hashMap.clear();
            TradingAccounts[] all = getAll();
            for (TradingAccounts acc : all) {
                hashMap.put(acc.getAccount().trim(), acc);
            }
        }
        ....
    }

    Production code.
    При рефакторинге LEGACY приложения утерян вызов
    TradingAccounts.clear()
    По факту это привело к ошибке, т.к. этот справочник всегда оставался пустым.
    Кто бы мог догадаться, что метод clear загружает данные из БД...

    Запостил: anurin, 30 Марта 2018

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

    • прежде чем рефакторить LEGACY приложение нужно написать тесты
      Ответить
      • Протестировал твой анус, теперь полгода его рефакторить будешь.
        Ответить

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