refactoring
This commit is contained in:
1 parent
1871b98963
commit
b5a7a01ddf
1 file changed
+6
-2
@@ -48,20 +48,24 @@ namespace api.Services
|
|||||||
|
|
||||||
await personExists(username);
|
await personExists(username);
|
||||||
|
|
||||||
Person updatedPerson = Person.Create(username: personDto.Username, realName: personDto.RealName, birthday: personDto.Birthday);
|
Person updatedPerson = Person.Create(username: username, realName: personDto.RealName, birthday: personDto.Birthday);
|
||||||
await _personRepository.UpdateAsync(updatedPerson);
|
await _personRepository.UpdateAsync(updatedPerson);
|
||||||
}
|
}
|
||||||
|
|
||||||
public async Task DeletePersonAsync(string username)
|
public async Task DeletePersonAsync(string username)
|
||||||
{
|
{
|
||||||
await personExists(username);
|
await personExists(username);
|
||||||
|
|
||||||
|
await _personRepository.DeleteAsync(username);
|
||||||
}
|
}
|
||||||
|
|
||||||
private async Task<Person> personExists(string username)
|
private async Task<Person> personExists(string username)
|
||||||
{
|
{
|
||||||
|
username = Person.NormalizeUsername(username);
|
||||||
|
|
||||||
Person? person = await _personRepository.GetByUsernameAsync(username);
|
Person? person = await _personRepository.GetByUsernameAsync(username);
|
||||||
|
|
||||||
if (person == null) throw new KeyNotFoundException("Person not found");
|
if (person == null) throw new KeyNotFoundException($"Person '{username}' not found");
|
||||||
|
|
||||||
return person;
|
return person;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in new issue
Block a user