Changeset 9627

Timestamp:

04/17/08 21:00:52 (5 years ago)

Author:

ewigkeit

Message:

lang/java/sabotter: ImageRegistryを使うように変更。ついでにスペルミスも修正。

Files:

• lang/java/sabotter/trunk/src/jp/xet/eclipse/sabotter/views/LabelProcessor.java (modified) (15 diffs)

lang/java/sabotter/trunk/src/jp/xet/eclipse/sabotter/views/LabelProcessor.java

@@ -9 +9 @@
 import java.text.SimpleDateFormat;
 import java.util.Date;
-import java.util.HashMap;
 import java.util.Locale;
-import java.util.Map;
 
 import jp.xet.eclipse.sabotter.SabotterPlugin;
@@ -21 +19 @@
 import org.apache.commons.httpclient.HttpMethod;
 import org.apache.commons.httpclient.methods.GetMethod;
+import org.eclipse.jface.resource.ImageRegistry;
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.graphics.Color;
@@ -33 +32 @@
 /**
  * 表示文字列等の加工（色づけ等）クラス。
+ * 
  * @author daisuke
  */
 public class LabelProcessor {
-        
+
         private TwitterManager manager;
 
@@ -48 +48 @@
 
         /** アイコン画像のキャッシュ */
-        private Map<String, Image> cashe = new HashMap<String, Image>();
-        
+        private ImageRegistry cache = new ImageRegistry();
+
         /** プリファレンス */
         SabotterPreference pref = SabotterPlugin.getPreference();
@@ -55 +55 @@
         /**
          * コンストラクタ。
+         * 
          * @param manager
          * @category instance creation
@@ -64 +65 @@
         /**
          * 表示する名前を取得する。
+         * 
          * @param status
          * @return 名前
@@ -82 +84 @@
         /**
          * 表示するメッセージを取得する。
+         * 
          * @param status
          * @return メッセージ
@@ -91 +94 @@
         /**
          * 表示する日時を取得する。
+         * 
          * @param status
          * @return 日時
@@ -107 +111 @@
         /**
          * 表示するアイコンを取得する
+         * 
          * @param status
          * @return アイコン
          */
         public Image getImage(Status status) {
-                Image image = null;
                 if (status == null) {
-                        return image;
+                        return null;
                 }
                 String url = encode(status.getUser().getProfileImageURL().toString());
-                if (cashe.get(url) == null) {
+                if (cache.get(url) == null) {
                         HttpClient client = new HttpClient();
 
@@ -126 +130 @@
                         }
 
+                        Image tmpImage = null;
+
                         try {
                                 HttpMethod method = new GetMethod(url);
                                 client.executeMethod(method);
                                 InputStream is = method.getResponseBodyAsStream();
-                                image = new Image(null, is);
-                                cashe.put(url, image);
+                                tmpImage = new Image(null, is);
+                                cache.put(url, processImage(tmpImage));
                         } catch (HttpException e) {
                                 SabotterPlugin.handleException(e);
@@ -139 +145 @@
                                 SabotterPlugin.handleException(e, SabotterPlugin.LOG
                                                 | SabotterPlugin.STACKTRACE);
+                        } finally {
+                                if ((tmpImage != null) && !tmpImage.isDisposed()) {
+                                        tmpImage.dispose();
+                                }
                         }
-                } else {
-                        image = cashe.get(url);
-                }
-
-                return processImage(image);
+                }
+
+                return cache.get(url);
         }
 
         /**
          * APIから取得したURLのエンコーディングを行う。（本来はAPI側で処理されているべきなんだが）
+         * 
          * @param url
          * @return エンコーディング結果。
@@ -168 +177 @@
         /**
          * 画像を設定の通りに加工する。
+         * 
          * @param image
          * @return 加工された画像
@@ -215 +225 @@
         /**
          * 背景色を取得する。
+         * 
          * @param status
          * @return 背景色
@@ -233 +244 @@
         /**
          * 文字色を取得する。
+         * 
          * @param status
          * @return 文字色
@@ -246 +258 @@
         /**
          * フォントを取得する。
+         * 
          * @param status
          * @return フォント
          */
         public Font getFont(Status status) {
-                if(manager.isUnread(status)) {
+                if (manager.isUnread(status)) {
                         // なんか微妙だな。。
                         return new Font(null, new FontData("", 9, SWT.BOLD));