Вキ
Значит, что Ben Lesh нам врет?)
Size: a a a
2019 December 14
RC
Мне нужнл патчить ключ
И
Мне нужнл патчить ключ
Ключ на 9 ?
Дядя Богдан ?
Дядя Богдан ?
KA
ага, бестпрактис от какого-то левого чела, не автора библиотеки
АМ
ребята, вот есть у меня тип
и есть ещё тип
и ещё
дело в том, что в
export type PageComponent = {
id: string;
name: string;
description: string;
groupId: string;
status: PageComponentStatus;
createdAt: string;
updatedAt: string;
}и есть ещё тип
export type PageIncident = {
id: string;
name: string;
message: string;
components: PageIncidentComponent[];
createdAt: string;
}и ещё
export type PageIncidentComponent = {
id: string;
status: PageComponentStatus;
}дело в том, что в
PageIncident в массив components приходит укороченная версия PageComponent'а и я создал для этого отдельный тип PageIncidentComponent. Так вот, а можно всё же сделать вот так?:АМ
export type PageIncident = {
id: string;
name: string;
message: string;
components: PageComponent[];
createdAt: string;
}АМ
ну, тобишь, суть в том, что есть некоторые объекты, например, User, но в некоторых других данных приходит укороченный объект User, в котором меньше данных, чем в полном User — так вот нужно ли для каждого такого случая плодить типы, чтобы описать модификации User? Ибо тогда у меня будут типы User, и, например, всякие UserShort, UserExtended и т.п.
Вキ
export type PageIncident = {
id: string;
name: string;
message: string;
components: PageComponent[];
createdAt: string;
}Лучше сделать что-то такого:
https://www.typescriptlang.org/docs/handbook/advanced-types.html#discriminated-unions
Вот тут есть пример, как это может выглядеть:
https://github.com/thekiba/angular-platform-cloud/blob/master/projects/platform-cloud/src/lib/shared/api.ts
И вот так использоваться:
https://github.com/thekiba/angular-platform-cloud/blob/master/projects/platform-cloud/src/lib/server/server-renderer-adapter.ts#L138
https://github.com/thekiba/angular-platform-cloud/blob/master/projects/platform-cloud/src/lib/browser/browser-main.ts#L55-L67
https://www.typescriptlang.org/docs/handbook/advanced-types.html#discriminated-unions
Вот тут есть пример, как это может выглядеть:
https://github.com/thekiba/angular-platform-cloud/blob/master/projects/platform-cloud/src/lib/shared/api.ts
И вот так использоваться:
https://github.com/thekiba/angular-platform-cloud/blob/master/projects/platform-cloud/src/lib/server/server-renderer-adapter.ts#L138
https://github.com/thekiba/angular-platform-cloud/blob/master/projects/platform-cloud/src/lib/browser/browser-main.ts#L55-L67
Вキ
В последней ссылке пример может быть не очень показательным, так как суть заключается в том, что мы должны сокрыть все детали и работать через один общий метод или тип
АМ
почитал — не, дискриминэйтед юнионс это не то — там берут несколько типов и объединяют их под одним названием, а потом через if или switch/case работают с каждым отдельно
АМ
давайте другой пример приведу
АМ
вот есть у вас юзеры, тип вы назовёте User, скорее всего. А теперь у вас есть страница забаненных юзеров, но в массиве юзеров вам приходит только id и bannedAt поля для каждого юзера, там не приходят first_name и last_name, например. Так вот вы создадите отдельный тип для этого случая — BannedUser — или будете использовать уже существующий тип User, и ничего, что тут в данных приходит неполный объект User, а только несколько полей из всего объекта?
АМ
и напишите в коллбеке
loadSuccess(users: BannedUser[]) или просто loadSuccess(users: User[])?И
вот есть у вас юзеры, тип вы назовёте User, скорее всего. А теперь у вас есть страница забаненных юзеров, но в массиве юзеров вам приходит только id и bannedAt поля для каждого юзера, там не приходят first_name и last_name, например. Так вот вы создадите отдельный тип для этого случая — BannedUser — или будете использовать уже существующий тип User, и ничего, что тут в данных приходит неполный объект User, а только несколько полей из всего объекта?
забанненый юзер = есть отметка bannedAt
вот и всё, хз зачем дополнительный тип вводить
вот и всё, хз зачем дополнительный тип вводить
АМ
забанненый юзер = есть отметка bannedAt
вот и всё, хз зачем дополнительный тип вводить
вот и всё, хз зачем дополнительный тип вводить
нет, забаненный юзер это не «есть отметка bannedAt», а «есть только id и bannedAt и больше никаких полей из обычного User»
AK
А зачем поля прятать?
AK
Я бы не делал отдельный тип
YS
Если бэк уже такой и шлет юзера с 2мя полями
YS
то почему бы и не создать тип для этого