Remove db.close() to speed up DB requests.

app/src/main/java/com/blogspot/e_kanivets/moneytracker/repo/base/BaseRepo.java

@@ -14,6 +14,8 @@
 
 /**
  * Base implementation of {@link IRepo}.
+ * No need to call db.close() at all, because SQLiteOpenHelper manage it for us + cache instances.
+ * It will speed up all DB operations.
  * Created on 2/15/16.
  *
  * @author Evgenii Kanivets
@@ -43,8 +45,6 @@ public T create(@Nullable T instance) {
 
         long id = db.insert(getTable(), null, contentValues(instance));
 
-        db.close();
-
         if (id == -1) {
             Log.d(TAG, "Couldn't create record : " + instance);
             return null;
@@ -74,8 +74,6 @@ public T update(@Nullable T instance) {
         String[] args = new String[]{Long.valueOf(instance.getId()).toString()};
         long rowsAffected = db.update(getTable(), contentValues(instance), "id=?", args);
 
-        db.close();
-
         if (rowsAffected == 0) {
             Log.d(TAG, "Couldn't update record : " + instance);
             return null;
@@ -101,8 +99,6 @@ public boolean delete(@Nullable T instance) {
         String[] args = new String[]{Long.toString(instance.getId())};
         long rowsAffected = db.delete(getTable(), "id=?", args);
 
-        db.close();
-
         Log.d(TAG, instance + (rowsAffected == 0 ? " didn't " : " ") + "deleted");
 
         return rowsAffected != 0;
@@ -118,7 +114,6 @@ public List<T> readWithCondition(@Nullable String condition, @Nullable String[]
         List<T> recordList = getListFromCursor(cursor);
 
         cursor.close();
-        db.close();
 
         return recordList;
     }