Refactor Music module

CMakeLists.txt

@@ -44,6 +44,7 @@ set(FAKER_SOURCES
     src/modules/movie/Movie.cpp
     src/modules/movie/MovieData.cpp
     src/modules/music/Music.cpp
+    src/modules/music/MusicData.cpp
     src/modules/number/Number.cpp
     src/modules/person/Person.cpp
     src/modules/phone/Phone.cpp

include/faker-cxx/Music.h

@@ -1,6 +1,6 @@
 #pragma once
 
-#include <string>
+#include <string_view>
 
 namespace faker
 {
@@ -16,7 +16,7 @@ class Music
      * Music::artist() // "Nirvana"
      * @endcode
      */
-    static std::string artist();
+    static std::string_view artist();
 
     /**
      * @brief Returns a random music genre.
@@ -27,7 +27,7 @@ class Music
      * Music::genre() // "Rock"
      * @endcode
      */
-    static std::string genre();
+    static std::string_view genre();
 
     /**
      * @brief Returns a random song name.
@@ -38,6 +38,6 @@ class Music
      * Music::songName() // "Light My Fire"
      * @endcode
      */
-    static std::string songName();
+    static std::string_view songName();
 };
 }

src/modules/music/Music.cpp

@@ -1,24 +1,22 @@
 #include "faker-cxx/Music.h"
 
-#include "data/Artists.h"
-#include "data/Genres.h"
-#include "data/SongNames.h"
+#include "MusicData.h"
 #include "faker-cxx/Helper.h"
 
 namespace faker
 {
-std::string Music::artist()
+std::string_view Music::artist()
 {
-    return Helper::arrayElement<std::string>(artists);
+    return Helper::arrayElement(music::artists);
 }
 
-std::string Music::genre()
+std::string_view Music::genre()
 {
-    return Helper::arrayElement<std::string>(musicGenres);
+    return Helper::arrayElement(music::musicGenres);
 }
 
-std::string Music::songName()
+std::string_view Music::songName()
 {
-    return Helper::arrayElement<std::string>(songNames);
+    return Helper::arrayElement(music::songNames);
 }
 }