Small improvements to OO slides

bernhardmgruber · hageboeck · bernhardmgruber · commit 998790c71d3c · 2023-03-07T15:38:15.000+01:00
Co-authored-by: Stephan Hageboeck &lt;stephan.hageboeck@cern.ch&gt;
diff --git a/talk/objectorientation/advancedoo.tex b/talk/objectorientation/advancedoo.tex
@@ -53,9 +53,9 @@
 
 \begin{frame}[fragile]
   \frametitlecpp[98]{Inheritance privacy and polymorphism}
-  \begin{block}{Only public inheritance is visible to code outside the class}
+  \begin{block}{Only public base classes are visible to outside code}
     \begin{itemize}
-    \item private and protected are not
+    \item private and protected bases are not
     \item this may restrict usage of polymorphism
     \end{itemize}
   \end{block}
@@ -187,7 +187,7 @@
       \end{itemize}
     \item they also imply extra storage and an extra indirection
       \begin{itemize}
-      \item practically the object stores a pointer to the correct method
+      \item practically, the object stores a pointer to the correct method
       \item in a so-called ``virtual table'' (``vtable'')
       \end{itemize}
     \end{itemize}
@@ -491,9 +491,12 @@
   \frametitlecpp[98]{Virtual inheritance}
   \begin{block}{Solution}
     \begin{itemize}
-    \item inheritance can be {\it virtual} or not
-    \item {\it virtual} inheritance will ``share'' parents
-    \item standard inheritance will replicate them
+    \item inheritance can be \cppinline{virtual} or not
+    \begin{itemize}
+      \item \cppinline{virtual} inheritance will ``share'' parents
+      \item standard inheritance will replicate them
+    \end{itemize}
+    \item most derived class will call the virtual base class's constructor
     \end{itemize}
     \begin{cppcode}
       class Text : public virtual Drawable {...};
@@ -528,19 +531,19 @@
 
 \begin{frame}[fragile]
   \frametitlecpp[98]{Multiple inheritance advice}
-  \begin{block}{Do not use multiple inheritance}
+  \begin{goodpractice}{Avoid multiple inheritance}
     \begin{itemize}
     \item Except for inheriting from interfaces
-    \item and for rare special cases
+    \item And for rare special cases
     \end{itemize}
-  \end{block}
+  \end{goodpractice}
   \pause
-  \begin{alertblock}{Do not use diamond shapes}
+  \begin{goodpractice}{Absolutely avoid diamond-shaped inheritance}
     \begin{itemize}
     \item This is a sign that your architecture is not correct
     \item In case you are tempted, think twice and change your mind
     \end{itemize}
-  \end{alertblock}
+  \end{goodpractice}
 \end{frame}
 
 \begin{frame}[fragile]
@@ -556,10 +559,3 @@
     \end{itemize}
   \end{exerciseWithShortcut}
 \end{frame}
-
-\begin{frame}[fragile]
-  \frametitlecpp[98]{Virtual inheritance}
-  \begin{alertblock}{Warning}
-    in case of virtual inheritance it is the most derived class that calls the virtual base class's constructor
-  \end{alertblock}
-\end{frame}
diff --git a/talk/objectorientation/allocations.tex b/talk/objectorientation/allocations.tex
@@ -47,7 +47,7 @@
   \frametitlecpp[98]{Object allocation on the stack}
   \begin{block}{On the stack}
     \begin{itemize}
-    \item objects are created when declared (constructor called)
+    \item objects are created on variable definition (constructor called)
     \item objects are destructed when out of scope (destructor is called)
     \end{itemize}
   \end{block}
diff --git a/talk/objectorientation/constructors.tex b/talk/objectorientation/constructors.tex
@@ -24,7 +24,7 @@
     \end{cppcode*}
     \columnbreak
     \begin{cppcode*}{gobble=2,firstnumber=10}
-      // note special notation for
+      // note: special notation for
       // initialization of members
       MyFirstClass() : a(0) {}
 
@@ -81,7 +81,7 @@
 \end{frame}
 
 \begin{frame}[fragile]
-  \frametitlecpp[98]{Copy constructor}
+  \frametitlecpp[11]{Copy constructor}
   \begin{block}{Concept}
     \begin{itemize}
     \item special constructor called for replicating an object
@@ -137,7 +137,7 @@
     \end{itemize}
   \end{block}
   \begin{cppcode}
-    void print( const Vector & v )
+    void print(const Vector & v)
       std::cout<<"printing v elements...\n";
     }
 
@@ -219,15 +219,17 @@
     \begin{itemize}
     \item avoid having to re-declare parent's constructors
     \item by stating that we inherit all parent constructors
+    \item derived class can add more constructors
     \end{itemize}
   \end{block}
   \begin{exampleblock}{Practically}
     \begin{cppcode}
       struct BaseClass {
-        BaseClass(int value);
+        BaseClass(int a);
       };
       struct DerivedClass : BaseClass {
         using BaseClass::BaseClass;
+        DerivedClass(int a, int b);
       };
       DerivedClass a{5};
     \end{cppcode}
diff --git a/talk/objectorientation/inheritance.tex b/talk/objectorientation/inheritance.tex
@@ -51,14 +51,14 @@
 
 \begin{frame}[fragile]
   \frametitlecpp[98]{Managing access to class members}
-  \begin{block}{{\it public} \color{white} / {\it private} keywords}
+  \begin{block}{\texttt{public} / \texttt{private} keywords}
     \begin{description}
-      \item[private] allows access only within the class
-      \item[public] allows access from anywhere
+      \item[\texttt{private}] allows access only within the class
+      \item[\texttt{public}] allows access from anywhere
     \end{description}
     \begin{itemize}
-       \item The default for \texttt{class} is {\it private}
-       \item A \texttt{struct} is just a \texttt{class} that defaults to {\it public} access
+       \item The default for \texttt{class} is \cppinline{private}
+       \item A \cppinline{struct} is just a class that defaults to \cppinline{public} access
     \end{itemize}
   \end{block}
   \pause
@@ -90,7 +90,7 @@
 
 \begin{frame}[fragile]
   \frametitlecpp[98]{Managing access to class members(2)}
-  \begin{block}{Solution is {\it protected} keyword}
+  \begin{block}{Solution is \texttt{protected} keyword}
     Gives access to classes inheriting from base class
   \end{block}
   \begin{multicols}{2}
@@ -125,10 +125,10 @@
     It influences the privacy of inherited members for external code.\\
     The code of the class itself is not affected
     \begin{description}
-    \item[public] privacy of inherited members remains unchanged
-    \item[protected] inherited public members are seen as protected
-    \item[private] all inherited members are seen as private \\
-      this is the default for \texttt{class} if nothing is specified
+    \item[\texttt{public}] privacy of inherited members remains unchanged
+    \item[\texttt{protected}] inherited public members are seen as protected
+    \item[\texttt{private}] all inherited members are seen as private \\
+      this is the default for classes if nothing is specified
     \end{description}
   \end{block}
   \pause
@@ -137,7 +137,7 @@
     \item only public members of public inheritance are accessible
     \end{itemize}
   \end{block}
-  \begin{block}{Net result for grand child code}
+  \begin{block}{Net result for code in derived classes}
     \begin{itemize}
     \item only public and protected members of public and protected parents are accessible
     \end{itemize}
diff --git a/talk/objectorientation/objectsclasses.tex b/talk/objectorientation/objectsclasses.tex
@@ -94,7 +94,6 @@
     \begin{itemize}
     \item usually in .cpp, outside of class declaration
     \item using the class name as ``namespace''
-    \item when reference/pointer to the object is needed, use \cppinline{this} keyword
     \end{itemize}
   \end{block}
   \begin{cppcode}
