Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hello-world: update tutorial to account for emptyStringIsComparedByValue() #282

Closed
jtigger opened this issue Feb 9, 2017 · 3 comments
Closed
Labels

Comments

@jtigger
Copy link
Contributor

jtigger commented Feb 9, 2017

hello-world contains a blow-by-blow tutorial. We recently added a test case to help catch folks using the equals operator to detect an empty string (i.e. #280).

The tutorial needs an adjustment:

  • the section titles assume there are three test cases; there are now four.
  • this (new) second test needs some crisp explanation about the difference between comparing by value vs. comparing the reference without invoking too many concepts.
  • the tutorial avoids the pitfall of using == for string comparison, let's keep that; this is about clarifying why this test matters.
  • the audience of the tutorial are Early Programmers — people who are relatively newish to programming, but this is not their very first attempt at writing code.
@stkent
Copy link
Contributor

stkent commented Feb 10, 2017

FYI in light of exercism/problem-specifications#520 we may want to hold off on this change.

@jtigger
Copy link
Contributor Author

jtigger commented Feb 13, 2017

doh!!!! :) Man, there goes many hours of effort into the rubbish bin. :-}

@jtigger
Copy link
Contributor Author

jtigger commented Feb 22, 2017

Okay... the tutorial, as is, will serve as inspiration as we flesh-out the early part of the track.

  • hello-world is/will be greatly simplified.
  • looking at the current difficulty graph, we probably need to smooth that ramp out with some well-known early exercises.
  • this tutorial could then be spread out across those exercises.

In light of that, this issue is no longer valid.

@jtigger jtigger closed this as completed Feb 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants