strip module bug

[Poppy]

I'm using versions 2.5.2 and 2.5.1 of python and have encountered a
potential bug. Not sure if I'm misunderstanding the usage of the strip
function but here's my example.

var = "detail.xml"
print var.strip(".xml") ### expect to see 'detail', but get 'detai'
var = "overview.xml"
print var.strip(".xml") ### expect and get 'overview'

I have a work around using the replace function which happens to be the
better choice for my script anyhow. But am curious about the strip module.
Any thoughts? Is it removing the 'l' in detail because the strip function
text ends in 'l'?

[Peter Otten]

Poppy wrote:

>
I'm using versions 2.5.2 and 2.5.1 of python and have encountered a
potential bug. Not sure if I'm misunderstanding the usage of the strip
function but here's my example.

var = "detail.xml"
print var.strip(".xml") ### expect to see 'detail', but get 'detai'
var = "overview.xml"
print var.strip(".xml") ### expect and get 'overview'

I have a work around using the replace function which happens to be the
better choice for my script anyhow. But am curious about the strip module.
Any thoughts? Is it removing the 'l' in detail because the strip function
text ends in 'l'?

The behaviour you intend, stripping a suffix, is achieved by

>>var = "detail.xml"
suffix = ".xml"
if var.endswith(suffix):

.... var = var[:-len(suffix)]
....

The var.strip(chars) /method/ does something different. It treats chars as a
set of characters and removes any of these characters from the end and the
beginning of the var string, i. e.

"detail.xml" d is not in ".xml" -remove no further chars from the
beginning
"detail.xml" l is in ".xml", remove it
"detail.xm" m is in ".xml", remove it
"detail.x" x is in ".xml", remove it
"detail." . is in ".xml", remove it
"detail" l is in ".xml", remove it
"detai" i is not in ".xml", we're done

Peter

[bearophileHUGS]

Poppy:

var = "detail.xml"
print var.strip(".xml") ### expect to see 'detail', but get 'detai'
var = "overview.xml"
print var.strip(".xml") ### expect and get 'overview'

Python V.2.5 is not flawless, but you can't find bugs so easily. I've
found only two bugs in about three years of continuous usage (while I
have found about 27 new different bugs in the DMD compiler in about
one year of usage).

str.strip() isn't a module, and generally in Python it's called
"method", in this case a method of str (or unicode).

str.strip() as optional argument doesn't take the string you want to
remove, but a string that represent the set of chars you want to strip
away from both ends, so:

>>"detail.xml".strip("lmx.")

'detai'

>>"detail.xml".strip("abcdlz")

'etail.xm'

To strip the ending ".xml" you can use the str.partition() method, or
an alternative solution:

>>var = "detail.xml"
suffix = ".xml"
if var.endswith(suffix):

.... var2 = var[: -len(suffix)]
....

>>var2

'detail'

Bye,
bearophile

[Tim Chase]

I'm using versions 2.5.2 and 2.5.1 of python and have encountered a

potential bug. Not sure if I'm misunderstanding the usage of the strip
function but here's my example.

var = "detail.xml"
print var.strip(".xml") ### expect to see 'detail', but get 'detai'
var = "overview.xml"
print var.strip(".xml") ### expect and get 'overview'

..strip() takes a *set* of characters to remove, as many as are found:
_______ _____

>>'xqxqxqxyxxyxqxqx'.strip('xq')

'yxxy'

Using .replace() may work, but might have some odd side effects:

>>'something.xml.zip'.replace('.xml')

'something.zip'

>>'this file has .xml in its name.xml'.replace('.xml')

'this file has in its name'

This also has problems if your filename and extension differ in
case (removing ".xml" from "file.XML")

If you want to remove the extension, then I recommend using the
python builtin:

>>import os
results = os.path.splitext('this has .xml in its name.xml')
results

('this has .xml in its name', '.xml')

>>fname, ext = results

Or, if you know what the extension is:

>>fname = 'this file has .xml in its name.xml'
ext = '.xml'
fname[:-len(ext)]

'this file has .xml in its name'

Just a few ideas that are hopefully more robust.

-tkc

[Tim Roberts]

"Poppy" <zn****************@yahoo.comwrote:

>
Thanks Steven and Tim, I understand the strip module a lot more today.

It's NOT a module. It is a method of string objects.
--
Tim Roberts, ti**@probo.com
Providenza & Boekelheide, Inc.