Beats: Move deserialization logic from BeatFactory class

mixxxdj · Aug 19, 2021 · 8299a06 · 8299a06
1 parent 6ae4c0a
commit 8299a06
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 25 deletions.
diff --git a/src/library/dao/trackdao.cpp b/src/library/dao/trackdao.cpp
@@ -1235,7 +1235,7 @@ bool setTrackBeats(const QSqlRecord& record, const int column, Track* pTrack) {
     QString beatsSubVersion = record.value(column + 2).toString();
     QByteArray beatsBlob = record.value(column + 3).toByteArray();
     bool bpmLocked = record.value(column + 4).toBool();
-    const mixxx::BeatsPointer pBeats = BeatFactory::loadBeatsFromByteArray(
+    const mixxx::BeatsPointer pBeats = mixxx::Beats::fromByteArray(
             pTrack->getSampleRate(), beatsVersion, beatsSubVersion, beatsBlob);
     if (pBeats) {
         if (bpmLocked) {

diff --git a/src/track/beatfactory.cpp b/src/track/beatfactory.cpp
@@ -12,25 +12,6 @@ const QString kRoundingVersion = QStringLiteral("V4");
 
 } // namespace
 
-mixxx::BeatsPointer BeatFactory::loadBeatsFromByteArray(
-        mixxx::audio::SampleRate sampleRate,
-        const QString& beatsVersion,
-        const QString& beatsSubVersion,
-        const QByteArray& beatsSerialized) {
-    if (beatsVersion == BEAT_GRID_1_VERSION ||
-        beatsVersion == BEAT_GRID_2_VERSION) {
-        auto pGrid = mixxx::BeatGrid::fromByteArray(sampleRate, beatsSubVersion, beatsSerialized);
-        qDebug() << "Successfully deserialized BeatGrid";
-        return pGrid;
-    } else if (beatsVersion == BEAT_MAP_VERSION) {
-        auto pMap = mixxx::BeatMap::fromByteArray(sampleRate, beatsSubVersion, beatsSerialized);
-        qDebug() << "Successfully deserialized BeatMap";
-        return pMap;
-    }
-    qDebug() << "BeatFactory::loadBeatsFromByteArray could not parse serialized beats.";
-    return mixxx::BeatsPointer();
-}
-
 mixxx::BeatsPointer BeatFactory::makeBeatGrid(
         mixxx::audio::SampleRate sampleRate,
         mixxx::Bpm bpm,

diff --git a/src/track/beatfactory.h b/src/track/beatfactory.h
@@ -11,11 +11,6 @@ class Track;
 
 class BeatFactory {
   public:
-    static mixxx::BeatsPointer loadBeatsFromByteArray(
-            mixxx::audio::SampleRate sampleRate,
-            const QString& beatsVersion,
-            const QString& beatsSubVersion,
-            const QByteArray& beatsSerialized);
     static mixxx::BeatsPointer makeBeatGrid(
             mixxx::audio::SampleRate sampleRate,
             mixxx::Bpm bpm,

diff --git a/src/track/beats.cpp b/src/track/beats.cpp
@@ -1,7 +1,37 @@
 #include "track/beats.h"
 
+#include "track/beatgrid.h"
+#include "track/beatmap.h"
+
 namespace mixxx {
 
+// static
+mixxx::BeatsPointer Beats::fromByteArray(
+        mixxx::audio::SampleRate sampleRate,
+        const QString& beatsVersion,
+        const QString& beatsSubVersion,
+        const QByteArray& beatsSerialized) {
+    mixxx::BeatsPointer pBeats = nullptr;
+    if (beatsVersion == BEAT_GRID_1_VERSION || beatsVersion == BEAT_GRID_2_VERSION) {
+        pBeats = mixxx::BeatGrid::fromByteArray(sampleRate, beatsSubVersion, beatsSerialized);
+    } else if (beatsVersion == BEAT_MAP_VERSION) {
+        pBeats = mixxx::BeatMap::fromByteArray(sampleRate, beatsSubVersion, beatsSerialized);
+    } else {
+        qWarning().nospace() << "Failed to deserialize Beats (" << beatsVersion
+                             << "): Invalid beats version";
+        return nullptr;
+    }
+
+    if (!pBeats) {
+        qWarning().nospace() << "Failed to deserialize Beats (" << beatsVersion
+                             << "): Parsing failed";
+        return nullptr;
+    }
+
+    qDebug().nospace() << "Successfully deserialized Beats (" << beatsVersion << ")";
+    return pBeats;
+}
+
 int Beats::numBeatsInRange(audio::FramePos startPosition, audio::FramePos endPosition) const {
     audio::FramePos lastPosition = audio::kStartFramePos;
     int i = 1;

diff --git a/src/track/beats.h b/src/track/beats.h
@@ -30,6 +30,12 @@ class Beats {
   public:
     virtual ~Beats() = default;
 
+    static mixxx::BeatsPointer fromByteArray(
+            mixxx::audio::SampleRate sampleRate,
+            const QString& beatsVersion,
+            const QString& beatsSubVersion,
+            const QByteArray& beatsSerialized);
+
     enum class BpmScale {
         Double,
         Halve,