w3 q1

main.cpp

@@ -1,18 +1,20 @@
 #include <iostream>
 #include <vector>
-#include <algorithm>
 #include <iomanip>
+#include "stats.h"
 
 using string = std::string;
 
 //Placw in header!
 template <typename T>
-void printm(T m){
+void printm(T m)
+{
     std::cout << m;
 }
 
 template <typename T>
-void printlm(T m){
+void printlm(T m)
+{
     printm(m);
     printm("\n");
 }
@@ -21,45 +23,39 @@ typedef std::vector<double>::size_type vtype;
 
 double promptd(string,bool*);
 
-int main() {
+int main()
+{
     std::setprecision(3);
-  printlm("Calculate median and average from sequence:");
-  std::vector<double> v{};
+    printlm("Calculate the average and median of scores:");
     bool s {};
-  do {
-    auto d = promptd("Enter a number (Enter non-number to complete entry) : ",&s);
+    std::vector<double> v {};
+    do
+    {
+        auto d = promptd("Enter a score (Enter a non number to finish entry) : ",&s);
         if (s) v.push_back(d);
     }
     while(s);
-  sort(v.begin(), v.end());
-  printm("The count is: ");
-  printlm(static_cast<int>(v.size()));
-  printm("The median is: ");
-  int halfway {static_cast<int>(v.size()) / 2};
-  if (static_cast<int>(v.size())%2 == 1)
-    printlm(v.at(static_cast<vtype>(halfway)));
-  else
-    printlm((v.at(static_cast<vtype>(halfway - 1)) + v.at(static_cast<vtype>(halfway))) / 2.0);
-  double avg {};
-  if (v.size() > 2) {
-    for (vtype i = 1; i < v.size() - 1; ++i) avg += v.at(i);
-  } else {
-    for (vtype i = 0; i < v.size(); ++i) avg += v.at(i);
-  }
-  avg /= static_cast<int>(v.size());
-  printm("The average is: ");
-  printlm(avg);
+    printm("Score median is: ");
+    printlm(med(v, false));
+    printm("Score median (No Extremes) is: ");
+    printlm(med(v, true));
+    printm("Score average is: ");
+    printlm(avg(v, false));
+    printm("Score average (No Extremes) is: ");
+    printlm(avg(v, true));
     return 0;
 }
 
-double promptd(string message, bool *status) {
-  double tr {};
+double promptd(string message, bool *status)
+{
+    double tr {0.0};
     *status = false;
     std::cout << message;
-  if (!(std::cin >> tr)){
+    if (!(std::cin >> tr))
+    {
         std::cin.clear();
         std::cin.ignore();
-    return 0;
+        return 0.0;
     }
     *status = true;
     return tr;

stats.cpp

@@ -0,0 +1,21 @@
+#include "stats.h"
+#include <vector>
+#include <algorithm>
+
+double med(std::vector<double> &scores,bool ignoreExtremes) {
+    const auto len = scores.size() - (ignoreExtremes && scores.size() > 2) ? 2 : 0;
+    sort(scores.begin(), scores.end());
+    using vec_size = std::vector<double>::size_type;
+    const vec_size middle = (vec_size) (len / 2);
+    if (middle%2 == 1)
+        return scores[middle];
+    else
+        return (scores[middle - 1] + scores[middle]) / 2.0;
+}
+
+double avg(const std::vector<double> &scores,bool ignoreExtremes) {
+    auto sum = 0.0;
+    using vec_size = std::vector<double>::size_type;
+    for (vec_size i = (ignoreExtremes && scores.size() > 2) ? 1 : 0; i < scores.size() - (ignoreExtremes && scores.size() > 2) ? 1 : 0; i++) sum += scores[i];
+    return sum / static_cast<double>(scores.size() - (ignoreExtremes && scores.size() > 2) ? 2 : 0);
+}

stats.h

@@ -0,0 +1,9 @@
+#ifndef STATS_H_INCLUDED
+#define STATS_H_INCLUDED
+
+#include <vector>
+
+double med(std::vector<double> &scores,bool ignoreExtremes);
+double avg(const std::vector<double> &scores,bool ignoreExtremes);
+
+#endif // STATS_H_INCLUDED