ansaurus

Question

Answer 1

A:

Is it always a comma then a space at the end?

substr($string, 0, -2)

Ergo Summary 2010-10-26 14:31:27

given the subject line says "remove last string IF its ', '" I assume not

Paul 2010-10-26 14:37:25

Answer 2

+6 A:

rtrim() function:

rtrim($string,', ');

but how are you defining the string? It may be that you can build it without the comma and space.

EDIT

$interests = array();
if(isset($_POST['accomodation']))   $interests[] = 'Accomodation'; 
if(isset($_POST['dance']))          $interests[] = 'Dance Lessons'; 
if(isset($_POST['vacation']))       $interests[] = 'Vacation planning'; 
if(isset($_POST['group']))          $interests[] = 'Group Vacation'; 
if(isset($_POST['inprivate']))      $interests[] = 'Private Vacation'; 
if(isset($_POST['land']))           $interests[] = 'Land purchase/lease'; 
if(isset($_POST['all']))            $all    = 'All'; 

$interest = implode(', ',$interests);
echo $interest;

Mark Baker 2010-10-26 14:32:05

Will also trim things like ",,,,," and " , , , " but maybe that's what's wanted...

Paul 2010-10-26 14:38:58

i update the code, to make it more understanding :)

GusDe CooL 2010-10-26 14:43:11

already try this `rtrim($string,', ');` but i didn't remove the last `,` only remove the last space :(

GusDe CooL 2010-10-26 14:50:18

Consider using `implode` if you want to concatenate with a separator.

Alin Purcaru 2010-10-26 14:50:22

@GusDe CooL - If $interest = rtrim($interest,', '); didn't remove the comma, then either you're using a very old version of PHP (earlier than 4.1.0) or you're not calling rtrim() correctly, or you have a very broken version of PHP

Mark Baker 2010-10-26 14:56:29

the `$interest = rtrim($interest,', ');` did do task, what i do in the first that only `rtrim($interest,', ');` as the first i see your code. thanks you very much for your help :)

GusDe CooL 2010-10-26 15:22:42

Answer 3

+1 A:

$string = preg_replace( "/,\s*$/","",$string);

Should do the trick

Cfreak 2010-10-26 14:32:27

this would def work- though it is usually better practice to use PHP string functions in place of RegEx where possible- as they execute faster. However, the difference is fairly miniscule

Ergo Summary 2010-10-26 14:34:00

@Ergo you're absolutely right. I just happen to like regexes :)

Cfreak 2010-10-26 14:43:50

@Cfreak, know what you mean, I have a bit of a thing for them too....(probably the fact they work across languages etc..)

Ergo Summary 2010-10-26 14:54:26

Answer 4

+4 A:

$string = preg_replace('/\s*,\s*$/', '', $string);

or, way cooler:

$string = rtrim($string, " ,");

Note that it does not matter the order of the characters in the pattern string.

@You last update. This changes some things. You could put all your variables in one array and then implode it. Like so:

$items = array();
$items[] = $accomodation = 'Accomodation';
$items[] = $dance = 'Accomodation';
...
$result = implode(', ', $items)

Alin Purcaru 2010-10-26 14:33:12

Thanks you very much.. the completed code really help much..

GusDe CooL 2010-10-26 14:54:20

Answer 5

A:

Often times, you can avoid the trailing comma altogether by changing the way you build the string. For example:

$count = 0;
foreach ($this as $that) {

   if ($count != 0) {
      $string .= ',';
   }

   $string .= $that['stuff'];
   $count++;
}

Would remove the possibility of any trailing comma at the end, no matter the combination of results.

bpeterson76 2010-10-26 16:15:47

Mh, I have seen this pattern and used it myself, but see @[Mark Baker]s answer: assigning the items to an array and join()ing them with the desired separator is a much better and cleaner way to do this.

Max 2010-10-26 16:19:47

ansaurus

tags:

views:

answers:

Remove Last String if its ', '

related questions