Вынесены общее исключение индекса и принадлежности строки

Author: Mr-Rm

src/OneScript.Core/Exceptions/RuntimeException.cs

@@ -182,7 +182,15 @@ public static RuntimeException IncorrectOffset()
             return new RuntimeException(
                 "Неправильное смещение внутри коллекции",
                 "Incorrect offset within collection");
+        }
+
+        public static RuntimeException IndexOutOfRange()
+        {
+            return new RuntimeException(
+                "Значение индекса выходит за пределы диапазона",
+                "Index is out of range");
         }
+
         #endregion
     }
 }

src/OneScript.StandardLibrary/Binary/BinaryDataBuffer.cs

@@ -246,7 +246,7 @@ public void WriteInt64(int position, BslValue value, BslValue byteOrder = null)
         private void WriteBitwiseOp(int position, BinaryDataBuffer buffer, int number, Func<byte, byte, byte> op)
         {
             if(position < 0)
-                throw new IndexOutOfRangeException("Значение индекса выходит за границы диапазона");
+                throw RuntimeException.IndexOutOfRange();
 
             try
             {

src/OneScript.StandardLibrary/Collections/ArrayImpl.cs

@@ -99,7 +99,7 @@ public void Add(IValue value = null)
         public void Insert(int index, IValue value = null)
         {
             if (index < 0)
-                throw IndexOutOfBoundsException();
+                throw RuntimeException.IndexOutOfRange();
 
             if (index > _values.Count)
                 Extend(index - _values.Count);
@@ -128,7 +128,7 @@ public IValue Find(IValue what)
         public void Remove(int index)
         {
             if (index < 0 || index >= _values.Count)
-                throw IndexOutOfBoundsException();
+                throw RuntimeException.IndexOutOfRange();
 
             _values.RemoveAt(index);
         }
@@ -143,7 +143,7 @@ public int UpperBound()
         public IValue Get(int index)
         {
             if (index < 0 || index >= _values.Count)
-                throw IndexOutOfBoundsException();
+                throw RuntimeException.IndexOutOfRange();
 
             return _values[index];
         }
@@ -152,7 +152,7 @@ public IValue Get(int index)
         public void Set(int index, IValue value)
         {
             if (index < 0 || index >= _values.Count)
-                throw IndexOutOfBoundsException();
+                throw RuntimeException.IndexOutOfRange();
 
             _values[index] = value;
         }
@@ -234,10 +234,5 @@ public static ArrayImpl Constructor(FixedArrayImpl fixedArray)
         {
             return new ArrayImpl(fixedArray);
         }
-
-        private static RuntimeException IndexOutOfBoundsException()
-        {
-            return new RuntimeException("Значение индекса выходит за пределы диапазона");
-        }
     }
 }

src/OneScript.StandardLibrary/Collections/ValueList/ValueListImpl.cs

@@ -212,7 +212,7 @@ private int IndexByValue(BslValue item)
                 }
 
                 if (index < 0 || index >= _items.Count())
-                    throw new RuntimeException("Значение индекса выходит за пределы диапазона");
+                    throw RuntimeException.IndexOutOfRange();
             }
 
             return index;

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

@@ -504,7 +504,7 @@ private int IndexByValue(BslValue item)
             {
                 index = IndexOf(row);
                 if (index == -1)
-                    throw new RuntimeException("Строка не принадлежит таблице значений");
+                    throw ValueTableException.RowDoesntBelongTo();
             }
             else
             {
@@ -518,7 +518,7 @@ private int IndexByValue(BslValue item)
                 }
 
                 if (index < 0 || index >= _rows.Count)
-                    throw new RuntimeException("Значение индекса выходит за пределы диапазона");
+                    throw RuntimeException.IndexOutOfRange();
             }
 
             return index;
@@ -760,5 +760,24 @@ public IValue GetField(string name)
         {
             return Columns.FindColumnByName(name);
         }
-    }
+    }
+
+    public class ValueTableException : RuntimeException
+    {
+        public ValueTableException(BilingualString message, Exception innerException) : base(message,
+            innerException)
+        {
+        }
+
+        public ValueTableException(BilingualString message) : base(message)
+        {
+        }
+
+        public static ValueTableException RowDoesntBelongTo()
+        {
+            return new ValueTableException(new BilingualString(
+                "Строка не принадлежит таблице значений",
+                "Row does not belong to table"));
+        }
+    }
 }

src/ScriptEngine/Machine/Contexts/StackTraceCollectionContext.cs

@@ -53,15 +53,9 @@ public override IValue GetIndexedValue(IValue index)
             var idx = (int)index.AsNumber();
 
             if (idx < 0 || idx >= Count())
-                throw IndexOutOfBoundsException();
+                throw RuntimeException.IndexOutOfRange();
 
             return _frames[idx];
         }
-
-        private static RuntimeException IndexOutOfBoundsException()
-        {
-            return new RuntimeException("Значение индекса выходит за пределы диапазона");
-        }
-
     }
 }

tests/binary-objects.os

@@ -277,7 +277,7 @@
 		Буфер = ПолучитьБуферДвоичныхДанныхИзHexСтроки("01 02 03");
 		Буфер.ЗаписатьПобитовоеИсключительноеИли(-1, ВторойБуфер, 3);
 	Исключение
-		юТест.ПроверитьВхождение(ОписаниеОшибки(), "Значение индекса выходит за границы диапазона");
+		юТест.ПроверитьВхождение(ОписаниеОшибки(), "Значение индекса выходит за пределы диапазона");
 	КонецПопытки;
 
 КонецПроцедуры