AS
ребят я тут скидывал кусок с switch case мне сказали что выглядит как говнокод, посоветуйте как это должно быть что бы не было говнокодом)
у тебя тут просто словарь ключ-значение, обычный объект сгодится
Size: a a a
2021 February 08
I
у тебя тут просто словарь ключ-значение, обычный объект сгодится
где его хранить если он мне нужен в разных компонентах, и как его вызывать в html?
AS
где его хранить если он мне нужен в разных компонентах, и как его вызывать в html?
в сервисе. делаешь объект приватным, а из сервиса торчит метод get чего нибудь.
статик не надо
статик не надо
AS
в шаблоне уже можно обращаться к сервису через пайп. т.е. вызываешь пайп, он инжектит сервис и берет из него метод
AS
это нормально, когда сервис у нас отвечает за логику, а пайп служит связующим звеном между темплейтом и сервисом, это его роль.
OS
Почитайте для чего нужен пайп) И поймёте, какие иногда дают советы :)
Шта?)
OS
@dopusteam вы мне подсказывали через пайп сделать выбор класса, подскажите логику внутри пайпа, if else или switch case(сказали говнокод)
Ну там до этого советовали положить это в словарь и из него брать если есть или возвращать по умолчанию значение, если нет в словаре
Так и сделайте, наверное )
Так и сделайте, наверное )
AS
Пайпа используется для изменения отображаемого значения. Какие связи... Судя по скрину, этот код возвращает тип какого-нибудь динамического объекта или что-то вроде того.
S
Я все правильно сделал? Подправьте если не сложно
OS
Пайпа используется для изменения отображаемого значения. Какие связи... Судя по скрину, этот код возвращает тип какого-нибудь динамического объекта или что-то вроде того.
Там был контекст ещё к вопросу)
Там нужно было класс установить в зависимости от значения
Не то, что бы православно пайп так юзать, но мне кажется хорошо подходит
Там нужно было класс установить в зависимости от значения
Не то, что бы православно пайп так юзать, но мне кажется хорошо подходит
AS
Там был контекст ещё к вопросу)
Там нужно было класс установить в зависимости от значения
Не то, что бы православно пайп так юзать, но мне кажется хорошо подходит
Там нужно было класс установить в зависимости от значения
Не то, что бы православно пайп так юзать, но мне кажется хорошо подходит
Ну не красиво тут пайпу юзать как-то
SC
Sem
прогугли решето Эратосфена)
AS
В этом случае лучше словарь + класс со статическим методом. Никакие сервисы не нужны
ll
можно даже без класса
OS
В этом случае лучше словарь + класс со статическим методом. Никакие сервисы не нужны
Согласен, это тоже вариант
Сервисы не предлагали, предлагал в пайп вынести преобразование
Спорить не буду, на вкус и цвет, по мне - так нормально заходит тут пайп,)
Сервисы не предлагали, предлагал в пайп вынести преобразование
Спорить не буду, на вкус и цвет, по мне - так нормально заходит тут пайп,)
I
Если всегда как обычно, то одно красиво будет наоборот некрасивым. В антураж не впишется)
ну хочется же писать хороший код .
AS
можно даже без класса
Да, просто статический метод)
I
В этом случае лучше словарь + класс со статическим методом. Никакие сервисы не нужны
во, у меня есть метод в утилсах, мне там же создать словать и в этом методе вместо свитч кейсов просто по ключу брать значение правильно?
ll
Да, просто статический метод)
красивее не получится, пайп ваще бессмысленен