null-safety; some cleanup; fix test

Author: Mr-Rm

src/OneScript.StandardLibrary/Json/JSONReader.cs

@@ -41,7 +41,6 @@ public override bool Read()
     }
 
     /// <summary>
-    /// 
     /// Предназначен для последовательного чтения JSON-данных из файла или строки.
     /// </summary>
     [ContextClass("ЧтениеJSON", "JSONReader")]
@@ -51,7 +50,6 @@ public class JSONReader : AutoContext<JSONReader>
         private JsonReaderInternal _reader;
 
         /// <summary>
-        /// 
         /// Возвращает true если для объекта чтения json был задан текст для парсинга.
         /// </summary>
         private bool IsOpen() => _reader != null;
@@ -72,49 +70,25 @@ public static JSONReader Constructor()
         }
 
         /// <summary>
-        /// 
         /// Указывает на позицию, находящуюся сразу после прочитанного значения.
         /// При ошибке чтение остается на позиции последнего успешно считанного символа.
         /// </summary>
         /// <value>Число (Number), Неопределено (Undefined)</value>
         [ContextProperty("ТекущаяПозиция", "CurrentPosition")]
         public IValue CurrentPosition
-        {
-            get
-            {
-                if (IsOpen())
-                {
-                    return ValueFactory.Create(_reader.LinePosition);
-                }
-
-                return ValueFactory.Create(); // Неопределено 
-            }
-        }
+            => IsOpen() ? ValueFactory.Create(_reader.LinePosition) : ValueFactory.Create();
 
         /// <summary>
-        /// 
         /// Указывает на позицию сразу после прочитанного значения.
         /// Например, перед чтением первого элемента - 0, после чтения первого элемента -1 .
         /// </summary>
         /// <value>Число (Number), Неопределено (Undefined)</value>
         [ContextProperty("ТекущаяСтрока", "CurrentLine")]
         public IValue CurrentLine
-        {
-            get
-            {
-                if (IsOpen())
-                {
-                    return ValueFactory.Create(_reader.LineNumber);
-                }
-
-                return ValueFactory.Create(); // Неопределено
-            }
-        }
+            => IsOpen() ? ValueFactory.Create(_reader.LineNumber) : ValueFactory.Create();
 
         /// <summary>
-        /// 
         /// Содержит текущее значение:
-        /// 
         ///  - Число - если ТипТекущегоЗначения имеет значение Число;
         ///  - Строка - если ТипТекущегоЗначения имеет одно из следующих значений:
         ///    - Комментарий,
@@ -161,15 +135,13 @@ public IValue CurrentValue
 
                     default:
                         throw JSONReaderException.CannotGetValue();
-                        ;
                 }
             }
         }
 
         public object ReaderValue => _reader.Value;
 
         /// <summary>
-        /// 
         /// Тип текущего значения в документе JSON во внутреннем формате.
         /// null - если чтение еще не началось или достигнут конец файла.
         /// </summary>
@@ -184,7 +156,6 @@ public JsonToken CurrentJsonTokenType
         }
 
         /// <summary>
-        /// 
         /// Тип текущего значения в документе JSON.
         /// Неопределено - если чтение еще не началось или достигнут конец файла.
         /// </summary>
@@ -218,10 +189,8 @@ public JSONValueTypeEnum CurrentValueType
         }
 
         /// <summary>
-        /// 
         /// Завершает чтение текста JSON из файла или строки.
         /// </summary>
-        ///
         [ContextMethod("Закрыть", "Close")]
         public void Close()
         {
@@ -233,10 +202,10 @@ public void Close()
         }
 
         /// <summary>
-        /// 
-        /// Открывает JSON-файл для чтения данным объектом. Если перед вызовом данного метода уже производилось чтение JSON из другого файла или строки, то чтение прекращается и объект инициализируется для чтения из указанного файла.
+        /// Открывает JSON-файл для чтения данным объектом.
+        /// Если перед вызовом данного метода уже производилось чтение JSON из другого файла или строки,
+        /// то чтение прекращается и объект инициализируется для чтения из указанного файла.
         /// </summary>
-        ///
         /// <param name="JSONFileName">
         /// Имя файла, содержащего текст JSON. </param>
         /// <param name="encoding">
@@ -249,11 +218,9 @@ public void OpenFile(string JSONFileName, IValue encoding = null)
             StreamReader _fileReader;
 
             try
-            {
-                if (encoding != null)
-                    _fileReader = FileOpener.OpenReader(JSONFileName, TextEncodingEnum.GetEncoding(encoding));
-                else
-                    _fileReader = FileOpener.OpenReader(JSONFileName, System.Text.Encoding.UTF8);
+            {
+                var enc = encoding != null ? TextEncodingEnum.GetEncoding(encoding) : System.Text.Encoding.UTF8;
+                _fileReader = FileOpener.OpenReader(JSONFileName, enc);
             }
             catch (Exception e)
             {
@@ -280,7 +247,9 @@ public void OpenStream(IStreamWrapper streamContext, IValue encoding = null)
         {
             Close();
 
-            var stream = streamContext.GetUnderlyingStream();
+            var stream = streamContext?.GetUnderlyingStream() 
+                ?? throw new ArgumentNullException(nameof(streamContext));
+
             var enc = encoding != null ? TextEncodingEnum.GetEncoding(encoding) : System.Text.Encoding.UTF8;
 
             _reader = new JsonReaderInternal(new StreamReader(stream, enc, leaveOpen:true))
@@ -293,7 +262,6 @@ public void OpenStream(IStreamWrapper streamContext, IValue encoding = null)
         /// Если текущее значение – начало массива или объекта, то пропускает его содержимое и конец.
         /// Для остальных типов значений работает аналогично методу Прочитать().
         /// </summary>
-        ///
         [ContextMethod("Пропустить", "Skip")]
         public bool Skip()
         {
@@ -310,24 +278,18 @@ public bool Skip()
         /// <summary>
         /// Выполняет чтение значения JSON.
         /// </summary>
-        ///
         [ContextMethod("Прочитать", "Read")]
         public bool Read()
         {
             CheckIfOpen();
             return _reader.Read();
-
         }
 
         /// <summary>
-        /// 
         /// Устанавливает строку, содержащую текст JSON для чтения данным объектом. Если перед вызовом данного метода уже производилось чтение JSON из другого файла или строки, то чтение прекращается и объект инициализируется для чтения из указанной строки.
         /// </summary>
-        ///
         /// <param name="JSONString">
         /// Строка, содержащая текст в формате JSON. </param>
-        ///
-        ///
         [ContextMethod("УстановитьСтроку", "SetString")]
         public void SetString(string JSONString)
         {

tests/json/test-json_reader.os

@@ -249,6 +249,7 @@
     БДД = ПолучитьБуферДвоичныхДанныхИзСтроки("{""ответ"":42}");   
 	Поток = Новый ПотокВПамяти(БДД);
 	Чтение = Новый ЧтениеJSON;
+	Чтение.ОткрытьПоток(Поток);
 	Чтение.Закрыть();
 	Чтение.ОткрытьПоток(Поток); // Поток должен остаться открытым
 	Json = ПрочитатьJSON(Чтение);