From 593e4a11edb42cf7311ee341b0dbb1e7b1497c1c Mon Sep 17 00:00:00 2001 From: Miklos Vajna Date: Mon, 26 Nov 2012 09:30:08 +0100 Subject: android: extract duplicated casts to helper methods Change-Id: Ib77731839ad9e46626a7a07d4b2c6c7e32d4bba1 --- .../libreoffice/android/examples/DocumentLoader.java | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) (limited to 'android/experimental/DocumentLoader') diff --git a/android/experimental/DocumentLoader/src/org/libreoffice/android/examples/DocumentLoader.java b/android/experimental/DocumentLoader/src/org/libreoffice/android/examples/DocumentLoader.java index fcf9fae0116c..4817bac8cb73 100644 --- a/android/experimental/DocumentLoader/src/org/libreoffice/android/examples/DocumentLoader.java +++ b/android/experimental/DocumentLoader/src/org/libreoffice/android/examples/DocumentLoader.java @@ -145,6 +145,16 @@ public class DocumentLoader Bundle extras; + PageViewer getPageViewerAt(int index) + { + return (PageViewer)flipper.getChildAt(index); + } + + PageViewer getCurrentPageViewer() + { + return (PageViewer)flipper.getCurrentView(); + } + class GestureListener extends GestureDetector.SimpleOnGestureListener { @@ -156,7 +166,7 @@ public class DocumentLoader { Log.i(TAG, "onFling: " + event1 + " " + event2); if (event1.getX() - event2.getX() > 120) { - if (((PageViewer)flipper.getCurrentView()).currentPageNumber == documentContext.pageCount-1) + if (getCurrentPageViewer().currentPageNumber == documentContext.pageCount-1) return false; Animation inFromRight = new TranslateAnimation(Animation.RELATIVE_TO_SELF, 1, Animation.RELATIVE_TO_SELF, 0, @@ -172,10 +182,10 @@ public class DocumentLoader flipper.showNext(); - ((PageViewer)flipper.getChildAt((flipper.getDisplayedChild() + PAGECACHE_PLUSMINUS) % PAGECACHE_SIZE)).display(((PageViewer)flipper.getCurrentView()).currentPageNumber + PAGECACHE_PLUSMINUS); + getPageViewerAt((flipper.getDisplayedChild() + PAGECACHE_PLUSMINUS) % PAGECACHE_SIZE).display(getCurrentPageViewer().currentPageNumber + PAGECACHE_PLUSMINUS); return true; } else if (event2.getX() - event1.getX() > 120) { - if (((PageViewer)flipper.getCurrentView()).currentPageNumber == 0) + if (getCurrentPageViewer().currentPageNumber == 0) return false; Animation inFromLeft = new TranslateAnimation(Animation.RELATIVE_TO_SELF, -1, Animation.RELATIVE_TO_SELF, 0, @@ -191,7 +201,7 @@ public class DocumentLoader flipper.showPrevious(); - ((PageViewer)flipper.getChildAt((flipper.getDisplayedChild() + PAGECACHE_SIZE - PAGECACHE_PLUSMINUS) % PAGECACHE_SIZE)).display(((PageViewer)flipper.getCurrentView()).currentPageNumber - PAGECACHE_PLUSMINUS); + getPageViewerAt((flipper.getDisplayedChild() + PAGECACHE_SIZE - PAGECACHE_PLUSMINUS) % PAGECACHE_SIZE).display(getCurrentPageViewer().currentPageNumber - PAGECACHE_PLUSMINUS); return true; } @@ -938,7 +948,7 @@ public class DocumentLoader public Object onRetainNonConfigurationInstance() { ArrayList ret = new ArrayList(2); ret.add(bootstrapContext); - documentContext.currentPageNumber = ((PageViewer)flipper.getCurrentView()).currentPageNumber; + documentContext.currentPageNumber = getCurrentPageViewer().currentPageNumber; ret.add(documentContext); return ret; } -- cgit