Update Parser.java

Parser.java

@@ -1,42 +1,84 @@
+import java.io.BufferedOutputStream;
 import java.io.File;
 import java.io.FileInputStream;
+import java.io.FileNotFoundException;
 import java.io.FileOutputStream;
 import java.io.IOException;
+
 /**
- * This class is thread safe.
+ * This class is immutable.
  */
 public class Parser {
-  private File file;
-  public synchronized void setFile(File f) {
-    file = f;
-  }
-  public synchronized File getFile() {
-    return file;
-  }
-  public String getContent() throws IOException {
-    FileInputStream i = new FileInputStream(file);
-    String output = "";
-    int data;
-    while ((data = i.read()) > 0) {
-      output += (char) data;
+
+    private final File file;
+    private String content;
+    private String contentWithoutUnicode;
+
+    public Parser(File file) {
+        this.file = file;
     }
-    return output;
-  }
-  public String getContentWithoutUnicode() throws IOException {
-    FileInputStream i = new FileInputStream(file);
-    String output = "";
-    int data;
-    while ((data = i.read()) > 0) {
-      if (data < 0x80) {
-        output += (char) data;
-      }
+
+    public String getContent() {
+        if (content != null) return content;
+        content = getContentInternal(true);
+        return content;
     }
-    return output;
-  }
-  public void saveContent(String content) throws IOException {
-    FileOutputStream o = new FileOutputStream(file);
-    for (int i = 0; i < content.length(); i += 1) {
-      o.write(content.charAt(i));
+
+    public String getContentWithoutUnicode() {
+        if (contentWithoutUnicode != null) return contentWithoutUnicode;
+        contentWithoutUnicode = getContentInternal(false);
+        return contentWithoutUnicode;
+    }
+
+    private String getContentInternal(boolean unicode) {
+        FileInputStream fis = loadStream();
+        StringBuilder sb = new StringBuilder();
+        int data;
+        try {
+            while ((data = fis.read()) > 0) {
+                if (unicode) sb.append(data);
+                else if (data < 0x80) {
+                    sb.append(data);
+                }
+            }
+        } catch (IOException e) {
+            closeStream(fis);
+            throw new RuntimeException(/* appropriate message */);
+        }
+        closeStream(fis);
+        return sb.toString();
+    }
+
+    private FileInputStream loadStream() {
+        FileInputStream fis;        
+        try {
+            fis = new FileInputStream(file);
+        } catch (FileNotFoundException e) {
+            throw new RuntimeException(/* appropriate message */);
+        }
+        return fis;
+    }
+
+    private void closeStream(FileInputStream fis) {
+        try {
+            fis.close();
+        } catch (IOException e) {
+            throw new RuntimeException(/* appropriate message */);
+        }
+    }
+    public void saveContent(String content) {
+        BufferedOutputStream bos = null;
+        try {
+            bos = new BufferedOutputStream(new FileOutputStream(file));
+        } catch (FileNotFoundException e) {
+            throw new RuntimeException(/* appropriate message */);
+        }
+
+        try {
+            bos.write(content.getBytes());
+            bos.close();
+        } catch (IOException e) {
+            throw new RuntimeException(/* appropriate message */);
+        }
     }
-  }
 }