chore: remove duplicate api functions

CHANGELOG.md

@@ -9,6 +9,24 @@ All notable changes to this project will be documented in this file
 * removed `structure` module
 * removed `book.translator` method, use `person.fullName` instead
 * removed `book.isbn` method, use `commerce.isbn` instead
+* removed `commerce.price` method, use `finance.amount` instead
+* removed `commerce.productId` method, use `string.alphanumeric` instead
+* removed `commerce.productRating` method, use `number.decimal` instead
+* removed `commerce.discountAmount` method, use `number.decimal` instead
+* removed `commerce.discountPercentage` method, use `number.decimal` instead
+* removed `commerce.orderNumber` method, use `string.numeric` instead
+* removed `commerce.discountCode` method, use `string.alphanumeric` instead
+* removed `git.author` method, use `person.fullName` instead
+* removed `helper.shuffle` method, use standard library shuffle instead
+* removed `person.middleName` method, use `person.firstName` instead
+* removed `weather.temperature` method, use `number.decimal` instead
+* removed `weather.pressure` method, use `number.decimal` instead
+* removed `weather.visibility` method, use `number.decimal` instead
+* removed `weather.windSpeed` method, use `number.decimal` instead
+* removed `weather.uvIndex` method, use `number.integer` instead
+* removed `weather.humidity` method, use `number.integer` instead
+* removed `weather.cloudCover` method, use `number.integer` instead
+
 * changed std::string to std::string_view in where possible
 
 ### Features

include/faker-cxx/Commerce.h

@@ -18,18 +18,6 @@ class Commerce
      */
     static std::string_view department();
 
-    /**
-     * @brief Generates a random price between the given bounds (inclusive).
-
-     * @param min The lower bound for the price. Defaults to `0`.
-     * @param max The upper bound for the price. Defaults to `1000`.
-     *
-     * @code
-     * Commerce::price() // "88.62"
-     * @endcode
-     */
-    static std::string price(double min = 0, double max = 1000);
-
     /**
      * @brief Generates a random sku by default only with digits.
      *
@@ -130,17 +118,6 @@ class Commerce
      */
     static std::string ISBN10();
 
-    /**
-     * @brief Returns a random product ID.
-     *
-     * @returns productId.
-     *
-     * @code
-     * Commerce::productId() // "ABCD123456"
-     * @endcode
-     */
-    static std::string productId();
-
     /**
      * @brief Returns a random payment type.
      *
@@ -196,17 +173,6 @@ class Commerce
      */
     static std::string_view productReview();
 
-    /**
-     * @brief Returns a random product rating (0-5).
-     *
-     * @returns productRating.
-     *
-     * @code
-     * Commerce::productRating() // 4.1
-     * @endcode
-     */
-    static double productRating();
-
     /**
      * @brief Returns a random discount type.
      *
@@ -218,50 +184,6 @@ class Commerce
      */
     static std::string_view discountType();
 
-    /**
-     * @brief Returns random discount code within the specified range of 6 to 12 characters.
-     *
-     * @returns discount code consists of uppercase letters and numbers.
-     *
-     * @code
-     * Commerce::discountCode() // "VNM15DEC1"
-     * @endcode
-     */
-    static std::string discountCode();
-
-    /**
-     * @brief Returns a random discount amount within the specified range of 10 to 1000.
-     *
-     * @returns discountAmount.
-     *
-     * @code
-     * Commerce::discountAmount() // 57.80
-     * @endcode
-     */
-    static double discountAmount();
-
-    /**
-     * @brief Returns a random discount percentage within the specified range of 1. to 90.
-     *
-     * @returns discountPercentage.
-     *
-     * @code
-     * Commerce::discountPercentage() // 35.50
-     * @endcode
-     */
-    static double discountPercentage();
-
-    /**
-     * @brief Returns a random order number.
-     *
-     * @returns orderNumber.
-     *
-     * @code
-     * Commerce::orderNumber() // 0123456
-     * @endcode
-     */
-    static std::string orderNumber();
-
     /**
      * @brief Returns a random order status.
      *

include/faker-cxx/Git.h

@@ -84,18 +84,5 @@ class Git
      * @endcode
      */
     static std::string commitSha(unsigned length = 40);
-
-    /**
-     * @brief Returns a random author name and email.
-     *
-     * @param
-     *
-     * @returns Author.
-     *
-     * @code
-     * Git::author // {Author.name = "Rachel McLaughlin", Author.email = "[email protected]"}
-     * @endcode
-     */
-    static Author author();
 };
 }

include/faker-cxx/Helper.h

@@ -61,12 +61,14 @@ class Helper
     static auto arrayElement(It start, It end) -> decltype(*::std::declval<It>())
     {
         size_t size = end - start;
+
         if (size == 0)
         {
             throw std::invalid_argument{"Range [start,end) is empty."};
         }
 
-        const auto index = Number::integer<size_t>(size - 1);
+        const std::integral auto index = Number::integer<size_t>(size - 1);
+
         return start[index];
     }
 
@@ -145,7 +147,9 @@ class Helper
         }
 
         T item;
+
         std::sample(data.begin(), data.end(), &item, 1, pseudoRandomGenerator);
+
         return item;
     }
 
@@ -188,41 +192,26 @@ class Helper
         }
 
         const std::integral auto targetWeightValue = Number::integer<unsigned>(1, sumOfWeights);
+
         unsigned currentSum = 0;
+
         size_t currentIdx = 0;
 
         while (currentIdx < data.size())
         {
             currentSum += data[currentIdx].weight;
+
             if (currentSum >= targetWeightValue)
+            {
                 break;
+            }
+
             currentIdx++;
         }
 
         return data.at(currentIdx).value;
     }
 
-    /**
-     * @brief Returns shuffled vector.
-     *
-     * @tparam T an element type of the vector.
-     *
-     * @param data The vector.
-     *
-     * @return Vector with shuffled elements.
-     *
-     * @code
-     * Helper::shuffle<std::string>(std::vector<std::string>{{"hello"}, {"world"}}) // {{"world"}, {"hello"}}
-     * @endcode
-     */
-    template <class T>
-    static std::vector<T> shuffle(std::vector<T> data)
-    {
-        std::shuffle(std::begin(data), std::end(data), pseudoRandomGenerator);
-
-        return data;
-    }
-
     /**
      * @brief Returns shuffled std::string
      *
@@ -265,6 +254,7 @@ class Helper
         }
 
         std::vector<typename T::key_type> keys;
+
         for (const auto& entry : object)
         {
             keys.push_back(entry.first);
@@ -296,6 +286,7 @@ class Helper
         {
             return callback();
         }
+
         return TResult();
     }
 

include/faker-cxx/Person.h

@@ -47,21 +47,6 @@ class Person
     static std::string_view lastName(std::optional<Country> country = std::nullopt,
                                      std::optional<Sex> sex = std::nullopt);
 
-    /**
-     * @brief Returns a random middle name.
-     *
-     * @param country The local country. Defaults to `Country::England`.
-     * @param sex The optional sex to use.
-     *
-     * @returns Middle name starting with a capital letter.
-     *
-     * @code
-     * Person::middleName() // "Васильевич"
-     * @endcode
-     */
-    static std::string_view middleName(std::optional<Country> country = std::nullopt,
-                                       std::optional<Sex> sex = std::nullopt);
-
     /**
      * @brief Returns a random full name.
      *

include/faker-cxx/Weather.h

@@ -7,95 +7,6 @@ namespace faker
 class Weather
 {
 public:
-    struct Temperature
-    {
-        double metric, imperial;
-    };
-
-    /**
-     * @brief Generates a random temperature
-     *
-     * @return Temperature object with metric and imperial temperatures
-     *
-     * @code
-     * Weather::temperature(); // Temperature.metric = 10, Temperature.imperial = 50
-     * @endcode
-     */
-    static Temperature temperature();
-
-    struct Pressure
-    {
-        double metric;
-        double imperial;
-    };
-
-    /**
-     * @brief Generated a random pressure
-     *
-     * @return Pressure object with metric and imperial pressures
-     *
-     * @code
-     * Weather::pressure(); // Pressure.metric = 1000, Pressure.imperial = 14.5
-     * @endcode
-     */
-    static Pressure pressure();
-
-    struct Visibility
-    {
-        double metric;
-        double imperial;
-    };
-
-    /**
-     * @brief Generated a random visibility
-     *
-     * @return Visibility object with metric and imperial visibility
-     *
-     * @code
-     * Weather::visibility(); // Visibility.metric = 10.0, Visibility.imperial = 6.2
-     * @endcode
-     */
-    static Visibility visibility();
-
-    struct WindSpeed
-    {
-        double metric;
-        double imperial;
-    };
-
-    /**
-     * @brief Generated a random wind speed
-     *
-     * @return WindSpeed object with metric and imperial wind speed
-     *
-     * @code
-     * Weather::windSpeed(); // WindSpeed.metric = 10.0, WindSpeed.imperial = 6.2
-     * @endcode
-     */
-    static WindSpeed windSpeed();
-
-    /**
-     * @brief Generated a random uvIndex
-     *
-     * @return Int A random uvIndex
-     *
-     * @code
-     * Weather::uvIndex(); // 10
-     * @endcode
-     */
-    static int uvIndex();
-
-    /**
-     * @brief Generated a random humidity percentage
-     *
-     * @return Int A random humidity percentage
-     *
-     * @code
-     * Weather::humidity(); // 10
-     * @endcode
-     */
-    static int humidity();
-
     /**
      * @brief Generated a random weather description
      *
@@ -106,16 +17,5 @@ class Weather
      * @endcode
      */
     static std::string_view weatherDescription();
-
-    /**
-     * @brief Generated a random cloud cover percentage
-     *
-     * @return Int A random cloud cover percentage
-     *
-     * @code
-     * Weather::cloudCover(); // 10
-     * @endcode
-     */
-    static int cloudCover();
 };
 }