mavka-ukr / mavka

Сучасна українська мова програмування
https://мавка.укр
Other
54 stars 4 forks source link

Імплементація модулів запуску #28

Closed dkostmii closed 1 year ago

dkostmii commented 1 year ago

Заімплементовано модулі запуску, читаючи усі файли в папці startup-modules та завантажуючи кожен з них у головний контекст Мавки.

Для прикладу було створено модулі, що містять дію Привітатись() та імперативний конструктор Дата().

Усі змінні усередині модулю переносяться у головний контекст Мавки методом перебору.

kohutd commented 1 year ago

image

Ймовірно краще використати "день" замість "дата"

kohutd commented 1 year ago

image

дії пишемо з маленької (і загалом її не варто включати під час запуску)

Якщо це лише для прикладу, то ок

kohutd commented 1 year ago

image image

мені не вдалось запустити, показує помилку

kohutd commented 1 year ago

Я побачив щойно, що ви говорили про назви "дата" і "день", але все-ж, вважаю що краще використовувати зрозумілі і звичні назви

dkostmii commented 1 year ago

image image

мені не вдалось запустити, показує помилку

Це спричинено тим, що у Windows шлях починається зі знаку / також, для прикладу /C:/. Наразі перший знак шляху видаляється.

dkostmii commented 1 year ago

image

дії пишемо з маленької (і загалом її не варто включати під час запуску)

Якщо це лише для прикладу, то ок

Забув про специфікацію назв. Пропоную додати відповідний розділ у документацію.

kohutd commented 1 year ago

image дії пишемо з маленької (і загалом її не варто включати під час запуску) Якщо це лише для прикладу, то ок

Забув про специфікацію назв. Пропоную додати відповідний розділ у документацію.

Підтримую пропозицію

kohutd commented 1 year ago

image image мені не вдалось запустити, показує помилку

Це спричинено тим, що у Windows шлях починається зі знаку / також, для прикладу /C:/. Наразі перший знак шляху видаляється.

треба знайти рішення яке задовільнятиме обидві системи. хоча js наче такі речі мав би сам вирішувати, дивно

dkostmii commented 1 year ago

image image мені не вдалось запустити, показує помилку

Це спричинено тим, що у Windows шлях починається зі знаку / також, для прикладу /C:/. Наразі перший знак шляху видаляється.

треба знайти рішення яке задовільнятиме обидві системи. хоча js наче такі речі мав би сам вирішувати, дивно

Використовувати регулярний вираз для перевірки, чи шлях починається з літери диску: /^\/[A-Z]:\/.

Він визначає, чи шлях схожий на /C:/решта-шляху

dkostmii commented 1 year ago

Виправлено проблему зі шляхом, та розширено імперативний конструктор усіма необхідними полями структури Дата разом з перевіркою цих полів.