к/р 2 #7
к/р 2 #7
Conversation
yurii-litvinov
left a comment
yurii-litvinov
left a comment
There was a problem hiding this comment.
DIffClasses не поддерживается, и CI сломан
Test2.sln
Outdated
| @@ -0,0 +1,27 @@ | |||
| | |||
| Microsoft Visual Studio Solution File, Format Version 12.00 | |||
There was a problem hiding this comment.
Какой-то лишний .sln, из-за которого CI ломается
Tasks/Test2/Test2/Test2.csproj
Outdated
| <Project Sdk="Microsoft.NET.Sdk"> | ||
|
|
||
| <PropertyGroup> | ||
| <TargetFramework>net8.0</TargetFramework> |
| @@ -0,0 +1,17 @@ | |||
| public class TestClass1 | |||
| { | |||
| public int testField1 | |||
There was a problem hiding this comment.
Точки с запятой печатаются только после полей и только во вложенных классах? Интересная избирательность :)
| public int testField1 | ||
| private static string testField2 | ||
|
|
||
| public int TestMethod1(float attribute1) |
There was a problem hiding this comment.
И для методов я бы просто {} выводил (и return default для не-void), чтобы повысить шансы на компилируемость
|
|
||
| using System.Reflection; | ||
|
|
||
| public static class Reflector |
Tasks/Test2/Test2/Reflector.cs
Outdated
|
|
||
| public static class Reflector | ||
| { | ||
| private const string Tab = " "; |
There was a problem hiding this comment.
Я бы назвал Indent или как-то так, тем более что это четыре пробела
| else if (method.IsPrivate) | ||
| { | ||
| signature += "private "; | ||
| } |
There was a problem hiding this comment.
protected не поддерживаете? Правильно, protected зло.
| } | ||
|
|
||
| signature += classInfo.Name; | ||
| return signature; |
There was a problem hiding this comment.
Ещё неплохо бы наследование и реализацию интерфейсов
| foreach (var method in someClass.GetMethods()) | ||
| { | ||
| await PrintMethod(method, numberOfTabs + 1, writer); | ||
| } |
There was a problem hiding this comment.
Конструкторы не поддерживаются вроде
| public static float TestMethod2() | ||
| private string TestMethod3(string attribute1, int attribute2) | ||
|
|
||
| private class NestedClass |
There was a problem hiding this comment.
+1 за поддержку вложенных классов, больше почти никто с ними не возился.
2 participants
No description provided.