Как обработать ошибку выборки элемента (что делать если элемент не найден)?

Question

[Назарий Лазарчук]

Дано:

• приложение на TypeScript / JavaScript, Node.js
  
• база данных (в моем случае SQLite 3, хотя неважно)
  

Делаю запрос к базе данных
SELECT * FROM companies WHERE id = ?

Если элемент с такое id существует в таблице то мне приходить объект, а если нет то приходит `undefined`.
Как будет правильно в таком случае:

1. Каждый раз делать проверку на undefined. Т.e:
   

   const company = await CompanyModel.getById(228);
   if (!company) console.log('Такой компании не существует');

   
   
   
2. Или выбросить какую-нибудь ошибку и обработать ее (как например 404 в вебе)
   
   

   try {
     const company = await CompanyModel.getById(228);
   } catch (err) {
     console.log('Что-то пошло не так!');
     console.error(err); // хз какого типа Error, через какой инстанс его создать?
   }

   
   
   

Я ближе ко второму варианту, как минимум потому что мне всегда придется обрабатывать ошибки любых запросов, так почему бы и не обработать `undefined`.
Так же, как аргумент использование null/undefined это очень плохо, как минимум это усложняет код (например вот пример с книги одного гуру по джаве https://www.yegor256.com/2014/05/13/why-null-is-ba...)

Comments

[Сергей Горностаев]

Функциональное программирование и ООП тут при чём?

[Назарий Лазарчук]

Сергей Горностаев, я оставил теги, потому что там реализация фп + ооп. Могу убрать впринципе

[Сергей Горностаев]

Назарий Лазарчук, с тегами лучше быть предельно конкретным, так как модераторы за нарушение пункта 3.1 правил сносят вопросы без предупреждения.

[WbICHA]

Назарий Лазарчук, а тс тут каким боком?

[Назарий Лазарчук]

WbICHA,

приложение на TypeScript / JavaScript, Node.js

[WbICHA]

Назарий Лазарчук, и что? Давай ещё все библиотеки из проекта сюда приплети. Какая проблема с тайпскриптом в данном вопросе?

Answer 1

[Alex Step]

Вот так

const company = await CompanyModel.getById(228);
if (!company) throw new Error('Такой компании не существует');

getById не должен выбрасывать исключение если запись не найдена пусть просто возвращает null

Comments

[Назарий Лазарчук]

а почему? Есть какие-то аргументы, ссилки, best-practice?

[Alex Step]

Назарий Лазарчук, ну а почему js метод массива find не выбрасывает ошибку в случае если элемент не найден?
Потому что это не исключение, а вполне ожидаемое поведение. Так работают методы получения во всех orm.

При таком подходе код будет во многих местах проще, без try / catch можно будет проверить наличие чего либо.

Решать является ли отсутствие элемента ошибкой или нет нужно уровнем выше. Это не дело orm.

[Alex Step]

Назарий Лазарчук, в статье что ты привел в пример модель базы данных возвращает 0 в случае если пользователь не найден.

У тебя CompanyModel судя по названию это модель

И вот уже функция в котором эта модель используется, в зависимости от контекста решает, ошибка это или нет .

[Назарий Лазарчук]

в статье что ты привел в пример модель базы данных возвращает 0

не совсем.
Там в примере автор считает что модель должна отдавать либо ошибку, либо переоприделение обьекта `Employee` в которого методы выдают ошибку. Но очень как-то сложно реализовано

[Назарий Лазарчук]

Alex Step, да, CompanyModel это моя самописная модель в которой идёт запрос в БД.

Answer 2

[Владимир Коротенко]

Смотрите тут все просто id генерируется базой и у него есть 5 состояний

существует - просто отдаете
не существует - выводите сообщение мол нет такой записи и пишете в блокнот для возможного бана
удален - сообщаете что пост удален, пользователя в блокнот
доступен вам - просто кидаете ошибку 404, пишете в блокнот
доступен спецслужбам- 404 и репорт спецслужбам

Comments

[Назарий Лазарчук]

Спасибо, звучит логично.

удален - сообщаете что пост удален, пользователя в блокнот

- как я понимаю вы имеете ввиду что это если есть реализация "soft delete" записи с таблицы.
И тогда получаеться что правельнее будет сгенерировать и кинуть нужную ошибку для каждого состояния, верно?

[Назарий Лазарчук]

Или для "не существует" всё-таки использовать null?

[Владимир Коротенко]

Назарий Лазарчук, лучше для каждого состояния

Answer 3

[Назарий Лазарчук]

Вот что у меня получилось в итоге. Вроде всё правильно понял, нет?

import { CompanyDbModel } from 'db/models';
 
class DbRowNotFoundError extends Error {}

// где-то дальше в асинхронной функции
try {
  const company = await CompanyDbModel.getById(228);
} catch (err) {
  if (err instanceof DbRowNotFoundError) {
    return console.log('Поля с таким id в таблице нет');
  }
}

Comments

[Назарий Лазарчук]

исплементация метода getById:

getById(id: number) {
        return new Promise((resolve, reject) => {
            db.get('SELECT * FROM companies WHERE id = ?', [id], (err, row) => {
                if (err) reject(err);
                if (typeof row === 'undefined') {
                    return reject(new DbRowNotFoundError());
                }
                resolve(row);
            });
        });
    },