Си диез / Говнокод #2580 Ссылка на оригинал

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
//Dictionary used to check if eventValues are unique
Dictionary<string, string> uniqueEventValues = new Dictionary<string, string>();

//... заполняем коллекцию

try {
    uniqueEventValues.Add(eventValue, "X");
}
catch (Exception ex) {
    if (ex.Message.Equals("An item with the same key has already been added.")) {
        msgAddEventParams.Text = "EventValues must be unique. " + eventValue + " is duplicate.";
        msgAddEventParams.Visible = true;
        return;
    }
}

мы не ищем легких путей!

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

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

  • вот удивится автор запустив программу в русской локали...
    Ответить
    • с таким кодом, не дойдёт до русских локалей 🙂
      Ответить
    • автор не в курсе, что есть ContainsKey... а сравнивать по Exception:Message - это вообще шик.
      Ответить
      • ContainsKey не всегда хорош.
        Он шерстит всю коллекцию, чтобы потом всю коллекцию прошерстить ещё раз (в методе Add). Если коллекция оч большая, лучше надеяться на исключения. Как ни странно - будет быстрее, ибо по коллекции проход только один.

        Алсо, использование String.Equals выдаёт жабакодера.
        Ответить

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

Из-за тебя ушел bormand, guest!

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


    8