Skip to content

Conversation

@ErikSchierboom
Copy link
Member

This PR adds four new exercises:

  1. nth-prime
  2. palindrome-products
  3. binary-search-tree
  4. robot-simulator
  5. simple-linked-list

These are all open exercises as listed in #2.

I hope someone would be willing to give some feedback on the design. In particular, I struggled with the design of the palindrome products.

@ErikSchierboom ErikSchierboom changed the title Add 3 new exercises Add 4 new exercises Feb 19, 2016
@ErikSchierboom ErikSchierboom changed the title Add 4 new exercises Add 5 new exercises Feb 19, 2016
using System.Linq;

public class BinarySearchTree<T>
where T : IComparable
Copy link
Contributor

Choose a reason for hiding this comment

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

The exercise only talks about using numbers so there's no point in making this generic. I would keep it simple and just make this non-generic and use ints.

Copy link
Member Author

Choose a reason for hiding this comment

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

Agreed.

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@kytrinyx
Copy link
Member

@ErikSchierboom @bressain I'm going to leave this discussion to the two of you since I don't know C#. If you are happy with any individual exercises, it might be worth sticking them in separate PRs or cherry-picking them while you keep working on and discussing the others. Or not :)

@bressain
Copy link
Contributor

Looks good, thanks for doing these.

bressain added a commit that referenced this pull request Feb 22, 2016
@bressain bressain merged commit 596c45d into exercism:master Feb 22, 2016
@ErikSchierboom ErikSchierboom deleted the new-exercises branch February 22, 2016 18:48
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.

3 participants