AO
Если честно, то папка не попадает ни под какой cohesion, потому что это просто папка. Сложность абсолютно на пустом месте.
я того же мнения. это просто папка. но если её и надо приткнуть под какой-то вариант cohesion, то это logical, я считаю
Size: a a a
2020 May 03
AO
а не в каждом модуле
а если у вас есть компоненты, которые много раз используются компонентами модуля? и не используются больше нигде. я вижу, что у вас есть папка ui. но вы можете переименовать её в shared, и это будет то же самое, только с другим именем
AL
да. то есть, наличие shared вам ясно говорит о том, какое предназначение у сервиса. тогда чем плох такой подход, почему нельзя делать shared? ангуляр как раз рекомендует вносить ясность
да ничего в этом плохого нет, у меня на проекте так и используется, это очень удобно.
G
а если у вас есть компоненты, которые много раз используются компонентами модуля? и не используются больше нигде. я вижу, что у вас есть папка ui. но вы можете переименовать её в shared, и это будет то же самое, только с другим именем
да, имеено так. Но создавая папку shared, вы кладете в неё не только компоненты, но и сервисы, пайпы и все все. Я просто это все все все вынес в корень модуля
G
я пока привел скрин, но еще не определился. Блин заварил кашу, самому уже страшно от масштаба. Шел третий день обсуждения файловой структуры приложения )))))
AL
блин кто мне может пояснить почему процедура работает в хром и не работает в edge и ei? полифилы я уже включил, но все равно не работает. кто сталкивался?
AL
весь бля гугл перерыл...может кто еще какие заветные места знает?
MA
Artem Laschenkov
блин кто мне может пояснить почему процедура работает в хром и не работает в edge и ei? полифилы я уже включил, но все равно не работает. кто сталкивался?
Ошибки есть? Дебаггер пробовали?
AL
Ошибки есть? Дебаггер пробовали?
да втом то и фишка что в консоль ничего не летит
AO
короче, выводы)
1. делайте feature модули, потому что это правильно с точки зрения cohesion
2. файловую и папочную структуру внутри этих модулей делайте так, как советует ангуляровская дока, потому что это масштабируется и легко читается
3. при этом делайте всё так, чтобы комфортно было с этим всем работать, даже если это нарушает одно из 100 правил. потому что в любом случае всё это делается для людей
4. идеальной по всем параметрам структуры не существует :)
как-то так, наверно)
1. делайте feature модули, потому что это правильно с точки зрения cohesion
2. файловую и папочную структуру внутри этих модулей делайте так, как советует ангуляровская дока, потому что это масштабируется и легко читается
3. при этом делайте всё так, чтобы комфортно было с этим всем работать, даже если это нарушает одно из 100 правил. потому что в любом случае всё это делается для людей
4. идеальной по всем параметрам структуры не существует :)
как-то так, наверно)
MA
Artem Laschenkov
да втом то и фишка что в консоль ничего не летит
Ну, значит с помощью дебаггера ищите тот минимальный кусок кода, где поведение различается, воспроизводите изолироаванно и ищите проблему.
AL
Ну, значит с помощью дебаггера ищите тот минимальный кусок кода, где поведение различается, воспроизводите изолироаванно и ищите проблему.
блин, а почему оно в одном браузере работает а в другом нет? из за чего? я вычитал что ie не поддерживает формат es5, для этого нужно включить полифилы, я их включил
MA
Artem Laschenkov
блин, а почему оно в одном браузере работает а в другом нет? из за чего? я вычитал что ie не поддерживает формат es5, для этого нужно включить полифилы, я их включил
Потому что браузеры разные ) Нюансов может быть много, надо искать конкретный кусок кода.
G
а так разве можно было? модули располагать за пределами папки app
S
а так разве можно было? модули располагать за пределами папки app
Они подключаются к аппе как плагины
S
Внутри них не должно быть зависимостей от того что в аппе написано
AS
а теперь добавим 5 видов пайпов и несколько директив.
S
а теперь добавим 5 видов пайпов и несколько директив.
Например
S
а теперь добавим 5 видов пайпов и несколько директив.
И долго ждал?)
AS
И долго ждал?)
фиг знает, открыл тг, оно на енто показывало :)