Re: stripping unwanted chars from string

2006-05-04 Thread Alex Martelli
Edward Elliott <[EMAIL PROTECTED]> wrote:

> I'm looking for the "best" way to strip a large set of chars from a filename
> string (my definition of best usually means succinct and readable).   I
> only want to allow alphanumeric chars, dashes, and periods.  This is what I
> would write in Perl (bless me father, for I have sinned...):
> 
> $filename =~ tr/\w.-//cd, or equivalently 
> $filename =~ s/[^\w.-]//
> 
> I could just use re.sub like the second example, but that's a bit overkill.
> I'm trying to figure out if there's a good way to do the same thing with
> string methods.  string.translate seems to do what I want, the problem is
> specifying the set of chars to remove.  Obviously hardcoding them all is a
> non-starter.

(untested code, but, the general idea shd be correct)...:

class KeepOnly(object):
allchars = ''.join(chr(i) for i in xrange(256))
identity = string.maketrans('', '')

def __init__(self, chars_to_keep):
self.chars_to_delete = self.allchars.translate(
self.identity, chars_to_keep)

def __call__(self, some_string):
return some_string.translate(self.identity,
self.chars_to_delete)


Alex
-- 
http://mail.python.org/mailman/listinfo/python-list


Re: stripping unwanted chars from string

2006-05-04 Thread John Machin
On 4/05/2006 4:30 PM, Edward Elliott wrote:
> Bryan wrote:
>>  >>> keepchars = set(string.letters + string.digits + '-.')
> 
> Now that looks a lot better.  Just don't forget the underscore. :)
> 

*Looks* better than the monkey business. Perhaps I should point out to 
those of the studio audience who are huddled in an ASCII bunker (if any) 
that string.letters provides the characters considered to be alphabetic 
in whatever the locale is currently set to. There is no guarantee that 
the operating system won't permit filenames containing other characters, 
ones that the file's creator would quite reasonably consider to be 
alphabetic. And of course there are languages that have characters that 
one would not want to strip but can scarcely be described as alphanumeric.

 >>> import os
 >>> os.listdir(u'.')
[u'\xc9t\xe9_et_hiver.doc', u'\u041c\u043e\u0441\u043a\u0432\u0430.txt', 
u'\u5f20\u654f.txt']

 >>> import string
 >>> string.letters
'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ'

Doing
import locale; locale.setlocale(locale.LC_ALL, '')
would make string.letters work (for me) with the first file above, but 
that's all.
-- 
http://mail.python.org/mailman/listinfo/python-list


Re: stripping unwanted chars from string

2006-05-04 Thread bruno at modulix
Edward Elliott wrote:
> Bryan wrote:
> 
>> >>> keepchars = set(string.letters + string.digits + '-.')
> 
> 
> Now that looks a lot better.  Just don't forget the underscore. :)
> 
You may also want to have a look at string.translate() and
string.maketrans()

-- 
bruno desthuilliers
python -c "print '@'.join(['.'.join([w[::-1] for w in p.split('.')]) for
p in '[EMAIL PROTECTED]'.split('@')])"
-- 
http://mail.python.org/mailman/listinfo/python-list


Re: stripping unwanted chars from string

2006-05-03 Thread Edward Elliott
Bryan wrote:
>  >>> keepchars = set(string.letters + string.digits + '-.')

Now that looks a lot better.  Just don't forget the underscore. :)

-- 
http://mail.python.org/mailman/listinfo/python-list


Re: stripping unwanted chars from string

2006-05-03 Thread Edward Elliott
John Machin wrote:
> [expletives deleted] and it was wrong anyway (according to your
> requirements);
> using \w would keep '_' which is *NOT* alphanumeric.

Actually the perl is correct, the explanation was the faulty part.  When in
doubt, trust the code.  Plus I explicitly allowed _ further down, so the
mistake should have been fairly obvious.


>  >>> alphabet = 'qwertyuiopasdfghjklzxcvbnm' # Look, Ma, no thought
> required!! Monkey see, monkey type.

I won't dignify that with a response.  The code that is, I could give a toss
about the comments.  If you enjoy using such verbose, error-prone
representations in your code, god help anyone maintaining it.  Including
you six months later.  Quick, find the difference between these sets at a
glance:

'qwertyuiopasdfghjklzxcvbnm'
'abcdefghijklmnopqrstuvwxyz'
'abcdefghijklmnopprstuvwxyz'
'abcdefghijk1mnopqrstuvwxyz'
'qwertyuopasdfghjklzxcvbnm' # no fair peeking

And I won't even bring up locales.


>  >>> keepchars = set(alphabet + alphabet.upper() + '1234567890-.')
>  >>> fixer = lambda x: ''.join(c for c in x if c in keepchars)

Those darn monkeys, always think they're so clever! ;)
if "you can" == "you should": do(it)
else: do(not)


>> Sadly I can find no such beast.  Anyone have any insight?  As of now,
>> regexes look like the best solution.
> 
> I'll leave it to somebody else to dredge up the standard riposte to your
> last sentence :-)

If the monstrosity above is the best you've got, regexes are clearly the
better solution.  Readable trumps inscrutable any day.


> One point on your requirements: replacing unwanted characters instead of
> deleting them may be better -- theoretically possible problems with
> deleting are: (1) duplicates (foo and foo_ become the same) (2) '_' 
> becomes '' which is not a valid filename. 

Which is why I perform checks for emptiness and uniqueness after the strip. 
I decided long ago that stripping is preferable to replacement here.


> And a legibility problem: if 
> you hate '_' and ' ' so much, why not change them to '-'?

_ is allowed.  And I do prefer -, but not for legibility.  It doesn't
require me to hit Shift.

 
> Oh and just in case the fix was accidentally applied to a path:
> 
> keepchars.update(os.sep)
> if os.altsep: keepchars.update(os.altsep)

Nope, like I said this is strictly a filename.  Stripping out path
components is the first thing I do.  But thanks for pointing out these
common pitfalls for members of our studio audience.  Tell him what he's
won, Johnny! ;)

-- 
http://mail.python.org/mailman/listinfo/python-list


Re: stripping unwanted chars from string

2006-05-03 Thread Bryan

>  >>> keepchars = set(alphabet + alphabet.upper() + '1234567890-.')

or

 >>> keepchars = set(string.letters + string.digits + '-.')

bryan

-- 
http://mail.python.org/mailman/listinfo/python-list


Re: stripping unwanted chars from string

2006-05-03 Thread John Machin
On 4/05/2006 1:36 PM, Edward Elliott wrote:
> I'm looking for the "best" way to strip a large set of chars from a filename
> string (my definition of best usually means succinct and readable).   I
> only want to allow alphanumeric chars, dashes, and periods.  This is what I
> would write in  (bless me father, for I have sinned...):

[expletives deleted] and it was wrong anyway (according to your 
requirements);
using \w would keep '_' which is *NOT* alphanumeric.

> I could just use re.sub like the second example, but that's a bit overkill. 
> I'm trying to figure out if there's a good way to do the same thing with
> string methods.  string.translate seems to do what I want, the problem is
> specifying the set of chars to remove.  Obviously hardcoding them all is a
> non-starter.
> 
> Working with chars seems to be a bit of a pain.  There's no equivalent of
> the range function, one has to do something like this:
> 
 [chr(x) for x in range(ord('a'), ord('z')+1)]
> ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o',
> 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z']

 >>> alphabet = 'qwertyuiopasdfghjklzxcvbnm' # Look, Ma, no thought 
required!! Monkey see, monkey type.
 >>> keepchars = set(alphabet + alphabet.upper() + '1234567890-.')
 >>> fixer = lambda x: ''.join(c for c in x if c in keepchars)
 >>> fixer('[EMAIL PROTECTED]')
'qwe456.--Howzat'
 >>>

> 
> Do that twice for letters, once for numbers, add in a few others, and I get
> the chars I want to keep.  Then I'd invert the set and call translate. 
> It's a mess and not worth the trouble.  Unless there's some way to expand a
> compact representation of a char list and obtain its complement, it looks
> like I'll have to use a regex.
> 
> Ideally, there would be a mythical charset module that works like this:
> 
 keep = charset.expand (r'\w.-') # or r'a-zA-Z0-9_.-'

Where'd that '_' come from?

 toss = charset.invert (keep)
> 
> Sadly I can find no such beast.  Anyone have any insight?  As of now,
> regexes look like the best solution.

I'll leave it to somebody else to dredge up the standard riposte to your 
last sentence :-)

One point on your requirements: replacing unwanted characters instead of 
deleting them may be better -- theoretically possible problems with 
deleting are: (1) duplicates (foo and foo_ become the same) (2) '_' 
becomes '' which is not a valid filename. And a legibility problem: if 
you hate '_' and ' ' so much, why not change them to '-'?

Oh and just in case the fix was accidentally applied to a path:

keepchars.update(os.sep)
if os.altsep: keepchars.update(os.altsep)

HTH,
John
-- 
http://mail.python.org/mailman/listinfo/python-list


stripping unwanted chars from string

2006-05-03 Thread Edward Elliott
I'm looking for the "best" way to strip a large set of chars from a filename
string (my definition of best usually means succinct and readable).   I
only want to allow alphanumeric chars, dashes, and periods.  This is what I
would write in Perl (bless me father, for I have sinned...):

$filename =~ tr/\w.-//cd, or equivalently 
$filename =~ s/[^\w.-]//

I could just use re.sub like the second example, but that's a bit overkill. 
I'm trying to figure out if there's a good way to do the same thing with
string methods.  string.translate seems to do what I want, the problem is
specifying the set of chars to remove.  Obviously hardcoding them all is a
non-starter.

Working with chars seems to be a bit of a pain.  There's no equivalent of
the range function, one has to do something like this:

>>> [chr(x) for x in range(ord('a'), ord('z')+1)]
['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o',
'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z']

Do that twice for letters, once for numbers, add in a few others, and I get
the chars I want to keep.  Then I'd invert the set and call translate. 
It's a mess and not worth the trouble.  Unless there's some way to expand a
compact representation of a char list and obtain its complement, it looks
like I'll have to use a regex.

Ideally, there would be a mythical charset module that works like this:

>>> keep = charset.expand (r'\w.-') # or r'a-zA-Z0-9_.-'
>>> toss = charset.invert (keep)

Sadly I can find no such beast.  Anyone have any insight?  As of now,
regexes look like the best solution.

-- 
http://mail.python.org/mailman/listinfo/python-list