Skip to content

Lesson5#6

Open
abercromb wants to merge 2 commits intomasterfrom
lesson5
Open

Lesson5#6
abercromb wants to merge 2 commits intomasterfrom
lesson5

Conversation

@abercromb
Copy link
Owner

No description provided.

case add, get;
}

protocol AbstractCar {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не нужно добавлять Abstract

{
switch action {

case .add where volumeState == .full:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

молодец избавился от вложенности

Comment on lines +58 to +59


Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем тут пустые строки?

Comment on lines +113 to +120
self.brand = brand;
self.year = year;
self.trunkVolume = trunkVolume;
self.engineState = engineState;
self.doorState = doorState;
self.volumeState = volumeState;
self.maxSpeed = maxSpeed;
self.power = power;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

избавляйся от привычки точек с запятыми

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants