Jawa / Говнокод #24037 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 anurin, (Updated )

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

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?


    8