Дмитрий Леонтьев #221

Dary0302 · 2024-12-11T08:25:56Z

No description provided.

ObjectPrinting/Tests/ObjectPrinterAcceptanceTests.cs

@@ -1,27 +1,71 @@
-using NUnit.Framework;
+using System;


ObjectPrinting/PrintingConfig.cs

-    public class PrintingConfig<TOwner>
+    public int MaxNestingLevel { get; set; } = 5;
+    private readonly HashSet<Type> excludeTypes = new(ReferenceEqualityComparer.Instance);
+    private readonly HashSet<string> excludeProperties = [];


ObjectPrinting/PropertyPrintingConfig.cs

+    public PrintingConfig<TOwner> OriginalConfig { get; }
+    public MemberInfo? PropertyName { get; }
+
+    public PropertyPrintingConfig(PrintingConfig<TOwner> config)


ObjectPrinting/PropertyPrintingConfig.cs

+
+    public PrintingConfig<TOwner> Using(Func<TPropType, string> serializeFunc)
+    {
+        if (string.IsNullOrEmpty(PropertyName?.Name))


ObjectPrinting/PropertyPrintingConfig.cs

+        return OriginalConfig;
+    }
+
+    public PrintingConfig<TOwner> TrimmedToLength(int length)


ObjectPrinting/Tests/ObjectPrinterAcceptanceTests.cs

+namespace ObjectPrinting.Tests;
+
+[TestFixture]
+public class ObjectPrinterAcceptanceTests


Dary0302 · 2024-12-11T14:06:47Z

ObjectPrinting/PrintingConfig.cs

+            return serializer.DynamicInvoke(obj)?.ToString()!;
+        }
+
+        if (type.IsPrimitive || type == typeof(string) || type == typeof(Guid))


А мы их выше обработаем, нет?
if (serializersToType.TryGetValue(type, out var serializer)) { return serializer.DynamicInvoke(obj)?.ToString()!; }

Добавил такой тест, он успешно исключает этот тип

Dary0302 · 2024-12-11T16:25:51Z

ObjectPrinting/PrintingConfig.cs

+                continue;
+            }
+
+            if (propertyTrim.TryGetValue(propertyName, out var trimLength) && propertyValue is string stringValue)


Тогда вопрос, а в каком порядке нужно будет обрабатывать значение?

ObjectPrinting/PrintingConfig.cs

+            var propertyType = propertyInfo.PropertyType;
+            var propertyName = propertyInfo.Name;
+
+            if (propertySerializers.TryGetValue(propertyName, out var propertySerializer))


…intingConfig)

ObjectPrinting.Test/PrintingConfigTests.cs

+    public void Exclude_ExcludeType()
+    {
+        const string unexpectedName = nameof(Person.Name);
+        const string unexpectedSurname = nameof(Person.Name);


shiyois · 2024-12-12T08:54:39Z

ObjectPrinting.Test/PrintingConfigTests.cs

+    [TestCase(0)]
+    [TestCase(1)]
+    [TestCase(2)]
+    public void TrimmedTo_TrimmingName(int length)


(optional) Еще так можно [Values(0, 1, 2)] int length

ObjectPrinting.Test/PrintingConfigTests.cs

+        FluentActions.Invoking(() => ObjectPrinter.For<Person>()
+                .SetMaxNestingLevel(levelNesting))
+                .Should().Throw<ArgumentException>()
+                .WithMessage("Max nesting level must be greater than or equal to 0.");


shiyois · 2024-12-12T09:08:10Z

ObjectPrinting/PrintingConfig.cs

+        return PrintToString(obj, 1);
+    }
+
+    private string PrintToString(object? obj, int nestingLevel)


Что на счет обработки циклических ссылок между объектами?
Не увидел тестов на такое поведение

SetMaxNestingLevel_WithDifferentNestingLevels, а как же этот тест?

SetMaxNestingLevel_WithDifferentNestingLevels, а как же этот тест?

Тут ты рекурсию обрабатываешь вызовов, я говорю про случай циклических ссылок, когда объект указывает сам на себя

Нашел баг. Пишем что циклическая ссылка, если это не циклическая ссылка))
Пример теста который сейчас не проходит:

var result = ObjectPrinter.For<List<Person>>() .PrintToString([firstPerson, firstPerson]); result.Should().NotContain("It is not possible to print an object with a circular reference.");

…ultiple serializations for the same type

added a solution to the task

52d1247