Skip to content

Comments

Семёнов Дмитрий#256

Open
dm7672 wants to merge 5 commits intokontur-courses:masterfrom
dm7672:master
Open

Семёнов Дмитрий#256
dm7672 wants to merge 5 commits intokontur-courses:masterfrom
dm7672:master

Conversation

@dm7672
Copy link

@dm7672 dm7672 commented Nov 22, 2025

No description provided.

@dm7672
Copy link
Author

dm7672 commented Nov 22, 2025

@Zuguki

var person = new Person { Id = Guid.NewGuid(), Name = "Петя", Height = 1.82, Age = 19 };
string s = ObjectPrinter.For<Person>().PrintToString(person);

s.Should().Contain("Person");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

на будущее: лучше в тестах писать так проверки:

            s.Should().Contain(nameof(Person));
            s.Should().Contain(nameof(Person.Name));
            s.Should().Contain(person.Name);

так как у нас могут меняться названия свойств, и если указывать так, то в тестах тоже будет меняться

PrintObject(root, 0, null);
return sb.ToString();

void PrintObject(object? obj, int nestingLevel, MemberInfo? currentMember)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем тут делать метод в методе?

@dm7672 dm7672 requested a review from Zuguki November 24, 2025 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants