Skip to content

5splitPR#1

Open
mozhnov wants to merge 3 commits intomainfrom
5splitBranch
Open

5splitPR#1
mozhnov wants to merge 3 commits intomainfrom
5splitBranch

Conversation

@mozhnov
Copy link
Copy Markdown
Owner

@mozhnov mozhnov commented Aug 25, 2024

No description provided.

Copy link
Copy Markdown

@ivannizh ivannizh left a comment

Choose a reason for hiding this comment

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

Привет!

Отличная работа! Осталось только исправить некоторые моменты, и задание будет принято.

Comment thread main.go Outdated
Comment thread main.go Outdated
func (s Swimming) meanSpeed() float64 {
// вставьте ваш код ниже
...
return float64(s.LengthPool) * float64(s.CountPool) / MInKm / s.Duration.Hours()
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Также следует проверить длительность на нулевое значение.

Comment thread main.go
Copy link
Copy Markdown

@ivannizh ivannizh left a comment

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