removed map_keys and map_values

Author: jkalias

include/map.h

@@ -57,72 +57,6 @@ class map
     {
     }
 
-    // Performs the functional `map` algorithm for keys, in which every element of the resulting
-    // vector is the output of applying the transform function on every key of this instance.
-    //
-    // example:
-    //      const fcpp::map<std::string, int> ages({{"jake", 32}, {"mary", 26}, {"david", 40}});
-    //      const auto initials = ages.map_keys<char>([](const auto& name) {
-    //          return name[0];
-    //      });
-    //
-    // outcome:
-    //      initials -> fcpp::vector<char>({'d', 'j', 'm'})
-    //
-    // is equivalent to:
-    //      const fcpp::map<std::string, int> ages({{"jake", 32}, {"mary", 26}, {"david", 40}});
-    //      fcpp::vector<char> initials;
-    //      for (const auto& element : ages) {
-    //          initials.insert_back(element.first[0]);
-    //      }
-#ifdef CPP17_AVAILABLE
-    template <typename U, typename Transform, typename = std::enable_if_t<std::is_invocable_r_v<U, Transform, TKey>>>
-#else
-    template <typename U, typename Transform>
-#endif
-    vector<U> map_keys(Transform && transform) const
-    {
-        vector<U> transformed;
-        transformed.reserve(size());
-        for (const auto& element : m_map) {
-            transformed.insert_back(transform(element.first));
-        }
-        return transformed;
-    }
-
-    // Performs the functional `map` algorithm for values, in which every element of the resulting
-    // vector is the output of applying the transform function on every value of this instance.
-    //
-    // example:
-    //      const fcpp::map<std::string, int> ages({{"jake", 32}, {"mary", 26}, {"david", 40}});
-    //      const auto labels = ages.map_values<std::string>([](const auto& age) {
-    //          return std::to_string(age);
-    //      });
-    //
-    // outcome:
-    //      labels -> fcpp::vector<std::string>({"40", "32", "26"})
-    //
-    // is equivalent to:
-    //      const fcpp::map<std::string, int> ages({{"jake", 32}, {"mary", 26}, {"david", 40}});
-    //      fcpp::vector<std::string> labels;
-    //      for (const auto& element : ages) {
-    //          labels.insert_back(std::to_string(element.second));
-    //      }
-#ifdef CPP17_AVAILABLE
-    template <typename U, typename Transform, typename = std::enable_if_t<std::is_invocable_r_v<U, Transform, TValue>>>
-#else
-    template <typename U, typename Transform>
-#endif
-    vector<U> map_values(Transform && transform) const
-    {
-        vector<U> transformed;
-        transformed.reserve(size());
-        for (const auto& element : m_map) {
-            transformed.insert_back(transform(element.second));
-        }
-        return transformed;
-    }
-
     // Performs the functional `map` algorithm from one map to another map, in which every key/value
     // pair of the resulting map is the output of applying the transform function on every key/value
     // pair of this instance. If two source elements produce equivalent result keys, the first

tests/map_test.cc

@@ -64,26 +64,6 @@ TEST(MapTest, AccessOperator)
     EXPECT_EQ(0, persons["john"]);
 }
 
-TEST(MapTest, MapValues)
-{
-    const map<std::string, int> persons({{"jake", 32}, {"mary", 26}, {"david", 40}});
-    const auto mapped = persons.map_values<std::string>([](const int& age) {
-        return std::to_string(age);
-    });
-
-    EXPECT_EQ(vector<std::string>({"40", "32", "26"}), mapped);
-}
-
-TEST(MapTest, MapKeys)
-{
-    const map<std::string, int> persons({{"jake", 32}, {"mary", 26}, {"david", 40}});
-    const auto mapped = persons.map_keys<char>([](const std::string& name) {
-        return name[0];
-    });
-
-    EXPECT_EQ(vector<char>({'d', 'j', 'm'}), mapped);
-}
-
 TEST(MapTest, MapTo)
 {
     const map<std::string, int> persons({{"jake", 32}, {"mary", 26}, {"david", 40}});