хорошее имя для метода, который добавляет в контейнер, если не aleady там

Question

Какое хорошее название для метода, который добавляет что-то в контейнер, если его там еще нет, т.е.

void AddCustomerToList(CustomerList list, Customer customer)

но это имя неправильно указывает на то, что оно не будет добавлено, если его там еще нет. Какое имя лучше? AddCustomerToListIfNotThereAlready? EnsureCustomerInList

Answer 1

Измените CustomerList на CustomerSet, тогда это очевидно.

add(CustomerSet set, Customer customer);

Answer 2

Я бы выбрал что-то вроде AddIfMissing, хотя мне нравится идея переименования в Set, поскольку это действительно так.

public static class ListExtensions
{
      public static void AddIfMissing<T>( this List<T> list, T item )
      {
            if (!list.Contains(item))
            {
                list.Add( item );
            }
      }
}

Answer 3

bool TryAdd(CustomerList list, Customer customer)

Answer 4

AddIfNotPresent

Answer 5

Обычно «put» используется вместо «add» для передачи этого, , но Я согласен с Чейзом, что вы должны просто вызвать это «add» и использовать «set» вместо «list». Если, конечно, контейнер не поддерживает обе операции (что было бы странно).

Answer 6

Ты не сказал, что делаешь, если его там нет. Предполагая, что ответ «ничего», я бы пошел с.

bool InsertIfNew (CustomerList list, Customer customer)

Метод возвращает true, если он был «вставлен», и false, если он уже был там. Чтобы вызывающая сторона могла выполнять альтернативную логику, если запись уже была там. Возможно, вы не захотите этого делать, но кто-то может и у вас уже есть знания в рутине.

Answer 7

Может быть, просто назовите его AddCustomerToList и не заставляйте его проверять, не существует ли он уже, вместо этого в конце вызовите другой метод RemoveMultipleOccurences (..).

Answer 8

по-моему, вы задаете этот вопрос, так как вы разрабатываете ОО неоптимально:

void AddCustomerToList(CustomerList list, Customer customer)

ответственность за обеспечение присутствия клиента должна быть назначена CustomerList.

В этом случае вы называете свой метод:

• Add в случае, если вы укажете в документации, что клиент будет добавлен, только если нет
• AddIfNotPresent или PutIfNotPresent в противном случае.

Я предпочитаю последнее, так как оно более с автодокументами .

Answer 9

Сделать это двумя способами. IsCustomerPresent () AddCustomer (). Затем, если хотите, вы можете создать метод AddCustomerIfNotAlreadyPresent (), который просто вызывает вашу слабо связанную логику.

Answer 10

Вы можете сделать так, как это делает универсальный список , и создать функцию ContinsCustomer, чтобы проверить, существует ли она сначала, а затем использовать AddCustomerToList, если она возвращает false.