PHP Classes

I would replace preg_split for str_split to get some better e...

Recommend this page to a friend!

      Saff Request  >  All threads  >  I would replace preg_split for...  >  (Un) Subscribe thread alerts  
Subject:I would replace preg_split for...
Summary:Package rating comment
Messages:2
Author:RicardoSaracino
Date:2011-04-18 12:45:31
Update:2011-04-19 13:22:04
 

 

RicardoSaracino rated this package as follows:

Utility: Sufficient
Consistency: Sufficient
Examples: Good

  1. I would replace preg_split for...   Reply   Report abuse  
Picture of RicardoSaracino RicardoSaracino - 2011-04-18 12:45:31
I would replace preg_split for str_split to get some better efficiency from the __get

mysql_real_escape_string is a bad idea as it depends on a mysql installation. perhaps use addslashes to keep it from breaking on other systems or have way to register a callback for escaping using addslashes as the default

  2. Re: I would replace preg_split for...   Reply   Report abuse  
Picture of Zulfugar Ismayilzadeh Zulfugar Ismayilzadeh - 2011-04-19 13:22:04 - In reply to message 1 from RicardoSaracino
I'll use explode in the bext versions of this class.
We used mysql_real_escape_string, because we have mysql platform. In next version i'll check, if mysql platform exists we'll use mysql_real_escape_string, else addslashes.