Function split() is deprecated , preg_split(): No ending delimiter ',' found

php split delimiter preg-split
happybeauty picture happybeauty · May 19, 2015 · Viewed 7.4k times · Source

I have a PHP script written 10 years ago. Now we moved the script to new server and it's not working. The line that has problem is:

$p_industry = split(',', $member['p_industry']);

The testing email receive this error message:

Function split() is deprecated .

I researched this website and then I replaced the script with

$p_industry = preg_split(',', $member['p_industry']);

Then the testing email receive this different error message:

preg_split(): No ending delimiter ',' found

When I change script to

$p_industry = explode(',', $member['p_industry']);

I did not receive any email for error message. But the script seems not working either. It seems not working in a way that it doesn't even send error message to testing email.

What should I change to the script? Can you give me specific answer?

Answer

pavel picture pavel · May 19, 2015

Preg_* functions has to have delimiters around pattern. I use ~.

$p_industry = preg_split('~,~', $member['p_industry']);

Related questions

In PHP, how can I split a string by whitespace, commas, and newlines at the same time

Possible Duplicate: How to split a string by multiple delimiters in PHP? What would be the most efficient way to split a string by three delimiters at the same time, specifically a single white space, newline, and commas?

Read More
PHP: Split string into array, like explode with no delimiter

I have a string such as: "0123456789" and need to split EACH character into an array. I for the hell of it tried: explode('', '123545789'); But it gave me the obvious: Warning: No delimiter defined in explode) .. How would …

Read More
PHP: Split a string in to an array foreach char

I am making a method so your password needs at least one captial and one symbol or number. I was thinking of splitting the string in to lose chars and then use preggmatch to count if it contains one capital …

Read More