Кр 3 - #28
Кр 3#28Palezehvat wants to merge 3 commits into
Conversation
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
- CI не работает, там в тестах что-то с переводами строк не сложилось.
- Параметры-типы в методах, кстати, не поддерживаются, но это ладно :)
|
|
||
| namespace Kr3; | ||
|
|
||
| public class Reflector |
| using System.Linq.Expressions; | ||
| using System.Reflection; | ||
|
|
||
| namespace Kr3; |
There was a problem hiding this comment.
namespace лучше писать первой строкой в файле, хотя дело вкуса
| private static string GetStaticOrNotFromClass(Type someClass) | ||
| { | ||
| return someClass.IsAbstract && someClass.IsSealed ? "static " : ""; | ||
| } |
There was a problem hiding this comment.
Тут и ниже лучше через => писать
| return "internal "; | ||
| } | ||
| return ""; | ||
| } |
There was a problem hiding this comment.
Так-то MethodInfo и ConstructorInfo — потомки MethodBase (потому что они суть методы), и IsPrivate/IsPublic/и т.д. — это свойства MethodBase. Так что один из этих двух методов не нужен.
| return ""; | ||
| } | ||
|
|
||
| private void WriteParameters(StreamWriter writer, ParameterInfo[] parameters) |
There was a problem hiding this comment.
Можно было бы сделать решение несколько более ООПшным, если бы разделить читалку структуры класса и писалку результатов в разные классы.
| } | ||
| private static void WriteDifferentFields(StreamWriter writer, |
There was a problem hiding this comment.
| } | |
| private static void WriteDifferentFields(StreamWriter writer, | |
| } | |
| private static void WriteDifferentFields(StreamWriter writer, |
| return methodB != null; | ||
| } | ||
|
|
||
| private static void WriteDifferentMethods(StreamWriter writer, |
There was a problem hiding this comment.
Ну и механизм диффов, видимо, можно было бы в отдельный класс вынести. Тут у Вас принцип единственности ответственности настолько нарушен, насколько это возможно.
| @@ -0,0 +1,6 @@ | |||
| class TestClassSum | |||
There was a problem hiding this comment.
Не выкладывайте ничего в bin/Debug. А то я вот перешёл на .NET 8 (и Вам советую, .NET 7 уже пару месяцев как ненужное старьё), и у меня эти файлы не находятся.
| else if (someField.IsAssembly) | ||
| { | ||
| return "internal "; | ||
| } |
There was a problem hiding this comment.
Кстати, есть ещё protected internal :)
3 participants
No description provided.