From: Kai Moritz <kai@coolibri.de>
Date: Wed, 3 Aug 2011 16:45:19 +0000 (+0200)
Subject: Optimierung in CacheControlInterceptor: unnötige Exception vermieden
X-Git-Tag: 1.0.1~3
X-Git-Url: http://juplo.de/gitweb/?a=commitdiff_plain;h=a88c4289488b6d238ae4c3c6d9b1e559735178c4;p=percentcodec

Optimierung in CacheControlInterceptor: unnötige Exception vermieden
---

diff --git a/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/CacheControlInterceptor.java b/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/CacheControlInterceptor.java
index 641bda34..a830e3c6 100644
--- a/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/CacheControlInterceptor.java
+++ b/cachecontrol/src/main/java/de/halbekunst/juplo/cachecontrol/CacheControlInterceptor.java
@@ -30,172 +30,167 @@ public class CacheControlInterceptor implements HandlerInterceptor {
 
   @Override
   public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception {
-    try {
-      Cacheable cacheable = (Cacheable)handler;
+    if (!(handler instanceof Cacheable))
+      return true;
 
-      long now = System.currentTimeMillis();
+    Cacheable cacheable = (Cacheable) handler;
 
-      /**
-       * Alle Antworten (insbesondere auch 304) sollen nach dem {@plainlink
-       * http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.18 RFC 2616,
-       * Abschnitt 14.18} einen Date-Header enthalten
-       */
-      response.setDateHeader(HEADER_DATE, now);
+    long now = System.currentTimeMillis();
 
-      /** Prüfen, ob der Handler willig ist, den Request zu verarbeiten */
-      if (!cacheable.accepts(request)) {
-        response.sendError(HttpServletResponse.SC_NOT_FOUND);
-        return false;
-      }
+    /**
+     * Alle Antworten (insbesondere auch 304) sollen nach dem {@plainlink
+     * http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.18 RFC 2616,
+     * Abschnitt 14.18} einen Date-Header enthalten
+     */
+    response.setDateHeader(HEADER_DATE, now);
 
-      /** Nichts weiter unternehmen, wenn der Handler dies nicht will */
-      if (!cacheable.isGenerateCacheHeaders(request))
-        return true;
-
-      String url = null;
-      if (log.isDebugEnabled()) {
-        if (request.getQueryString() == null) {
-          url = request.getRequestURI();
-        }
-        else {
-          StringBuilder builder = new StringBuilder();
-          builder.append(request.getRequestURI());
-          builder.append('?');
-          builder.append(request.getQueryString());
-          url = builder.toString();
-        }
-      }
+    /** Prüfen, ob der Handler willig ist, den Request zu verarbeiten */
+    if (!cacheable.accepts(request)) {
+      response.sendError(HttpServletResponse.SC_NOT_FOUND);
+      return false;
+    }
 
-      int cacheSeconds = cacheable.getCacheSeconds(request);
-      if (cacheSeconds < 0) {
-        log.debug("{}: caching disabled!", url);
-        response.setDateHeader(HEADER_DATE, now);
-        response.setDateHeader(HEADER_EXPIRES, 0);
-        response.addHeader(HEADER_PRAGMA, "no-cache");
-        response.addHeader(HEADER_CACHE_CONTROL, "private");
-        response.addHeader(HEADER_CACHE_CONTROL, "no-cache");
-        response.addHeader(HEADER_CACHE_CONTROL, "no-store");
-        response.addHeader(HEADER_CACHE_CONTROL, "max-age=0");
-        response.addHeader(HEADER_CACHE_CONTROL, "s-max-age=0");
-        return true;
-      }
+    /** Nichts weiter unternehmen, wenn der Handler dies nicht will */
+    if (!cacheable.isGenerateCacheHeaders(request))
+      return true;
 
-      long ifModifiedSince = -1;
-      try {
-        ifModifiedSince = request.getDateHeader(HEADER_IF_MODIFIED_SINCE);
+    String url = null;
+    if (log.isDebugEnabled()) {
+      if (request.getQueryString() == null) {
+        url = request.getRequestURI();
       }
-      catch (Exception e) {
-        log.error("Exception while fetching If-Modified-Since: {}", e);
+      else {
+        StringBuilder builder = new StringBuilder();
+        builder.append(request.getRequestURI());
+        builder.append('?');
+        builder.append(request.getQueryString());
+        url = builder.toString();
       }
+    }
 
-      long lastModified = cacheable.getLastModified(request);
+    int cacheSeconds = cacheable.getCacheSeconds(request);
+    if (cacheSeconds < 0) {
+      log.debug("{}: caching disabled!", url);
+      response.setDateHeader(HEADER_DATE, now);
+      response.setDateHeader(HEADER_EXPIRES, 0);
+      response.addHeader(HEADER_PRAGMA, "no-cache");
+      response.addHeader(HEADER_CACHE_CONTROL, "private");
+      response.addHeader(HEADER_CACHE_CONTROL, "no-cache");
+      response.addHeader(HEADER_CACHE_CONTROL, "no-store");
+      response.addHeader(HEADER_CACHE_CONTROL, "max-age=0");
+      response.addHeader(HEADER_CACHE_CONTROL, "s-max-age=0");
+      return true;
+    }
 
-      /**
-       * Sicherstellen, dass der Wert keine Millisekunden enthält, da die
-       * Zeitangabe aus dem Modified-Since-Header keine Millisekunden enthalten
-       * kann und der Test unten dann stets fehlschlagen würde!
-       */
-      lastModified = lastModified - (lastModified % 1000);
+    long ifModifiedSince = -1;
+    try {
+      ifModifiedSince = request.getDateHeader(HEADER_IF_MODIFIED_SINCE);
+    }
+    catch (Exception e) {
+      log.error("Exception while fetching If-Modified-Since: {}", e);
+    }
 
-      String ifNoneMatch = request.getHeader(HEADER_IF_NONE_MATCH);
-      String eTag = cacheable.getETag(request);
+    long lastModified = cacheable.getLastModified(request);
 
-      /**
-       * 304-Antworten sollen nach dem {@plainlink
-       * http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.3.5 RFC
-       * 2616, Abschnitt 10.3.5} einen ETag-Header enthalten, wenn auch die
-       * 200-Antwort einen enthalten hätte.
-       */
-      if (eTag != null) {
-        response.setHeader(HEADER_ETAG, eTag);
-      }
+    /**
+     * Sicherstellen, dass der Wert keine Millisekunden enthält, da die
+     * Zeitangabe aus dem Modified-Since-Header keine Millisekunden enthalten
+     * kann und der Test unten dann stets fehlschlagen würde!
+     */
+    lastModified = lastModified - (lastModified % 1000);
 
+    String ifNoneMatch = request.getHeader(HEADER_IF_NONE_MATCH);
+    String eTag = cacheable.getETag(request);
 
-      if (ifModifiedSince >= lastModified && lastModified > 0) {
-        /**
-         * request.getDateHeader liefert die Zeit als long, oder -1, wenn der
-         * Header nicht existiert. D.h., wenn "If-Modified-Since" nicht gesetzt
-         * ist, wird die komplette Seite ausgeliefert.
-         * Der zusätzliche Test, ob lastModified größer 0 ist, ist nötig, um
-         * Fehler auszuschließen, wenn die Implementierung von Cachable
-         * negative Werte für Last-Modified zurückliefert.
-         */
-        if (log.isDebugEnabled())
-          log.debug("{}: Not modified since {}", url, new Date(ifModifiedSince));
-
-        if (ifNoneMatch == null) {
-          /** Neue Anfrage oder HTTP/1.0 Client! */
-          log.debug("{}: ETag nicht gesetzt -> 304", url);
-          response.setStatus(HttpServletResponse.SC_NOT_MODIFIED);
-          return false;
-        }
-      }
+    /**
+     * 304-Antworten sollen nach dem {@plainlink
+     * http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.3.5 RFC
+     * 2616, Abschnitt 10.3.5} einen ETag-Header enthalten, wenn auch die
+     * 200-Antwort einen enthalten hätte.
+     */
+    if (eTag != null)
+      response.setHeader(HEADER_ETAG, eTag);
+
+
+    if (ifModifiedSince >= lastModified && lastModified > 0) {
+      /**
+       * request.getDateHeader liefert die Zeit als long, oder -1, wenn der
+       * Header nicht existiert. D.h., wenn "If-Modified-Since" nicht gesetzt
+       * ist, wird die komplette Seite ausgeliefert.
+       * Der zusätzliche Test, ob lastModified größer 0 ist, ist nötig, um
+       * Fehler auszuschließen, wenn die Implementierung von Cachable
+       * negative Werte für Last-Modified zurückliefert.
+       */
+      if (log.isDebugEnabled())
+        log.debug("{}: Not modified since {}", url, new Date(ifModifiedSince));
 
-      if (ifNoneMatch != null && ifNoneMatch.equals(eTag)) {
-        log.debug("{}: ETag {} not changed -> 304 ", url, ifNoneMatch);
+      if (ifNoneMatch == null) {
+        /** Neue Anfrage oder HTTP/1.0 Client! */
+        log.debug("{}: ETag nicht gesetzt -> 304", url);
         response.setStatus(HttpServletResponse.SC_NOT_MODIFIED);
         return false;
       }
+    }
+
+    if (ifNoneMatch != null && ifNoneMatch.equals(eTag)) {
+      log.debug("{}: ETag {} not changed -> 304 ", url, ifNoneMatch);
+      response.setStatus(HttpServletResponse.SC_NOT_MODIFIED);
+      return false;
+    }
 
 
-      log.debug("{}: first up!", url);
+    log.debug("{}: first up!", url);
 
-      /** HTTP/1.1-Caching-Header richtig setzen!! */
-      response.setDateHeader(HEADER_LAST_MODIFIED, lastModified);
+    /** HTTP/1.1-Caching-Header richtig setzen!! */
+    response.setDateHeader(HEADER_LAST_MODIFIED, lastModified);
 
-      /** Cache-Control für HTTP/1.1-Clients generieren */
-      Map<String, String> cacheControl = new HashMap<String, String>(cacheable.getCacheControl(request));
+    /** Cache-Control für HTTP/1.1-Clients generieren */
+    Map<String, String> cacheControl = new HashMap<String, String>(cacheable.getCacheControl(request));
 
+    /**
+     * Wenn eins JSESSIONID in der URL enthalten ist, darf die Anfrage nur vom
+     * Browser gecached werden!
+     */
+    if (request.isRequestedSessionIdFromURL())
+      cacheControl.put("private", null);
+
+    if (cacheControl.containsKey("private")) {
       /**
-       * Wenn eins JSESSIONID in der URL enthalten ist, darf die Anfrage nur vom
-       * Browser gecached werden!
+       * HTTP/1.0 Caches davon abhalten, die Ressource zu cachen (vgl.: RFC
+       * 2616, {@plainlink
+       * http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9.3
+       * Abschnitt 14.9.3} und {@plainlink
+       * http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.32
+       * Abschnitt 14.32})
        */
-      if (request.isRequestedSessionIdFromURL()) {
-        cacheControl.put("private", null);
-      }
-
-      if (cacheControl.containsKey("private")) {
-        /**
-         * HTTP/1.0 Caches davon abhalten, die Ressource zu cachen (vgl.: RFC
-         * 2616, {@plainlink
-         * http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9.3
-         * Abschnitt 14.9.3} und {@plainlink
-         * http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.32
-         * Abschnitt 14.32})
-         */
-        response.setDateHeader(HEADER_EXPIRES, 0l);
-        response.addHeader(HEADER_PRAGMA, "no-cache");
-      }
-      else {
-        /**
-         * Hier muss nicht geprüft werden, ob cacheSeconds > 0 gilt, da in diesem
-         * Fall oben bereits No-Cache-Header generiert und <code>false</code>
-         * zurückgeliefert werden!
-         *
-         * Den Wert als <code>max-age</code> zu den Schlüssel-Wert-Paaren für den
-         * <code>Cache-Control</code>-Header hinzufügen und einen entsprechenden
-         * <code>Expires</code>-Header für HTTP/1.0-Clients setzen.
-         */
-        cacheControl.put("max-age", Integer.toString(cacheSeconds));
-        response.setDateHeader(HEADER_EXPIRES, (now + (long)cacheSeconds * 1000));
-      }
+      response.setDateHeader(HEADER_EXPIRES, 0l);
+      response.addHeader(HEADER_PRAGMA, "no-cache");
+    } else {
+      /**
+       * Hier muss nicht geprüft werden, ob cacheSeconds > 0 gilt, da in diesem
+       * Fall oben bereits No-Cache-Header generiert und <code>false</code>
+       * zurückgeliefert werden!
+       *
+       * Den Wert als <code>max-age</code> zu den Schlüssel-Wert-Paaren für den
+       * <code>Cache-Control</code>-Header hinzufügen und einen entsprechenden
+       * <code>Expires</code>-Header für HTTP/1.0-Clients setzen.
+       */
+      cacheControl.put("max-age", Integer.toString(cacheSeconds));
+      response.setDateHeader(HEADER_EXPIRES, (now + (long) cacheSeconds * 1000));
+    }
 
-      StringBuilder builder = new StringBuilder();
-      for (Entry<String, String> entry : cacheControl.entrySet()) {
-        builder.setLength(0);
-        builder.append(entry.getKey());
-        if (entry.getValue() != null) {
-          builder.append('=');
-          builder.append(entry.getValue());
-        }
-        response.addHeader(HEADER_CACHE_CONTROL, builder.toString());
+    StringBuilder builder = new StringBuilder();
+    for (Entry<String, String> entry : cacheControl.entrySet()) {
+      builder.setLength(0);
+      builder.append(entry.getKey());
+      if (entry.getValue() != null) {
+        builder.append('=');
+        builder.append(entry.getValue());
       }
-
-      return true;
-    }
-    catch (ClassCastException e) {
-      return true;
+      response.addHeader(HEADER_CACHE_CONTROL, builder.toString());
     }
+
+    return true;
   }
 
   @Override