fixup! feat: add exercise for pointer concept

Author: vaeng

exercises/concept/speedywagon/.docs/hints.md

@@ -38,3 +38,9 @@ int sum = (sensor_array + i)->activity; // Access activity using pointer arithme
 - Use the `->` operator to access the `activity` member of the `pillar_men_sensor` struct.
 - Think carefully about what should happen if the sensor's activity level is `0`.
   Should the alarm trigger?
+
+## 4. Checking the data for anomalies with the `uv_alarm` function
+
+- Use the `&` operator to pass a pointer to the sensor's data array into the `uv_light_heuristic` function.
+- Ensure you correctly check for a null pointer before accessing the sensor's data.
+- Compare the result of `uv_light_heuristic` with the sensor's `activity` value to determine if the alarm should trigger.

exercises/concept/speedywagon/.docs/instructions.md

@@ -6,7 +6,7 @@ In the course of this effort, you've spent years maintaining the Foundation's te
 
 However, in recent times, the sensors that track Pillar Men activities are malfunctioning.
 The Foundation's systems are old, and the code interacts with a legacy C++ library that cannot be updated.
-Your task is to implement three core functions that monitor Pillar Men sensor activity using an old-fashioned pointer-based library.
+Your task is to implement four core functions that monitor Pillar Men sensor activity using an old-fashioned pointer-based library.
 
 The Foundation's operations rely on you.
 
@@ -81,11 +81,25 @@ bool alarm = alarm_control(&db);
 // alarm => true
 ```
 
+## 4. Checking the data for anomalies with the `uv_alarm` function
+
+In this task, you will implement the `uv_alarm` function to determine whether an alarm should be triggered based on UV light exposure levels and sensor activity.
+The `uv_alarm` function should use the provided `uv_light_heuristic` function, which operates on a vector of data and returns a value based on certain thresholds.
+This is a mockup version of the complex code that will run during production, please don't change the interface.
+
+### Task
+
+Define the `uv_alarm` function in the `speedywagon` namespace. It should:
+
+- Take a pointer to a `pillar_men_sensor` _struct_ as its parameter.
+- Return `false` if the sensor pointer is null.
+- Call the `uv_light_heuristic` function, passing the address of the sensor's `data` array.
+- Return `true` if the value returned by `uv_light_heuristic` is greater than the `sensor->activity` level, otherwise return `false`.
+
 ## Wrapping Up
 
 You’ve been entrusted with an essential task for the Speedywagon Foundation.
 By testing for valid sensor connections, counting activity, and implementing alarm controls, you’ve ensured that the Foundation's battle against the Pillar Men can continue uninterrupted.
 
 As a modern C++ engineer, you’d prefer using smart pointers, but alas, legacy code demands respect for the old ways.
 The fate of humanity may rest on these pointers, so proceed carefully, and may the Hamon energy guide you.
-

exercises/concept/speedywagon/.meta/exemplar.cpp

@@ -2,6 +2,19 @@
 
 namespace speedywagon {
 
+int uv_light_heuristic(std::vector<int>* data_array) {
+    double avg{};
+    for (auto element : *data_array) {
+        avg += element;
+    }
+    avg /= data_array->size();
+    int uv_index{};
+    for (auto element : *data_array) {
+        if (element > avg) ++uv_index;
+    }
+    return uv_index;
+}
+
 bool connection_check(pillar_men_sensor* sensor) { return sensor != nullptr; }
 
 int activity_counter(pillar_men_sensor* sensor_array, int capacity) {
@@ -17,4 +30,9 @@ bool alarm_control(pillar_men_sensor* sensor) {
     return sensor->activity > 0;
 }
 
+bool uv_alarm(pillar_men_sensor* sensor) {
+    if (sensor == nullptr) return false;
+    return uv_light_heuristic(&(sensor->data)) > sensor->activity;
+}
+
 }  // namespace speedywagon

exercises/concept/speedywagon/.meta/exemplar.h

@@ -11,10 +11,14 @@ struct pillar_men_sensor {
     std::vector<int> data{};
 };
 
+int uv_light_heuristic(std::vector<int>* data_array);
+
 bool connection_check(pillar_men_sensor* sensor);
 
 int activity_counter(pillar_men_sensor* sensor_array, int capacity);
 
 bool alarm_control(pillar_men_sensor* sensor);
 
+bool uv_alarm(pillar_men_sensor* sensor);
+
 }  // namespace speedywagon

exercises/concept/speedywagon/speedywagon.cpp

@@ -1,3 +1,21 @@
 #include "speedywagon.h"
 
-namespace speedywagon {}  // namespace speedywagon
+namespace speedywagon {
+
+// Enter your code below:
+
+// Please don't change the interface of the uv_light_heuristic function
+int uv_light_heuristic(std::vector<int>* data_array) {
+    double avg{};
+    for (auto element : *data_array) {
+        avg += element;
+    }
+    avg /= data_array->size();
+    int uv_index{};
+    for (auto element : *data_array) {
+        if (element > avg) ++uv_index;
+    }
+    return uv_index;
+}
+
+}  // namespace speedywagon

exercises/concept/speedywagon/speedywagon.h

@@ -11,4 +11,6 @@ struct pillar_men_sensor {
     std::vector<int> data{};
 };
 
+int uv_light_heuristic(std::vector<int>* data_array);
+
 }  // namespace speedywagon

exercises/concept/speedywagon/speedywagon_test.cpp

@@ -46,4 +46,16 @@ TEST_CASE("alarm_control: works correctly for nullptr", "[task_3]") {
     REQUIRE(speedywagon::alarm_control(&santana));
 }
 
+TEST_CASE("uv_alarm: works correctly for nullptr", "[task_4]") {
+    speedywagon::pillar_men_sensor* wham{nullptr};
+    REQUIRE_FALSE(speedywagon::uv_alarm(wham));
+}
+
+TEST_CASE("uv_alarm: works correctly with mock data", "[task_4]") {
+    speedywagon::pillar_men_sensor wham{0, "Rome", {1, 605, 313, 4000}};
+    REQUIRE(speedywagon::uv_alarm(&wham));
+    wham.activity = 9001;
+    REQUIRE_FALSE(speedywagon::uv_alarm(&wham));
+}
+
 #endif