More reckless refactoring and code style tweaks -- mostly adding braces around condit...
[zxing.git] / android / src / com / google / zxing / client / android / DecodeThread.java
old mode 100644 (file)
new mode 100755 (executable)
index 2c4b904..5835ecf
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2008 Google Inc.
+ * Copyright (C) 2008 ZXing authors
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
 
 package com.google.zxing.client.android;
 
-import android.app.Application;
-import android.graphics.Bitmap;
-import android.os.Debug;
+import android.content.SharedPreferences;
+import android.os.Bundle;
 import android.os.Handler;
 import android.os.Looper;
 import android.os.Message;
+import android.preference.PreferenceManager;
 import com.google.zxing.BarcodeFormat;
 import com.google.zxing.DecodeHintType;
-import com.google.zxing.MonochromeBitmapSource;
 import com.google.zxing.MultiFormatReader;
 import com.google.zxing.ReaderException;
 import com.google.zxing.Result;
 
-import java.io.File;
-import java.io.FileNotFoundException;
-import java.io.FileOutputStream;
-import java.io.IOException;
-import java.io.OutputStream;
-import java.util.Date;
 import java.util.Hashtable;
 import java.util.Vector;
 
 /**
- * This thread does all the heavy lifting of decoding the images. It can also save images to flash
- * for debugging purposes.
- *
- * @author dswitkin@google.com (Daniel Switkin)
+ * This thread does all the heavy lifting of decoding the images.
  */
 final class DecodeThread extends Thread {
 
-  public Handler handler;
-
-  private final BarcodeReaderCaptureActivity activity;
-  private final CameraManager cameraManager;
-  private Hashtable<DecodeHintType, Object> hints;
-  private Handler cameraThreadHandler;
-  private int methodTraceCount;
-  private boolean tracing;
-
-  DecodeThread(BarcodeReaderCaptureActivity activity, CameraManager cameraManager) {
-    this.activity = activity;
-    this.cameraManager = cameraManager;
-    methodTraceCount = 0;
-    tracing = false;
+  public static final String BARCODE_BITMAP = "barcode_bitmap";
+
+  public Handler mHandler;
+  private final CaptureActivity mActivity;
+  private final MultiFormatReader mMultiFormatReader;
+
+  DecodeThread(CaptureActivity activity, String mode) {
+    mActivity = activity;
+    mMultiFormatReader = new MultiFormatReader();
+
+    // The prefs can't change while the thread is running, so pick them up once here.
+    if (mode == null || mode.length() == 0) {
+      SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(activity);
+      boolean decode1D = prefs.getBoolean(PreferencesActivity.KEY_DECODE_1D, true);
+      boolean decodeQR = prefs.getBoolean(PreferencesActivity.KEY_DECODE_QR, true);
+      if (decode1D && decodeQR) {
+        setDecodeAllMode();
+      } else if (decode1D) {
+        setDecode1DMode();
+      } else if (decodeQR) {
+        setDecodeQRMode();
+      }
+    } else {
+      if (mode.equals(Intents.Scan.PRODUCT_MODE)) {
+        setDecodeProductMode();
+      } else if (mode.equals(Intents.Scan.ONE_D_MODE)) {
+        setDecode1DMode();
+      } else if (mode.equals(Intents.Scan.QR_CODE_MODE)) {
+        setDecodeQRMode();
+      } else {
+        setDecodeAllMode();
+      }
+    }
   }
 
   @Override
   public void run() {
     Looper.prepare();
-    handler = new Handler() {
+    mHandler = new Handler() {
+      @Override
       public void handleMessage(Message message) {
         switch (message.what) {
           case R.id.decode:
-            captureAndDecode();
-            break;
-          case R.id.save:
-            captureAndSave();
+            decode((byte[]) message.obj, message.arg1, message.arg2);
             break;
           case R.id.quit:
             Looper.myLooper().quit();
             break;
-          case R.id.set_decode_all_mode:
-            setDecodeAllMode();
-            break;
-          case R.id.set_decode_1D_mode:
-            setDecode1DMode();
-            break;
-          case R.id.set_decode_QR_mode:
-            setDecodeQRMode();
-            break;
-          case R.id.toggle_tracing:
-            tracing = !tracing;
-            break;
         }
       }
     };
     Looper.loop();
   }
 
-  public void setCameraThreadHandler(Handler cameraThreadHandler) {
-    this.cameraThreadHandler = cameraThreadHandler;
-  }
-
-  private void setDecodeAllMode() {
-    hints = null;
+  private void setDecodeProductMode() {
+    Hashtable<DecodeHintType, Object> hints = new Hashtable<DecodeHintType, Object>(3);
+    Vector<BarcodeFormat> vector = new Vector<BarcodeFormat>();
+    vector.addElement(BarcodeFormat.UPC_A);
+    vector.addElement(BarcodeFormat.UPC_E);
+    vector.addElement(BarcodeFormat.EAN_13);
+    vector.addElement(BarcodeFormat.EAN_8);
+    hints.put(DecodeHintType.POSSIBLE_FORMATS, vector);
+    mMultiFormatReader.setHints(hints);
   }
 
   // TODO: This is fragile in case we add new formats. It would be better to have a new enum
   // value which represented all 1D formats.
   private void setDecode1DMode() {
-    hints = new Hashtable<DecodeHintType, Object>(3);
+    Hashtable<DecodeHintType, Object> hints = new Hashtable<DecodeHintType, Object>(3);
     Vector<BarcodeFormat> vector = new Vector<BarcodeFormat>();
     vector.addElement(BarcodeFormat.UPC_A);
     vector.addElement(BarcodeFormat.UPC_E);
@@ -115,107 +113,69 @@ final class DecodeThread extends Thread {
     vector.addElement(BarcodeFormat.CODE_39);
     vector.addElement(BarcodeFormat.CODE_128);
     hints.put(DecodeHintType.POSSIBLE_FORMATS, vector);
+    mMultiFormatReader.setHints(hints);
   }
 
   private void setDecodeQRMode() {
-    hints = new Hashtable<DecodeHintType, Object>(3);
+    Hashtable<DecodeHintType, Object> hints = new Hashtable<DecodeHintType, Object>(3);
     Vector<BarcodeFormat> vector = new Vector<BarcodeFormat>();
     vector.addElement(BarcodeFormat.QR_CODE);
     hints.put(DecodeHintType.POSSIBLE_FORMATS, vector);
+    mMultiFormatReader.setHints(hints);
   }
 
-  private void captureAndDecode() {
-    Date startDate = new Date();
-    Bitmap bitmap = cameraManager.captureStill();
-    // Let the CameraThread know it can resume previews while the decoding continues in parallel.
-    Message restart = Message.obtain(cameraThreadHandler, R.id.decode_started);
-    restart.sendToTarget();
+  /**
+   * Instead of calling setHints(null), which would allow new formats like ITF to sneak in, we
+   * explicitly set which formats are available.
+   */
+  private void setDecodeAllMode() {
+    Hashtable<DecodeHintType, Object> hints = new Hashtable<DecodeHintType, Object>(3);
+    Vector<BarcodeFormat> vector = new Vector<BarcodeFormat>();
+    vector.addElement(BarcodeFormat.UPC_A);
+    vector.addElement(BarcodeFormat.UPC_E);
+    vector.addElement(BarcodeFormat.EAN_13);
+    vector.addElement(BarcodeFormat.EAN_8);
+    vector.addElement(BarcodeFormat.CODE_39);
+    vector.addElement(BarcodeFormat.CODE_128);
+    vector.addElement(BarcodeFormat.QR_CODE);
+    hints.put(DecodeHintType.POSSIBLE_FORMATS, vector);
+    mMultiFormatReader.setHints(hints);
+  }
 
-    if (tracing) {
-      Debug.startMethodTracing("/sdcard/ZXingDecodeThread" + methodTraceCount);
-      methodTraceCount++;
-    }
+  /**
+   * Decode the data within the viewfinder rectangle, and time how long it took. For efficiency,
+   * reuse the same reader objects from one decode to the next.
+   *
+   * @param data   The YUV preview frame.
+   * @param width  The width of the preview frame.
+   * @param height The height of the preview frame.
+   */
+  private void decode(byte[] data, int width, int height) {
+    long start = System.currentTimeMillis();
     boolean success;
     Result rawResult = null;
+    YUVMonochromeBitmapSource source = new YUVMonochromeBitmapSource(data, width, height,
+        CameraManager.get().getFramingRect());
     try {
-      MonochromeBitmapSource source = new RGBMonochromeBitmapSource(bitmap);
-      rawResult = new MultiFormatReader().decode(source, hints);
+      rawResult = mMultiFormatReader.decodeWithState(source);
       success = true;
     } catch (ReaderException e) {
       success = false;
     }
-    if (tracing) {
-      Debug.stopMethodTracing();
-    }
-    Date endDate = new Date();
+    long end = System.currentTimeMillis();
 
     if (success) {
-      Message message = Message.obtain(cameraThreadHandler, R.id.decode_succeeded, rawResult);
-      message.arg1 = (int) (endDate.getTime() - startDate.getTime());
+      Message message = Message.obtain(mActivity.mHandler, R.id.decode_succeeded, rawResult);
+      message.arg1 = (int) (end - start);
+      Bundle bundle = new Bundle();
+      bundle.putParcelable(BARCODE_BITMAP, source.renderToBitmap());
+      message.setData(bundle);
       message.sendToTarget();
     } else {
-      Message message = Message.obtain(cameraThreadHandler, R.id.decode_failed);
+      Message message = Message.obtain(mActivity.mHandler, R.id.decode_failed);
+      message.arg1 = (int) (end - start);
       message.sendToTarget();
     }
   }
 
-  /**
-   * This is a debugging feature used to take photos and save them as JPEGs using the exact camera
-   * setup as in normal decoding. This is useful for building up a library of test images.
-   */
-  private void captureAndSave() {
-    Bitmap bitmap = cameraManager.captureStill();
-    OutputStream outStream = getNewPhotoOutputStream();
-    if (outStream != null) {
-      bitmap.compress(Bitmap.CompressFormat.JPEG, 80, outStream);
-      try {
-        outStream.close();
-      } catch (IOException e) {
-      }
-      Message success = Message.obtain(cameraThreadHandler, R.id.save_succeeded);
-      success.sendToTarget();
-    } else {
-      Message failure = Message.obtain(cameraThreadHandler, R.id.save_failed);
-      failure.sendToTarget();
-    }
-  }
-
-  /**
-   * We prefer to write to the SD Card because it has more space, and is automatically mounted as a
-   * drive over USB. If it's not present, fall back to the package's private file area here:
-   *
-   * /data/data/com.google.zxing.client.android/files
-   *
-   * @return A stream which represents the new file where the photo will be saved.
-   */
-  private OutputStream getNewPhotoOutputStream() {
-    File sdcard = new File("/sdcard");
-    if (sdcard.exists()) {
-      File barcodes = new File(sdcard, "barcodes");
-      if (!barcodes.exists()) {
-        if (!barcodes.mkdir()) {
-          return null;
-        }
-      }
-      String fileName = getNewPhotoName();
-      try {
-        return new FileOutputStream(new File(barcodes, fileName));
-      } catch (FileNotFoundException e) {
-      }
-    } else {
-      Application application = activity.getApplication();
-      String fileName = getNewPhotoName();
-      try {
-        return application.openFileOutput(fileName, 0);
-      } catch (FileNotFoundException e) {
-      }
-    }
-    return null;
-  }
-
-  private String getNewPhotoName() {
-    Date now = new Date();
-    return "capture" + now.getTime() + ".jpg";
-  }
-
 }