PHP preg_replace is not matching entire pattern

Question

I'm stuck trying to get the PHP preg_replace to work properly. I want to find all matches of a pattern and replace them with a string. But, for some reason, it's finding only partial matches and replacing all of them. I'm trying to remove the "password" from every line of a text file. The password is always at the end of each line, contains 4 to 8 alpha-numeric characters, and always follows two pipe characters.

Example:

$data = 'A00000001|A00000001|FirstName|LastName|email@address|Role||password'.PHP_EOL;
$data .= 'B00000002|B00000002|FirstName|LastName|email@address|Role||password'.PHP_EOL;
$delim = '|';

$newData = preg_replace("/".$delim.$delim."[a-zA-Z0-9]{4,8}/", $delim.$delim, $data);
echo $newData;

Output:

||||||1|||||||||1|||||||||e||||||||||||e||m||a||i||l||@||a||d||d||r||e||s||s|||||R||o||l||e|||||||||||| ||||||2|||||||||2|||||||||e||||||||||||e||m||a||i||l||@||a||d||d||r||e||s||s|||||R||o||l||e|||||||||||| ||

I've tried many variations with different groupings using parenthesis, putting back to back [a-zA-Z0-9] patterns instead of {#}. I've tried adding line start ^ and end $ to my pattern. I'm stuck. I know this will end up being something simple to that I'm just overlooking. That's why I need some fresh eyes on this.

Relevant info: stackoverflow.com/questions/1531456/…

Tim Pietzcker
– Tim Pietzcker

2013-06-26 15:42:48 +00:00
Commented Jun 26, 2013 at 15:42 — Tim Pietzcker
– Tim Pietzcker, Commented Jun 26, 2013 at 15:42

Anirudha · Accepted Answer · 2013-06-26 15:41:54Z

3

You should use this regex

/(?<=\|\|)[a-zA-Z]{4,8}$/

You need to escape | since it represents OR in regex

$ marks the end of string

(?<=\|\|) is a zero width lookbehind

answered Jun 26, 2013 at 15:41

Anirudha

32.9k8 gold badges71 silver badges90 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Swede Over a year ago

When I used your regex pattern, it only removed the "password" in the 2nd line. I'm assuming this is due to the lookbehind portion. You're correct about escaping the pipe. I knew it would be something simple. Thank you.

Adam · Accepted Answer · 2013-06-26 15:48:59Z

0

Looks like you can just escape your delimiters.

$newData = preg_replace('/\'.$delim.'\'.$delim.'[a-zA-Z0-9]{4,8}/', $delim.$delim, $data);

answered Jun 26, 2013 at 15:48

Adam

8024 silver badges11 bronze badges

1 Comment

Swede Over a year ago

Perfect, thanks. Although, I had to use double escapes with my double-quoted code: $newData = preg_replace("/\\".$delim."\\".$delim."[a-zA-Z0-9]{4,8}/", $delim.$delim, $data);

Denis de Bernardy · Accepted Answer · 2013-06-26 15:48:16Z

0

I'm trying to remove the "password" from every line of a text file.

In this case, anchor the regex after properly escaping your delimiter. Assuming the delim shouldn't be kept either, you could use:

preg_replace('/\|.*?$/', '', $data);

If it should, use a look-behind or:

preg_replace('/\|.*?$/', '|', $data);

On a separate note: this looks like an SQL dump or a CSV file. If so, consider using whichever variation of COPY ... DELIMITER ... your RDBMS offers instead:

http://www.postgresql.org/docs/current/static/sql-copy.html

You could then create a temporary table, import, drop the column, do whatever else you need to do, and populate the final tables as needed once you're done.

answered Jun 26, 2013 at 15:48

Denis de Bernardy

79.1k14 gold badges138 silver badges158 bronze badges

1 Comment

Swede Over a year ago

Thanks Denis. I fixed it by escaping the pipe char. This is manipulating a csv file that I don't have control over how it's being made. This is a temporary fix until it can be fixed properly on the other party's side.

Collectives™ on Stack Overflow

PHP preg_replace is not matching entire pattern

3 Answers 3

1 Comment

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related