правки по замечаниям coderabbit; добавлены тесты

Author: Mr-Rm

src/OneScript.Core/Exceptions/RuntimeException.cs

@@ -180,8 +180,8 @@ public static RuntimeException InvalidEncoding(string encoding)
         public static RuntimeException IncorrectOffset()
         {
             return new RuntimeException(
-                $"Неправильное смещение внутри коллекции",
-                $"Incorrect offset within collection");
+                "Неправильное смещение внутри коллекции",
+                "Incorrect offset within collection");
         }
         #endregion
     }

src/OneScript.Language/Localization/BilingualString.cs

@@ -73,12 +73,6 @@ public static string Localize(string russian, string english)
 
             return russian;
         }
-
-        public static bool BiEquals(string str, string lang1, string lang2)
-        {
-            StringComparison comparison = StringComparison.CurrentCultureIgnoreCase;
-            return string.Equals(str, lang1, comparison) || string.Equals(str, lang2, comparison);
-        }
     }
 
     public static class BilingualStringExtension

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

@@ -5,19 +5,20 @@ This Source Code Form is subject to the terms of the
 at http://mozilla.org/MPL/2.0/.
 ----------------------------------------------------------*/
 
-using System;
-using System.Collections.Generic;
-using System.Linq;
 using OneScript.Contexts;
-using OneScript.StandardLibrary.Collections.Indexes;
 using OneScript.Exceptions;
 using OneScript.Execution;
+using OneScript.Localization;
+using OneScript.StandardLibrary.Collections.Exceptions;
+using OneScript.StandardLibrary.Collections.Indexes;
 using OneScript.Types;
 using OneScript.Values;
 using ScriptEngine.Machine;
 using ScriptEngine.Machine.Contexts;
-using OneScript.StandardLibrary.Collections.Exceptions;
-using OneScript.Localization;
+using System;
+using System.Collections.Generic;
+using System.Data.Common;
+using System.Linq;
 
 namespace OneScript.StandardLibrary.Collections.ValueTable
 {
@@ -242,39 +243,51 @@ public int IndexOf(IValue row)
         [ContextMethod("Итог", "Total")]
         public IValue Total(IValue columnIndex)
         {
-            var Column = Columns.GetColumnByIIndex(columnIndex);
-            bool has_data = false;
-            decimal result = 0;
+            var column = Columns.GetColumnByIIndex(columnIndex);
 
-            var types = Column.ValueType.Types();
+            var types = column.ValueType.Types();
             if (types.Count() == 1)  // единственный тип
             {
-                foreach (var row in _rows)
-                {
-                    try
-                    {
-                        result += row.Get(Column).AsNumber();
-                        has_data = true;
-                    }
-                    catch (RuntimeException)
-                    { }
-                }
+                return TotalAllAsNumber(column);
             }
-            else if (types.Count() == 0 // нет типов
+            
+            if (types.Count() == 0 // нет типов
                 || types.Any(x => ((BslTypeValue)x).TypeValue == BasicTypes.Number)) // среди типов есть Число
             {
-                foreach (var row in _rows)
+                return TotalNumbersOnly(column);
+            }
+
+            // несколько типов и нет типа Число
+            return ValueFactory.Create();
+        }
+
+        private IValue TotalAllAsNumber(ValueTableColumn column)
+        {
+            decimal result = 0;
+            foreach (var row in _rows)
+            {
+                try
                 {
-                    var current_value = row.Get(Column);
-                    if (current_value.SystemType == BasicTypes.Number)
-                    {
-                        result += current_value.AsNumber();
-                        has_data = true;
-                    }
-                }
-            }
- 
-            return has_data ? ValueFactory.Create(result) : ValueFactory.Create();
+                    result += row.Get(column).AsNumber();
+                }
+                catch (RuntimeException)
+                { }
+            }
+            return ValueFactory.Create(result);
+        }
+
+        private IValue TotalNumbersOnly(ValueTableColumn column)
+        {
+            decimal result = 0;
+            foreach (var row in _rows)
+            {
+                var current_value = row.Get(column);
+                if (current_value.SystemType == BasicTypes.Number)
+                {
+                    result += current_value.AsNumber();
+                }
+            }
+            return ValueFactory.Create(result);
         }
 
         /// <summary>

tests/valuetable.os

@@ -67,6 +67,10 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_Строка");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаЧисло");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаДата");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаБезЧисел");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаЧислоБезЧисел");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаДатаБезЧисел");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьИтогПоКолонкеБезОписанияТипов_БезЧисел");
 
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьИсключениеПриНеверноИмениКолонкиВОтборе");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьЗапятуюВИменахКолонок_БезПустых");
@@ -287,15 +291,15 @@
 	Исключение
 		Ошибка = ИнформацияОбОшибке().Описание;
 	КонецПопытки;
-	юТест.ПроверитьРавенство(Ошибка, "Неверное значение аргумента номер 2");
+	юТест.ПроверитьРавенство(Ошибка, "Неправильное смещение внутри коллекции");
 
 	Ошибка = "Сдвиг за пределы вверх";
 	Попытка
 		Т.Сдвинуть(1, -3);
 	Исключение
 		Ошибка = ИнформацияОбОшибке().Описание;
 	КонецПопытки;
-	юТест.ПроверитьРавенство(Ошибка, "Неверное значение аргумента номер 2");
+	юТест.ПроверитьРавенство(Ошибка, "Неправильное смещение внутри коллекции");
 
 	Ошибка = "Сдвиг с неверным индексом";
 	Попытка
@@ -1121,6 +1125,46 @@
 	юТест.ПроверитьРавенство(ТЗ.Итог("Тест"), Неопределено);	
 КонецПроцедуры
 
+Процедура ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаБезЧисел() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест", Новый ОписаниеТипов("Строка"));
+	ТЗ.Добавить().Тест = "без";	
+	ТЗ.Добавить().Тест = "числовых";	
+	ТЗ.Добавить().Тест = "значений";
+
+	юТест.ПроверитьРавенство(ТЗ.Итог("Тест"), 0);	
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаЧислоБезЧисел() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест", Новый ОписаниеТипов("Строка,Число"));
+	ТЗ.Добавить().Тест = "без";	
+	ТЗ.Добавить().Тест = "числовых";	
+	ТЗ.Добавить().Тест = "значений";
+
+	юТест.ПроверитьРавенство(ТЗ.Итог("Тест"), 0);	
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьИтогПоКолонкеСОписаниемТипов_СтрокаДатаБезЧисел() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест", Новый ОписаниеТипов("Строка,Дата"));
+	ТЗ.Добавить().Тест = "без";	
+	ТЗ.Добавить().Тест = "числовых";	
+	ТЗ.Добавить().Тест = "значений";
+
+	юТест.ПроверитьРавенство(ТЗ.Итог("Тест"), Неопределено);	
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьИтогПоКолонкеБезОписанияТипов_БезЧисел() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+	ТЗ.Добавить().Тест = "без";	
+	ТЗ.Добавить().Тест = "числовых";	
+	ТЗ.Добавить().Тест = "значений";
+
+	юТест.ПроверитьРавенство(ТЗ.Итог("Тест"), 0);	
+КонецПроцедуры
+
 Процедура ТестДолжен_ПроверитьИсключениеПриНеверноИмениКолонкиВОтборе() Экспорт
 	ТЗ = Новый ТаблицаЗначений();
 	ТЗ.Колонки.Добавить("Тест");