lesson 9 видео № 1 "Ввод/вывод":

AlbertoJava · AlbertoJava · commit 3908ebaa970f · 2019-10-07T00:02:14.000+03:00
&#1050;&#1083;&#1072;&#1089;&#1089; AbstractFileStorage, &#1084;&#1077;&#1090;&#1086;&#1076; doUpdate, doGet, doWrite, doRead
diff --git a/src/com/urise/webapp/storage/AbstractFileStorage.java b/src/com/urise/webapp/storage/AbstractFileStorage.java
@@ -3,34 +3,35 @@
 import com.urise.webapp.exception.StorageException;
 import com.urise.webapp.model.Resume;
 
-import java.io.File;
-import java.io.IOException;
+import java.io.*;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
 
 public abstract class AbstractFileStorage extends AbstractStorage {
     private File directory;
+    protected abstract void doWrite(Resume resume, OutputStream os) throws IOException;
+    protected abstract Resume doRead(InputStream is)throws IOException;
+
 
     protected AbstractFileStorage(File directory) {
         Objects.requireNonNull(directory,"directory must not null");
-        if (directory.isDirectory()){
+        if (!directory.isDirectory()){
             throw new IllegalArgumentException(directory.getAbsolutePath() +  " is not directory");
         }
-        if (directory.canRead()||directory.canWrite()){
+        if (!directory.canRead()||!directory.canWrite()){
             throw new IllegalArgumentException(directory.getAbsolutePath() +  " is not readable/writable");
         }
         this.directory=directory;
     }
 
     @Override
     protected void doUpdate(Resume resume, Object searchKey) {
-        //&#1082;&#1072;&#1082; doSave &#1090;&#1086;&#1083;&#1100;&#1082;&#1086; &#1074; &#1089;&#1091;&#1097;&#1077;&#1089;&#1090;&#1074;&#1091;&#1102;&#1097;&#1080;&#1081;
-        File f = (File)searchKey;
+        File file = (File)searchKey;
         try {
-            doWrite(resume,f);
+            doWrite(resume, new BufferedOutputStream(new FileOutputStream(file)));
         } catch (IOException e) {
-            throw new StorageException("IO error", f.getName(), e);
+            throw new StorageException("File write error", resume.getUuid(), e);
         }
     }
 
@@ -43,18 +44,22 @@ protected boolean isExist(Object file) {
     protected void doSave(Resume resume, Object file) {
         File f =((File)file);
         try {
+
             f.createNewFile();
-            doWrite (resume,f);
         } catch (IOException e) {
-            throw new StorageException("IO error", f.getName(), e);
+            throw new StorageException("Couldn't create file " + f.getAbsolutePath(), f.getName(), e);
         }
+        doUpdate(resume,f);
     }
 
-    protected abstract void doWrite(Resume resume, File f) throws IOException;
-
     @Override
     protected Resume doGet(Object searchKey) {
-         return doRead((File)searchKey);
+        File file = (File) searchKey;
+        try {
+            return doRead(new BufferedInputStream(new FileInputStream(file)));
+        } catch (IOException e) {
+            throw  new StorageException("File read error",file.getName(),e);
+        }
     }
 
     @Override
@@ -78,13 +83,11 @@ protected List<Resume> doCopyAll() {
             return resumes;
         }
             for (File f : directory.listFiles()) {
-                resumes.add(doRead(f));
+                resumes.add(doGet(f));
             }
         return resumes;
     }
 
-    protected abstract Resume doRead(File f);
-
     @Override
     public void clear() {
         //&#1087;&#1086;&#1083;&#1091;&#1095;&#1080;&#1090;&#1100; &#1074;&#1089;&#1077; &#1092;&#1072;&#1081;&#1083;&#1099;&#1080;&#1079; &#1082;&#1072;&#1090;&#1072;&#1083;&#1086;&#1075;&#1072; &#1080; &#1091;&#1076;&#1072;&#1083;&#1080;&#1090;&#1100;
