Вキ
Для этого параметр по дефолту придуман
А проверять кто будет?)
Size: a a a
2020 May 22
L
А проверять кто будет?)
тест
Вキ
Тест такое не покроет нормально
С
Может покажешь)
private fetchAllCategories(lobId): Observable<PresentationCategoryDTO[]> {
return this.xhrTool.marshall(this.http.get(Api.path('/categories/tree'),
{params: { lobId, deep: 'true' }}));
}
private searchPresentations(lobId, tags: string[], searchPhrase: string, languages: string): Observable<PresentationCategoryDTO[]> {
const params: { tags: string; text: string; language: string} = {
tags: encodeURIComponent(tags.join(',')),
language: encodeURIComponent(languages),
text: encodeURIComponent(searchPhrase),
};
return this.xhrTool.marshall(this.http.get(Api.path('/categories/tree'),
{params: { lobId, ...params, deep: 'true' }}));
}Вキ
private fetchAllCategories(lobId): Observable<PresentationCategoryDTO[]> {
return this.xhrTool.marshall(this.http.get(Api.path('/categories/tree'),
{params: { lobId, deep: 'true' }}));
}
private searchPresentations(lobId, tags: string[], searchPhrase: string, languages: string): Observable<PresentationCategoryDTO[]> {
const params: { tags: string; text: string; language: string} = {
tags: encodeURIComponent(tags.join(',')),
language: encodeURIComponent(languages),
text: encodeURIComponent(searchPhrase),
};
return this.xhrTool.marshall(this.http.get(Api.path('/categories/tree'),
{params: { lobId, ...params, deep: 'true' }}));
}А где проверка?
L
можно врача
С
в общем-то проверка есть в другом метода ахахаха
Вキ
в общем-то проверка есть в другом метода ахахаха
Она тут должна быть
L
@SuperPowerful
это что такое?)
this.xhrTool.marshall
С
getPresentationCategories(lobId, tags: string[], searchPhrase: string, languages: string): Observable<any> {
const responseObs = tags.length > 0 || searchPhrase !== '' || languages !== '' ?
this.searchPresentations(lobId, tags, searchPhrase, languages) :
this.fetchAllCategories(lobId)
;
return responseObs.pipe(
map((response: any) => response.content),
map((categoriesDTO: PresentationCategoryDTO[]) => {
return categoriesDTO;
}),
);
}С
@SuperPowerful
это что такое?)
this.xhrTool.marshall
это костыль по поимке ошибок.
Вキ
в общем-то проверка есть в другом метода ахахаха
V
короче смотри объясняю на пальцах как это все дело должно работать у тебя когда вызывается какой то метод и уверенно то что у тебя какие то параметры должны быть про валидированных обязательную тоо. я прям в этом методе должна быть поляризации если это валидация вынесены куда то отдельно и это может привести такой ситуации то что ты просто забудешь вызвать эту валидацию перед вызвать метод это первый. второе это немножко усложнит твой кот который ты будешь писать вне этого метода то есть тебе придется вызвать не один метод два метода что гораздо сложно сложнее вот и как я уже. приводил один пример это все сводится к тому то что просто надеемся то что было забыто или не было забыто поэтому проверку лучше поместить в этот метод
L
это костыль по поимке ошибок.
return responseObs.pipe(
map((response: any) => response.content),
map((categoriesDTO: PresentationCategoryDTO[]) => {
return categoriesDTO;
}),
);
С
ладно. я уже тут придумала
L
А тут можно 1 мап
С