Шевелев Георгий #201
Шевелев Георгий #201
Conversation
ObjectPrinting/PrintingConfig.cs
Outdated
| private readonly HashSet<Type> finalTypes = new HashSet<Type> | ||
| { | ||
| return PrintToString(obj, 0); | ||
| typeof(int), typeof(double), typeof(float), | ||
| typeof(bool), typeof(long), typeof(string), | ||
| typeof(DateTime), typeof(TimeSpan), typeof(Guid) | ||
| }; |
There was a problem hiding this comment.
А как же остальные простые типы? Например, uint, decimal и тд?
На самом деле есть более простой способ узнать, является ли тип простым, не перечисляя все числовые вариации. Небольшая подсказка: посмотри в сторону Type
ObjectPrinting/PrintingConfig.cs
Outdated
| var identation = new string('\t', nestingLevel + 1); | ||
| return obj switch | ||
| { | ||
| IList collection => SerializeCollection(collection, nestingLevel), |
There was a problem hiding this comment.
Кажется, что проверять на список по интерфейсу IList не очень хорошая идея, так как есть ещё более базовые коллекции, которые не будут сериализованы, так как не наследуют IList
ObjectPrinting/PrintingConfig.cs
Outdated
| }; | ||
| } | ||
|
|
||
| private string SerializeCollection(IList collection, int nestingLevel) |
There was a problem hiding this comment.
Давай вытащим конкретные кейсы сериализации в отдельный класс-утилиту для сериализации
| [Test] | ||
| public void Print_WhenArray() | ||
| { | ||
| const string excepted = "Person\r\n\tParents = Person[] {\r\n\t\tPerson" + |
There was a problem hiding this comment.
Нам с тобой повезло оказаться на одной операционной системе и переварить \r\n на ней. Однако если твой сокомандник окажется на Linux или Mac, то этот тест развалится, так как на них для переноса используется другой символ \n, и строки не будут совпадать. Где-то ты уже используешь Environment.NewLine, давай и тут так же сделаем
|
|
||
| namespace ObjectPrintingTests | ||
| { | ||
| public class ObjectPrinting_Should |
There was a problem hiding this comment.
Сейчас если добавить в Person вложенный объект Person с именем Child (условно), то такой тест проходить не будет, так как настройка сериализации задела не только дочерний объект, но ещё и родительский
var result = ObjectPrinter.For<Person>()
.ChangeSerializationFor(e => e.Child.Age)
.To(s => "ВОЗРАСТ")
.PrintToString(person);
result.Should().Contain("ВОЗРАСТ", LessThan.Twice());
No description provided.