code: refactoring

project/main.cpp

@@ -3,6 +3,7 @@
 #include "histogram.h"
 #include "svg.h"
 #include "show_svg.h"
+#include <curl/curl.h>
 using namespace std;
 
 struct Input {
@@ -13,29 +14,39 @@ struct Input {
 };
 
 Input
-input_data() {
+input_data(istream& stream, bool prompt) {
+    size_t number_count;
+    if (prompt) {
+        cerr << "Enter number count: ";
+    }
+    stream >> number_count;
+
     Input in;
-    cerr << "Enter number count: ";
-    cin >> in.number_count;
+    in.numbers.resize(number_count);
+
+
+   if (prompt){
+            cerr << "Enter numbers: ";
+    }
+    for (size_t i = 0; i < number_count; i++)
+    {
+        stream >> in.numbers[i];
+    }
 
-    vector<double> numbers(in.number_count);
-    in.numbers.resize(in.number_count);
-    for (size_t i = 0; i < in.number_count; i++) {
-        cin >> in.numbers[i];
+    if (prompt){
+    cerr << "Enter number of bins: ";
     }
+        stream >> in.bin_count;
 
-    size_t bin_count;
-    cerr << "Enter bin count: ";
-    cin >> in.bin_count;
 
-    size_t max_count;
-    in.max_count = 0;
     return in;
 }
 
 int main()
 {
-    Input in = input_data();
+     curl_global_init(CURL_GLOBAL_ALL);
+    bool prompt = true;
+    auto in = input_data(cin, prompt);
     vector<size_t> bins = make_histogram(in.numbers, in.bin_count, in.number_count, in.max_count);
     show_histogram_svg(bins);
     return 0;

project/show_histogram.cpp

@@ -29,7 +29,7 @@ show_histogram_svg(const vector<size_t>& bins) {
         }
     }
     for (size_t bin : bins) {
-            const double bin_width = BLOCK_WIDTH * bin;
+            const double bin_width = (IMAGE_WIDTH - TEXT_WIDTH) * (bin) / (longest);
             if (bin == longest){
                 svg_text(TEXT_LEFT, top + TEXT_BASELINE, to_string(bin));
                 svg_rect(TEXT_WIDTH, top, IMAGE_WIDTH - TEXT_WIDTH, BIN_HEIGHT, "#000000", "#ff00a2");

project/unittest.cbp

@@ -33,6 +33,8 @@
 		</Compiler>
 		<Unit filename="histogram.cpp" />
 		<Unit filename="histogram_internal.h" />
+		<Unit filename="percent.cpp" />
+		<Unit filename="percent.h" />
 		<Unit filename="unittest.cpp" />
 		<Extensions>
 			<lib_finder disable_auto="1" />

project/unittest.cpp

@@ -2,7 +2,6 @@
 #define DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN
 #include "doctest.h"
 #include "histogram_internal.h"
-#include "percent.h"
 
 TEST_CASE("distinct empty") {
     double min = 0;