Beta
Lost In Time
17kayahr
Loading description...
Algorithms
View
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
This comment has been hidden.
Duplicate of binary search.
I've added a coffeescript translation if you would like to approve it.
Is approved. Thanks again!
The tests should report the values of both the tardisTime and amyTime
Done
Great.
Have a look at how I've used the Test.describe and Test.it spec methods to add the dates to the test output in the coffeescript version.
I copied your tests over to JavaScript. I like them, thanks!
This is a great looking kata. Is there a reason why the rescueAmy function could not just return the Date of Amy's location?
Yes. The mission is to control the TARDIS so it travels to the right time. Not to just calculate the time and return it. It's more lore friendly this way :-) And it can't be changed now anyway because then the already submitted solutions are all wrong.