From 96ca069a26db0f2d0f46fef8d026495d7a13bb05 Mon Sep 17 00:00:00 2001
From: fegger <flo.egger@icloud.com>
Date: Thu, 25 Jun 2026 12:10:16 +0200
Subject: [PATCH] Refactor ID retrieval logic in Database class

Simplify the return logic for getAlbumId, getArtistId, and getGenreId
functions by returning results directly when available or after
assignment.
---
 sound_serve.db      | Bin 45056 -> 49152 bytes
 src/Database.cpp    |   6 ++++--
 src/PlayerLocal.cpp |   1 -
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/sound_serve.db b/sound_serve.db
index 08668b69849087e4596056d0d65b05e2f0bab57c..e27ca1355cd973a65df5aafd56fd4ac293c95e3b 100644
GIT binary patch
delta 656
zcmZp8z|_#dJV9EJpMima2a1`1wCqG3V`Y8@z24uvoVyt~*>srsc=>km?&g=|{Kx*E
z^CFuLZy)zE?qwSrSFy@82Z*tYOG+}fRhJ|t<)oGrB_?MVLr9Lvcesv*c>0C7y1NGH
zaB(UX<Rn%)=a=S{C~*QsK_ZFCC7ETZN}Nvq{ywgbehPm6Aqsw>K0XRAu5ON@J|PMv
zMWv}Ah3TnzML=~>!zMrHo+s06DFU(C7i@E4PEu)ZF@$8AypU%aJ4~@NZ`ou!9^v}(
z{Gx0nO@$y=H`gFnKWEoq1*OEI60oR-Qf3NJ%HI!Yv5#wrD^O8zNRX#9$dPXTL9U+e
zehS{Mks45Upc(>pTd@*OGq^Ol4lsx@hy#Nb6=;jHiQ7v~KENkAnS-C#gt^gPkX>9_
znz1Pq;v<OTpl;-J4svw_x-7)e$;TD$^2wh3a=rzb$t9&lsqrP11;8*0ag6|nZEj{R
zx=>1LadJ^+K}lwQ9ztDOW=?8+L1IY;L}+pozX}WAOQv%h8&5H@X^TUhD8Ubj)ObS(
z&AoXyuO1@{|1t*MiH$mwCHW^XMonxC+ho9^u$V<afs2X%B?JFU{;&L(Hwy~f=AZmB
lo|lnj^Y?g8g+&Geixmn47AXi!3}9zv1O^G$X2F6l`~aD^y5#@>

delta 303
zcmZo@U~YK8G(lRBn}LA=gt>s235bOz>KH3>GwAjH;^kk)z`=EZnU9xm7w>Ma1N@iy
zC3#x;UUJRinzdO_z=2Dq*@=r?TvC#;Exsf%DJQk0C^0#^7(y~l_Fy||8|E0~?BN)s
zVPtBmsl&yoRFIQc>6~AhSE9t}=@+7-pp=+gl3A9jq~PT5@8jy|r{Lo1<{0V|qEJ#)
znmW0XUuJR%d$&xJF+bRv7Duo(i8)E7xy2BYW3n{IG@i1=qU4OkBCs`+cW{(3acOSe
z&8x@Aq%AOc0<X;E_ndr;+>^!lCo}3!oT$m9%?%XQVr1SV!0EvJmH#EsGk5tXzl`T)
eWZL{ao>O77LIb-1D<d#4I5!J6eBocDAOHYb992jF

diff --git a/src/Database.cpp b/src/Database.cpp
index ac9c321..b287255 100644
--- a/src/Database.cpp
+++ b/src/Database.cpp
@@ -252,7 +252,7 @@ std::optional<int> db::getAlbumId( const std::string& albumTitle ) {
     if (sqlite3_step( stmt ) == SQLITE_ROW ) {
         id = sqlite3_column_int( stmt, 0 );
     }
-
+    return id;
 }
 
 std::optional<int> db::getArtistId( const std::string& artistName ) {
@@ -264,8 +264,9 @@ std::optional<int> db::getArtistId( const std::string& artistName ) {
 
     std::optional<int> id;
     if (sqlite3_step( stmt ) == SQLITE_ROW ) {
-        id = sqlite3_column_int( stmt, 0 );
+         return sqlite3_column_int( stmt, 0 );
     }
+    return id;
 }
 
 std::optional<int> db::getGenreId( const std::string& genre ) {
@@ -279,4 +280,5 @@ std::optional<int> db::getGenreId( const std::string& genre ) {
     if (sqlite3_step( stmt ) == SQLITE_ROW ) {
         id = sqlite3_column_int( stmt, 0 );
     }
+    return id;
 }
diff --git a/src/PlayerLocal.cpp b/src/PlayerLocal.cpp
index e96b7ba..8530f8f 100644
--- a/src/PlayerLocal.cpp
+++ b/src/PlayerLocal.cpp
@@ -4,7 +4,6 @@
 #include "../include/Database.h"
 #include "../include/Containers.h"
 
-#include <memory>
 
 using db = Database;