Телеграмм чат группы nest_ru страница 3283

parrot.ru

Рейтинг популярных групп и каналов

В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

NestJS — русскоязычное сообщество

1310 membersпожаловаться на группу

2020 August 11

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

можно немного философии?
как назвать функцию?

getTaskByIdOrThrowNotFound(id: string): TaskModel {
  const found = this.tasks.find((task) => task.id === id)
  if (!found) {
    throw new NotFoundException();
  }
  return found
}

не хочу название getTaskById - в функции removeTask я просто её вызываю не ожидая результата, в другой функции вызываю уже ожидая return.

Здесь нарушается паттерн синглтон

источник

18:20пожаловаться #1

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

Идёт и гет и проверка

источник

18:20пожаловаться #2

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

Если функция называется типо get...and...or... значит она сложная и ее нужно декомпозироваьь)))

источник

18:20пожаловаться #3

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

В данном случае проверка и ошибка лишняя

источник

18:21пожаловаться #4

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

Проверку вынести в родителя, ошибку убрать

источник

18:21пожаловаться #5

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

И код будет чище и читабельнее

источник

18:21пожаловаться #6

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество

Maksim Zakharov

И код будет чище и читабельнее

так что ли?

источник

18:22пожаловаться #7

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

removeTaskById(id: string): void {
  this.getTaskByIdOrThrowNotFound(id)
  this.tasks = this.tasks.filter((task) => task.id !== id)
}

Причём вот это это код контроллера

источник

18:22пожаловаться #8

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество

Maksim Zakharov

Причём вот это это код контроллера

почему?

источник

18:22пожаловаться #9

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

так что ли?

Похоже

источник

18:23пожаловаться #10

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

Да норм

источник

18:23пожаловаться #11

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество

Maksim Zakharov

Да норм

ну а дублирование?

источник

18:23пожаловаться #12

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

Ну тогда сделай функцию checkTaskExist

источник

18:24пожаловаться #13

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

И в неё эти 2

источник

18:24пожаловаться #14

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество

Maksim Zakharov

Ну тогда сделай функцию checkTaskExist

она же будет как это выглядеть https://t.me/nest_ru/69999

Oleg Rizhkov in NestJS — русскоязычное сообщество

можно немного философии?
как назвать функцию?
getTaskByIdOrThrowNotFound(id: string): TaskModel {
const found = this.tasks.find((task) => task.id === id)
if (!found) {
throw new NotFoundException();
}
return found
}
не хочу название getTaskById - в функции removeTask я просто её вызываю не ожидая результата, в другой функции вызываю уже ожидая return.

источник

18:24пожаловаться #15

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

она же будет как это выглядеть https://t.me/nest_ru/69999

Oleg Rizhkov in NestJS — русскоязычное сообщество

можно немного философии?
как назвать функцию?
getTaskByIdOrThrowNotFound(id: string): TaskModel {
const found = this.tasks.find((task) => task.id === id)
if (!found) {
throw new NotFoundException();
}
return found
}
не хочу название getTaskById - в функции removeTask я просто её вызываю не ожидая результата, в другой функции вызываю уже ожидая return.

Тут уже да, вопрос про читабельность

источник

18:26пожаловаться #16

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

так что ли?

Тут код дублируется но он читабельный

источник

18:26пожаловаться #17

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

она же будет как это выглядеть https://t.me/nest_ru/69999

Oleg Rizhkov in NestJS — русскоязычное сообщество

можно немного философии?
как назвать функцию?
getTaskByIdOrThrowNotFound(id: string): TaskModel {
const found = this.tasks.find((task) => task.id === id)
if (!found) {
throw new NotFoundException();
}
return found
}
не хочу название getTaskById - в функции removeTask я просто её вызываю не ожидая результата, в другой функции вызываю уже ожидая return.

Тут кода меньше, но ты под капотом кидаешь ошибки, а это не очевидно

источник

18:26пожаловаться #18

OR

Oleg Rizhkov in NestJS — русскоязычное сообщество

Maksim Zakharov

Тут кода меньше, но ты под капотом кидаешь ошибки, а это не очевидно

то есть, лучше дублирование? не стоит всегда в dry ударяться?

источник

18:27пожаловаться #19

MZ

Maksim Zakharov in NestJS — русскоязычное сообщество

то есть, лучше дублирование? не стоит всегда в dry ударяться?

Если говорить про чистый код, командную работу, то лучше читабельность :)

источник

18:28пожаловаться #20