Update instructions.md #2128
Update instructions.md #2128
Conversation
Speed is distance per unit time, updated all references to speed units of meters to meters per second
|
Hello. Thanks for opening a PR on Exercism. We are currently in a phase of our journey where we have paused community contributions to allow us to take a breather and redesign our community model. You can learn more in this blog post. As such, all issues and PRs in this repository are being automatically closed. That doesn't mean we're not interested in your ideas, or that if you're stuck on something we don't want to help. The best place to discuss things is with our community on the Exercism Community Forum. You can use this link to copy this into a new topic there. Note: If this PR has been pre-approved, please link back to this PR on the forum thread and a maintainer or staff member will reopen it. |
| @@ -2,7 +2,7 @@ | |||
|
|
|||
| In this exercise you'll be organizing races between various types of remote controlled cars. Each car has its own speed and battery drain characteristics. | |||
|
|
|||
| Cars start with full (100%) batteries. Each time you drive the car using the remote control, it covers the car's speed in meters and decreases the remaining battery percentage by its battery drain. | |||
| Cars start with full (100%) batteries. Each time you drive the car using the remote control, it increases the car's speed in meters per second and decreases the remaining battery percentage by its battery drain. | |||
There was a problem hiding this comment.
To me this reads as: the speed is increased, which it isn't. I like the other two suggestions, but maybe change or revert this?
There was a problem hiding this comment.
You're right, increases the speed doesn't make sense if the car is already travelling. How about, "it causes the car to travel at its rated speed for 1 second and decreases the remaining battery percentage by its battery drain rate."? It's wordier but I think it helps lead the reader to distanceTravelled = speed * time.
There was a problem hiding this comment.
Hi, there is no time in this exercise, the speed is in distance per click of a button, not per unit of time.
How about
"Every time you drive the car using the remote control, the car travels distance equal to its current speed and its battery level drop by the the value of battery drain." Or something like this?
There was a problem hiding this comment.
Distance per press of a button is just distance. But the exercise is full of references to speed, and anywhere you talk about speed, you infer time by the definition of speed. My suggestion is either:
- replace all references to speed to distance travelled per use of the remote control, or
- define the amount of time the car travels at rated speed per use of the remote control so you have the information you need to convert rated speed to distance travelled, ex.
_totalDistance += _speed * 1;if the operating the remote control causes the car to travel at its rated speed for 1 second
There was a problem hiding this comment.
Every time you drive the car using the remote control, the car travels distance equal to its current speed and its battery level drop by the the value of battery drain
I like this suggestion.
There was a problem hiding this comment.
Every time you drive the car using the remote control, the car travels distance equal to its current speed and its battery level drop by the the value of battery drain
I like this suggestion.
"travels distance equal to its current speed" makes no sense, speed does not equal distance.
Speed is distance per unit time, updated all references to speed units of meters to meters per second