From 82863ad638c1230f9dc52606f51a6658a1c7307c Mon Sep 17 00:00:00 2001
From: Mark J Price <markjprice@gmail.com>
Date: Sat, 27 May 2023 07:09:42 +0100
Subject: [PATCH] Add improvement for page 233, 238

---
 docs/errata/README.md       |  2 +-
 docs/errata/improvements.md | 48 ++++++++++++++++++++++++++++++++++++-
 2 files changed, 48 insertions(+), 2 deletions(-)

diff --git a/docs/errata/README.md b/docs/errata/README.md
index c62f31c..0128c4d 100644
--- a/docs/errata/README.md
+++ b/docs/errata/README.md
@@ -4,7 +4,7 @@ If you find any mistakes in the seventh edition, *C# 11 and .NET 7 - Modern Cros
 
 [**Errata** (44 items)](errata.md): Typos, tool user interface changes, or mistakes in code that would cause a compilation error that prevents a successful build.
 
-[**Improvements** (34 items)](improvements.md): Changes to text or code that would improve the content. These are optional.
+[**Improvements** (35 items)](improvements.md): Changes to text or code that would improve the content. These are optional.
 
 [**Common Errors** (4 items)](common-errors.md): These are some of the most common errors that a reader might encounter when trying to get code in book tasks to work, or when trying to write your own code. 
 
diff --git a/docs/errata/improvements.md b/docs/errata/improvements.md
index da944bc..15bbaa1 100644
--- a/docs/errata/improvements.md
+++ b/docs/errata/improvements.md
@@ -1,4 +1,4 @@
-**Improvements** (34 items)
+**Improvements** (35 items)
 
 If you have suggestions for improvements, then please [raise an issue in this repository](https://github.com/markjprice/cs11dotnet7/issues) or email me at markjprice (at) gmail.com.
 
@@ -17,6 +17,7 @@ If you have suggestions for improvements, then please [raise an issue in this re
 - [Page 200 - Talking about OOP](#page-200---talking-about-oop)
 - [Page 235 - More about methods](#page-235---more-about-methods)
 - [Page 237 - Implementing functionality using methods](#page-237---implementing-functionality-using-methods)
+- [Page 238 - Implementing functionality using methods](#page-238---implementing-functionality-using-methods)
 - [Page 241 - Defining flight passengers](#page-241---defining-flight-passengers)
 - [Page 251 - Setting up a class library and console application](#page-251---setting-up-a-class-library-and-console-application)
 - [Page 254 - Calling methods using delegates](#page-254---calling-methods-using-delegates)
@@ -440,6 +441,51 @@ Console.WriteLine(result);
 
 In the 8th edition, I will add a similar explanation of the operator `??` to the **Chapter 3, Operating on variables** section.
 
+# Page 238 - Implementing functionality using methods
+
+In Step 4, you write a `for` statement that enumerates the children of a `Person` object named `lamech`, as shown in the following code:
+```cs
+for (int i = 0; i < lamech.Children.Count; i++)
+{
+  WriteLine(format: "{0}'s child #{1} is named \"{2}\".",
+    arg0: lamech.Name, arg1: i, arg2: lamech[i].Name);
+}
+```
+
+Previously on page 233 you define an indexer for the `Person` class that accepts an integer parameter, as shown in the following code:
+```cs
+// indexers
+public Person this[int index]
+{
+  get
+  {
+    return Children[index]; // pass on to the List<T> indexer
+  }
+  set
+  {
+    Children[index] = value;
+  }
+}
+```
+
+If you do not define the indexer, then you cannot use in it the `for` statement, and you will see the following compile error:
+```
+Error	CS0021	Cannot apply indexing with [] to an expression of type 'Person'
+
+```
+
+In the next edition, I will add a note in the section where you define the indexer to warn that it will be used later in the chapter, and I will add a comment in the `for` statement to warn the reader that use of the indexer requires that the indexer be defined, as shown in the following code:
+```cs
+for (int i = 0; i < lamech.Children.Count; i++)
+{
+  WriteLine(format: "{0}'s child #{1} is named \"{2}\".",
+    arg0: lamech.Name, arg1: i, 
+    // The following requires an indexer to be defined for Person:
+    // public Person this[int index]
+    arg2: lamech[i].Name);
+}
+```
+
 # Page 241 - Defining flight passengers
 
 > Thanks to [Masoud Nazari](https://github.com/MAS-OUD) for raising this [issue on 8 March 2023](https://github.com/markjprice/cs11dotnet7/issues/38).