Beefy Boxes and Bandwidth Generously Provided by pair Networks
The stupid question is the question not asked

Re: In search of a better way to trim string length

by theAcolyte (Pilgrim)
on Jul 19, 2004 at 08:09 UTC ( #375482=note: print w/replies, xml ) Need Help??

in reply to In search of a better way to trim string

sub trim_length { my($string, $desired_length) = @_; # edit: swiped idea from Tachyon if((length($string)) < $desired_length) { return $string; } my @words = split(" ", $string); my $clipped = ""; foreach(@words) { my $temp = $clipped; $clipped .= $_; if((length($clipped)) > $desired_length) { $clipped = $temp; last +; } } if((length($clipped)) == 0) { $clipped = substr($string, 0, $desired_length); } $clipped .= "..."; return $clipped }

Does the same as your code but with a little less effort. Breaks the string down into words, keeps adding words until it passes your minimum marker, then keeps the previous version. If the first word puts it over the minimum marker, then it goes back to the original $string and clips that.


Replies are listed 'Best First'.
Re^2: In search of a better way to trim string length
by kiat (Vicar) on Jul 19, 2004 at 08:47 UTC
    Thanks, theAcolyte!

    Ran your code. I think you need to add a space here:

    $clipped .= $_; # orginal $clipped .= "$_ "; # changed
    With your code, when I passed the sub a string such as "this is a very long sentence without spaces in between", it gets transformed to "thisisaverylongsentencewithout" i.e. the original spaces were gone.


      blah ... right you are! See what I get for posting untested code? Then again, I always post untested code ... :P

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://375482]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others examining the Monastery: (10)
As of 2016-10-25 08:20 GMT
Find Nodes?
    Voting Booth?
    How many different varieties (color, size, etc) of socks do you have in your sock drawer?

    Results (315 votes). Check out past polls.