fix EvilBeaver#1577: сортировка неупорядочиваемых типов +тест

Author: EvilBeaver

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

@@ -636,14 +636,15 @@ private class RowComparator : IComparer<ValueTableRow>
         {
             readonly List<ValueTableSortRule> Rules;
 
-            readonly GenericIValueComparer _comparer = new GenericIValueComparer();
+            readonly GenericIValueComparer _comparer;
 
-            public RowComparator(List<ValueTableSortRule> Rules)
+            public RowComparator(IBslProcess process, List<ValueTableSortRule> Rules)
             {
-                if (Rules.Count() == 0)
+                if (Rules.Count == 0)
                     throw RuntimeException.InvalidArgumentValue();
 
                 this.Rules = Rules;
+                 _comparer = new GenericIValueComparer(process);
             }
 
             private int OneCompare(ValueTableRow x, ValueTableRow y, ValueTableSortRule Rule)
@@ -661,7 +662,7 @@ public int Compare(ValueTableRow x, ValueTableRow y)
                 int i = 0, r;
                 while ((r = OneCompare(x, y, Rules[i])) == 0)
                 {
-                    if (++i >= Rules.Count())
+                    if (++i >= Rules.Count)
                         return 0;
                 }
 
@@ -678,9 +679,9 @@ public int Compare(ValueTableRow x, ValueTableRow y)
         /// </param>
         /// <param name="comparator">СравнениеЗначений - правила сравнения значений при наличии различных типов данных в колонке.</param>
         [ContextMethod("Сортировать", "Sort")]
-        public void Sort(string columns, IValue comparator = null)
+        public void Sort(IBslProcess process, string columns, IValue comparator = null)
         {
-            _rows.Sort(new RowComparator(GetSortRules(columns)));
+            _rows.Sort(new RowComparator(process, GetSortRules(columns)));
         }
 
         /// <summary>

src/ScriptEngine/Machine/GenericIValueComparer.cs

@@ -5,14 +5,29 @@ 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 OneScript.Values;
 using ScriptEngine.Machine.Contexts;
+using OneScript.Execution;
 
 namespace ScriptEngine.Machine
 {
     public class GenericIValueComparer : IEqualityComparer<IValue>, IComparer<IValue>
     {
+        private readonly IBslProcess _process;
+        private readonly Func<IValue, IValue, int> _comparer;
+
+        public GenericIValueComparer()
+        {
+            _comparer = CompareAsStrings;
+        }
+
+        public GenericIValueComparer(IBslProcess proc)
+        {
+            _process = proc;
+            _comparer = CompareByPresentations;
+        }
 
         public bool Equals(IValue x, IValue y)
         {
@@ -37,15 +52,25 @@ public int GetHashCode(IValue obj)
             return CLR_obj.GetHashCode();
         }
 
+        private int CompareAsStrings(IValue x, IValue y)
+        {
+            return x.ToString().CompareTo(y.ToString());
+        }
+
+        private int CompareByPresentations(IValue x, IValue y)
+        {
+            return ((BslValue)x).ToString(_process).CompareTo(((BslValue)y).ToString(_process));
+        }
+
         public int Compare(IValue x, IValue y)
         {
-            if (ReferenceEquals(x, default) && ReferenceEquals(y, default))
+           if (ReferenceEquals(x, y))
                 return 0;
 
-            if (x.SystemType == y.SystemType)
+            if (x is IComparable && x.SystemType == y.SystemType )
                 return x.CompareTo(y);
             else
-                return x.ToString().CompareTo(y.ToString());
+                return _comparer(x,y);
         }
     }
 }

tests/valuetable.os

@@ -43,6 +43,9 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьСверткуПоКолонкамСуммированияСРазнымиТипами");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьСверткуПоОднойКолонкеСРазнымиТипами");
 
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьСортировкуНеупорядочиваемыхТипов");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьСортировкуРазныхТипов");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьСортировкуПоПредставлению");
 	Возврат ВсеТесты;
 
 КонецФункции
@@ -779,3 +782,77 @@
 	юТест.ПроверитьРавенство( СтрТЗ2[КолонкаСуммы], Количество,
 	     СтрШаблон("Сумма колонки %1 для значения %2 типа %3", КолонкаСуммы, Значение, ТипЗнч(Значение)) );
 КонецПроцедуры
+
+// к issue #1577
+Процедура ТестДолжен_ПроверитьСортировкуНеупорядочиваемыхТипов() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+
+	ТЗ.Добавить().Тест = Неопределено;
+	ТЗ.Добавить().Тест = Неопределено;
+
+	ТЗ.Сортировать("Тест"); // не должно упасть
+
+	юТест.ПроверитьРавенство(ТЗ[0].Тест, Неопределено);
+	юТест.ПроверитьРавенство(ТЗ[1].Тест, Неопределено);
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьСортировкуРазныхТипов() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+
+	ТЗ.Добавить().Тест = Тип("Строка");
+	ТЗ.Добавить().Тест = "Стр2";
+	ТЗ.Добавить().Тест = "СтрЪ";
+	ТЗ.Добавить().Тест = Новый Структура;
+	ТЗ.Добавить().Тест = null;
+	ТЗ.Добавить().Тест = Тип("Булево");
+
+	ТЗ.Сортировать("Тест"); // не должно упасть
+
+	юТест.ПроверитьРавенство(ТЗ[0].Тест, Null);
+	юТест.ПроверитьРавенство(ТЗ[1].Тест, Тип("Булево"));
+	юТест.ПроверитьРавенство(ТЗ[2].Тест, "Стр2");
+	юТест.ПроверитьРавенство(ТЗ[3].Тест, Тип("Строка"));
+	юТест.ПроверитьРавенство(ТипЗнч(ТЗ[4].Тест), Тип("Структура"));
+	юТест.ПроверитьРавенство(ТЗ[5].Тест, "СтрЪ");
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьСортировкуПоПредставлению() Экспорт
+	ТЗ = Новый ТаблицаЗначений();
+	ТЗ.Колонки.Добавить("Тест");
+
+    ТекстКласса =
+    	"Процедура ОбработкаПолученияПредставления(Представление, СтандартнаяОбработка)
+    	|	СтандартнаяОбработка = Ложь;
+    	|	Представление = ""Представление0"";
+    	|КонецПроцедуры";
+    КлассИзСтроки0 = ЗагрузитьСценарийИзСтроки(ТекстКласса);
+
+    ТекстКласса =
+    	"Процедура ОбработкаПолученияПредставления(Представление, СтандартнаяОбработка)
+    	|	СтандартнаяОбработка = Ложь;
+    	|	Представление = ""Представление1"";
+    	|КонецПроцедуры";
+    КлассИзСтроки1 = ЗагрузитьСценарийИзСтроки(ТекстКласса);
+
+    ТекстКласса =
+    	"Процедура ОбработкаПолученияПредставления(Представление, СтандартнаяОбработка)
+    	|	СтандартнаяОбработка = Ложь;
+    	|	Представление = ""Представление3"";
+    	|КонецПроцедуры";
+    КлассИзСтроки3 = ЗагрузитьСценарийИзСтроки(ТекстКласса);
+
+	ТЗ.Добавить().Тест = КлассИзСтроки3;
+	ТЗ.Добавить().Тест = "Представление2";
+	ТЗ.Добавить().Тест = КлассИзСтроки0;
+	ТЗ.Добавить().Тест = КлассИзСтроки1;
+	
+	ТЗ.Сортировать("Тест");
+	юТест.ПроверитьРавенство(Строка(ТЗ[0].Тест), "Представление0");
+	юТест.ПроверитьРавенство(Строка(ТЗ[1].Тест), "Представление1");
+	юТест.ПроверитьРавенство(ТЗ[2].Тест, "Представление2");
+	юТест.ПроверитьРавенство(Строка(ТЗ[3].Тест), "Представление3");
+  
+КонецПроцедуры
+