Merge pull request EvilBeaver#1629 from Mr-Rm/v2/fix-vtrow

V2 Исправления к СтрокаТаблицыЗначений

Author: EvilBeaver

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

@@ -156,7 +156,13 @@ public ValueTableColumn FindColumnByName(string name)
             return _columns.Find(column => _namesComparer.Equals(name, column.Name));
         }
 
-        public ValueTableColumn FindColumnByIndex(int index) => _columns[index];
+        public ValueTableColumn FindColumnByIndex(int index)
+        {
+            if (index < 0 || index >= _columns.Count)
+                throw RuntimeException.IndexOutOfRange();
+            return _columns[index];
+        }
+
 
         public IEnumerator<ValueTableColumn> GetEnumerator()
         {
@@ -209,11 +215,7 @@ public ValueTableColumn GetColumnByIIndex(IValue index)
 
             if (index.SystemType == BasicTypes.Number)
             {
-                int i_index = Decimal.ToInt32(index.AsNumber());
-                if (i_index < 0 || i_index >= Count())
-                    throw RuntimeException.IndexOutOfRange();
-
-                return FindColumnByIndex(i_index);
+                return FindColumnByIndex(decimal.ToInt32(index.AsNumber()));
             }
 
             if (index is ValueTableColumn column)

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

@@ -16,7 +16,7 @@ namespace OneScript.StandardLibrary.Collections.ValueTable
     [ContextClass("СтрокаТаблицыЗначений", "ValueTableRow")]
     public class ValueTableRow : AutoContext<ValueTableRow>, ICollectionContext<IValue>, IDebugPresentationAcceptor
     {
-        private readonly Dictionary<IValue, IValue> _data = new Dictionary<IValue, IValue>();
+        private readonly Dictionary<ValueTableColumn, IValue> _data = new Dictionary<ValueTableColumn, IValue>();
         private readonly ValueTable _owner;
 
         public ValueTableRow(ValueTable owner)
@@ -26,7 +26,7 @@ public ValueTableRow(ValueTable owner)
 
         public int Count()
         {
-            return Owner().Columns.Count();
+            return _owner.Columns.Count();
         }
 
         public int Count(IBslProcess process) => Count();
@@ -43,12 +43,11 @@ public ValueTable Owner()
 
 		private IValue TryValue(ValueTableColumn Column)
 		{
-			IValue Value;
-			if (_data.TryGetValue(Column, out Value))
-			{
-				return Value;
-			}
-			return Column.ValueType.AdjustValue();
+            if (_data.TryGetValue(Column, out IValue Value))
+            {
+                return Value;
+            }
+            return Column.ValueType.AdjustValue();
 		}
 
         /// <summary>
@@ -59,19 +58,17 @@ private IValue TryValue(ValueTableColumn Column)
         [ContextMethod("Получить", "Get")]
         public IValue Get(int index)
         {
-            var C = Owner().Columns.FindColumnByIndex(index);
-            return TryValue(C);
+            return TryValue(_owner.Columns.FindColumnByIndex(index));
         }
 
         public IValue Get(IValue index)
         {
-            var C = Owner().Columns.GetColumnByIIndex(index);
-            return TryValue(C);
+            return TryValue(_owner.Columns.GetColumnByIIndex(index));
         }
 
-        public IValue Get(ValueTableColumn c)
+        public IValue Get(ValueTableColumn column)
         {
-            return TryValue(c);
+            return TryValue(column);
         }
 
         /// <summary>
@@ -82,31 +79,29 @@ public IValue Get(ValueTableColumn c)
         [ContextMethod("Установить", "Set")]
         public void Set(int index, IValue value)
         {
-            var C = Owner().Columns.FindColumnByIndex(index);
-            Set(C, value);
+            Set(_owner.Columns.FindColumnByIndex(index), value);
         }
 
         public void Set(IValue index, IValue value)
         {
-            var C = Owner().Columns.GetColumnByIIndex(index);
-            Set(C, value);
+            Set(_owner.Columns.GetColumnByIIndex(index), value);
         }
 
         public void Set(ValueTableColumn column, IValue value)
         {
-            Owner().Indexes.ElementRemoved(this);
+            _owner.Indexes.ElementRemoved(this);
             _data[column] = column.ValueType.AdjustValue(value);
-            Owner().Indexes.ElementAdded(this);
+            _owner.Indexes.ElementAdded(this);
         }
 
-        public void OnOwnerColumnRemoval(IValue column)
+        public void OnOwnerColumnRemoval(ValueTableColumn column)
         {
             _data.Remove(column);
         }
 
         public IEnumerator<IValue> GetEnumerator()
         {
-            foreach (var item in Owner().Columns)
+            foreach (var item in _owner.Columns)
             {
                 yield return TryValue(item);
             }
@@ -124,51 +119,41 @@ public override int GetPropCount()
 
         public override string GetPropName(int propNum)
         {
-            return Owner().Columns.GetPropName(propNum);
+            return _owner.Columns.GetPropName(propNum);
         }
 
         public override int GetPropertyNumber(string name)
         {
-            return Owner().Columns.GetPropertyNumber(name);
+            return _owner.Columns.GetPropertyNumber(name);
         }
 
-        public override bool IsPropReadable(int propNum)
-        {
-            return true;
-        }
+        public override bool IsPropReadable(int propNum) => true;
 
-        public override bool IsPropWritable(int propNum)
-        {
-            return true;
-        }
+        public override bool IsPropWritable(int propNum) => true; 
 
         public override IValue GetPropValue(int propNum)
         {
-            var C = Owner().Columns.FindColumnByIndex(propNum);
-            return TryValue(C);
+            return TryValue(_owner.Columns.FindColumnByIndex(propNum));
         }
 
 		public override void SetPropValue(int propNum, IValue newVal)
 		{
-			var C = Owner().Columns.FindColumnByIndex(propNum);
-            Set(C, newVal);
+            Set(_owner.Columns.FindColumnByIndex(propNum), newVal);
 		}
 
         private ValueTableColumn GetColumnByIIndex(IValue index)
         {
-            return Owner().Columns.GetColumnByIIndex(index);
+            return _owner.Columns.GetColumnByIIndex(index);
         }
 
         public override IValue GetIndexedValue(IValue index)
         {
-            ValueTableColumn C = GetColumnByIIndex(index);
-            return TryValue(C);
+            return TryValue(GetColumnByIIndex(index));
         }
 
 		public override void SetIndexedValue(IValue index, IValue val)
 		{
-			var C = GetColumnByIIndex(index);
-			_data[C] = C.ValueType.AdjustValue(val);
+            Set(GetColumnByIIndex(index), val);
 		}
 
         void IDebugPresentationAcceptor.Accept(IDebugValueVisitor visitor)

tests/ValueTableIndex.os

@@ -24,6 +24,7 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьПоискСИндексом");
 	ВсеТесты.Добавить("ТестДолжен_ПоведениеИндексовПриУдаленииКолонок");
 	ВсеТесты.Добавить("ТестДолжен_ПоведениеИндексовПриИзмененииЗначенийИндексированныхПолей");
+	ВсеТесты.Добавить("ТестДолжен_ПоведениеИндексовПриИзмененииЗначенийИндексированныхПолейПоИндексу");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьВыбросИсключений");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьПоискПоИндексированнойКолонкеСЗаданнымТипом");
 
@@ -179,6 +180,29 @@
 
 КонецПроцедуры
 
+Процедура ТестДолжен_ПоведениеИндексовПриИзмененииЗначенийИндексированныхПолейПоИндексу() Экспорт
+
+	Т = Новый ТаблицаЗначений;
+	Т.Колонки.Добавить("К1");
+	Т.Колонки.Добавить("К2");
+	Т.Индексы.Добавить("К2");
+	Т.Индексы.Добавить("К2, К1");
+	Т.Индексы.Добавить("К1");
+	Т.Добавить().К1 = 1;
+	Т.Добавить().К1 = 2;
+	Т.Добавить().К1 = 3;
+
+	Т[0]["К1"] = 3;
+	Т[1]["К1"] = 2;
+	Т[2]["К1"] = 1;
+
+	СтруктураПоиска = Новый Структура("К1", 1);
+	НайденныеСтроки = Т.НайтиСтроки(СтруктураПоиска);
+	юТест.ПроверитьРавенство(НайденныеСтроки.Количество(), 1);
+	юТест.ПроверитьРавенство(НайденныеСтроки[0], Т[2]);
+
+КонецПроцедуры
+
 Процедура ТестДолжен_ПроверитьВыбросИсключений() Экспорт
 
 	Т = Новый ТаблицаЗначений;

tests/valuetable.os

@@ -83,6 +83,8 @@
 	ВсеТесты.Добавить("ТестДолжен_ВызватьИсключениеНаПоискеИндексаКолонкиСНевернымПримитивнымТипом");
 	ВсеТесты.Добавить("ТестДолжен_ВызватьИсключениеНаПоискеИндексаКолонкиСНевернымОбъектнымТипом");
 
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьИсключениеПолученияКолонкиСНевернымНомером");
+	
 
 	Возврат ВсеТесты;
 
@@ -1288,3 +1290,30 @@
 
 	ВызватьИсключение "Должно было быть выдано исключение, но его не было";
 КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьИсключениеПолученияКолонкиСНевернымНомером() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	СтрокаТЗ = ТЗ.Добавить();
+	
+	БылоИсключение = Ложь;
+	Попытка
+		Рез = СтрокаТЗ.Получить(-1);
+	Исключение
+		Ошибка = ИнформацияОбОшибке().Описание;
+		юТест.ПроверитьНеРавенство(Найти(Ошибка,"Значение индекса выходит за пределы"), 0, "Неверный вид ошибки: "+Ошибка);
+		БылоИсключение = Истина;
+	КонецПопытки;
+	юТест.ПроверитьИстину(БылоИсключение, "Получение колонки с неверным номером");
+
+	БылоИсключение = Ложь;
+	Попытка
+		Рез = СтрокаТЗ[-1];
+	Исключение
+		Ошибка = ИнформацияОбОшибке().Описание;
+		юТест.ПроверитьНеРавенство(Найти(Ошибка,"Значение индекса выходит за пределы"), 0, "Неверный вид ошибки: "+Ошибка);
+		БылоИсключение = Истина;
+	КонецПопытки;
+	юТест.ПроверитьИстину(БылоИсключение, "Получение колонки по неверному индексу");
+
+КонецПроцедуры