feat (#746): Реализована возможность чтения из скрипта данных, переданных через stdin #1086

arkuznetsov · 2021-05-26T14:31:52Z

Реализована возможность чтения из скрипта данных, переданных через stdin.

test.os

Чтение = Новый ЧтениеТекста();
Чтение.Открыть(СтандартныйПотокВвода);
Сообщить(СокрЛП(Чтение.Прочитать()));

Запускаем:

> echo 1234 | oscript test.os

Вывод:

…ых, переданных через stdin.

EvilBeaver · 2021-05-26T14:40:27Z

src/ScriptEngine.HostedScript/Library/SystemGlobalContext.cs

+                ConsoleInput.CopyTo(StandartInput);
+                ConsoleInput.Close();
+            }
+            RegisterProperty("СтандартныйПотокВвода", () => StandartInput);


Давай сделаем это не глобальным свойством, а в классе Консоль? Не хочется занимать глобальный неймспейс без нужды... @nixel2007 @artbear @dmpas @Infactum - реквестирую консилиум.

А в таком случае, как должно выглядеть обращение из скрипта?

я могу поделиться примерами из java.

Обычно работа с потоком ввода идет через объект System.in, который держит ссылку на InputStream, у которого есть read-методы по чтению сырых байтов. При этом есть класс Console, ссылка на который лежит там же в System.console(). Там методы уже покрасивее, есть readLine, format, readPassword и прочая красота.

При этом запись в stdout может выполняться как из System.out так и из System.console().

System.in появился в 1.1, то есть грубо говоря на заре появления джавы. Console - уже в 1.6, то есть некоторого после взятия за голову.

Но на практике напрямую из потоков и/или консоли все равно никто не читает, а оборачивают обращения к System.in через вызовы Scanner (который умеет readInt, readFloat и прочее) либо в InputStreamReader.

Занимать глобальный нэймспэйс точно не хочется. Перенести поток в Консоль выглядит вполне логично и похоже на ту же джаву. Поток же все равно нельзя будет получить, если скрипт запущен не из терминала, да? помню, что обращение к Консоль падает при запуске в определенных вне-терминальных окружениях. Не будет ли здесь проблем?

EvilBeaver · 2021-05-26T14:41:28Z

src/ScriptEngine.HostedScript/Library/SystemGlobalContext.cs

+            MemoryStreamContext StandartInput = MemoryStreamContext.Constructor();
+            if (Console.IsInputRedirected)
+            {
+                GenericStream ConsoleInput = new GenericStream(Console.OpenStandardInput(), true);


Мне больше нравится API Console.OpenStandardInput() который вызывается, как метод-действие по требованию.

Ты предлагаешь, сделать типа:

Консоль = Новый Консоль(); Консоль.ПолучитьСтандартныйВвод();

?

Да. Консоль.ОткрытьПотокВвода(), например

arkuznetsov · 2021-05-27T09:20:43Z

Хм, чот на Linux не отрабатывает. :-(
@EvilBeaver @dmpas Что может быть?

EvilBeaver · 2021-05-27T15:28:53Z

Хм, чот на Linux не отрабатывает. :-(
@EvilBeaver @dmpas Что может быть?

Все что угодно. Как именно оно не отрабатывает? Ругается на что-то? Есть тексты ошибок?

arkuznetsov · 2021-05-27T15:44:31Z

Хм, чот на Linux не отрабатывает. :-(
@EvilBeaver @dmpas Что может быть?

Все что угодно. Как именно оно не отрабатывает? Ругается на что-то? Есть тексты ошибок?

Просто не получает данные с stdin. Без ошибок. Руками не запускал, просто тесты посмотрел.

EvilBeaver · 2021-05-28T08:06:07Z

Я не знаю. Предлагаю копать дальше. Погуглить известные баги в mono или core (смотря под чем гоняешь тесты)

arkuznetsov · 2021-05-28T09:09:37Z

Я не знаю. Предлагаю копать дальше. Погуглить известные баги в mono или core (смотря под чем гоняешь тесты)

Да, конечно, буду ковырять. А тесты я на твоем Женькинсе смотрел :-).

feat: ConsoleContext сделан синглтоном; Добавлено глобальное свойство "Консоль".

arkuznetsov · 2021-05-28T14:04:45Z

@EvilBeaver Переделал.

EvilBeaver · 2021-05-28T14:08:00Z

src/ScriptEngine.HostedScript/Library/SystemGlobalContext.cs

            RegisterProperty("FileStreams", () => FileStreams);

-			RegisterProperty("Символы", () => (IValue)Chars);
+            RegisterProperty("Символы", () => (IValue)Chars);


Вот сюда бы добавить свойство Консоль, чтобы был глобальный синглтон. @dmpas, что думаешь?

Вот сюда бы добавить свойство Консоль, чтобы был глобальный синглтон. @dmpas, что думаешь?

Уже сразу в тестах нарвался на 'Консоль = Новый Консоль()' боюсь, что много где может вылезти.

может не глобальный, а через враппер? типа Система?
я помню, что вы наоборот выносили из системной информации всякие методы и свойства наружу, но может просто было нехорошее имя?

EvilBeaver · 2021-05-28T14:09:45Z

src/ScriptEngine.HostedScript/Library/ConsoleContext.cs

-            return new ConsoleContext();
+            if (_console == null)
+            {
+                lock (_lock)


lock не нужен, см. статью Скита про синглоны. Нужно создать пустой статик конструктор и обычное статическое поле. https://csharpindepth.com/articles/singleton

Варианты 5 и 6

Опять же оставил конструктор, чтобы не ломать совместимость. С lock посмотрю.

nixel2007 · 2021-05-28T14:10:56Z

src/ScriptEngine.HostedScript/Library/ConsoleContext.cs

        public static ConsoleContext Constructor()
        {
-            return new ConsoleContext();
+            if (_console == null)


если это попытка в DLC, то в мультитреде оно работает не так, как ты ожидаешь.

https://csharpindepth.com/articles/singleton

# Conflicts: # src/ScriptEngine.HostedScript/Library/ConsoleContext.cs

arkuznetsov · 2021-07-25T07:10:25Z

src/ScriptEngine.HostedScript/Library/ConsoleContext.cs

+        /// Возвращает или задает кодировку консоли, используемую при чтении входных данных.
+        /// </summary>
+        /// <returns>КодировкаТекста</returns>
+        [ContextProperty("КодировкаВыходногоПотока", "InputEncoding")]


Наверное "КодировкаВходногоПотока".

Нет, тут именно Выходного. Их там две. Но английский синоним забыл поменять после копипасты. Спасибо

Угу, не туда посмотрел.

Artem Kuznetsov added 2 commits May 26, 2021 17:28

feat (EvilBeaver#746): Реализована возможность чтения из скрипта данн…

9e7b90a

…ых, переданных через stdin.

test: Поправлен тест.

2da24ef

EvilBeaver reviewed May 26, 2021

View reviewed changes

refactor: Получение stdin перенесено в объект ConsoleContext.

6365a70

feat: ConsoleContext сделан синглтоном; Добавлено глобальное свойство "Консоль".

EvilBeaver requested changes May 28, 2021

View reviewed changes

nixel2007 reviewed May 28, 2021

View reviewed changes

Andrey Ovsiankin and others added 4 commits July 24, 2021 20:46

Заготовка по EvilBeaver#746

f40bb11

# Conflicts: # src/ScriptEngine.HostedScript/Library/ConsoleContext.cs

fixed EvilBeaver#583 fixed EvilBeaver#746

0e259aa

Ошибочно вкоммитил флаг ошибок

2ef6c0b

Удален старый тест

75c2dd6

arkuznetsov commented Jul 25, 2021

View reviewed changes

arkuznetsov and others added 5 commits July 25, 2021 10:41

Merge branch 'EvilBeaver:develop' into feature/get-stdin

b0cfa9b

Проверка получения стандартного потока ввода на Linux

a2afd30

Проверка получения стандартного потока ввода на Linux (2)

6a05619

Откат изменений для отладки перенаправления ввода.

0267e63

Исправлен тест на Linux

7b95747

EvilBeaver approved these changes Jul 27, 2021

View reviewed changes

EvilBeaver merged commit 58aea76 into EvilBeaver:develop Jul 27, 2021

arkuznetsov deleted the feature/get-stdin branch July 28, 2021 13:20

feat (#746): Реализована возможность чтения из скрипта данных, переданных через stdin #1086

feat (#746): Реализована возможность чтения из скрипта данных, переданных через stdin #1086

Uh oh!

Conversation

arkuznetsov commented May 26, 2021

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

arkuznetsov commented May 27, 2021

Uh oh!

EvilBeaver commented May 27, 2021

Uh oh!

arkuznetsov commented May 27, 2021

Uh oh!

EvilBeaver commented May 28, 2021

Uh oh!

arkuznetsov commented May 28, 2021

Uh oh!

arkuznetsov commented May 28, 2021

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants