Skip to content
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

Merged

Conversation

skyfxllen
Copy link
Contributor

@skyfxllen skyfxllen commented Dec 3, 2024

@skyfxllen
Copy link
Contributor Author

Хотел спросить, где мне надо вызывать функцию createArrayOfPhotos в main.js или там, где она создана (data.js)?

@keksobot keksobot changed the title module5 Модуляция Dec 3, 2024
js/variables.js Outdated
@@ -0,0 +1,29 @@
const descriptions = ["Вся красота мира в одной картинке",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличный вариант с выносом констант в отдельный файл) Только имя файла лучше поменять, это ведь не совсем переменные. Или константы или перечисления/словари

Copy link
Contributor Author

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';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Прям все импортировать нет необходимости. Тут лучше вызывать функцию генерации из data.js, значит ее нужно импортировать, а остальное и не нужно

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Понял, сейчас исправил

@Danila100
Copy link

Хотел спросить, где мне надо вызывать функцию createArrayOfPhotos в main.js или там, где она создана (data.js)?

Лучше в main.js. Потом это изменится, но сейчас так. data.js это просто модуль с функциями, сам по себе он ничего не делает

@keksobot keksobot merged commit 1ab907f into htmlacademy-univer-javascript-1:master Dec 3, 2024
1 check passed
@Danila100
Copy link

PR принят 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants