Зарипов Кирилл #210
Зарипов Кирилл #210
Conversation
ObjectPrinting/PrintingConfig.cs
Outdated
|
|
||
| private readonly HashSet<Type> excludedTypes = []; | ||
| private readonly HashSet<string> excludedProperties = []; | ||
| private readonly HashSet<object> processedObjects = []; |
There was a problem hiding this comment.
Не уверен, но скорее всего в конструктор HashSet'а нужно передать ReferenceEqualityComparer.Instance. Иначе будет использоваться Equals объекта и есть вероятность, что "совпадут" два одинаковых по состоянию, но разных по ссылке объекта
ObjectPrinting/PrintingConfig.cs
Outdated
| result = obj switch | ||
| { | ||
| IFormattable formattable => formattable.ToString(null, numberCulture) + Environment.NewLine, | ||
| string str when maxStringLength.HasValue => string.Concat( |
There was a problem hiding this comment.
Круто что знаешь про Span, но при наличии ограничения на длину, сама строка может быть сразу подходящей и тогда всё это преобразование будет избыточным. Кстати, что будет, если сравнить str.Length < (или >) int?(null)?
ObjectPrinting/PrintingExtensions.cs
Outdated
|
|
||
| namespace ObjectPrinting; | ||
|
|
||
| public static class PrintingExtensions |
There was a problem hiding this comment.
nit: Операция выглядит слишком специфичной, чтобы быть расширением. Я бы завел статические методы в ObjectPrinter
ObjectPrinting/PrintingConfig.cs
Outdated
| return this; | ||
| } | ||
|
|
||
| public PrintingConfig<TOwner> SetCulture(IFormatProvider culture) |
There was a problem hiding this comment.
Насколько понял постановку задачи, культуру должно быть можно задать для каждого типа в отдельности (хотя я плохо понимаю сценарий, в котором могло бы такое понадобиться)
No description provided.