CodeWars, Dog class #16
Conversation
CourseApp.Tests/GlobalUsings.cs
Outdated
| @@ -0,0 +1,3 @@ | |||
| #pragma warning disable SA1200 | |||
There was a problem hiding this comment.
Линтер указывает на ошибку в том, что using директивы расположены неправильно. Считает, что using должен располагаться после пространства имён. Я поправил, докинул в каждый класс с тестами "using Xuint" и удалил этот файл.
На будущее хотел бы спросить, есть ли какой-то более важный плюс у этого global, чем просто везде не указывать, одинаковые using'и?
| public string Name | ||
| { | ||
| get => _name; | ||
|
|
||
| set => _name = value; | ||
| } |
There was a problem hiding this comment.
Если вы внутри не меняете поведение - то сделайте стандартно
| public string Name | |
| { | |
| get => _name; | |
| set => _name = value; | |
| } | |
| public string Name {get; set; } |
CourseApp/DogClass/Dog.cs
Outdated
| public int Age | ||
| { | ||
| get => _age; | ||
|
|
||
| set => _age = value; | ||
| } |
There was a problem hiding this comment.
возраст я могу сделать отрицательным?
| set => _flock = value; | ||
| } | ||
|
|
||
| public void PrintInfo() |
There was a problem hiding this comment.
параметр flock? или метод PrintInfo?
| } | ||
| else | ||
| { | ||
| enemy.Health -= Damage + 2; |
There was a problem hiding this comment.
а вдруг у него иммунитет на данный вид атаки? почему вы вдруг у него без его ведома меняете здоровье?
| set => _health = value; | ||
| } | ||
|
|
||
| #pragma warning disable SA1201 |
There was a problem hiding this comment.
вот так себе идея отключать так предупреждения - если не нужны - отключите в конфигурации
| _name = names[_rand.Next(0, 4)]; | ||
| _health = _rand.Next(10, 100); | ||
| _damage = _rand.Next(5, 30); | ||
| _spellUsed = false; |
There was a problem hiding this comment.
и как в этом случае тест написать?
There was a problem hiding this comment.
Так герои то должны полностью без участия пользователя создаваться и палками друг в друга кидаться. Разве нет?
CourseApp/RpgSaga/Hero.cs
Outdated
| while (person1._health > 0 || person2._health > 0) | ||
| { | ||
| person2 = person1.Attack(person2); | ||
| Thread.Sleep(1000); |
There was a problem hiding this comment.
это зачем? не его это дело - можете паузу в самой игре добавлять
Dockerfile
Outdated
| ENTRYPOINT ["dotnet", "CourseApp.dll"]~ |
No description provided.