Sign in to follow this  

[web] PHP alternative to mysql_real_escape_string

This topic is 3859 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

I need to create a PHP function that does the same thing as mysql_real_escape_string. The reason being that I need to use this function more than I actually need to connect to the database. According to the PHP Manual Page, mysql_real_escape_string escapes the following characters: \x00, \n, \r, \, ', " and \x1a. Is this an equivalent function? function my_real_escape_string($value) { $search = array("\x00", "\n", "\r", "\\", "'", "\"", "\x1a"); $replace = array("\\x00", "\\n", "\\r", "\\\\" ,"\'", "\\\"", "\\\x1a"); return str_replace($search, $replace, $value); } Or is there a reason this isn't safe, etc?

Share this post


Link to post
Share on other sites
Nope, looks good.

PS: You can save yourself a few headaches writing that by using single quoted strings. They don't need to be escaped by backslashes. I.e:


$search = array("\x00", "\n", "\r", '\', "'", '"', "\x1a");
$replace = array('\x00', '\n', '\r', '\\' ,"\'", '\"', '\x1a');


That's especially useful if you're dealing with multiple levels of backslash escaping (e.g. in some regular expessions) where you would otherwise end up with madness like "\\\\\\\\" for a simple backslash :-)

Share this post


Link to post
Share on other sites
Thanks Sander,

I'm having a little bit of trouble with the new code. In the top line you have '\' and it's causing an error as it thinks there is an open ended string. An elegant solution? Or must I replace '\\' with '\\\\'?

Share this post


Link to post
Share on other sites
Oh, ofcourse. Silly me.

In a single quoted string, only the ' needs to be escaped. Not the rest like in double quoted strings. So the '\' should be '\\'. It's very easy to test. If you echo('\n') you will get \n as output, but when you echo("\n") you will get a newline.

See http://www.php.net/manual/en/language.types.string.php and compare single quoted strings to double quoted strings. To revise my example:


$search = array("\x00", "\n", "\r", '\\', "'", '"', "\x1a");
$replace = array('\x00', '\n', '\r', '\\\' ,"\'", '\"', '\x1a');


It does exactly the same as your code does, but IMHO it's a bit easier on the eyes. Whatever floats your boat I guess :-)

Share this post


Link to post
Share on other sites
Thanks again, and for the link. The echo() example really made things clear. I've always assumed that single and double quoted strings worked the same... now I know different. I didn't mean to sound picky about your code. I always appreciate smaller more readable code since that's what keeps my boat from sinking. [smile]

Share this post


Link to post
Share on other sites
That method won't work.

Let's say someone enters a NULL character. You want it to be stored in the string as \x00. However, your code will do this:

NULL => \x00
\x00 => \\x00

This is because calling str_replace with an array is no different from calling it multiple times. You need to use strtr.


return strtr($text, array(
"\x00" => '\x00',
"\n" => '\n',
"\r" => '\r',
'\\' => '\\\\',
"'" => "\'",
'"' => '\"',
"\x1a" => '\x1a'
));



strtr translates the biggest strings first and doesn't translates text that has already been translated. (You may want to double check that the above table is accurate.)

As an aside, I don't really recommend using this method at all. I much prefer using PDO and prepared statements.

Share this post


Link to post
Share on other sites
Quote:
I much prefer using PDO and prepared statements.


The OP doesn't want to use this in combination with a database. That's why he needs to roll his own.

Share this post


Link to post
Share on other sites
Quote:
The OP doesn't want to use this in combination with a database. That's why he needs to roll his own.
Yes I do, sometimes, want to use this for database queries. Let me explain a little..

I have my database function wrapped in a class for caching purposes.

class->sql_query($query); // Simple example

The query may contain data that has been escaped with mysql_real_escape_string.

The class uses the query string to query the database and to create an ID for caching events.

A database connection is only made when data is not found in the cache. So it seems a waste to make a connection when it is only needed for mysql_real_escape_string.

I was hoping to make a replacement function for mysql_real_escape_string to avoid unnecessary connections to my mysql server.

So should I use the strstr to implement this function? What are the dangers?

Share this post


Link to post
Share on other sites
Ah. In that case try lazy evaluation. If the data is in the cache then you do not need to query the database. If you do not query the batabase, you do not need to escape variables. Ergo: You only need mysql_real_escape_string() because *if* you escape something, you have a database connection.

This probably means that you need to modify your wrapper class to accept something like:

Quote:
$wrapper->sql_query('SELECT * FROM table WHERE col1=?, col2=?', $var1, $var2);


The sql_query() function checks the cache. If it's not there, make a database connection, escape $var1 and $var2, insert then in the query, run the query and cache the result.

Share this post


Link to post
Share on other sites

This topic is 3859 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this