add StarRating::verifyStarCount() helper

mixxxdj · Jan 19, 2024 · 6e4b816 · 6e4b816
1 parent 09b4cce
commit 6e4b816
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/src/library/starrating.cpp b/src/library/starrating.cpp
@@ -13,8 +13,7 @@ StarRating::StarRating(
         int maxStarCount)
         : m_starCount(starCount),
           m_maxStarCount(maxStarCount) {
-    DEBUG_ASSERT(m_starCount >= kMinStarCount);
-    DEBUG_ASSERT(m_starCount <= m_maxStarCount);
+    DEBUG_ASSERT(verifyStarCount(m_starCount));
     // 1st star cusp at 0° of the unit circle whose center is shifted to adapt the 0,0-based paint area
     m_starPolygon << QPointF(1.0, 0.5);
     for (int i = 1; i < 5; ++i) {

diff --git a/src/library/starrating.h b/src/library/starrating.h
@@ -34,9 +34,15 @@ class StarRating {
     int maxStarCount() const {
         return m_maxStarCount;
     }
+
+    bool verifyStarCount(int starCount) {
+        return starCount >= kMinStarCount && starCount <= m_maxStarCount;
+    }
+
     void setStarCount(int starCount) {
-        DEBUG_ASSERT(starCount >= kMinStarCount);
-        DEBUG_ASSERT(starCount <= m_maxStarCount);
+        VERIFY_OR_DEBUG_ASSERT(verifyStarCount(starCount)) {
+            return;
+        }
         m_starCount = starCount;
     }
 

diff --git a/src/widget/wstarrating.cpp b/src/widget/wstarrating.cpp
@@ -52,8 +52,7 @@ QSize WStarRating::sizeHint() const {
 }
 
 void WStarRating::slotSetRating(int starCount) {
-    if (starCount == m_starCount) {
-        // Unchanged
+    if (starCount == m_starCount || !m_visualStarRating.verifyStarCount(starCount)) {
         return;
     }
     m_starCount = starCount;