Create PR #25
Create PR #25
Conversation
rpgsaga/saga/src/mouse.ts
Outdated
| DIED = "Died" | ||
| } | ||
| export class Mouse { | ||
| _heathpool: number = 100; |
There was a problem hiding this comment.
а _heathpool не должен быть приватным или protected
rpgsaga/saga/src/mouse.ts
Outdated
| @@ -0,0 +1,42 @@ | |||
| export enum Mode{ | |||
There was a problem hiding this comment.
enum можно вынести в отдельный файл
rpgsaga/saga/src/mouse.ts
Outdated
| this._heathpool-=10 | ||
| if (this._heathpool > 0) { | ||
| return `Вы ударили мышь, теперь у неё ${this._heathpool} здоровья` | ||
| }else{ |
rpgsaga/saga/src/mouse.ts
Outdated
| return `Вы ударили мышь, теперь у неё ${this._heathpool} здоровья` | ||
| }else{ | ||
| this._mode = Mode.DIED | ||
| return "Мышь сдохла" |
There was a problem hiding this comment.
линтер точно не заработал - нужна ; в конце строки
| _healthpool: number = 100; | ||
| private _age: number; | ||
| _color: string; | ||
| _mode: Mode = Mode.ALIVE; |
There was a problem hiding this comment.
и извне могу ее убить - поле то публичное
| this._color = color; | ||
| }; | ||
| set year(year: number) { | ||
| this._age = year > 0 && year < 4 ? year : 0 |
There was a problem hiding this comment.
при неправильном значении бросайте exception
| constructor(color: string) { | ||
| this._color = color; | ||
| }; | ||
| sound(): string { |
There was a problem hiding this comment.
тогда сделайте его абстрактным
| export class Animal { | ||
| _mode: Mode = Mode.ALIVE; | ||
| _color: string; | ||
| _healthpool: number = 100; |
| } | ||
|
|
||
| sound(): string { | ||
| return `Фыр-фыр`; |
| set year(year: number) { | ||
| this._age = year > 0 && year < 20 ? year : 0; | ||
| }; |
There was a problem hiding this comment.
а вот тут интересно - у вас у всех есть возраст - но нет в базовом классе -> я могу создать животьное у которого не будет этого года - что может привести к неожиданному поведению
| this.year = year; | ||
| }; | ||
| set year(year: number) { | ||
| this._age = year > 0 && year < 25 ? year : 0; |
There was a problem hiding this comment.
при неправильном годе - бросайте exception
кроме того вы можете определить базовое поведение и хранить сколько лет живет то или иное животное - и исходя из этого сделать этот метод вообще в базовом классе
No description provided.