скорректирован тип исключения; мекие правки

Author: Mr-Rm

src/OneScript.StandardLibrary/Collections/ValueTable/ValueTableColumnCollection.cs

@@ -131,7 +131,7 @@ public IValue Find(string name)
         public void Delete(IValue column)
         {
             var vtColumn = GetColumnByIIndex(column);
-            _owner.ForEach((ValueTableRow x)=>
+            _owner.ForEach((ValueTableRow x) =>
             {
                 x.OnOwnerColumnRemoval(vtColumn);
             });
@@ -156,10 +156,7 @@ public ValueTableColumn FindColumnByName(string name)
             return _columns.Find(column => _namesComparer.Equals(name, column.Name));
         }
 
-        public ValueTableColumn FindColumnByIndex(int index)
-        {
-            return _columns[index];
-        }
+        public ValueTableColumn FindColumnByIndex(int index) => _columns[index];
 
         public IEnumerator<ValueTableColumn> GetEnumerator()
         {
@@ -176,10 +173,7 @@ System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator()
 
         public override bool IsIndexed => true;
 
-        public override IValue GetIndexedValue(IValue index)
-        {
-            return GetColumnByIIndex(index);
-        }
+        public override IValue GetIndexedValue(IValue index) => GetColumnByIIndex(index);
 
         public override int GetPropertyNumber(string name)
         {
@@ -189,11 +183,8 @@ public override int GetPropertyNumber(string name)
             return idx;
         }
 
-        public override int GetPropCount()
-        {
-            return _columns.Count;
-        }
-        
+        public override int GetPropCount() => _columns.Count;
+
         public override string GetPropName(int propNum)
         {
             return FindColumnByIndex(propNum).Name;
@@ -204,39 +195,30 @@ public override IValue GetPropValue(int propNum)
             return FindColumnByIndex(propNum);
         }
 
-        public override bool IsPropWritable(int propNum)
-        {
-            return false;
-        }
-        
-        public override bool IsPropReadable(int propNum)
-        {
-            return true;
-        }
+        public override bool IsPropWritable(int propNum) => false;
+
+        public override bool IsPropReadable(int propNum) => true;
 
         public ValueTableColumn GetColumnByIIndex(IValue index)
         {
             if (index.SystemType == BasicTypes.String)
             {
-                ValueTableColumn Column = FindColumnByName(index.ToString());
-                if (Column == null)
-                    throw PropertyAccessException.PropNotFoundException(index.ToString());
-                return Column;
+                return FindColumnByName(index.ToString())
+                    ?? throw PropertyAccessException.PropNotFoundException(index.ToString());
             }
 
             if (index.SystemType == BasicTypes.Number)
             {
                 int i_index = Decimal.ToInt32(index.AsNumber());
                 if (i_index < 0 || i_index >= Count())
-                    throw RuntimeException.InvalidArgumentValue();
+                    throw RuntimeException.IndexOutOfRange();
 
-                ValueTableColumn Column = FindColumnByIndex(i_index);
-                return Column;
+                return FindColumnByIndex(i_index);
             }
 
-            if (index is ValueTableColumn)
+            if (index is ValueTableColumn column)
             {
-                return index as ValueTableColumn;
+                return column;
             }
 
             throw RuntimeException.InvalidArgumentType();
@@ -253,13 +235,12 @@ public int GetColumnNumericIndex(IValue index)
             {
                 int iIndex = Decimal.ToInt32(index.AsNumber());
                 if (iIndex < 0 || iIndex >= Count())
-                    throw RuntimeException.InvalidArgumentValue();
+                    throw RuntimeException.IndexOutOfRange();
 
                 return iIndex;
             }
 
-            var column = index as ValueTableColumn;
-            if (column != null)
+            if (index is ValueTableColumn column)
             {
                 return IndexOf(column);
             }