]> gitweb.fperrin.net Git - Dictionary.git/blobdiff - src/com/hughes/android/dictionary/DictionaryActivity.java
go
[Dictionary.git] / src / com / hughes / android / dictionary / DictionaryActivity.java
index 5e251b7f234cc1c207fa9f1c79f3b591d808a04f..922b9728cda321c11c62614ba554414496ce8025 100644 (file)
@@ -9,6 +9,7 @@ import java.text.SimpleDateFormat;
 import java.util.Date;\r
 import java.util.concurrent.Executor;\r
 import java.util.concurrent.Executors;\r
+import java.util.concurrent.ThreadFactory;\r
 import java.util.concurrent.atomic.AtomicBoolean;\r
 \r
 import android.app.ListActivity;\r
@@ -50,8 +51,10 @@ import android.widget.Toast;
 import com.hughes.android.dictionary.engine.Dictionary;\r
 import com.hughes.android.dictionary.engine.Index;\r
 import com.hughes.android.dictionary.engine.PairEntry;\r
+import com.hughes.android.dictionary.engine.PairEntry.Pair;\r
 import com.hughes.android.dictionary.engine.RowBase;\r
 import com.hughes.android.dictionary.engine.TokenRow;\r
+import com.hughes.android.dictionary.engine.TransliteratorManager;\r
 import com.hughes.android.util.PersistentObjectCache;\r
 \r
 public class DictionaryActivity extends ListActivity {\r
@@ -68,7 +71,12 @@ public class DictionaryActivity extends ListActivity {
   \r
   // package for test.\r
   final Handler uiHandler = new Handler();\r
-  private final Executor searchExecutor = Executors.newSingleThreadExecutor();\r
+  private final Executor searchExecutor = Executors.newSingleThreadExecutor(new ThreadFactory() {\r
+    @Override\r
+    public Thread newThread(Runnable r) {\r
+      return new Thread(r, "searchExecutor");\r
+    }\r
+  });\r
   private SearchOperation currentSearchOperation = null;\r
 \r
   EditText searchText;\r
@@ -111,11 +119,15 @@ public class DictionaryActivity extends ListActivity {
     prefs.remove(C.INDEX_INDEX);\r
     prefs.remove(C.SEARCH_TOKEN);\r
     prefs.commit();\r
+    Log.d(LOG, "Removed default dictionary prefs.");\r
   }\r
 \r
   @Override\r
   public void onCreate(Bundle savedInstanceState) {\r
+    ((DictionaryApplication)getApplication()).applyTheme(this);\r
+    \r
     super.onCreate(savedInstanceState);\r
+    Log.d(LOG, "onCreate:" + this);\r
     \r
     final SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this);\r
     \r
@@ -143,14 +155,32 @@ public class DictionaryActivity extends ListActivity {
       return;\r
     }\r
 \r
+    indexIndex = prefs.getInt(C.INDEX_INDEX, 0) % dictionary.indices.size();\r
+    Log.d(LOG, "Loading index.");\r
+    index = dictionary.indices.get(indexIndex);\r
+    setListAdapter(new IndexAdapter(index));\r
+\r
     // Pre-load the collators.\r
     searchExecutor.execute(new Runnable() {\r
       public void run() {\r
         final long startMillis = System.currentTimeMillis();\r
-        Log.d(LOG, "Constructing index for lang=" + index.sortLanguage.getSymbol());\r
+        \r
+        TransliteratorManager.init(new TransliteratorManager.Callback() {\r
+          @Override\r
+          public void onTransliteratorReady() {\r
+            uiHandler.post(new Runnable() {\r
+              @Override\r
+              public void run() {\r
+                onSearchTextChange(searchText.getText().toString());\r
+              }\r
+            });\r
+          }\r
+        });\r
+        \r
         for (final Index index : dictionary.indices) {\r
           Log.d(LOG, "Starting collator load for lang=" + index.sortLanguage.getSymbol());\r
-          final com.ibm.icu.text.Collator c = index.sortLanguage.getCollator();\r
+          \r
+          final com.ibm.icu.text.Collator c = index.sortLanguage.getCollator();          \r
           if (c.compare("pre-print", "preppy") >= 0) {\r
             Log.e(LOG, c.getClass()\r
                 + " is buggy, lookups may not work properly.");\r
@@ -161,18 +191,17 @@ public class DictionaryActivity extends ListActivity {
       }\r
     });\r
     \r
-    indexIndex = prefs.getInt(C.INDEX_INDEX, 0) % dictionary.indices.size();\r
-    index = dictionary.indices.get(indexIndex);\r
-    setListAdapter(new IndexAdapter(index));\r
-    \r
+\r
     setContentView(R.layout.dictionary_activity);\r
     searchText = (EditText) findViewById(R.id.SearchText);\r
     langButton = (Button) findViewById(R.id.LangButton);\r
     \r
     searchText.requestFocus();\r
     searchText.addTextChangedListener(searchTextWatcher);\r
-    searchText.setText(prefs.getString(C.SEARCH_TOKEN, ""));\r
-    Log.d(LOG, "Trying to restore searchText=" + searchText.getText());\r
+    final String search = prefs.getString(C.SEARCH_TOKEN, "");\r
+    searchText.setText(search);\r
+    searchText.setSelection(0, search.length());\r
+    Log.d(LOG, "Trying to restore searchText=" + search);\r
     \r
     final Button clearSearchTextButton = (Button) findViewById(R.id.ClearSearchTextButton);\r
     clearSearchTextButton.setOnClickListener(new OnClickListener() {\r
@@ -212,6 +241,7 @@ public class DictionaryActivity extends ListActivity {
         if (!searchText.isFocused()) {\r
           // TODO: don't do this if multi words are entered.\r
           final RowBase row = (RowBase) getListAdapter().getItem(position);\r
+          Log.d(LOG, "onItemSelected: " + row.index());\r
           final TokenRow tokenRow = row.getTokenRow(true);\r
           searchText.setText(tokenRow.getToken());\r
         }\r
@@ -229,18 +259,17 @@ public class DictionaryActivity extends ListActivity {
     wordList = new File(prefs.getString(getString(R.string.wordListFileKey),\r
         getString(R.string.wordListFileDefault)));\r
     saveOnlyFirstSubentry = prefs.getBoolean(getString(R.string.saveOnlyFirstSubentryKey), false);\r
-    if (prefs.getBoolean(getString(R.string.vibrateOnFailedSearchKey), true)) {\r
+    //if (prefs.getBoolean(getString(R.string.vibrateOnFailedSearchKey), true)) {\r
       // vibrator = (Vibrator) getSystemService(Context.VIBRATOR_SERVICE);\r
-    }\r
+    //}\r
     Log.d(LOG, "wordList=" + wordList + ", saveOnlyFirstSubentry=" + saveOnlyFirstSubentry);\r
   }\r
   \r
-  \r
   @Override\r
   protected void onResume() {\r
     super.onResume();\r
   }\r
-\r
+  \r
   @Override\r
   protected void onPause() {\r
     super.onPause();\r
@@ -253,11 +282,31 @@ public class DictionaryActivity extends ListActivity {
       return;\r
     }\r
     setDictionaryPrefs(this, dictIndex, indexIndex, searchText.getText().toString());\r
+    \r
+    // Before we close the RAF, we have to wind the current search down.\r
+    if (currentSearchOperation != null) {\r
+      Log.d(LOG, "Interrupting search to shut down.");\r
+      final SearchOperation searchOperation = currentSearchOperation;\r
+      currentSearchOperation = null;\r
+      searchOperation.interrupted.set(true);\r
+      synchronized (searchOperation) {\r
+        while (!searchOperation.done) {\r
+          try {\r
+            searchOperation.wait();\r
+          } catch (InterruptedException e) {\r
+            Log.d(LOG, "Interrupted.", e);\r
+          }\r
+        }\r
+      }\r
+    }\r
+    \r
     try {\r
+      Log.d(LOG, "Closing RAF.");\r
       dictRaf.close();\r
     } catch (IOException e) {\r
       Log.e(LOG, "Failed to close dictionary", e);\r
     }\r
+    dictRaf = null;\r
   }\r
 \r
   // --------------------------------------------------------------------------\r
@@ -312,7 +361,7 @@ public class DictionaryActivity extends ListActivity {
     searchText.removeTextChangedListener(searchTextWatcher);\r
     searchText.setText(dest.token);\r
     jumpToRow(index.sortedIndexEntries.get(destIndexEntry).startRow);\r
-    searchText.removeTextChangedListener(searchTextWatcher);\r
+    searchText.addTextChangedListener(searchTextWatcher);\r
   }\r
 \r
   // --------------------------------------------------------------------------\r
@@ -323,37 +372,37 @@ public class DictionaryActivity extends ListActivity {
   public boolean onCreateOptionsMenu(final Menu menu) {\r
     \r
     {\r
-    final MenuItem preferences = menu.add(getString(R.string.preferences));\r
-    preferences.setOnMenuItemClickListener(new OnMenuItemClickListener() {\r
-      public boolean onMenuItemClick(final MenuItem menuItem) {\r
-        startActivity(new Intent(DictionaryActivity.this,\r
-            PreferenceActivity.class));\r
-        return false;\r
-      }\r
-    });\r
+      final MenuItem preferences = menu.add(getString(R.string.preferences));\r
+      preferences.setOnMenuItemClickListener(new OnMenuItemClickListener() {\r
+        public boolean onMenuItemClick(final MenuItem menuItem) {\r
+          startActivity(new Intent(DictionaryActivity.this,\r
+              PreferenceActivity.class));\r
+          return false;\r
+        }\r
+      });\r
     }\r
 \r
     {\r
-    final MenuItem dictionaryList = menu.add(getString(R.string.dictionaryList));\r
-    dictionaryList.setOnMenuItemClickListener(new OnMenuItemClickListener() {\r
-      public boolean onMenuItemClick(final MenuItem menuItem) {\r
-        startActivity(DictionaryListActivity.getIntent(DictionaryActivity.this));\r
-        startActivity(DictionaryListActivity.getIntent(DictionaryActivity.this));\r
-        return false;\r
-      }\r
-    });\r
+      final MenuItem dictionaryList = menu.add(getString(R.string.dictionaryList));\r
+      dictionaryList.setOnMenuItemClickListener(new OnMenuItemClickListener() {\r
+        public boolean onMenuItemClick(final MenuItem menuItem) {\r
+          startActivity(DictionaryListActivity.getIntent(DictionaryActivity.this));\r
+          finish();\r
+          return false;\r
+        }\r
+      });\r
     }\r
 \r
     {\r
-      final MenuItem dictionaryList = menu.add(getString(R.string.editDictionary));\r
-      dictionaryList.setOnMenuItemClickListener(new OnMenuItemClickListener() {\r
+      final MenuItem dictionaryEdit = menu.add(getString(R.string.editDictionary));\r
+      dictionaryEdit.setOnMenuItemClickListener(new OnMenuItemClickListener() {\r
         public boolean onMenuItemClick(final MenuItem menuItem) {\r
           final Intent intent = DictionaryEditActivity.getIntent(dictIndex);\r
           startActivity(intent);\r
           return false;\r
         }\r
       });\r
-      }\r
+    }\r
 \r
     return true;\r
   }\r
@@ -433,6 +482,10 @@ public class DictionaryActivity extends ListActivity {
       }\r
       return true;\r
     }\r
+    if (keyCode == KeyEvent.KEYCODE_BACK) {\r
+      Log.d(LOG, "Clearing dictionary prefs.");\r
+      DictionaryActivity.clearDictionaryPrefs(this);\r
+    }\r
     return super.onKeyDown(keyCode, event);\r
   }\r
 \r
@@ -454,7 +507,18 @@ public class DictionaryActivity extends ListActivity {
     final Index.IndexEntry searchResult = searchOperation.searchResult;\r
     Log.d(LOG, "searchFinished: " + searchOperation + ", searchResult=" + searchResult);\r
 \r
-    jumpToRow(searchResult.startRow);\r
+    currentSearchOperation = null;\r
+\r
+    uiHandler.postDelayed(new Runnable() {\r
+      @Override\r
+      public void run() {\r
+        if (currentSearchOperation == null) {\r
+          jumpToRow(searchResult.startRow);\r
+        } else {\r
+          Log.d(LOG, "More coming, waiting for currentSearchOperation.");\r
+        }\r
+      }\r
+    }, 50);\r
     \r
 //    if (!searchResult.success) {\r
 //      if (vibrator != null) {\r
@@ -464,6 +528,7 @@ public class DictionaryActivity extends ListActivity {
 //      searchText.setSelection(searchResult.longestPrefixString.length());\r
 //      return;\r
 //    }\r
+    \r
   }\r
   \r
   private final void jumpToRow(final int row) {\r
@@ -481,6 +546,8 @@ public class DictionaryActivity extends ListActivity {
 \r
     Index.IndexEntry searchResult;\r
     \r
+    boolean done = false;\r
+    \r
     SearchOperation(final String searchText, final Index index) {\r
       this.searchText = searchText.trim();\r
       this.index = index;\r
@@ -492,18 +559,25 @@ public class DictionaryActivity extends ListActivity {
 \r
     @Override\r
     public void run() {\r
-      searchStartMillis = System.currentTimeMillis();\r
-      searchResult = index.findInsertionPoint(searchText, interrupted);\r
-      Log.d(LOG, "searchText=" + searchText + ", searchDuration="\r
-          + (System.currentTimeMillis() - searchStartMillis) + ", interrupted="\r
-          + interrupted.get());\r
-      if (!interrupted.get()) {\r
-        uiHandler.post(new Runnable() {\r
-          @Override\r
-          public void run() {            \r
-            searchFinished(SearchOperation.this);\r
-          }\r
-        });\r
+      try {\r
+        searchStartMillis = System.currentTimeMillis();\r
+        searchResult = index.findInsertionPoint(searchText, interrupted);\r
+        Log.d(LOG, "searchText=" + searchText + ", searchDuration="\r
+            + (System.currentTimeMillis() - searchStartMillis) + ", interrupted="\r
+            + interrupted.get());\r
+        if (!interrupted.get()) {\r
+          uiHandler.post(new Runnable() {\r
+            @Override\r
+            public void run() {            \r
+              searchFinished(SearchOperation.this);\r
+            }\r
+          });\r
+        }\r
+      } finally {\r
+        synchronized (this) {\r
+          done = true;\r
+          this.notifyAll();\r
+        }\r
       }\r
     }\r
   }\r
@@ -551,7 +625,7 @@ public class DictionaryActivity extends ListActivity {
     private View getView(PairEntry.Row row, ViewGroup parent) {\r
       final TableLayout result = new TableLayout(parent.getContext());\r
       final PairEntry entry = row.getEntry();\r
-      final int rowCount = entry.pairs.length;\r
+      final int rowCount = entry.pairs.size();\r
       for (int r = 0; r < rowCount; ++r) {\r
         final TableRow tableRow = new TableRow(result.getContext());\r
 \r
@@ -577,7 +651,8 @@ public class DictionaryActivity extends ListActivity {
         column2.setWidth(1);\r
 \r
         // TODO: color words by gender\r
-        final String col1Text = index.swapPairEntries ? entry.pairs[r].lang2 : entry.pairs[r].lang1;\r
+        final Pair pair = entry.pairs.get(r);\r
+        final String col1Text = index.swapPairEntries ? pair.lang2 : pair.lang1;\r
         column1.setText(col1Text, TextView.BufferType.SPANNABLE);\r
         final Spannable col1Spannable = (Spannable) column1.getText();\r
         \r
@@ -589,7 +664,7 @@ public class DictionaryActivity extends ListActivity {
           startPos += token.length();\r
         }\r
 \r
-        final String col2Text = index.swapPairEntries ? entry.pairs[r].lang1 : entry.pairs[r].lang2;\r
+        final String col2Text = index.swapPairEntries ? pair.lang1 : pair.lang2;\r
         column2.setText(col2Text, TextView.BufferType.NORMAL);\r
 \r
         result.addView(tableRow);\r
@@ -612,6 +687,10 @@ public class DictionaryActivity extends ListActivity {
   // --------------------------------------------------------------------------\r
 \r
   void onSearchTextChange(final String text) {\r
+    if (dictRaf == null) {\r
+      Log.d(LOG, "searchText changed during shutdown, doing nothing.");\r
+      return;\r
+    }\r
     if (!searchText.isFocused()) {\r
       Log.d(LOG, "searchText changed without focus, doing nothing.");\r
       return;\r