unnecessary null check removed.

legalimpurity · Jul 14, 2018 · 84f1fc0 · 84f1fc0
1 parent dd965d2
commit 84f1fc0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/...rc/main/java/com/legalimpurity/asynctaskwrittenfromscratch/MyAsyncTaskCode/MyAsyncTask.kt b/...rc/main/java/com/legalimpurity/asynctaskwrittenfromscratch/MyAsyncTaskCode/MyAsyncTask.kt
@@ -14,14 +14,12 @@ abstract class MyAsyncTask<ResultTypeWeWantOnPostExecute>{
     fun execute() {
         onPreExecute()
 
-        ExecutorProvider.backgroundTaskExecutor?.let {
-            val future: Future<ResultTypeWeWantOnPostExecute> = it.submit(Callable<ResultTypeWeWantOnPostExecute>
-            {
-                doInBackground()
-            })
-            val valueFromDoInBackground = future.get()
+        val future: Future<ResultTypeWeWantOnPostExecute> = ExecutorProvider.backgroundTaskExecutor.submit(Callable<ResultTypeWeWantOnPostExecute>
+        {
+            doInBackground()
+        })
+        val valueFromDoInBackground = future.get()
 
-            onPostExecute(valueFromDoInBackground)
-        }
+        onPostExecute(valueFromDoInBackground)
     }
 }
diff --git a/...m/legalimpurity/asynctaskwrittenfromscratch/MyAsyncTaskCode/executors/ExecutorProvider.kt b/...m/legalimpurity/asynctaskwrittenfromscratch/MyAsyncTaskCode/executors/ExecutorProvider.kt
@@ -7,8 +7,8 @@ object ExecutorProvider
     private val NUMBER_OF_CORES = Runtime.getRuntime().availableProcessors()
     private var backgroundTasksThreadFactory: ThreadFactory = MyThreadFactory()
 
-    var backgroundTaskExecutor: ThreadPoolExecutor? = null
-    var mainThreadTaskExecutor: Executor? = null
+    var backgroundTaskExecutor: ThreadPoolExecutor
+    var mainThreadTaskExecutor: Executor
 
     init {