Conversation
just-musha
left a comment
just-musha
left a comment
There was a problem hiding this comment.
Привет! Код корректный, форматирвоаноие гуд, тесты зеленые - супер. Давай ещё в формулах константы применим, а то сейчас не очень читаемо получается, не понятно что за числа. И работа будет принята!
There was a problem hiding this comment.
Файлы с настройками IDE не нужно добавлять в коммит. Они мб у каждого свои
There was a problem hiding this comment.
Можешь удалить .idea из коммита, и после этого прописать ее в .gitignore файле, в корне проекта
| // Преобразуем скорость из км/ч в м/с | ||
| speedMS := meanSpeed(action, duration) * kmhInMsec | ||
| // Преобразуем рост в метры | ||
| heightM := height / 100 |
There was a problem hiding this comment.
Вместо 100 лучше использовать специальную константу, так более читаемо получается, нет вопросов что это за число и что оно значит
| heightM := height / 100 | ||
|
|
||
| // Формула для расчета калорий | ||
| calories := (0.035*weight + math.Pow(speedMS, 2)/heightM*0.029*weight) * duration * 60 |
There was a problem hiding this comment.
Аналогично здесь - вместо чисел тебе нужно использовать специальные константы, они уже заведены в файле
| // ваш код здесь | ||
| ... | ||
| speed := swimmingMeanSpeed(lengthPool, countPool, duration) | ||
| return (speed + swimmingCaloriesMeanSpeedShift) * swimmingCaloriesWeightMultiplier * weight * duration |
There was a problem hiding this comment.
О, вот тут исиользуешь константы - это хорошо
2 participants
No description provided.