Телеграмм чат группы nodejs

Size: a a a

Node.js — русскоговорящее сообщество

2021 January 14

Oleg Shpatacovski in Node.js — русскоговорящее сообщество

Подскажите, проверка кода ругается:

Possible iteration over unexpected (custom / inherited) members, probably missing hasOwnProperty check

На этот код:

  const tmp = []
  for (const key in obj) {
    tmp.push({
      id: key,
      name: nameMatrix[key]?.name ?? '',
      short_name: nameMatrix[key]?.shortName ?? '',
      value: obj[key]
    })
  }

я же правильно пониматю, что optional chaining там закрывает проблему. Или я что-то не учел?

тебе IDE рекомендует проверить наличие поле key в обьекте nameMatrix

источник

19:08пожаловаться #1

Oleg Shpatacovski in Node.js — русскоговорящее сообщество

nameMatrix.hasOwnProperty(key)

источник

19:09пожаловаться #2

Oleg Shpatacovski in Node.js — русскоговорящее сообщество

и obj.hasOwnProperty(key)

источник

19:11пожаловаться #3

Nikita in Node.js — русскоговорящее сообщество

Правильно ли использовать HTTP коды ответа для сообщения об ошибке приложения в API? По типу 404 если не найдена запись в бд, или 401 если токен не передан в запросе. Лучше ли это того чтобы отвечать всегда 200 а саму ошибку (или ее отсутствие) указывать в теле ответа?

источник

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

Oleg Shpatacovski in Node.js — русскоговорящее сообщество

Если не найдена запись то присылай пустой обьект

источник

19:22пожаловаться #5

Oleg Shpatacovski in Node.js — русскоговорящее сообщество

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

источник

19:23пожаловаться #6

Pavel in Node.js — русскоговорящее сообщество

Nikita