[C#] Issues with Randomness concept and "Roll the Die!" exercise instructions

In the “Roll the Die!” exercise, line 3 of the instructions says “Continuing the theme of the wizards and warriors game,” even though the "Wizards and Warriors exercise is not a prerequisite to unlocking the exercise (in fact, it is tied to the Inheritance concept which is much farther down the concept tree).

Line 19 of the instructions also says “Note that spell strength must be a real number (not an integer)”, even though these kinds of numbers were always referred to as floating-point numbers previously in the syllabus, not real numbers. (Line 13 of hints.md also says “real numbers” instead of floating-point numbers.)

The about.md says “Random.Double()” on line 13 instead of “Random.NextDouble()”. The next line also says “Next...(), Double() and Sample()”, which should be “Next(), NextDouble(), and Sample()”. It also calls the items of the list a “number of issues” even though some of them, especially the last point, are not issues.

(These files also all have multiple sentences per line, when the Exercism markdown specification states that there should be only one sentence per line.)

If a PR would be accepted to fix these issues, I would be happy to create one.

1 Like

Given the .NET Core 3.1 references inside the URLs, I’m thinking the docs are pretty old to begin with and some of the inconsistencies might stem from the syllabus changing over time as things were rewritten. A general and systematic overhaul of the prose might be warranted if Erik is interested similar to what Isaac has been doing to the Go and Swift concepts.

The whole learning track needs reorganising. The order of concepts seems arbitrary (for one thing, namespaces and constants are at the end!), the style is inconsistent, some basic things are never mentioned (eg the difference between structs and classes), there is a notable lack of simple examples, some of the links should be in the resources list, and I’m pretty sure a couple of them led me to a casino. >.>

I’ll be happy to take PR’s to update/streamline things

1 Like

A PR has been opened here.

2 Likes