Merge pull request android#11 from DmitriyZaitsev/dz/todo-databinding-1

JoseAlcerreca · JoseAlcerreca · commit 5ca3f05e2d29 · 2016-03-30T18:43:54.000+01:00
Use binded fields instead of finding them by id again
diff --git a/todoapp/app/src/main/java/com/example/android/architecture/blueprints/todoapp/tasks/TasksFragment.java b/todoapp/app/src/main/java/com/example/android/architecture/blueprints/todoapp/tasks/TasksFragment.java
@@ -93,10 +93,8 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container,
 
         tasksFragBinding.setActionHandler(mPresenter);
 
-        View root = tasksFragBinding.getRoot();
-
         // Set up tasks view
-        ListView listView = (ListView) root.findViewById(R.id.tasks_list);
+        ListView listView = tasksFragBinding.tasksList;
 
         mListAdapter = new TasksAdapter(new ArrayList<Task>(0), mPresenter);
         listView.setAdapter(mListAdapter);
@@ -114,8 +112,7 @@ public void onClick(View v) {
         });
 
         // Set up progress indicator
-        final ScrollChildSwipeRefreshLayout swipeRefreshLayout =
-                (ScrollChildSwipeRefreshLayout) root.findViewById(R.id.refresh_layout);
+        final ScrollChildSwipeRefreshLayout swipeRefreshLayout = tasksFragBinding.refreshLayout;
         swipeRefreshLayout.setColorSchemeColors(
                 ContextCompat.getColor(getActivity(), R.color.colorPrimary),
                 ContextCompat.getColor(getActivity(), R.color.colorAccent),
@@ -126,6 +123,8 @@ public void onClick(View v) {
 
         setHasOptionsMenu(true);
 
+        View root = tasksFragBinding.getRoot();
+
         return root;
     }
 
