Comments on: JSP/Java - strip_tags() PHP like function http://www.stratulat.com/blog/jspjava-strip_tags-php-like-function This is my personal blog Fri, 5 Dec 2008 11:30:10 +0000 http://wordpress.org/?v=2.6.5 By: Remus Stratulat http://www.stratulat.com/blog/jspjava-strip_tags-php-like-function#comment-421 Remus Stratulat Tue, 04 Nov 2008 08:06:10 +0000 http://www.stratulat.com/wordpress/?p=56#comment-421 At a first glance it seems this method is removing all allowed tags and keeps the others and that is opposite of what it should do. Please tell me if I am wrong. At a first glance it seems this method is removing all allowed tags and keeps the others and that is opposite of what it should do.

Please tell me if I am wrong.

]]> By: Mauro Gonzalez http://www.stratulat.com/blog/jspjava-strip_tags-php-like-function#comment-420 Mauro Gonzalez Mon, 03 Nov 2008 15:50:24 +0000 http://www.stratulat.com/wordpress/?p=56#comment-420 this should work as well. <div class="csharpcode"><pre class="alt"><span class="lnum"> 1: </span> <span class="kwrd">public</span> <span class="kwrd">static</span> String stripTags(String text, String allowed) {</pre><pre><span class="lnum"> 2: </span> String pattern = <span class="str">""</span>;</pre><pre class="alt"><span class="lnum"> 3: </span> String[] allow = allowed.split(<span class="str">","</span>);</pre><pre><span class="lnum"> 4: </span> <span class="kwrd">if</span> (allow.length > 0) {</pre><pre class="alt"><span class="lnum"> 5: </span> StringBuffer sb = <span class="kwrd">new</span> StringBuffer();</pre><pre><span class="lnum"> 6: </span> sb.append(allow[0]);</pre><pre class="alt"><span class="lnum"> 7: </span> <span class="kwrd">for</span> (<span class="kwrd">int</span> i = 1; i < allow.length; i++) {</pre><pre><span class="lnum"> 8: </span> sb.append(<span class="str">"|"</span>);</pre><pre class="alt"><span class="lnum"> 9: </span> sb.append(allow[i]);</pre><pre><span class="lnum"> 10: </span> }</pre><pre class="alt"><span class="lnum"> 11: </span> pattern = <span class="str">""</span>;</pre><pre><span class="lnum"> 12: </span> } <span class="kwrd">else</span> {</pre><pre class="alt"><span class="lnum"> 13: </span> pattern = <span class="str">""</span>;</pre><pre><span class="lnum"> 14: </span> }</pre><pre class="alt"><span class="lnum"> 15: </span> <span class="kwrd">return</span> text.replaceAll(pattern, <span class="str">""</span>);</pre><pre><span class="lnum"> 16: </span> } </pre></div> this should work as well.

   1:      public static String stripTags(String text, String allowed) {
   2:          String pattern = "";
   3:          String[] allow = allowed.split(",");
   4:          if (allow.length > 0) {
   5:              StringBuffer sb = new StringBuffer();
   6:              sb.append(allow[0]);
   7:              for (int i = 1; i < allow.length; i++) {
   8:                  sb.append("|");
   9:                  sb.append(allow[i]);
  10:              }
  11:              pattern = "";
  12:          } else {
  13:              pattern = "";
  14:          }
  15:          return text.replaceAll(pattern, "");
  16:      }
]]> By: Remus Stratulat [Member] http://www.stratulat.com/blog/jspjava-strip_tags-php-like-function#comment-25 Remus Stratulat [Member] Thu, 22 Jun 2006 10:05:29 +0000 http://www.stratulat.com/wordpress/?p=56#comment-25 Thank you for your observations. I've corrected them out. Thank you for your observations. I’ve corrected them out.

]]> By: David Sickmiller [Visitor] http://www.stratulat.com/blog/jspjava-strip_tags-php-like-function#comment-24 David Sickmiller [Visitor] Thu, 22 Jun 2006 09:22:06 +0000 http://www.stratulat.com/wordpress/?p=56#comment-24 Additionally, two backslashes (\\) need to be added before each occurrence of "s" in the regex pattern. Additionally, two backslashes (\\) need to be added before each occurrence of “s” in the regex pattern.

]]> By: David Sickmiller [Visitor] http://www.stratulat.com/blog/jspjava-strip_tags-php-like-function#comment-23 David Sickmiller [Visitor] Thu, 22 Jun 2006 09:11:34 +0000 http://www.stratulat.com/wordpress/?p=56#comment-23 You have a mistake with a variable name on line 13. It should be changed to this:<br /> <code>if(Arrays.binarySearch(tag_list, tag) < 0) {</code><br /> <br /> It's also worth noting that this implementation is case sensitive. You have a mistake with a variable name on line 13. It should be changed to this:
if(Arrays.binarySearch(tag_list, tag) < 0) {

It’s also worth noting that this implementation is case sensitive.

]]>