Improving notes on inheriting and modifiers.

Author: aubertc

source/code/projects/FileCountProgram/FileCountProgram/Program.cs

@@ -5,21 +5,20 @@ class Program
 {
   static void Main()
   {
-
     string filePath = Path.Combine(
       new DirectoryInfo(
         Directory.GetCurrentDirectory()
       ).ToString(),
       "Program.cs"
     );
-    
+
     /*
-     * On older versions, you may need to use the following instead, 
+     * On older versions, you may need to use the following instead,
      * as the program is executed in /bin/Debug, we need to go
      * two folders up, where Program.cs is located: note the ".Parent"
      * bits in the code that follows.
      */
-    
+
     /*
     
     string filePath = Path.Combine(
@@ -30,7 +29,7 @@ static void Main()
     );
     
     */
-    
+
     if (!File.Exists(filePath))
     {
       Console.WriteLine(

source/code/projects/FileLineCount/FileLineCount/Program.cs

@@ -9,7 +9,10 @@ static void Main(string[] args)
     string directoryPath = AppDomain
       .CurrentDomain
       .BaseDirectory;
-    string filePath = Path.Combine(directoryPath, "dummy.txt");
+    string filePath = Path.Combine(
+      directoryPath,
+      "dummy.txt"
+    );
     // Creating a generator for random numbers.
     Random gen = new Random();
     int linePopulated = gen.Next(20, 100);
@@ -37,7 +40,7 @@ static void Main(string[] args)
       line = sr.ReadLine();
       while (line != null)
       {
-       count++;
+        count++;
         line = sr.ReadLine();
       }
       sr.Close();
@@ -48,11 +51,15 @@ static void Main(string[] args)
     }
     // We can do a simple check to make sure
     // we populated as many lines as we counted.
-    if(count != linePopulated){
+    if (count != linePopulated)
+    {
       throw new Exception("Something was wrong!");
     }
-    else{
-      Console.WriteLine($"The file located at {filePath} has {count} lines.");
+    else
+    {
+      Console.WriteLine(
+        $"The file located at {filePath} has {count} lines."
+      );
     }
   }
 }

source/code/projects/InheritanceAndAccessModifiers/InheritanceAndAccessModifiers/FMember.cs

@@ -1,17 +1,23 @@
-class FMember : Person{
-    public string Relationship;
-    public FMember(string relationP, string nameP){
-        // This implicitely calls
-        // Person()
-        // which exists to create first
-        // a person object.
-        Relationship = relationP;
-        // the following
-        // name = nameP;
-        // gives
-        // InheritanceAndAccessModifiers/FMember.cs(5,9): error CS0122: 'Person.name' is inaccessible due to its protection level
-        // so we have to use the public method instead:
-        SetName(nameP);
+class FMember : Person
+{
+  public string Relationship;
 
-    }
+  public FMember(string relationP, string nameP)
+  {
+    // This implicitely calls
+    // Person()
+    // which exists to create first
+    // a person object.
+    Relationship = relationP;
+    // the following
+    // name = nameP;
+    // gives
+    // error CS0122: 'Person.name' is inaccessible due to its protection level
+    // so we have to use the public method instead:
+    SetName(nameP);
+    // the following
+    // NewName(nameP);
+    // gives
+    //  error CS0122: 'Person.NewName(string)' is inaccessible due to its protection level
+  }
 }

source/code/projects/InheritanceAndAccessModifiers/InheritanceAndAccessModifiers/Person.cs

@@ -1,7 +1,28 @@
-public class Person{
-    private string name;
-    public string GetName(){return name;}
-    public Person(){}
-    public void SetName(string nameP){name = nameP;}
+public class Person
+{
+  private string name;
+
+  public string GetName()
+  {
+    return name;
+  }
+
+  public Person() { }
+
+  public void SetName(string nameP)
+  {
+    if (name == null)
+    {
+      name = nameP;
+    }
+    else
+    {
+      NewName(nameP);
+    }
+  }
+
+  private void NewName(string newnameP)
+  {
+    name = newnameP + $" (previous name: {name})";
+  }
 }
-    

source/code/projects/InheritanceAndAccessModifiers/InheritanceAndAccessModifiers/Program.cs

@@ -1,5 +1,15 @@
-class Program{
-    public static void Main(){
-        FMember test = new FMember("Cousin", "Samuel");
-    }
+using System;
+
+class Program
+{
+  public static void Main()
+  {
+    Person test1 = new Person();
+    test1.SetName("Patricia");
+    Console.WriteLine(test1.GetName());
+    test1.SetName("Patty");
+    Console.WriteLine(test1.GetName());
+
+    FMember test = new FMember("Cousin", "Samuel");
+  }
 }

source/code/projects/Properties_errors/Properties_errors/Derived.cs

@@ -30,16 +30,20 @@ public string Attr1
   }
 
 */
-  
+
   // A similar issue is to have a setter that does nothing:
-  
-  public string Attr2 {get{return attr1;} set{}}
-  
-  public string GetAttr2(){
+
+  public string Attr2
+  {
+    get { return attr1; }
+    set { }
+  }
+
+  public string GetAttr2()
+  {
     return attr1;
   }
-      
-  
+
   // Misconception #2
   public void Test(string argP)
   {

source/code/projects/Properties_errors/Properties_errors/Program.cs

@@ -9,7 +9,7 @@ static void Main()
 
     test1.Test(null);
     test1.Test();
-    
+
     test1.Attr2 = "This is the parameter.";
     Console.WriteLine(test1.GetAttr2());
 

source/code/projects/Species/Species/Bat.cs

@@ -3,16 +3,17 @@
 class Bat : Mammal
 {
   private double flightSpeed;
-    /* 
-     * Having a property for the 
-     * flightSpeed attribute was 
-     * not required per the UML 
-     * diagram, but it was not 
-     * disallowed either.
-     * We use this approach here
-     * to make the code more 
-     * modular.
-     */
+
+  /*
+   * Having a property for the
+   * flightSpeed attribute was
+   * not required per the UML
+   * diagram, but it was not
+   * disallowed either.
+   * We use this approach here
+   * to make the code more
+   * modular.
+   */
   public double FlightSpeed
   {
     get { return flightSpeed; }

source/code/projects/Species/Species/Program.cs

@@ -136,7 +136,13 @@ static void Main()
     }
     try
     {
-      Bat test4 = new Bat("LC", 10, "https://en.wikipedia.org/wiki/whatever", 2, -10);
+      Bat test4 = new Bat(
+        "LC",
+        10,
+        "https://en.wikipedia.org/wiki/whatever",
+        2,
+        -10
+      );
     }
     catch (Exception e)
     {

source/lectures/oop/inheritance.md

@@ -79,7 +79,6 @@ If there was a `SetNOW` method in both the `Bike` and the `Vehicle` method, we c
 
 The following example tries to make it clear that `private` attributes *cannot* be accessed from classes inheriting.
 
-
 ```{download="./code/projects/InheritanceAndAccessModifiers.zip"}
 !include code/projects/InheritanceAndAccessModifiers/InheritanceAndAccessModifiers/Person.cs
 ```