Resolve all warnings

This commit is contained in:
2023-10-30 22:08:02 +01:00
parent 08983ba30d
commit c337c3dfd5
5 changed files with 16 additions and 19 deletions

View File

@@ -64,8 +64,8 @@ namespace GerbilManagerWebAPI.Controllers
return NotFound(); return NotFound();
} }
Breeder updatedBreeder = existingItem with{ Breeder updatedBreeder = existingItem with {
Name = breederDto.Name Name = breederDto?.Name!
}; };
unitOfWork.BreederRepository.Update(updatedBreeder); unitOfWork.BreederRepository.Update(updatedBreeder);

View File

@@ -79,10 +79,10 @@ namespace GerbilManagerWebAPI.Controllers
} }
Gerbil updatedGerbil = existingItem with{ Gerbil updatedGerbil = existingItem with{
Breeder = unitOfWork.BreederRepository.GetByID(gerbilDto.Breeder), Breeder = unitOfWork.BreederRepository.GetByID(gerbilDto?.Breeder ?? Guid.Empty),
Litter = unitOfWork.LitterRepository.GetByID(gerbilDto.Litter), Litter = unitOfWork.LitterRepository.GetByID(gerbilDto?.Litter ?? Guid.Empty),
Gender = (Gender) (int) gerbilDto.Gender, Gender = (Gender) (int) gerbilDto?.Gender!,
Name = gerbilDto.Name Name = gerbilDto?.Name!
}; };
unitOfWork.GerbilRepository.Update(updatedGerbil); unitOfWork.GerbilRepository.Update(updatedGerbil);

View File

@@ -81,7 +81,7 @@ namespace GerbilManagerWebAPI.Controllers
Strength = litterDto.Strength, Strength = litterDto.Strength,
Father = this.unitOfWork.GerbilRepository.GetByID(litterDto.Father ?? Guid.Empty), Father = this.unitOfWork.GerbilRepository.GetByID(litterDto.Father ?? Guid.Empty),
Mother = this.unitOfWork.GerbilRepository.GetByID(litterDto.Mother ?? Guid.Empty), Mother = this.unitOfWork.GerbilRepository.GetByID(litterDto.Mother ?? Guid.Empty),
Name = litterDto.Name Name = litterDto?.Name!
}; };
unitOfWork.LitterRepository.Update(updatedLitter); unitOfWork.LitterRepository.Update(updatedLitter);

View File

@@ -16,8 +16,8 @@ namespace GerbilManagerWebAPI.DAL
} }
public virtual IEnumerable<TEntity> Get( public virtual IEnumerable<TEntity> Get(
Expression<Func<TEntity, bool>> filter = null, Expression<Func<TEntity, bool>> filter = null!,
Func<IQueryable<TEntity>, IOrderedQueryable<TEntity>> orderBy = null, Func<IQueryable<TEntity>, IOrderedQueryable<TEntity>> orderBy = null!,
string includeProperties = "") string includeProperties = "")
{ {
IQueryable<TEntity> query = dbSet; IQueryable<TEntity> query = dbSet;
@@ -45,7 +45,7 @@ namespace GerbilManagerWebAPI.DAL
public virtual TEntity GetByID(object id) public virtual TEntity GetByID(object id)
{ {
return dbSet.Find(id); return dbSet?.Find(id)!;
} }
public virtual void Insert(TEntity entity) public virtual void Insert(TEntity entity)
@@ -55,8 +55,8 @@ namespace GerbilManagerWebAPI.DAL
public virtual void Delete(object id) public virtual void Delete(object id)
{ {
TEntity entityToDelete = dbSet.Find(id); TEntity entityToDelete = dbSet?.Find(id)!;
Delete(entityToDelete); if (entityToDelete is not null) Delete(entityToDelete);
} }
public virtual void Delete(TEntity entityToDelete) public virtual void Delete(TEntity entityToDelete)

View File

@@ -6,9 +6,9 @@ namespace GerbilManagerWebAPI.DAL
public class UnitOfWork : IDisposable public class UnitOfWork : IDisposable
{ {
private ApplicationContext context; private ApplicationContext context;
private GenericRepository<Gerbil> gerbilRepository; private GenericRepository<Gerbil> gerbilRepository = null!;
private GenericRepository<Litter> litterRepository; private GenericRepository<Litter> litterRepository = null!;
private GenericRepository<Breeder> breederRepository; private GenericRepository<Breeder> breederRepository = null!;
public GenericRepository<Gerbil> GerbilRepository public GenericRepository<Gerbil> GerbilRepository
{ {
@@ -46,10 +46,7 @@ namespace GerbilManagerWebAPI.DAL
} }
} }
public UnitOfWork(ApplicationContext context) public UnitOfWork(ApplicationContext context) => this.context = context;
{
this.context = context;
}
public void Save() public void Save()
{ {