-
Notifications
You must be signed in to change notification settings - Fork 272
Conversation
AkulinMitya
commented
Oct 14, 2025
@Pasha0666
Pasha0666
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Перва часть задачи сделана хорошо
Считаю new AssertionScope() что-то около чита. Ну и сильно много всего не стоит в него запихивать, т.к. читаемость от этого в итоге не растет
Часть про Царя тут тоже норм решение, но можно улучшить
И давай еще вести хорошие коммиты. Ну т.е. разбивать их по логическим кускам и давать внятное именование
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А что будет если появиться еще одно поле с окончанием Id?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Еще мое любимое про рекурсии тут есть
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А что будет если появиться еще одно поле с окончанием Id?
Тогда мы упустим это новое поле из сравнения. Ну то есть не очень надежное правило получилось конечно.
Переписать?
Про рекурсию это ты видимо имеешь ввиду случай когда у нас зациклены ссылки. Типо есть два царя и каждый является родителем другого. Хоть в теории такого не должно быть, но, как я понял, тесты с цикличными ссылками будут падать по дефолту и нужно добавить .IgnoringCyclicReferences() чтобы такие тесты проходили.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ага, переписать историю с Id
Про рекурсию прав, можешь не добавлять
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Я бы сказал зависит от, но в целом согласен)
No description provided.