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

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
  13. 13
  14. 14
  15. 15
  16. 16
//...
        Hashtable somehash = new Hashtable();
//...

        public String keyByValue (String value) {
            String s;

                while (somehash.elements().hasMoreElements()) {
                    s = (String) somehash.elements().nextElement(); //!!!

                    if (s.equals(value)) { //!!!
                     return s;
                    }
                }
            return null;
        }

Запостил: Cr0s Cr0s, (Updated )

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

  • ava Lure Of Chaos # Ссылка на оригинал 0

    знаете, как обойтись без цикла?

    конечно, Hashtable и цикл через Enumeration это здорово, но это все мелочи, просто старый код.
    Это кстати подтверждается отсутствие generics
    Ответить

  • ava Мистер Хэнки # Ссылка на оригинал 0

    Бред какой-то...
    Судя по названию метод должен возвращать ключ, соответствующий переданному значению value (или самый первый из ключей, если их несколько, содержащих value).
    Но судя по коду (метод elements() возвращает список не ключей, а значений) тут просто производится проверка наличия в хэше значения value, да и еще возврат переданного значения в случае успеха xD
    Ответить

    • ava Cr0s # Ссылка на оригинал 0 ⇈ 

      s = (String) somehash.elements().nextElement();

      Это самый смак, благодаря которому, метод образует бесконечный цикл.
      Ответить

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

Семь раз отмерь — один отрежь, guest!

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

    ↑
    8
    ↓