Oleg lifanov #54
Oleg lifanov #54
Conversation
CourseApp/Transport.cs
Outdated
| { | ||
| private int passangers; | ||
| private int dlina; | ||
| private int vodoizmeschenie; |
There was a problem hiding this comment.
Не ко всему транспорту применимо водоизмещение, тем более что есть класс ship,
CourseApp/Transport.cs
Outdated
| private int dlina; | ||
| private int vodoizmeschenie; | ||
|
|
||
| public Transport() |
There was a problem hiding this comment.
А заем пустой конструктор? другие не в даом классе определены - и этот в данном случае не нужен
CourseApp/Ship.cs
Outdated
| Vodoizmeschenie = 100; | ||
| } | ||
|
|
||
| public Ship(int b) |
There was a problem hiding this comment.
Смените имя переменой - что за b?
CourseApp/Ship.cs
Outdated
| Vodoizmeschenie = 100; | ||
| } | ||
|
|
||
| public Ship(int a, int b, int c) |
There was a problem hiding this comment.
То же самое - что за а б и c ? они должны нести нормальную смысловую нагрузку
CourseApp/Ship.cs
Outdated
|
|
||
| public Ship(int b) | ||
| { | ||
| Passangers = 25; |
There was a problem hiding this comment.
Этот конструктор должен вызывать более общий вид конструктора, пробросив в него параметры
There was a problem hiding this comment.
Простите но не понимаю что здесь требуется сделать
jskonst
left a comment
jskonst
left a comment
There was a problem hiding this comment.
Чем Ship от Kater отличаются? - почти ничем, может только info и vid - остальное - в базовый класс, если чем-то еще отличаются - ну ок. Добавьте submarine
| using CourseApp; | ||
|
|
||
| namespace CourseApp.Test | ||
| namespace CourseApp.Tests |
There was a problem hiding this comment.
Переименуйте файлы тестов - чтобы было ясно кого ои тестируют
| [Fact] | ||
| public void Test10() | ||
| { | ||
| Ship admiral = new Ship(); |
There was a problem hiding this comment.
Имя теста должно отражать то, что тест проверяет
CourseApp.Tests/UnitTest2.cs
Outdated
| public void Test10() | ||
| { | ||
| Ship admiral = new Ship(); | ||
| var passangers = admiral.Passangers; |
There was a problem hiding this comment.
Уберите эти промежуточные переменные - зачем они вам?
CourseApp.Tests/UnitTest4.cs
Outdated
| [Fact] | ||
| public void Test15() | ||
| { | ||
| Ship admiral = new Ship(); |
There was a problem hiding this comment.
Что за тест и что проверяет? не массив ли тут хотели?
2 participants
No description provided.