Коллекция HashSet
построена на базе HashMap
. В обоих случаях, для метода contains()
и add()
выполняется итерация по элементам коллекции. Для HashSet
/ HashMap
, в общем случае, это быстрые операции, которые выполняются за O(1)
, т.е. за константное время. Статья в тему.
Тем не менее, лишнего обхода коллекции можно избежать, исключив метод contains()
поскольку метод add(E) и так проверяет наличие элемента и возвращает boolean:
if ( !a.add(object) ) {
throw new IllegalArgumentException("...");
}
return object;
Также, я бы подумал нужно ли вообще возвращать что-то из метода. Из приведенного кода не видно смысла возвращать ссылку на объект, которая передана методу как аргумент. Если элемент не добавится, кидается RuntimeException
, этого достаточно.
В целом, обычный код, без явного code smell. По одному методу не всегда можно судить об эффективности алгоритма. Если вам в этом месте нужен экспепшн - кидайте эксепшн. Это определяется только логикой вашего приложения.