GG
методах*
Size: a a a
2020 August 11
GG
впринципе решил просто мокая рес
GG
но незнаю насколько это норм
OR
можно немного философии?
как назвать функцию?
как назвать функцию?
getTaskByIdOrThrowNotFound(id: string): TaskModel {
const found = this.tasks.find((task) => task.id === id)
if (!found) {
throw new NotFoundException();
}
return found
}
не хочу название getTaskById - в функции removeTask я просто её вызываю не ожидая результата, в другой функции вызываю уже ожидая return.MZ
можно немного философии?
как назвать функцию?
как назвать функцию?
getTaskByIdOrThrowNotFound(id: string): TaskModel {
const found = this.tasks.find((task) => task.id === id)
if (!found) {
throw new NotFoundException();
}
return found
}
не хочу название getTaskById - в функции removeTask я просто её вызываю не ожидая результата, в другой функции вызываю уже ожидая return.Не хочешь а надо) а зачем в другой функции просто так вызывать?
MZ
Думаю вопрос в этом
OR
Не хочешь а надо) а зачем в другой функции просто так вызывать?
removeTaskById(id: string): void {
this.getTaskByIdOrThrowNotFound(id)
this.tasks = this.tasks.filter((task) => task.id !== id)
}MZ
Так, а может убрать экзепшен?
OR
Не хочешь а надо) а зачем в другой функции просто так вызывать?
getTaskById - плохо, потому что эта функция делает не только это.
MZ
Не найдено в массиве это ошибка разве?
MZ
Или бизнес кейс
OR
Не найдено в массиве это ошибка разве?
да, 404.
OR
ну юзер может же отправить другой id.
MZ
Я правильно понимаю что это таск менеджер для пользователей
MZ
И это удаление таска
OR
Я правильно понимаю что это таск менеджер для пользователей
+
MZ
Тогда тут не нужна ошибка и 404
OR
MZ
Ищем таску, проверяем есть ли, если да - удаляем, если нет - кидаем (внимание) БИЗНЕс ошибку
MZ
Типо
{
error: таски нет
}
{
error: таски нет
}