Как избежать дублированности наименования?

Question

[lucky4]

Есть продукт, каждый новый продукт должен отличаться названием. Как написать функцию, которая будет просматривать не совпадает ли найменование нового товара с любым другим товаром ?

public async Task<ProductDto> AddAsync(ProductDto item)
        {
            Product product = _mapper.Map<Product>(item);

            if (!await _unitOfWork.ProductRepository.IsDublicated(product.Label))
            {
                throw new DataDublication("Label already exists.");
            }

            await _unitOfWork.ProductRepository.AddAsync(product);

            return _mapper.Map<ProductDto>(product);
        }

public async Task<bool> IsDublicated(string label)
        {
            bool flag = false;

            var check = await _context.Set<Product>()
                .FirstOrDefaultAsync(property => property.Label.ToLower() == label.ToLower());

            return flag;
        }

Знаю, что проблема в функции IsDublicated, что возврощает все время false. Но, не понимаю, как мне закинуть соотвествующий статус после выполнения...

Answer 1

[Илья]

IsDublicated все время возвращает false потому что написана неверно. Вы инициализируете флаг потом возвращаете его без изменений.
Попробуйте так

public async Task<bool> IsDublicated(string label)
{
     var existedProduct = await _context.Set<Product>()
                .FirstOrDefaultAsync(property => property.Label.ToLower() == label.ToLower());
     return existedProduct != null;
}

И в функции AddAsync у вас неверное условие. Должно быть:

if (await _unitOfWork.ProductRepository.IsDublicated(product.Label))
{
     throw new DataDublication("Label already exists.");
}

Comments

[lucky4]

Да, Вы правы. Я когда заливал проблемкУ, забросил старый вариант...

public async Task<bool> IsDublicated(string label)
        {
            bool flag = false;

            var check = await _context.Set<Product>()
                .FirstOrDefaultAsync(property => property.Label.ToLower() == label.ToLower());

            if (check.Label == label)
            {
                return flag = true;
            }

            return flag;
        }

Тут я неправильно сформулировался и вопрос, и описание. Сейчас вроде как выглядит правильно, но когда FirstOrDefaultAsync добавляет того чего нет в БД, он возвращает нулл же и идет ошибка Object reference not set to an instance of an object. ДА! Я понял Вас, СПАСИБО! Я копался, что мне другое возвращаться и не додумался до != null. Я пытался, что-то с GetAll возвращать и тд))))

Answer 2

[Владимир Коротенко]

Выбирайте из продуктов по StartWith после парусите индекс выбирайте максимальный и вставляйте добавив 1

Comments

[lucky4]

спасибо! да зачем так усложнять), если будет дублированная запись - ошибку вынести и все.
тут проблема с FirstOrDefaultAsync больше. Он когда видит, что нет такой записи - становится нуллом, что призводит к ошибке.

[Владимир Коротенко]

lucky4, я про дубли не услышал постучитесь в контакты возможно помогу