Listing 2.20 and 2.21's code may need to be switched

Open
#878 0 comments 0 reactions 1 assignee View on GitHub

@BenjaminMichaelis is already working on this.

Since Apr 2, 2026.

Assessment

This issue has not been assessed yet.

Description

The title for Listing 2.19 and Listing 2.20 are identical though the one for Listing 2.20 appears to be the one that's incorrect.
It appears the code for Listing 2.21 needs to instead be swapped with the code in Listing 2.20 based on both the text surrounding both examples and the Output 2.11 content which appears to align with what is currently the code of Listing 2.20.

Despite the above aligning with the text, the test names also don't line up with the content so I'm not sure if the text even should be in that order to begin with given the previous commits to those sections were movement from 2.09.

If someone can confirm the final intended order / changes desired I'm happy to submit a PR to correct this!
Really appreciate the book and it's availability online @MarkMichaelis !

Dominant language
C#
Stars
419
Forks
173
Avg merge
3m
Merged PRs (30d)
3

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from IntelliTect/EssentialCSharp

All issues in IntelliTect/EssentialCSharp

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.