From d62ca9f87906be6f046fe9d07afb8bfb69c166e3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?louiz=E2=80=99?= <louiz@louiz.org>
Date: Fri, 16 Mar 2018 01:11:47 +0100
Subject: Use if constexpr to make things a lot more readable

---
 src/database/row.hpp | 52 +++++++++++++++++++++++++---------------------------
 1 file changed, 25 insertions(+), 27 deletions(-)

(limited to 'src/database/row.hpp')

diff --git a/src/database/row.hpp b/src/database/row.hpp
index 3703ff7..194a9c5 100644
--- a/src/database/row.hpp
+++ b/src/database/row.hpp
@@ -29,41 +29,39 @@ struct Row
     return col.value;
   }
 
-  template <bool Coucou=true>
-  void save(std::unique_ptr<DatabaseEngine>& db, typename std::enable_if<!is_one_of<Id, T...> && Coucou>::type* = nullptr)
+  void save(std::unique_ptr<DatabaseEngine>& db)
   {
-    this->insert(*db);
-  }
-
-  template <bool Coucou=true>
-  void save(std::unique_ptr<DatabaseEngine>& db, typename std::enable_if<is_one_of<Id, T...> && Coucou>::type* = nullptr)
-  {
-    const Id& id = std::get<Id>(this->columns);
-    if (id.value == Id::unset_value)
+    if constexpr(is_one_of<Id, T...>)
       {
-        this->insert(*db);
-        if (db->last_inserted_rowid >= 0)
-          std::get<Id>(this->columns).value = static_cast<Id::real_type>(db->last_inserted_rowid);
+        const Id& id = std::get<Id>(this->columns);
+        if (id.value == Id::unset_value)
+          {
+            this->insert(*db);
+            if (db->last_inserted_rowid >= 0)
+              std::get<Id>(this->columns).value = static_cast<Id::real_type>(db->last_inserted_rowid);
+          }
+        else
+          this->update(*db);
       }
     else
-      this->update(*db);
+      this->insert(*db);
   }
 
  private:
-  template <bool Coucou=true>
-  void insert(DatabaseEngine& db, typename std::enable_if<is_one_of<Id, T...> && Coucou>::type* = nullptr)
-  {
-    InsertQuery query(this->table_name, this->columns);
-    // Ugly workaround for non portable stuff
-    query.body += db.get_returning_id_sql_string(Id::name);
-    query.execute(db, this->columns);
-  }
-
-  template <bool Coucou=true>
-  void insert(DatabaseEngine& db, typename std::enable_if<!is_one_of<Id, T...> && Coucou>::type* = nullptr)
+  void insert(DatabaseEngine& db)
   {
-    InsertQuery query(this->table_name, this->columns);
-    query.execute(db, this->columns);
+    if constexpr(is_one_of<Id, T...>)
+      {
+        InsertQuery query(this->table_name, this->columns);
+        // Ugly workaround for non portable stuff
+        query.body += db.get_returning_id_sql_string(Id::name);
+        query.execute(db, this->columns);
+      }
+    else
+      {
+        InsertQuery query(this->table_name, this->columns);
+        query.execute(db, this->columns);
+      }
   }
 
   void update(DatabaseEngine& db)
-- 
cgit v1.2.3