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.

Sign up for GitHub

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

Jump to bottom

В рефлектор добавлен метод ИзвестныеТипы() #1368

Merged
merged 4 commits into from
Oct 19, 2023
Merged

В рефлектор добавлен метод ИзвестныеТипы() #1368

merged 4 commits into from
Oct 19, 2023

Conversation

sfaqer
Copy link
Contributor

@sfaqer sfaqer commented Oct 18, 2023 •
edited
Loading

Добавлен метод который возвращает все зарегистрированные типы в виде:
ТаблицаЗначений<Имя, Значение, Примитивный, Пользовательский, Коллекция>

@sfaqer
В рефлектор добавлен метод ИзвестныеТипы()
21f5777 
Добавлен метод который возвращает все зарегистрированные типы в виде Массив<Тип>
@nixel2007
Copy link
Collaborator

Может вместо голого типа возвращать ещё какие-то метаданные? Типа пользовательский ли это тип (из сценария) или движковый?

@sfaqer
Известные типы теперь возвращают не массив типов а массив структур, к…
31a7d0f 
…оторый содержит информацию о самом типе, его имени, является ли тип примитивным и является ли тип созданным пользователем
@sfaqer
Copy link
Contributor Author

sfaqer commented Oct 18, 2023

Может вместо голого типа возвращать ещё какие-то метаданные? Типа пользовательский ли это тип (из сценария) или движковый?

Типа того?

@dmpas
Copy link
Collaborator

dmpas commented Oct 18, 2023

Вечный спор между Массивом Структур и Таблицей Значений.

@sfaqer
Copy link
Contributor Author

sfaqer commented Oct 18, 2023

Вечный спор между Массивом Структур и Таблицей Значений.

Можно в теории кстати мапу сделать, с типом на структуру с описанием типа, вероятно так даже удобнее будет

ИзвестныеТипы().Получить(Тип).Пользовательский

@sfaqer
Возвращаемое значение ИзвестныеТипы, переделано с массива структур на…
3a4bf91 
… соответствие Типа на Структуру описания типа
@nixel2007
Copy link
Collaborator

Выглядит симпатично, а хватит ли этого для осени - надо пробовать)

@EvilBeaver
Copy link
Owner

Может вместо голого типа возвращать ещё какие-то метаданные? Типа пользовательский ли это тип (из сценария) или движковый?

А зачем? Какую задачу вы планируете этим решить? (C)

EvilBeaver
EvilBeaver requested changes Oct 18, 2023
View reviewed changes
src/OneScript.StandardLibrary/Reflector.cs Outdated Show resolved Hide resolved
src/ScriptEngine/Machine/DefaultTypeManager.cs Show resolved Hide resolved
src/OneScript.StandardLibrary/Reflector.cs Outdated Show resolved Hide resolved
@sfaqer
1. Исправлен тип возвращаемого значения в конструкторе рефлектора
417bd9d 
2. RegisteredTypes теперь возвращает список только для чтения
3. Соответствие заменена на ТаблицуЗначений, добавлен признак "Коллекция"
4. Добавлена возможность передать фильтр для отбора известных типов
@sfaqer sfaqer requested a review from EvilBeaver October 18, 2023 11:57
@sfaqer
Copy link
Contributor Author

sfaqer commented Oct 18, 2023

@EvilBeaver Поправил по замечаниям =)

@sfaqer sfaqer mentioned this pull request Oct 18, 2023
Merged
@EvilBeaver
Copy link
Owner

@sfaqer спасибо, кажется, что огнищще!

@EvilBeaver EvilBeaver merged commit 212d189 into EvilBeaver:develop Oct 19, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EvilBeaver EvilBeaver Awaiting requested review from EvilBeaver

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sfaqer @nixel2007 @dmpas @EvilBeaver