Changeset 12919

Timestamp:

03/01/09 13:53:55 (3 years ago)

Author:

sys

Message:

Optimisations algo encode Url

File:

• trunk/Jav_FW/com/scenari/s/fw/utils/HUrl.java (modified) (8 diffs)

trunk/Jav_FW/com/scenari/s/fw/utils/HUrl.java

@@ -39 +39 @@
 package com.scenari.s.fw.utils;
 
-import java.io.ByteArrayOutputStream;
-import java.io.IOException;
-import java.io.OutputStreamWriter;
+import java.io.CharArrayWriter;
 import java.io.UnsupportedEncodingException;
 import java.net.URLDecoder;
+import java.net.URLEncoder;
+import java.nio.charset.Charset;
+import java.nio.charset.IllegalCharsetNameException;
+import java.nio.charset.UnsupportedCharsetException;
 import java.util.BitSet;
 
@@ -75 +77 @@
          * Instancie une URL avec ses paramètres "aa/bb/cc?p=v1&pp=v2".
          */
-        public HUrl(String pUrlAvecParams) throws Exception{
+        public HUrl(String pUrlAvecParams) throws Exception {
                 super();
                 hInit(pUrlAvecParams);
@@ -162 +164 @@
          * 
          */
-        public void hInit(String pUrlAvecParams) throws Exception{
+        public void hInit(String pUrlAvecParams) throws Exception {
                 if (pUrlAvecParams != null) {
                         int vOffs = pUrlAvecParams.indexOf('?');
@@ -185 +187 @@
          * @param pParams java.lang.String
          */
-        public void hInit(String pUrlSansParams, String pParams) throws Exception{
+        public void hInit(String pUrlSansParams, String pParams) throws Exception {
                 if (pUrlSansParams == null) {
                         fUrl = "";
@@ -373 +375 @@
         /**
          * 
-         * Encode tous les caractères sauf les "/" et ":". Les espaces sont transformés
+         * Encode tous les caractères sauf les "/" et ":". 
+         * Correposnd à encodeURI() en JS.
+         * Les espaces sont transformés
          * en "%20" et non en "+". (le client webdav Windows ne gère pas le +).
          */
-        public static String hEncode(String s, String enc) throws UnsupportedEncodingException {
+        public static String hEncode(CharSequence s, String enc) throws UnsupportedEncodingException {
                 return xEncode(s, enc, sDontNeedEncodingUrl);
         }
@@ -382 +386 @@
         /**
          * 
-         * Encode tous les caractères sauf les "/" et ":". Les espaces sont transformés
+         * Encode tous les caractères. 
+         * Correposnd à encodeURIComponent() en JS.
+         * Les espaces sont transformés
          * en "%20" et non en "+". (le client webdav Windows ne gère pas le +).
          */
-        public static String hEncodeParam(String s, String enc) throws UnsupportedEncodingException {
+        public static String hEncodeParam(CharSequence s, String enc) throws UnsupportedEncodingException {
                 return xEncode(s, enc, sDontNeedEncodingParam);
         }
 
         /**
-         * 
-         * Encode tous les caractères qui ne sont pas dans le tableau pDontNeedToEncode. 
-         * Les espaces sont transformés en "%20" et non en "+". (le client webdav Windows ne gère pas le +).
-         */
-        public static String xEncode(String s, String enc, BitSet pDontNeedToEncode) throws UnsupportedEncodingException {
-
-                boolean needToChange = false;
-                boolean wroteUnencodedChar = false;
-                int maxBytesPerChar = 10; // rather arbitrary limit, but safe for now
-                StringBuilder out = new StringBuilder(s.length());
-                ByteArrayOutputStream buf = new ByteArrayOutputStream(maxBytesPerChar);
-
-                OutputStreamWriter writer = new OutputStreamWriter(buf, enc);
-
-                for (int i = 0; i < s.length(); i++) {
-                        int c = s.charAt(i);
+         * <p>Encode tous les caractères qui ne sont pas dans le tableau pDontNeedToEncode. 
+         * Les espaces sont transformés en "%20" et non en "+". (le client webdav Windows ne gère pas le +).</p>
+         * 
+         * <p>Par rapport à l'implémentation java {@link URLEncoder}, cette version est plus otptimisée si
+         * statistiquement les cas de présence de caractères à encoder (en %XX) sont faibles 
+         * (instanciation juste à temps des buffers et objets d'encodings).</p>
+         */
+        public static String xEncode(CharSequence s, String enc, BitSet pDontNeedToEncode) throws UnsupportedEncodingException {
+                if (enc == null) throw new NullPointerException("charsetName");
+
+                StringBuffer out = null;
+                CharArrayWriter charArrayWriter = null;
+                Charset charset = null;
+
+                for (int i = 0; i < s.length();) {
+                        int c = (int) s.charAt(i);
                         //System.out.println("Examining character: " + c);
                         if (pDontNeedToEncode.get(c)) {
                                 // SYS 2003-03-09. Mise en commentaire pour transformer l'espace en %20
-                                //if (c == ' ') {
-                                //      c = '+';
-                                //      needToChange = true;
-                                //}
+                                //                if (c == ' ') {
+                                //                    c = '+';
+                                //                    needToChange = true;
+                                //                }
                                 //System.out.println("Storing: " + c);
-                                out.append((char) c);
-                                wroteUnencodedChar = true;
+                                if (out != null) out.append((char) c);
+                                i++;
                         } else {
+                                if (out == null) {
+                                        //1er car à encoder détecté
+                                        try {
+                                                charset = Charset.forName(enc);
+                                        } catch (IllegalCharsetNameException e) {
+                                                throw new UnsupportedEncodingException(enc);
+                                        } catch (UnsupportedCharsetException e) {
+                                                throw new UnsupportedEncodingException(enc);
+                                        }
+                                        out = new StringBuffer(s.length() + 32);
+                                        if (i > 0) out.append(s, 0, i);
+                                        charArrayWriter = new CharArrayWriter();
+                                }
                                 // convert to external encoding before hex conversion
-                                try {
-                                        if (wroteUnencodedChar) { // Fix for 4407610
-                                                writer = new OutputStreamWriter(buf, enc);
-                                                wroteUnencodedChar = false;
-                                        }
-                                        writer.write(c);
+                                do {
+                                        charArrayWriter.write(c);
                                         /*
                                          * If this character represents the start of a Unicode
                                          * surrogate pair, then pass in two characters. It's not
-                                         * clear what should be done if a bytes reserved in the 
+                                         * clear what should be done if a bytes reserved in the
                                          * surrogate pairs range occurs outside of a legal
-                                         * surrogate pair. For now, just treat it as if it were 
+                                         * surrogate pair. For now, just treat it as if it were
                                          * any other character.
                                          */
                                         if (c >= 0xD800 && c <= 0xDBFF) {
                                                 /*
-                                                 System.out.println(Integer.toHexString(c) 
-                                                 + " is high surrogate");
-                                                 */
+                                                  System.out.println(Integer.toHexString(c)
+                                                  + " is high surrogate");
+                                                */
                                                 if ((i + 1) < s.length()) {
-                                                        int d = s.charAt(i + 1);
+                                                        int d = (int) s.charAt(i + 1);
                                                         /*
-                                                         System.out.println("\tExamining " 
-                                                         + Integer.toHexString(d));
-                                                         */
+                                                          System.out.println("\tExamining "
+                                                          + Integer.toHexString(d));
+                                                        */
                                                         if (d >= 0xDC00 && d <= 0xDFFF) {
                                                                 /*
-                                                                 System.out.println("\t" 
-                                                                 + Integer.toHexString(d) 
-                                                                 + " is low surrogate");
-                                                                 */
-                                                                writer.write(d);
+                                                                  System.out.println("\t"
+                                                                  + Integer.toHexString(d)
+                                                                  + " is low surrogate");
+                                                                */
+                                                                charArrayWriter.write(d);
                                                                 i++;
                                                         }
                                                 }
                                         }
-                                        writer.flush();
-                                } catch (IOException e) {
-                                        buf.reset();
-                                        continue;
-                                }
-                                byte[] ba = buf.toByteArray();
+                                        i++;
+                                } while (i < s.length() && !pDontNeedToEncode.get((c = (int) s.charAt(i))));
+
+                                charArrayWriter.flush();
+                                String str = new String(charArrayWriter.toCharArray());
+                                byte[] ba = str.getBytes(charset.name());
                                 for (int j = 0; j < ba.length; j++) {
                                         out.append('%');
@@ -475 +490 @@
                                         out.append(ch);
                                 }
-                                buf.reset();
-                                needToChange = true;
-                        }
-                }
-
-                return (needToChange ? out.toString() : s);
+                                charArrayWriter.reset();
+                        }
+                }
+
+                return (out != null ? out.toString() : s.toString());
         }
 
@@ -525 +539 @@
                 return decodeURI(pUri, pOffset, pOffsetEnd, pEncoding, new StringBuilder(pOffsetEnd - pOffset));
         }
-        
+
         /**
          * Decode une URL.