-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Модуляция #5
Модуляция #5
Conversation
Хотел спросить, где мне надо вызывать функцию createArrayOfPhotos в main.js или там, где она создана (data.js)? |
js/variables.js
Outdated
@@ -0,0 +1,29 @@ | |||
const descriptions = ["Вся красота мира в одной картинке", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Отличный вариант с выносом констант в отдельный файл) Только имя файла лучше поменять, это ведь не совсем переменные. Или константы или перечисления/словари
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Поменял
js/main.js
Outdated
"Лишь тот, кто странствует, открывает новые пути", | ||
"Зарядитесь нашим теплом", | ||
"Жизнь лучше, когда ты смеешься",]; | ||
import './variables.js'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Прям все импортировать нет необходимости. Тут лучше вызывать функцию генерации из data.js, значит ее нужно импортировать, а остальное и не нужно
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Понял, сейчас исправил
Лучше в main.js. Потом это изменится, но сейчас так. data.js это просто модуль с функциями, сам по себе он ничего не делает |
1ab907f
into
htmlacademy-univer-javascript-1:master
PR принят 👍🏻 |
🎓 Модуляция