2

Является ли это Best Practice, можно вообще так писать, а именно скажем если элемент уже в коллекции существует выбрасывать исключение с сообщением.

static private Set<A> a = new HashSet<>();   
public A save(A object) {
    if (!a.contains(object)) {
        a.add(object);
        return object;
    } else {
        throw new IllegalArgumentException("This object exists in the system already!");
    }
}
2
  • Если это удовлетворяет бизнес-логике, и метод не находится под высокой нагрузкой, то почему бы и нет.
    – Nofate
    5 апр 2016 в 16:57
  • Писать можно, но не думаю что это Best Practice.
    – Russtam
    5 апр 2016 в 17:29

1 ответ 1

4

Коллекция HashSet построена на базе HashMap. В обоих случаях, для метода contains() и add() выполняется итерация по элементам коллекции. Для HashSet / HashMap, в общем случае, это быстрые операции, которые выполняются за O(1), т.е. за константное время. Статья в тему.

Тем не менее, лишнего обхода коллекции можно избежать, исключив метод contains() поскольку метод add(E) и так проверяет наличие элемента и возвращает boolean:

if ( !a.add(object) ) {
    throw new IllegalArgumentException("...");
}
return object;

Также, я бы подумал нужно ли вообще возвращать что-то из метода. Из приведенного кода не видно смысла возвращать ссылку на объект, которая передана методу как аргумент. Если элемент не добавится, кидается RuntimeException, этого достаточно.

В целом, обычный код, без явного code smell. По одному методу не всегда можно судить об эффективности алгоритма. Если вам в этом месте нужен экспепшн - кидайте эксепшн. Это определяется только логикой вашего приложения.


2
  • 1
    это ответ не совсем на тот вопрос, что был у ТС как мне кажется.
    – Nofate
    5 апр 2016 в 20:11
  • 1
    возврат аргумента - это мелкое удобство, можно писать save( otherObject.getSomeA() ).someAMethod().anotherMethod(). Экономия точек с запятой :) Но в вопросе очень не хватает примера использования.
    – zRrr
    6 апр 2016 в 11:54

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.