#6[1h]. Add TransferController to keep track on transfers.

Author: Evgenii Kanivets

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/TransferActivity.java

@@ -9,8 +9,10 @@
 import com.blogspot.e_kanivets.moneytracker.R;
 import com.blogspot.e_kanivets.moneytracker.activity.base.BaseActivity;
 import com.blogspot.e_kanivets.moneytracker.controller.AccountController;
+import com.blogspot.e_kanivets.moneytracker.controller.TransferController;
 import com.blogspot.e_kanivets.moneytracker.helper.DbHelper;
 import com.blogspot.e_kanivets.moneytracker.model.Account;
+import com.blogspot.e_kanivets.moneytracker.model.Transfer;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -21,7 +23,9 @@ public class TransferActivity extends BaseActivity {
     @SuppressWarnings("unused")
     private static final String TAG = "TransferActivity";
 
-    private AccountController accountController;
+    private TransferController transferController;
+
+    private List<Account> accountList;
 
     @Bind(R.id.spinner_from)
     AppCompatSpinner spinnerFrom;
@@ -39,7 +43,12 @@ protected int getContentViewId() {
 
     @Override
     protected boolean initData() {
-        accountController = new AccountController(new DbHelper(TransferActivity.this));
+        DbHelper dbHelper = new DbHelper(TransferActivity.this);
+
+        AccountController accountController = new AccountController(dbHelper);
+        transferController = new TransferController(dbHelper, accountController);
+
+        accountList = accountController.getAccounts();
 
         return super.initData();
     }
@@ -48,8 +57,6 @@ protected boolean initData() {
     protected void initViews() {
         super.initViews();
 
-        List<Account> accountList = accountController.getAccounts();
-
         List<String> accounts = new ArrayList<>();
         for (Account account : accountList) {
             accounts.add(account.getTitle());
@@ -88,6 +95,25 @@ public boolean onOptionsItemSelected(MenuItem item) {
     }
 
     private void doTransfer() {
+        Account fromAccount = accountList.get(spinnerFrom.getSelectedItemPosition());
+        Account toAccount = accountList.get(spinnerTo.getSelectedItemPosition());
+
+        int fromAmount = -1;
+        try {
+            fromAmount = Integer.parseInt(etFromAmount.getText().toString());
+        } catch (NumberFormatException e) {
+            e.printStackTrace();
+        }
+
+        int toAmount = -1;
+        try {
+            toAmount = Integer.parseInt(etToAmount.getText().toString());
+        } catch (NumberFormatException e) {
+            e.printStackTrace();
+        }
 
+        Transfer transfer = new Transfer(System.currentTimeMillis(), fromAccount.getId(),
+                toAccount.getId(), fromAmount, toAmount);
+        transferController.create(transfer);
     }
 }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/controller/AccountController.java

@@ -9,6 +9,7 @@
 import com.blogspot.e_kanivets.moneytracker.helper.DbHelper;
 import com.blogspot.e_kanivets.moneytracker.model.Account;
 import com.blogspot.e_kanivets.moneytracker.model.Record;
+import com.blogspot.e_kanivets.moneytracker.model.Transfer;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -62,14 +63,14 @@ public List<Account> getAccounts() {
     }
 
     @Nullable
-    public Account read(int id) {
+    public Account read(long id) {
         Account account = null;
 
         SQLiteDatabase db = dbHelper.getReadableDatabase();
 
         // Read accounts table from db
         Cursor cursor = db.query(DbHelper.TABLE_ACCOUNTS, null, "id=?",
-                new String[]{Integer.toString(id)}, null, null, null);
+                new String[]{Long.toString(id)}, null, null, null);
 
         if (cursor.moveToFirst()) {
             // Get indexes of columns
@@ -94,7 +95,7 @@ public Account read(int id) {
         return account;
     }
 
-    public void updateAccountById(Account account) {
+    public void update(Account account) {
         SQLiteDatabase db = dbHelper.getWritableDatabase();
 
         ContentValues contentValues = new ContentValues();
@@ -148,7 +149,7 @@ public boolean recordAdded(@Nullable Record record) {
 
         Log.d(TAG, "recordAdded: " + account);
 
-        updateAccountById(account);
+        update(account);
 
         return true;
     }
@@ -172,7 +173,7 @@ public boolean recordDeleted(@Nullable Record record) {
                 break;
         }
 
-        updateAccountById(account);
+        update(account);
 
         return true;
     }
@@ -183,4 +184,21 @@ public boolean recordUpdated(@Nullable Record oldRecord, @Nullable Record newRec
 
         return recordDeleted(oldRecord) && recordAdded(newRecord);
     }
+
+    public boolean transferDone(@Nullable Transfer transfer) {
+        if(transfer == null) return false;
+
+        Account fromAccount = read(transfer.getFromAccountId());
+        Account toAccount = read(transfer.getToAccountId());
+
+        if (fromAccount == null || toAccount == null) return false;
+
+        fromAccount.take(transfer.getFromAmount());
+        toAccount.put(transfer.getToAmount());
+
+        update(fromAccount);
+        update(toAccount);
+
+        return true;
+    }
 }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/controller/TransferController.java

@@ -0,0 +1,48 @@
+package com.blogspot.e_kanivets.moneytracker.controller;
+
+import android.content.ContentValues;
+import android.database.sqlite.SQLiteDatabase;
+import android.support.annotation.Nullable;
+import android.util.Log;
+
+import com.blogspot.e_kanivets.moneytracker.helper.DbHelper;
+import com.blogspot.e_kanivets.moneytracker.model.Transfer;
+
+/**
+ * Controller class to encapsulate transfer handling logic.
+ * Created by evgenii_kanivets on 2/10/16.
+ */
+public class TransferController {
+    @SuppressWarnings("unused")
+    private static final String TAG = "TransferController";
+
+    private DbHelper dbHelper;
+    private AccountController accountController;
+
+    public TransferController(DbHelper dbHelper, AccountController accountController) {
+        this.dbHelper = dbHelper;
+        this.accountController = accountController;
+    }
+
+    public boolean create(@Nullable Transfer transfer) {
+        if (transfer == null) return false;
+
+        SQLiteDatabase db = dbHelper.getWritableDatabase();
+
+        ContentValues contentValues = new ContentValues();
+        contentValues.put(DbHelper.TIME_COLUMN, transfer.getTime());
+        contentValues.put(DbHelper.FROM_ACCOUNT_ID_COLUMN, transfer.getFromAccountId());
+        contentValues.put(DbHelper.TO_ACCOUND_ID_COLUMN, transfer.getToAccountId());
+        contentValues.put(DbHelper.FROM_AMOUNT_COLUMN, transfer.getFromAmount());
+        contentValues.put(DbHelper.TO_AMOUNT_COLUMN, transfer.getToAmount());
+
+        long id = db.insert(DbHelper.TABLE_TRANSFERS, null, contentValues);
+        Log.d(TAG, "created transfer with id = " + id);
+
+        db.close();
+
+        accountController.transferDone(transfer);
+
+        return true;
+    }
+}

app/src/main/java/com/blogspot/e_kanivets/moneytracker/fragment/AccountsFragment.java

@@ -134,6 +134,10 @@ public void onActivityResult(int requestCode, int resultCode, Intent data) {
                     update();
                     break;
 
+                case REQUEST_TRANSFER:
+                    update();
+                    break;
+
                 default:
                     break;
             }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/model/Transfer.java

@@ -0,0 +1,64 @@
+package com.blogspot.e_kanivets.moneytracker.model;
+
+/**
+ * Entity class to represent transfer operation between accounts.
+ * Created on 2/10/16.
+ *
+ * @author Evgenii Kanivets
+ *
+ * + ID_COLUMN + " INTEGER PRIMARY KEY AUTOINCREMENT,"
++ TIME_COLUMN + " INTEGER,"
++ FROM_ACCOUNT_ID_COLUMN + " INTEGER,"
++ TO_ACCOUND_ID_COLUMN + " INTEGER,"
++ FROM_AMOUNT_COLUMN + " INTEGER,"
++ TO_AMOUNT_COLUMN + " INTEGER);");
+ */
+public class Transfer {
+    private long id;
+    private long time;
+    private long fromAccountId;
+    private long toAccountId;
+    private int fromAmount;
+    private int toAmount;
+
+    public Transfer(long id, long time, long fromAccountId, long toAccountId, int fromAmount, int toAmount) {
+        this.id = id;
+        this.time = time;
+        this.fromAccountId = fromAccountId;
+        this.toAccountId = toAccountId;
+        this.fromAmount = fromAmount;
+        this.toAmount = toAmount;
+    }
+
+    public Transfer(long time, long fromAccountId, long toAccountId, int fromAmount, int toAmount) {
+        this.time = time;
+        this.fromAccountId = fromAccountId;
+        this.toAccountId = toAccountId;
+        this.fromAmount = fromAmount;
+        this.toAmount = toAmount;
+    }
+
+    public long getId() {
+        return id;
+    }
+
+    public long getTime() {
+        return time;
+    }
+
+    public long getFromAccountId() {
+        return fromAccountId;
+    }
+
+    public long getToAccountId() {
+        return toAccountId;
+    }
+
+    public int getFromAmount() {
+        return fromAmount;
+    }
+
+    public int getToAmount() {
+        return toAmount;
+    }
+}