list() and stripslashes()

sethadam1

it used to be that I would have long code like this:

$array = explode("::",$string); 
$name = trim(stripslashes($array[0]));
$address = trim(stripslashes($array[1]));
$phone = trim(stripslashes($array[2]));

now, I use list to get something like this:

list($name,$address,$phone) = explode("::",$string);

the question is, there a compact way to run stripslashes() and trim() inside the list function? Anyone know of a simpler more concise way?

nemonoman

list($name,$address,$phone) = explode("::",trim(stripslashes($string));

sethadam1

Originally posted by nemonoman
list($name,$address,$phone) = explode("::",trim(stripslashes($string));

??

Well, the strislashes should work, guess I should've seen that, but the trim will strip space from the ends of the WHOLE STRING, and I need space removed from around each element within the string after it's exploded.

nemonoman

Where do these values come from? why don't you trim them before you concatenate them?

You are trying to mix array actions and concatenated string actions. So your code will be messy as a result. You can just live withit, or maybe create at strip_trim_array funcition of your very own to be included in later PHP versions...

swr

The function you are looking for is [man]array_map[/man].

$array = explode('::', $string);
$array = array_map('stripslashes', $array);
$array = array_map('trim', $array);
list($name,$address,$phone) = $array;

nemonoman

OK. Instead of the original 4 lines of code, array_map lets the coder manage the problem with only...4 lines of code.

nemonoman

OK, in one line:

list($name,$address,$phone) = explode("::",stripslashes(trim(preg_replace('/[ ]*::/', "::", $string))));

swr

If it's lines that you're counting, you could do it in one:

list($name, $address, $phone) = array_map('trim', array_map('stripslashes', explode('::', $string)));

I still think the four-line version using array_map is nicer than the four-line version that calls stripslashes and trim on each individual element.

sethadam1

Thanks guys - array_map is awesome!

I love the preg_replace too.

Thanks,

Weedpacket

Of course, don't assume that "fewer lines of code" equates to "faster running" or "easier to read" or even "smaller memory footprint". The first (and maybe the third) should be determined by experiment, and the second by eyeball.