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

Size: a a a

Angular - русскоговорящее сообщество

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

2021 May 20

Anton Shvets in Angular - русскоговорящее сообщество

опять свитчмап

источник

10:11пожаловаться #1

Daniil Baunov in Angular - русскоговорящее сообщество

я бы открытие диалога вынес в отдельный метод типа openOverwriteYearConfirmationDialog. Метод бы принимал колбэк с параметром ok.

А вот setYearConditionally выглядит немного бредово. Особенно часть "conditionally"

Текст бы тоже в константы вынести за пределы методов.

источник

10:27пожаловаться #2

Daniil Baunov in Angular - русскоговорящее сообщество

Вообще, мне кажется, по хорошему бы все это дело реактивно делать, а не подписываться на каждый Observable внутри. Так было бы локаничнее

источник

10:29пожаловаться #3

Alex Inkin in Angular - русскоговорящее сообщество

Сенсей красава, реально когда 3 и больше строчек — это скорее исключение в хорошем коде. Но это не значит, что надо дробить на подфункции насильно. Просто флажочек, что может зря ты код написал ) лично у меня так происходит, что чем опытнее я становлюсь — тем меньше кода пишу.

источник

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

Daniil Baunov in Angular - русскоговорящее сообщество

+ с именованием какие-то проблемы на мой взгляд. checkExistingNote почему-то вместо того чтобы проверять существование заметки (и возвращать boolean) - задает значение year в дате (и возвращает void)

источник

10:38пожаловаться #5

Daniil Baunov in Angular - русскоговорящее сообщество

не думаю, что стоит пытаться сокращать объем кода в ущерб логике

источник

10:40пожаловаться #6

Alex Inkin in Angular - русскоговорящее сообщество

Я бы точно сделал, чтобы диалог boolean возвращал. Можно написать вот так, если не нравится if в сабскрайбе:

this.checkYear(year).pipe(
  switchMap(exists => exists 
    ? this.openDialog()
    : EMPTY
  ),
  filter(Boolean),
).subscribe(() => {
  this.date.setValue(year);
});

источник

10:41пожаловаться #7

Alex Inkin in Angular - русскоговорящее сообщество

Да, над именами надо подумать, длинные и не понятные.

источник

10:42пожаловаться #8

АК

Андрей Картошка... in Angular - русскоговорящее сообщество

Привет

источник

10:53пожаловаться #9

АК