python os.path.exists failure

K

koranthala

Hi all,
My code is as follows:

path = r'C:/"Program Files"/testfolder/2.3/test.txt'
if os.path.lexists(path):
print 'Path Exists'
else:
print 'No file found in path - %s' %path
print Popen(path, stdout=PIPE, shell=True).stdout.read()

The output comes as
No file found in path - C:/"Program Files"/testfolder/2.3/test.txt
but the test.txt file is opened.

The issue, I guess, is that the double quotes inside is failing the
check. But without the double quotes, Popen fails.
One solution, I can think is to check without double quotes, and then
using some code, put the double quotes back inside, but it looks quite
kludgy.

What is the usual solution to this?
 
B

Benjamin Kaplan

Hi all,
  My code is as follows:

path = r'C:/"Program Files"/testfolder/2.3/test.txt'
if os.path.lexists(path):
   print 'Path Exists'
else:
   print 'No file found in path - %s' %path
print Popen(path, stdout=PIPE, shell=True).stdout.read()

The output comes as
No file found in path - C:/"Program Files"/testfolder/2.3/test.txt
but the test.txt file is opened.

The issue, I guess, is that the double quotes inside is failing the
check. But without the double quotes, Popen fails.
One solution, I can think is to check without double quotes, and then
using some code, put the double quotes back inside, but it looks quite
kludgy.

Just out of curiosity, does 'C:/"Program FIles"/' even work on the
Windows command line? The usual procedure is to put the entire path in
quotes. r'"C:\Program Files\..."'.
 
R

Roel Schroeven

koranthala schreef:
Hi all,
My code is as follows:

path = r'C:/"Program Files"/testfolder/2.3/test.txt'
if os.path.lexists(path):
print 'Path Exists'
else:
print 'No file found in path - %s' %path
print Popen(path, stdout=PIPE, shell=True).stdout.read()

The output comes as
No file found in path - C:/"Program Files"/testfolder/2.3/test.txt
but the test.txt file is opened.

The issue, I guess, is that the double quotes inside is failing the
check. But without the double quotes, Popen fails.
One solution, I can think is to check without double quotes, and then
using some code, put the double quotes back inside, but it looks quite
kludgy.

You can put the double quotes around the whole path instead of just
around "Program Files" in the call to Popen().

The issue here is actually that os.path.exists() (and all other Python
functions) use the path exactly like you pass it; but with your call to
Popen(), the path is passed as an argument to the shell, and it needs to
be quoted for the shell to interpret that correctly.

So here's what I would do: first specify the path literally without
quotes, and quote it only when needed:

path = r'C:/Program Files/testfolder/2.3/test.txt'
if os.path.lexists(path):
print 'Path Exists'
else:
print 'No file found in path - %s' %path
print Popen('"%s"' % path, stdout=PIPE, shell=True).stdout.read()

Some other notes:
- Since you use forward slashes, there's no need to use a raw string
literal.
- You can just as well use os.path.exists() instead of
os.path.lexists(). The difference has to do with symbolic links, which
Windows doesn't have.
- I'm not sure what you expect the line with Popen to do. On my system,
it opens the specified text file in notepad and returns an empty string.
If that's what you want to do, it's easier with os.startfile().

--
The saddest aspect of life right now is that science gathers knowledge
faster than society gathers wisdom.
-- Isaac Asimov

Roel Schroeven
 
A

Aahz

path = r'C:/"Program Files"/testfolder/2.3/test.txt'
if os.path.lexists(path):
print 'Path Exists'
else:
print 'No file found in path - %s' %path
print Popen(path, stdout=PIPE, shell=True).stdout.read()

Avoiding shell=True is a Good Idea
--
Aahz ([email protected]) <*> http://www.pythoncraft.com/

[on old computer technologies and programmers] "Fancy tail fins on a
brand new '59 Cadillac didn't mean throwing out a whole generation of
mechanics who started with model As." --Andrew Dalke
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

No members online now.

Forum statistics

Threads
473,744
Messages
2,569,484
Members
44,903
Latest member
orderPeak8CBDGummies

Latest Threads

Top