]> 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 a1521fa2fecae8db0c4d9367576dc1812e97249d..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
@@ -18,7 +19,6 @@ import android.content.SharedPreferences;
 import android.graphics.Typeface;\r
 import android.os.Bundle;\r
 import android.os.Handler;\r
-import android.os.Vibrator;\r
 import android.preference.PreferenceManager;\r
 import android.text.ClipboardManager;\r
 import android.text.Editable;\r
@@ -27,16 +27,17 @@ import android.text.TextWatcher;
 import android.text.style.StyleSpan;\r
 import android.util.Log;\r
 import android.view.ContextMenu;\r
+import android.view.ContextMenu.ContextMenuInfo;\r
 import android.view.KeyEvent;\r
 import android.view.Menu;\r
 import android.view.MenuItem;\r
-import android.view.View;\r
-import android.view.ViewGroup;\r
-import android.view.ContextMenu.ContextMenuInfo;\r
 import android.view.MenuItem.OnMenuItemClickListener;\r
+import android.view.View;\r
 import android.view.View.OnClickListener;\r
+import android.view.ViewGroup;\r
 import android.view.inputmethod.InputMethodManager;\r
 import android.widget.AdapterView;\r
+import android.widget.AdapterView.AdapterContextMenuInfo;\r
 import android.widget.BaseAdapter;\r
 import android.widget.Button;\r
 import android.widget.EditText;\r
@@ -46,13 +47,14 @@ import android.widget.TableLayout;
 import android.widget.TableRow;\r
 import android.widget.TextView;\r
 import android.widget.Toast;\r
-import android.widget.AdapterView.AdapterContextMenuInfo;\r
 \r
 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
@@ -69,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
@@ -81,8 +88,10 @@ public class DictionaryActivity extends ListActivity {
 \r
   // Visible for testing.\r
   ListAdapter indexAdapter = null;\r
+  \r
+  final SearchTextWatcher searchTextWatcher = new SearchTextWatcher();\r
 \r
-  private Vibrator vibrator = null;\r
+  //private Vibrator vibrator = null;\r
   \r
   public DictionaryActivity() {\r
   }\r
@@ -110,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
@@ -128,18 +141,46 @@ public class DictionaryActivity extends ListActivity {
       dictionary = new Dictionary(dictRaf); \r
     } catch (Exception e) {\r
       Log.e(LOG, "Unable to load dictionary.", e);\r
+      if (dictRaf != null) {\r
+        try {\r
+          dictRaf.close();\r
+        } catch (IOException e1) {\r
+          Log.e(LOG, "Unable to close dictRaf.", e1);\r
+        }\r
+        dictRaf = null;\r
+      }\r
       Toast.makeText(this, getString(R.string.invalidDictionary, "", e.getMessage()), Toast.LENGTH_LONG);\r
       startActivity(DictionaryEditActivity.getIntent(dictIndex));\r
       finish();\r
       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
+        \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
-          final com.ibm.icu.text.Collator c = index.sortLanguage.getCollator();\r
+          Log.d(LOG, "Starting collator load for lang=" + index.sortLanguage.getSymbol());\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
@@ -150,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(new SearchTextWatcher());\r
-    searchText.setText(prefs.getString(C.SEARCH_TOKEN, ""));\r
-    Log.d(LOG, "Trying to restore searchText=" + searchText.getText());\r
+    searchText.addTextChangedListener(searchTextWatcher);\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
@@ -201,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
@@ -218,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
-      vibrator = (Vibrator) getSystemService(Context.VIBRATOR_SERVICE);\r
-    }\r
+    //if (prefs.getBoolean(getString(R.string.vibrateOnFailedSearchKey), true)) {\r
+      // vibrator = (Vibrator) getSystemService(Context.VIBRATOR_SERVICE);\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
@@ -238,12 +278,35 @@ public class DictionaryActivity extends ListActivity {
   @Override\r
   protected void onDestroy() {\r
     super.onDestroy();\r
+    if (dictRaf == null) {\r
+      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
@@ -293,11 +356,12 @@ public class DictionaryActivity extends ListActivity {
       // Down\r
       destIndexEntry = Math.min(tokenRow.referenceIndex + 1, index.sortedIndexEntries.size());\r
     }\r
-    \r
     final Index.IndexEntry dest = index.sortedIndexEntries.get(destIndexEntry);\r
-    searchText.setText(dest.token);\r
     Log.d(LOG, "onUpDownButton, destIndexEntry=" + dest.token);\r
-    //jumpToRow(index.sortedIndexEntries.get(destIndexEntry).startRow);\r
+    searchText.removeTextChangedListener(searchTextWatcher);\r
+    searchText.setText(dest.token);\r
+    jumpToRow(index.sortedIndexEntries.get(destIndexEntry).startRow);\r
+    searchText.addTextChangedListener(searchTextWatcher);\r
   }\r
 \r
   // --------------------------------------------------------------------------\r
@@ -308,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
@@ -418,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
@@ -439,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.longestPrefix.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
@@ -449,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
@@ -466,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
@@ -477,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
@@ -536,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
@@ -562,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
@@ -574,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
@@ -597,7 +687,12 @@ 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
     }\r
     Log.d(LOG, "onSearchTextChange: " + text);    \r