Исправлены ошибки

ageley · web-flow · commit 42fa14dd008f · 2019-11-18T11:50:13.000+03:00
- добавлено поле size для хранения количества резюме в базе
- в методе getAll возвращаемый массив переименован в resumes
- в методе delete удаляется последнее добавленное резюме, исправлена ошибка выхода за границы массива
- в методе save добавлена проверка максимальной длины массива, если база переполнена то новое резюме не сохраняется
diff --git a/src/ArrayStorage.java b/src/ArrayStorage.java
@@ -2,24 +2,24 @@
  * Array based storage for Resumes
  */
 public class ArrayStorage {
-    Resume[] storage = new Resume[10000];
+    Resume[] storage = new Resume[3];
+    private int size = size();
 
     void clear() {
-        int size = this.size();
         for (int i = 0; i < size; i++) {
             storage[i] = null;
         }
+        size = 0;
     }
 
     void save(Resume r) {
-        int size = this.size();
-        if (r.uuid != null) {
+        if (r.uuid != null & size < storage.length) {
             storage[size] = r;
+            size++;
         }
     }
 
     Resume get(String uuid) {
-        int size = this.size();
         if (uuid != null) {
             for (int i = 0; i < size; i++) {
                 if (storage[i].uuid.equals(uuid)) {
@@ -31,13 +31,13 @@ Resume get(String uuid) {
     }
 
     void delete(String uuid) {
-        int size = this.size();
         if (uuid != null) {
-            for (int i = 0; i < size; i++) {
+            for (int i = size - 1; i >= 0; i--) {
                 if (storage[i].uuid.equals(uuid)) {
-                    for (int k = i; k < size; k++) {
+                    for (int k = i; k < size - 1; k++) {
                         storage[k] = storage[k+1];
                     }
+                    size--;
                     break;
                 }
             }
@@ -48,12 +48,11 @@ void delete(String uuid) {
      * @return array, contains only Resumes in storage (without null)
      */
     Resume[] getAll() {
-        int size = this.size();
-        Resume[] storageAll = new Resume[size];
+        Resume[] resumes = new Resume[size];
         for (int i = 0; i < size; i++) {
-            storageAll[i] = storage[i];
+            resumes[i] = storage[i];
         }
-        return storageAll;
+        return resumes;
     }
 
     int size() {
