HomeworkV2 by Balashov2004 · Pull Request #250 · kontur-courses/fluent-api

Balashov2004 · 2025-11-20T16:06:15Z

No description provided.

HELLoWorlD01100 · 2025-11-20T16:15:14Z

+using System;
+using System.Collections.Generic;
+
+namespace ObjectPrinting;


namespace поломан
папку с большой буквы

namespace ObjectPrinting.Interface;

HELLoWorlD01100 · 2025-11-20T16:35:22Z

+    public HashSet<Type> ExcludedTypes { get; } = new();
+    public HashSet<string> ExcludedProperties { get; } = new();
+
+    public Dictionary<Type, Func<object, string>> TypeSerializers { get; } = new();
+    public Dictionary<string, Func<object, string>> PropertySerializers { get; } = new();
+    public Dictionary<string, int> TrimLengths { get; } = new();


Все это отображается в интелисенсе, что будет мешать изучению сервиса через fluent подход.
Плюсом, я могу в обход методов добавить в твои коллекции что угодно

не понял
они же должны вызываться то есть не могут быть private

HELLoWorlD01100 · 2025-11-20T16:45:39Z

+
+    public PrintingConfig<TOwner> Excluding<TProp>(Expression<Func<TOwner, TProp>> selector)
+    {
+        var name = ((MemberExpression)selector.Body).Member.Name;


По условиям задачи - "Исключение из сериализации конкретного свойства/поля".
А значит это исключение не должно влиять на другие типы с таким же названием поля. У тебя сейчас влияет

Под эту ситуацию нужны также тесты

HELLoWorlD01100 · 2025-11-20T16:57:48Z

+
+namespace ObjectPrinting;
+
+public interface IPrintingConfigInternal


Для чего нам в целом интерфейс?

HELLoWorlD01100 · 2025-11-20T17:05:31Z

+    public TypePrintingConfig<TOwner, TProp> Printing<TProp>()
+        => new TypePrintingConfig<TOwner, TProp>(this);


HELLoWorlD01100 · 2025-11-20T18:37:03Z

+
+namespace ObjectPrinting
+{
+    public static class ObjectTraversal


Давай назовем более понятно

Не очень понимаю смысл статичности в этом случае

исправил

HELLoWorlD01100 · 2025-11-20T18:48:23Z

+            foreach (var property in type.GetProperties(BindingFlags.Public | BindingFlags.Instance)
+                         .Where(p => p.GetIndexParameters().Length == 0))
+            {
+                if (config.ExcludedTypes.Contains(property.PropertyType)) continue;
+                if (config.ExcludedProperties.Contains(property.Name)) continue;
+
+                var value = property.GetValue(obj);
+                string printed = PrintMember(property.Name, value, property.PropertyType,
+                    config, level + 1, visited);
+
+                sbObj.Append(indentObj + "\t" + printed);
+            }
+
+            foreach (var field in type.GetFields(BindingFlags.Public | BindingFlags.Instance))
+            {
+                if (config.ExcludedTypes.Contains(field.FieldType)) continue;
+                if (config.ExcludedProperties.Contains(field.Name)) continue;
+
+                var value = field.GetValue(obj);
+                string printed = PrintMember(field.Name, value, field.FieldType,
+                    config, level + 1, visited);
+
+                sbObj.Append(indentObj + "\t" + printed);
+            }


Разбил

HELLoWorlD01100 · 2025-11-20T18:59:55Z

+            if (config.PropertySerializers.TryGetValue(memberName, out var serializer))
+            {
+                string val = serializer(value);
+                return memberName + " = " + val + "\n";
+            }
+
+            if (config.TypeSerializers.TryGetValue(memberType, out var typeSerializer))
+            {
+                string val = typeSerializer(value);
+                return memberName + " = " + val + "\n";
+            }
+
+            if (config.TrimLengths.TryGetValue(memberName, out var maxLen) && value is string s)
+            {
+                if (s.Length > maxLen)
+                    s = s.Substring(0, maxLen);
+
+                return memberName + " = " + s + "\n";
+            }


Потенциальное исключение, пример на пикрил

HELLoWorlD01100 · 2025-11-20T19:01:02Z

+            if (config.PropertySerializers.TryGetValue(memberName, out var serializer))
+            {
+                string val = serializer(value);
+                return memberName + " = " + val + "\n";
+            }
+
+            if (config.TypeSerializers.TryGetValue(memberType, out var typeSerializer))
+            {
+                string val = typeSerializer(value);
+                return memberName + " = " + val + "\n";
+            }


DRY. Вызвали функцию -> сериализовали

HELLoWorlD01100 · 2025-11-20T19:11:00Z

+            return sbObj.ToString();
+        }
+
+        private static string PrintMember(


Обрезание строки никогда не будет работать, если задана кастомная обработка строчки.

В случае с полем - более приоритетная обработка поля, тут ок.
Но вот трим и обработка строки - на одном уровне, это буквально синтактическое упрощение.

and other small fix

HELLoWorlD01100 · 2025-11-24T05:39:46Z

+
+public class TypeHelper
+{
+    public static bool IsSimpleType(Type type)


Лучше переделать в Extensions и вызывать прямо из типа.

typeof(...).IsSimpleType

HELLoWorlD01100 · 2025-11-24T05:40:40Z

+        if (type.IsPrimitive) return true;
+        if (type == typeof(string)) return true;
+        if (type == typeof(decimal)) return true;
+        if (type == typeof(DateTime)) return true;
+        if (type == typeof(TimeSpan)) return true;
+        if (type.IsEnum) return true;
+
+        return false;


return type.IsPrimitive ||
type == typeof(string) || ...

HELLoWorlD01100 · 2025-11-24T05:41:56Z

+public class ReferenceEqualityComparer : IEqualityComparer<Object>
+{
+    public new bool Equals(object x, object y) => ReferenceEquals(x, y);
+    public int GetHashCode(object obj) => RuntimeHelpers.GetHashCode(obj);


А obj.GetHashCode почему не подошел?

HELLoWorlD01100 · 2025-11-24T05:42:22Z

+
+namespace ObjectPrinting;
+
+public class ReferenceEqualityComparer : IEqualityComparer<Object>


https://learn.microsoft.com/en-us/dotnet/api/system.collections.generic.referenceequalitycomparer?view=net-9.0

HELLoWorlD01100 · 2025-11-24T05:50:27Z

+        foreach (var property in type.GetProperties(BindingFlags.Public | BindingFlags.Instance))
+        {
+            if (property.GetIndexParameters().Length > 0) continue;
+            if (config.ExcludedTypes.Contains(property.PropertyType)) continue;
+            if (config.ExcludedProperties.Contains(property.Name)) continue;
+
+            var value = property.GetValue(obj);
+            sb.Append(indent + "\t" +
+                      memberPrinter.PrintMember(property.Name, value, property.PropertyType, config, level + 1, visited, this));
+        }
+
+        foreach (var field in type.GetFields(BindingFlags.Public | BindingFlags.Instance))
+        {
+            if (config.ExcludedTypes.Contains(field.FieldType)) continue;
+            if (config.ExcludedProperties.Contains(field.Name)) continue;
+
+            var value = field.GetValue(obj);
+            sb.Append(indent + "\t" +
+                      memberPrinter.PrintMember(field.Name, value, field.FieldType, config, level + 1, visited, this));
+        }


DRY так и осталось

HELLoWorlD01100 · 2025-11-24T06:13:15Z

+    HashSet<Type> ExcludedTypes { get; }
+    HashSet<string> ExcludedProperties { get; }
+
+    Dictionary<Type, Func<object, string>> TypeSerializers { get; }
+    Dictionary<string, Func<object, string>> PropertySerializers { get; }
+    Dictionary<string, int> TrimLengths { get; }
+    public int? GlobalStringTrimLength { get; }


Я продолжаю мочь добавлять элементы напрямую в коллекции.

Зачем мне мучиться с контекстами, валидациями и т.д, если я могу просто написать TrimLength["Name"]=0

HELLoWorlD01100 · 2025-11-24T06:27:21Z

+    public PrintingConfig<TOwner> TrimStringsGlobal(int maxLength)
+    {
+        GlobalStringTrimLength = maxLength;
+        return this;
+    }


В целом так сделать можно. Но мы на этом уровне не работаем с конкретными типами.

Лучше, чтобы было типа такого: config.Printing().Trimmed(5)

Сделать это можно так:

public static PrintingConfig Trimmed(
this MemberPrintingConfig<TOwner, string> propConfig, int maxLen)

HELLoWorlD01100 · 2025-11-24T06:31:56Z

+    public IStringMemberConfig<TOwner> AsString()
+    {
+        return typeof(TProp) == typeof(string)
+            ? new StringMemberConfig<TOwner>(config, memberName)
+            : throw new InvalidOperationException("Property is not string");
+    }


С точки зрения интерфейса выглядит ультра странно. Я и так передал конкретное поле с конкретным типом. Но чтобы появилась пачка методов, я должен еще раз сказать тип.

Допустим, у нас появятся для других типов кастомные методы. Мы также будем дописывать AsGuid, AsEnum, AsInt?

Плюсом, у нас возможно исключение. Что вообще весь пользовательский опыт рушит

HELLoWorlD01100 · 2025-11-24T06:32:45Z

+    public PrintingConfig<TOwner> Using(Func<TProp, string> serializer)
+    {
+        config.PropertySerializers[memberName] = obj => serializer((TProp)obj);
+        return config;
+    }


HELLoWorlD01100 · 2025-11-24T07:02:34Z

+    public PrintingConfig<TOwner> Using(CultureInfo culture)
+    {
+        if (!typeof(IFormattable).IsAssignableFrom(typeof(TProp)))
+            throw new InvalidOperationException("Culture can be applied only to IFormattable");
+
+        config.TypeSerializers[typeof(TProp)] =
+            o => ((IFormattable)o).ToString(null, culture);
+
+        return config;
+    }


По итогу остается, что я могу вызвать культуру даже к типам, которые культуры не имеют.
А вдобавок я получу исключение

Balashov2004 added 3 commits November 18, 2025 17:30

first

484d8fe

beta

7f314ec

beta2

2e0bbd6

HELLoWorlD01100 suggested changes Nov 20, 2025

View reviewed changes

Balashov2004 added 10 commits November 21, 2025 17:20

fix some 2

a42948b

dry fix 1

6bd26bd

1

da663d6

fix name

ca5e647

123

d05a834

Add GlobalTrim

56499f6

fix Culture

edaaf3e

delete throw

5280988

add test

f638fe6

int -> var

437537c

and other small fix

HELLoWorlD01100 suggested changes Nov 24, 2025

View reviewed changes


		namespace ObjectPrinting;

		public interface IPrintingConfigInternal

		public TypePrintingConfig<TOwner, TProp> Printing<TProp>()
		=> new TypePrintingConfig<TOwner, TProp>(this);


		namespace ObjectPrinting;

		public class ReferenceEqualityComparer : IEqualityComparer<Object>

Conversation

Balashov2004 commented Nov 20, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Balashov2004 Nov 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Balashov2004 Nov 21, 2025 •

edited

Loading