[30m]. Refactor AppUtils.

Author: Evgenii Kanivets

app/src/main/AndroidManifest.xml

@@ -8,7 +8,7 @@
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
 
     <application
-        android:name=".util.MTApp"
+        android:name=".util.MtApp"
         android:allowBackup="true"
         android:icon="@drawable/ic_launcher"
         android:label="@string/app_name"

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

@@ -13,7 +13,7 @@
 import com.blogspot.e_kanivets.moneytracker.fragment.ExportFragment;
 import com.blogspot.e_kanivets.moneytracker.fragment.NavigationDrawerFragment;
 import com.blogspot.e_kanivets.moneytracker.fragment.RecordsFragment;
-import com.blogspot.e_kanivets.moneytracker.util.AppUtils;
+import com.blogspot.e_kanivets.moneytracker.util.PrefUtils;
 
 public class NavDrawerActivity extends AppCompatActivity
         implements NavigationDrawerFragment.NavigationDrawerCallbacks {
@@ -35,7 +35,7 @@ protected void onCreate(Bundle savedInstanceState) {
         setContentView(R.layout.activity_nav_drawer);
 
         /* Increment launch count */
-        AppUtils.addLaunchCount(NavDrawerActivity.this);
+        PrefUtils.addLaunchCount();
 
         mNavigationDrawerFragment = (NavigationDrawerFragment)
                 getSupportFragmentManager().findFragmentById(R.id.navigation_drawer);

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

@@ -34,7 +34,7 @@
 import com.blogspot.e_kanivets.moneytracker.model.Record;
 import com.blogspot.e_kanivets.moneytracker.ui.AppRateDialog;
 import com.blogspot.e_kanivets.moneytracker.ui.ChangeDateDialog;
-import com.blogspot.e_kanivets.moneytracker.util.AppUtils;
+import com.blogspot.e_kanivets.moneytracker.util.PrefUtils;
 
 import java.util.Calendar;
 import java.util.Date;
@@ -180,7 +180,7 @@ public void onItemSelected(AdapterView<?> parent, View view, int position, long
                 break;
         }
 
-        AppUtils.writeUsedPeriod(getActivity(), position);
+        PrefUtils.writeUsedPeriod(position);
 
         update();
 
@@ -250,9 +250,7 @@ public void onGlobalLayout() {
                     if (isFirst) {
                         isFirst = false;
                         listView.setSelection(listView.getCount() - 1);
-                        if (AppUtils.checkRateDialog(getActivity())) {
-                            showAppRateDialog();
-                        }
+                        if (PrefUtils.checkRateDialog()) showAppRateDialog();
                     }
                 }
             });
@@ -274,7 +272,7 @@ private void initActionBar() {
         Spinner spinner = (Spinner) customNav.findViewById(R.id.spinner_period);
         spinner.setAdapter(new ArrayAdapter<>(getActivity(),
                 android.R.layout.simple_list_item_1, getResources().getStringArray(R.array.array_periods)));
-        spinner.setSelection(AppUtils.readUsedPeriod(getActivity()));
+        spinner.setSelection(PrefUtils.readUsedPeriod());
         spinner.setOnItemSelectedListener(this);
 
         if (actionBar != null) {

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

@@ -2,7 +2,7 @@
 
 import com.blogspot.e_kanivets.moneytracker.controller.CategoryController;
 import com.blogspot.e_kanivets.moneytracker.helper.DbHelper;
-import com.blogspot.e_kanivets.moneytracker.util.MTApp;
+import com.blogspot.e_kanivets.moneytracker.util.MtApp;
 
 import java.io.Serializable;
 
@@ -30,7 +30,7 @@ public Record(int id, long time, int type, String title, int categoryId, int pri
         this.price = price;
         this.accountId = accountId;
 
-        category = new CategoryController(new DbHelper(MTApp.get())).getCategoryById(categoryId);
+        category = new CategoryController(new DbHelper(MtApp.get())).getCategoryById(categoryId);
     }
 
     public int getId() {

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

@@ -3,11 +3,9 @@
 import android.util.Pair;
 
 import com.blogspot.e_kanivets.moneytracker.R;
-import com.blogspot.e_kanivets.moneytracker.util.MTApp;
+import com.blogspot.e_kanivets.moneytracker.util.MtApp;
 
 import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 
@@ -110,11 +108,11 @@ private void fillSummaryReportList(int totalIncome, int totalExpense) {
         //Add summary row to list
         summaryReportList = new ArrayList<Pair<String, Integer>>();
         summaryReportList.add(new Pair<String, Integer>(
-                MTApp.get().getResources().getString(R.string.total_incomes) + " :", totalIncome));
+                MtApp.get().getResources().getString(R.string.total_incomes) + " :", totalIncome));
         summaryReportList.add(new Pair<String, Integer>(
-                MTApp.get().getResources().getString(R.string.total_expenses) + " :", totalExpense));
+                MtApp.get().getResources().getString(R.string.total_expenses) + " :", totalExpense));
         summaryReportList.add(new Pair<String, Integer>(
-                MTApp.get().getResources().getString(R.string.total) + " :", totalExpense + totalIncome));
+                MtApp.get().getResources().getString(R.string.total) + " :", totalExpense + totalIncome));
     }
 
     private void fillRecordList(HashMap<String, Record> recordMap) {

app/src/main/java/com/blogspot/e_kanivets/moneytracker/ui/AppRateDialog.java

@@ -9,11 +9,10 @@
 import android.widget.Button;
 
 import com.blogspot.e_kanivets.moneytracker.R;
-import com.blogspot.e_kanivets.moneytracker.util.AppUtils;
+import com.blogspot.e_kanivets.moneytracker.util.PrefUtils;
 import com.blogspot.e_kanivets.moneytracker.util.Constants;
 
 public class AppRateDialog extends AlertDialog {
-
     private Context context;
 
     public AppRateDialog(Context context) {
@@ -36,8 +35,8 @@ protected void onCreate(Bundle savedInstanceState) {
             @Override
             public void onClick(View view) {
                 context.startActivity(new Intent(Intent.ACTION_VIEW,
-                        Uri.parse(Constants.GP_MARKET + Constants.APP_NAME)));
-                AppUtils.appRated(context);
+                        Uri.parse(Constants.GP_MARKET + context.getPackageName())));
+                PrefUtils.appRated();
                 dismiss();
             }
         });
@@ -52,7 +51,7 @@ public void onClick(View view) {
         thanksButton.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View view) {
-                AppUtils.appRated(context);
+                PrefUtils.appRated();
                 dismiss();
             }
         });
@@ -66,4 +65,4 @@ public void dismiss() {
             e.printStackTrace();
         }
     }
-}
+}

app/src/main/java/com/blogspot/e_kanivets/moneytracker/util/AppUtils.java

@@ -2,21 +2,22 @@
 
 /**
  * Constants class. It's better to change it to interface.
- * Created by eugene on 29/08/14.
+ * Created on 29/08/14.
+ *
+ * @author Evgenii Kanivets
  */
-public class Constants {
-    public static final String APP_NAME = "com.blogspot.e_kanivets.moneytracker";
-    public static final String GP_MARKET = "market://details?id=";
-    public static final String APP_RATED = "app_rated";
-    public static final String LAUNCH_COUNT = "launch_count";
-    public static final int RATE_PERIOD = 5;
-    public static final String CONTRIBUTION = "contribution";
-    public static final String KEY_USED_PERIOD = "key_used_period";
+public interface Constants {
+    String GP_MARKET = "market://details?id=";
+    String APP_RATED = "app_rated";
+    String LAUNCH_COUNT = "launch_count";
+    int RATE_PERIOD = 5;
+    String CONTRIBUTION = "contribution";
+    String KEY_USED_PERIOD = "key_used_period";
 
-    public static final int DEFAULT_USED_PERIOD = 1;
+    int DEFAULT_USED_PERIOD = 1;
 
-    public static final String TITLE_PARAM_NAME = "title";
-    public static final String PRICE_PARAM_NAME = "price";
+    String TITLE_PARAM_NAME = "title";
+    String PRICE_PARAM_NAME = "price";
 
-    public static final String DEFAULT_EXPORT_FILE_NAME = "money_tracker.csv";
-}
+    String DEFAULT_EXPORT_FILE_NAME = "money_tracker.csv";
+}

app/src/main/java/com/blogspot/e_kanivets/moneytracker/util/Constants.java

@@ -8,11 +8,11 @@
  *
  * @author Evgenii Kanivets
  */
-public class MTApp extends Application {
+public class MtApp extends Application {
 
-    private static MTApp mtApp;
+    private static MtApp mtApp;
 
-    public static MTApp get() {
+    public static MtApp get() {
         return mtApp;
     }
 

app/src/main/java/com/blogspot/e_kanivets/moneytracker/util/MTApp.java

@@ -0,0 +1,52 @@
+package com.blogspot.e_kanivets.moneytracker.util;
+
+import android.content.Context;
+import android.content.SharedPreferences;
+
+/**
+ * Util class for application.
+ * Created on 02/09/14.
+ *
+ * @author Evgenii Kanivets
+ */
+public class PrefUtils {
+    public static void addLaunchCount() {
+        SharedPreferences preferences = getDefaultPrefs();
+        SharedPreferences.Editor editor = preferences.edit();
+        editor.putInt(Constants.LAUNCH_COUNT, preferences.getInt(Constants.LAUNCH_COUNT, 0) + 1);
+        editor.apply();
+    }
+
+    public static boolean checkRateDialog() {
+        SharedPreferences preferences = getDefaultPrefs();
+
+        boolean appRated = preferences.getBoolean(Constants.APP_RATED, false);
+        if (appRated) return false;
+
+        int launchCount = preferences.getInt(Constants.LAUNCH_COUNT, 0);
+        return launchCount % Constants.RATE_PERIOD == 0;
+    }
+
+    public static void appRated() {
+        SharedPreferences preferences = getDefaultPrefs();
+        SharedPreferences.Editor editor = preferences.edit();
+        editor.putBoolean(Constants.APP_RATED, true);
+        editor.apply();
+    }
+
+    public static int readUsedPeriod() {
+        SharedPreferences preferences = getDefaultPrefs();
+        return preferences.getInt(Constants.KEY_USED_PERIOD, Constants.DEFAULT_USED_PERIOD);
+    }
+
+    public static void writeUsedPeriod(int usedPeriod) {
+        SharedPreferences preferences = getDefaultPrefs();
+        SharedPreferences.Editor editor = preferences.edit();
+        editor.putInt(Constants.KEY_USED_PERIOD, usedPeriod);
+        editor.apply();
+    }
+
+    private static SharedPreferences getDefaultPrefs() {
+        return MtApp.get().getSharedPreferences(MtApp.get().getPackageName(), Context.MODE_PRIVATE);
+    }
+}