From dfb7d34f88efbb3eb7632ae628ccfd4576824477 Mon Sep 17 00:00:00 2001
From: ulrich
Date: Thu, 08 Apr 2021 10:37:10 +0000
Subject: [PATCH] REST-Konformitaet des StorageHandler verbessert

---
 src/de/uhilger/mediaz/api/StorageHandler.java |  182 ++++++++++++++++++++++++++++++++++++--------
 1 files changed, 147 insertions(+), 35 deletions(-)

diff --git a/src/de/uhilger/mediaz/api/StorageHandler.java b/src/de/uhilger/mediaz/api/StorageHandler.java
index 22d9de0..255380f 100644
--- a/src/de/uhilger/mediaz/api/StorageHandler.java
+++ b/src/de/uhilger/mediaz/api/StorageHandler.java
@@ -21,57 +21,88 @@
 import com.sun.net.httpserver.HttpExchange;
 import com.sun.net.httpserver.HttpHandler;
 import de.uhilger.mediaz.App;
+import static de.uhilger.mediaz.App.RB_EP_LISTE;
 import de.uhilger.mediaz.Server;
+import static de.uhilger.mediaz.Server.RB_SLASH;
 import de.uhilger.mediaz.store.FileStorage;
 import de.uhilger.mediaz.entity.Entity;
 import java.io.BufferedReader;
-import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.OutputStream;
+import java.util.List;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
 /**
- * HttpHandler fuer die Ablage von Entitaeten der Mediazentrale
+ * HttpHandler fuer die Verwaltung von Entitaeten der Mediazentrale
  * 
  * @author Ulrich Hilger
  * @version 1, 5.4.2021
  */
-public class StorageHandler implements HttpHandler  {
+public class StorageHandler extends AbstractHandler implements HttpHandler {
   
   private static final Logger logger = Logger.getLogger(StorageHandler.class.getName());
 
   
   /*
   
-    HTTP GET: lies einen Ablageort und schreibe JSON
-    HTTP PUT: schreibe einen neuen Ablageort auf die Platte
-    HTTP POST: schreibe Aenderungen auf die Platte
-    HTTP DELETE: loesche den Ablageort
+    PUT Die angegebene Ressource wird angelegt. Wenn die Ressource bereits 
+           existiert, wird sie geändert. 
+  
+    Das REST-Muster sieht je Entitaet fuenf Faelle vor (Beispiel Ablageort):
+    
+    1. HTTP GET Ablageort/[Name]  - Liefere den Ablageort namens [Name] als JSON
+    2. HTTP GET Ablageort/liste/  - Liefere eine Liste mit Namen aller Ablageorte
+                                     Es duerfen also keine Elemente namens "liste" angelegt werden
+    3. HTTP GET Ablageort/        - Liefere eine Liste aller Ablageort-Objekte als JSON
+    4. HTTP PUT Ablageort/[Name]  - Die angegebene Ressource wird angelegt. Wenn die Ressource 
+                                     bereits existiert, wird sie geändert. 
+    5. HTTP DELETE                - loesche den Ablageort
+    
+  
+  
+    falsch:
+    1. HTTP GET Ablageort/[Name]  - Liefere den Ablageort namens [Name] als JSON
+    2. HTTP GET Ablageort/liste/  - Liefere eine Liste mit Namen aller Ablageorte
+    2. HTTP GET Ablageort/  - Liefere eine Liste aller Ablageort-Objekte als JSON
+    3. HTTP PUT  - schreibe einen neuen Ablageort auf die Platte
+    4. HTTP POST  - schreibe Aenderungen auf die Platte
+    5. HTTP DELETE  - loesche den Ablageort
   
     Beispiele:
   
-    HTTP GET an /mz/api/store/Ablageort/Katalog
-    liest den Ablageort namens "Katalog"
+    HTTP GET an /mz/api/store/Ablageort/
+    liefert alle Ablageort-Objekte
   
-    HTTP POST an /mz/api/store/Ablageort
-    schreibt den neuen Ablageort im Body der Anfrage (Neu)
+    HTTP GET an /mz/api/store/Ablageort/liste/  
+    liefert eine Liste der Namen vorhandener Ablageorte
   
-    HTTP PUT an /mz/api/store/Ablageort
-    sucht den Ablageort mit dem Namen laut Body der Anfrage 
-    und schreibt den Inhalt aus der Anfrage in die Datei (Aenderung)
+    HTTP GET an /mz/api/store/Ablageort/[Name]
+    liest den Ablageort namens [Name]
   
-    HTTP DELETE an /mz/api/store/Ablageort/Katalog
-    löscht den Ablageort namens "Katalog"
+    HTTP PUT an /mz/api/store/Ablageort/[Name]
+    wenn [Name] existiert: Aenderung, sonst neu anlegen
+  
+    HTTP DELETE an /mz/api/store/Ablageort/[Name]
+    löscht den Ablageort namens [Name]
   
   */
   
+  /** Name der HTTP Methode GET */
   public static final String HTTP_GET = "GET";
+  
+  /** Name der HTTP Methode PUT */
   public static final String HTTP_PUT = "PUT";
+  
+  /** Name der HTTP Methode POST */
   public static final String HTTP_POST = "POST";
+  
+  /** Name der HTTP Methode DELETE */
   public static final String HTTP_DELETE = "DELETE";
+  
+  public static final String EP_LISTE = "liste/";
 
   @Override
   public void handle(HttpExchange e) throws IOException {
@@ -90,25 +121,86 @@
         break;
         
       case HTTP_PUT:
-        response = "PUT noch bauen.";
+        response = put(e);
         break;
         
       case HTTP_POST:
-        response = neu(e);
+        response = "nicht unterstuertzt.";
+        code = 404;
         break;
         
       case HTTP_DELETE:
-        response = "DELETE noch bauen.";
+        boolean geloescht = loeschen(e);
+        if(geloescht) {
+          response = "geloescht";
+        } else {
+          response = "nicht geloescht";
+        }
         break;
     }
-    logger.info(response);
+    logger.fine(response);
     e.sendResponseHeaders(code, response.length());
     OutputStream os = e.getResponseBody();
     os.write(response.getBytes());
     os.close();        
   }
   
+  private String put(HttpExchange e) throws IOException {
+    String path = e.getRequestURI().toString();
+    String[] elems = path.split(App.getRs(Server.RB_SLASH));
+    String type = elems[elems.length - 2];
+    String elemName = elems[elems.length - 1]; // alter Name, wenn Aenderung
+    if(!elemName.equalsIgnoreCase(App.getRs(RB_EP_LISTE))) {
+      FileStorage fs = new FileStorage(App.getInitParameter(App.getRs(App.RB_AP_CONF)));
+      Gson gson = new Gson();
+      logger.log(Level.FINE, "type: {0}, token: {1}", new Object[]{type, fs.typeFromName(type).getType().getTypeName()});
+      Object o = gson.fromJson(bodyLesen(e), fs.typeFromName(type).getType());
+      if(o instanceof Entity) {
+        Entity entity = (Entity) o;
+        if(fs.exists(type, elemName)) {
+          fs.delete(type, elemName);
+          fs.write(entity, true);
+        } else {
+          fs.write(entity, false);
+        }
+        return type + App.getRs(Server.RB_SLASH) + entity.getName();
+      } else {
+        return "Ungueltiges Objekt im Body.";
+      }
+    } else {
+      return "Ungueltiger Elementname: " + App.getRs(RB_EP_LISTE);
+    }
+  }
+  
+  /*
+  private String schreiben(FileStorage fs, String typ, String name, String body, boolean overwrite) {
+    Gson gson = new Gson();
+    logger.log(Level.FINE, "type: {0}, token: {1}", new Object[]{typ, fs.typeFromName(typ).getType().getTypeName()});
+    Object o = gson.fromJson(body, fs.typeFromName(typ).getType());
+    if(o instanceof Entity) {
+      Object antwortObjekt = fs.write((Entity) o, overwrite);
+      if(antwortObjekt instanceof File) {
+        File file = (File) antwortObjekt;
+        logger.log(Level.FINE, "Datei {0} geschrieben.", file.getAbsolutePath());
+        String filename = file.getName();
+        return typ + FileHandler.STR_BLANK + filename;
+      }
+    }    
+    return typ + FileHandler.STR_BLANK + " etwas Seltsames ist passiert.";
+  }
+  */
+  /*
   private String neu(HttpExchange e) throws IOException {
+    return schreiben(e, false);
+  }
+  
+  private String aendern(HttpExchange e) throws IOException {
+    return schreiben(e, true);
+  }
+  */
+  
+  /*
+  private String schreiben(HttpExchange e, boolean overwrite) throws IOException {
     String path = e.getRequestURI().toString();
     String[] elems = path.split(App.getRs(Server.RB_SLASH));
     String type = elems[elems.length - 1];
@@ -116,38 +208,58 @@
     String filename = ""; 
     FileStorage fs = new FileStorage(App.getInitParameter(App.getRs(App.RB_AP_CONF)));
     Gson gson = new Gson();
-    logger.log(Level.INFO, "type: {0}", type);
+    logger.log(Level.FINE, "type: {0}, token: {1}", new Object[]{type, fs.typeFromName(type).getType().getTypeName()});
     Object o = gson.fromJson(body, fs.typeFromName(type).getType());
     if(o instanceof Entity) {
-      Object antwortObjekt = fs.write((Entity) o);
+      Object antwortObjekt = fs.write((Entity) o, overwrite);
       if(antwortObjekt instanceof File) {
         File file = (File) antwortObjekt;
-        logger.log(Level.INFO, "Datei {0} geschrieben.", file.getAbsolutePath());
+        logger.log(Level.FINE, "Datei {0} geschrieben.", file.getAbsolutePath());
         filename = file.getName();
+        return type + FileHandler.STR_BLANK + filename;
       }
     }
-    return type + FileHandler.STR_BLANK + filename;
+    return type + FileHandler.STR_BLANK + " Operation 'neu' fuer bereits existierende Entitaet.";
   }
+  */
   
-  private void aendern() {
-    
+  /*
+  private String aendern(HttpExchange e) throws IOException {
+    return neu(e); // einstweilen wird einfach ueberschrieben
   }
+  */
   
-  private void loeschen() {
-    
-  }
-  
-  private String lesen(HttpExchange e) {
+  private boolean loeschen(HttpExchange e) {
     String path = e.getRequestURI().toString();
     String[] elems = path.split(App.getRs(Server.RB_SLASH));
     String type = elems[elems.length - 2];
     String elemName = elems[elems.length - 1];
     FileStorage fs = new FileStorage(App.getInitParameter(App.getRs(App.RB_AP_CONF)));
-    //Entity entity = fs.read(type, elemName);
-    //return entity;
-    return fs.readJson(type, elemName);
+    return fs.delete(type, elemName);
   }
   
+  private String lesen(HttpExchange e) {
+    String path = e.getRequestURI().toString();
+    String[] elems = path.split(App.getRs(Server.RB_SLASH));
+    FileStorage fs = new FileStorage(App.getInitParameter(App.getRs(App.RB_AP_CONF)));
+    if(path.endsWith(App.getRs(RB_SLASH))) {
+      List list = null;
+      if(elems[elems.length - 1].equalsIgnoreCase(App.getRs(RB_EP_LISTE))) {
+        String type = elems[elems.length - 2];
+        logger.fine(type);
+        list = fs.list(type);
+      } else {
+        String type = elems[elems.length - 1];
+        logger.fine(type);
+        list = fs.listObjects(type);
+      }
+      return jsonWithEnclosingType(list);
+    } else {
+      String type = elems[elems.length - 2];
+      String elemName = elems[elems.length - 1];
+      return fs.readJson(type, elemName);
+    }
+  }
   
   private String bodyLesen(HttpExchange e) throws IOException {
     InputStream is = e.getRequestBody();
@@ -160,7 +272,7 @@
     }
     r.close();
     String json = sb.toString();
-    logger.log(Level.INFO, "json: {0}", json);
+    logger.log(Level.FINE, "json: {0}", json);
     return json;
   }
 }

--
Gitblit v1.9.3