Insights added, tasks edited

Author: ziobron

module4/06-exercise-cars.md

@@ -6,14 +6,19 @@ ___
 
 ## Cars
 
-1. <!-- .element: class="fragment fade-in" --> Design proper abstraction (interfaces)
-2. <!-- .element: class="fragment fade-in" --> Apply inheritance
-3. <!-- .element: class="fragment fade-in" --> Fix encapsulation
-4. <!-- .element: class="fragment fade-in" --> Use polymorphism to represent every type of car, using a single pointer
-5. <!-- .element: class="fragment fade-in" --> Fix diamond problem
-6. <!-- .element: class="fragment fade-in" --> Fix potential memory leaks
-7. <!-- .element: class="fragment fade-in" --> Think about the way of keeping engines in cars. Should they be kept by a value, reference or a pointer (what kind of pointer)?
-8. <!-- .element: class="fragment fade-in" --> Is this code testable?
-
-### [View task in repo](https://github.com/coders-school/Cars.git)
+### [Repo](https://github.com/coders-school/Cars.git)
 <!-- .element: class="fragment fade-in" -->
+
+1. <!-- .element: class="fragment fade-in" --> Zaproponuj właściwą abstrakcję (interfejs)
+2. <!-- .element: class="fragment fade-in" --> Zastosuj dziedziczenie
+3. <!-- .element: class="fragment fade-in" --> Napraw enkapsulację
+4. <!-- .element: class="fragment fade-in" --> Użyj polimorfizmu, aby za pomocą wskaźnika na klasę bazową reprezentować dowolny typ samochodu
+
+___
+
+## Cars - pytania
+
+1. <!-- .element: class="fragment fade-in" --> Jak trzymać silniki? Przez wartość, referencję, czy wskaźnik?
+2. <!-- .element: class="fragment fade-in" --> Czy jest problem diamentowy?
+3. <!-- .element: class="fragment fade-in" --> Czy są wycieki pamięci?
+4. <!-- .element: class="fragment fade-in" --> Czy kod jest testowalny?

module4/07-insights.md

@@ -12,18 +12,20 @@ ___
 class A {
     virtual void f();
     virtual void g();
+    virtual ~A();
     int a;
+    double b;
 };
 ```
 
 ```text
-stos:   | *vptr | int a |
+stos:   | *vptr | int a | double b |
             |
             V
-sterta: | void (*f)() | void (*g)() |
-                 |             |
-                 V             V
-               A::f          A::g
+sterta: | void (*f)() | void (*g)() | destructor |
+                 |             |           |
+                 V             V           V
+               A::f          A::g        A::~()
 ```
 
 ___

module4/08-homework.md

@@ -8,9 +8,7 @@ ___
 
 You can work in groups or individually. Fork the Cars repo and submit a Pull Request after you have finished.
 
-1. <!-- .element: class="fragment fade-in" --> (4 XP) Create <code>InvalidGear</code> exception. It should be thrown when someone tries eg. change a gear from 5 to R. It should inherit from one of STL exceptions
-2. <!-- .element: class="fragment fade-in" --> (2 XP per fix) Fix interfaces to be easy to use correctly and hard to use incorrectly (like <code>accelerate(-999)</code>)
-3. <!-- .element: class="fragment fade-in" --> (10 XP - optional) Write a proper unit tests to this code
-4. <!-- .element: class="fragment fade-in" --> Read one of below articles. It will be useful for the next lesson
-
-  * <!-- .element: class="fragment fade-in" --> <a href="https://www.samouczekprogramisty.pl/solid-czyli-dobre-praktyki-w-programowaniu-obiektowym/">SOLID czyli dobre praktyki w programowaniu obiektowym</a>
+1. (5 XP) Utwórz wyjątek <code>InvalidGear</code>. Ma on być rzucany wtedy, gdy ktoś próbuje w niewłaściwy sposób zmienić bieg, np. z 5 na R. Ma on dziedziczyć po `std::logic_error`.
+2. (10 XP) Napisz testy jednostkowe do tego kodu. W szczególności przetestuj rzucanie powyższego wyjątku. Skonfiguruj odpowiednio CMake'a.
+3. (0 XP) Napraw interfejs, aby był łatwy w poprawnym użyciu i trudny do niewłaściwego użycia (np. <code>accelerate(-999)</code>). Bez punktów, bo ciężko zautomatyzować tak ogólne zadanie i każdy wymyśli tutaj coś innego.
+4. Przeczytaj artykuł [SOLID czyli dobre praktyki w programowaniu obiektowym](https://www.samouczekprogramisty.pl/solid-czyli-dobre-praktyki-w-programowaniu-obiektowym/)