comparing elements of a list with a string

[Shriphani]

Hello all,
I have a problem here. I have a list named list_of_files which
contains filenames with their timestamps attached to the name. If I
have a string "fstab", and I want to list out the files in whose names
the word fstab appears should I go about like this :

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if element.find(file) != -1:
date = ###
time = ####
return (date, time)

The major trouble is that the return statement causes it to exit after
attempt one. How do I use the yield statement here?

Regards,
Shriphani Palakodety

[Evan Klitzke]

On Tue, 2007-09-25 at 08:39 -0700, Shriphani wrote:

Hello all,
I have a problem here. I have a list named list_of_files which
contains filenames with their timestamps attached to the name. If I
have a string "fstab", and I want to list out the files in whose names
the word fstab appears should I go about like this :

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if element.find(file) != -1:
date = ###
time = ####
return (date, time)

The major trouble is that the return statement causes it to exit after
attempt one. How do I use the yield statement here?

You can just replace the return statement with a yield statement,
nothing fancy is required. Also, as long as you're using the string.find
method to search for the file, I'd replace it with "if file in element",
which is a bit more idiomatic.

--
Evan Klitzke <ev**@yelp.com>

[Matimus]

Shriphani wrote:

Hello all,
I have a problem here. I have a list named list_of_files which
contains filenames with their timestamps attached to the name. If I
have a string "fstab", and I want to list out the files in whose names
the word fstab appears should I go about like this :

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if element.find(file) != -1:
date = ###
time = ####
return (date, time)

The major trouble is that the return statement causes it to exit after
attempt one. How do I use the yield statement here?

I would just do this:

from glob import glob
def listallbackupfiles(filename):
return glob("/home/shriphani/backupdir/*%s*"%filename)
 

[Larry Bates]

Shriphani wrote:

Hello all,
I have a problem here. I have a list named list_of_files which
contains filenames with their timestamps attached to the name. If I
have a string "fstab", and I want to list out the files in whose names
the word fstab appears should I go about like this :

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if element.find(file) != -1:
date = ###
time = ####
return (date, time)

The major trouble is that the return statement causes it to exit after
attempt one. How do I use the yield statement here?

Regards,
Shriphani Palakodety

You should take a quick look at glob(). You may be able to use it to make life
a lot easier. Here is how you would do it if all your backup files begin with
fstab.

import glob
list_of_backup_files=glob.glob('/home/shriphani/backupdir/glob*')

If "fstab" can appear anywhere in the filename, this might not work for you.

-Larry

[Steve Holden]

Larry Bates wrote:

Shriphani wrote:

>Hello all,
I have a problem here. I have a list named list_of_files which
contains filenames with their timestamps attached to the name. If I
have a string "fstab", and I want to list out the files in whose names
the word fstab appears should I go about like this :

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if element.find(file) != -1:
date = ###
time = ####
return (date, time)

The major trouble is that the return statement causes it to exit after
attempt one. How do I use the yield statement here?

Regards,
Shriphani Palakodety

You should take a quick look at glob(). You may be able to use it to make life
a lot easier. Here is how you would do it if all your backup files begin with
fstab.

import glob
list_of_backup_files=glob.glob('/home/shriphani/backupdir/glob*')

If "fstab" can appear anywhere in the filename, this might not work for you.

I don't see why

list_of_backup_files=glob.glob('/home/shriphani/backupdir/*fstab*')

shouldn't work. However, t answer the OP's question about yield (which
nobody seems to have done fully yet):

1. Rewrite the function to become a generator function:

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if file in element: # tidied this up too
date = 1 ###
time = 2 ####
yield (date, time)

2. Create a generator by calling the function:

list_of_backup_files = listAllbackups("fstab")

3. Use the generator in an iterative context:

for file in list_of_backup_files:
# do something

regards
Steve
--
Steve Holden [phone] [phone]
Holden Web LLC/Ltd http://www.holdenweb.com
Skype: holdenweb http://del.icio.us/steve.holden

Sorry, the dog ate my .sigline

[byte8bits]

On Sep 25, 11:39 am, Shriphani <[e-mail]:

If I have a string "fstab", and I want to list out the files in whose names
the word fstab appears should I go about like this :

def listAllbackups(file):
list_of_files = os.listdir("/home/shriphani/backupdir")
for element in list_of_files:
if element.find(file) != -1:
date = ###
time = ####
return (date, time)

I would do something like this instead:

>>for root, dirs, files in os.walk('.'):

.... for f in files:
.... if 'text' in f:
.... print f
....
gimp-text-tool
gimp-text-tool.presets
text.py~
textwrap.pyc
textwrap.py
....

You can append the output to a list and return that list if you want
to encapsulate this in a function.

[Shriphani]

Hello,
Would that mean that if I wanted to append all the (date, time) tuples
to a list, I should do something like:

for file in list_of_backup_files:
some_list.append(file)
By the way I did this:

def listAllbackups(filename):
list_of_backups = glob(home+'/Desktop/backupdir/*%s*'%filename)
for element in list_of_back:
if element.find(file) != -1:
date_components = element.split('-')[-4:-1]
date = str(date_components[0]) + ":" + str(date_components[1]) +
":" + str(date_components[2])
time = element.split('-')[-1]
yield (date, time)
print listAllbackups('fstab')
I ran it in the terminal and I got:

<generator object at 0x81ed58c>

Why does it do that and not just print out all the values of (date,
time)
Regards,
Shriphani Palakodety

[Steve Holden]

Shriphani wrote:

Hello,
Would that mean that if I wanted to append all the (date, time) tuples
to a list, I should do something like:

for file in list_of_backup_files:
some_list.append(file)

That would be one way to do it (assuming you started with some_list as
an empty list). But a faster way would be to use a list comprehension
and say

some_list = [f for f in list_of_backup_files]

By the way I did this:

def listAllbackups(filename):
list_of_backups = glob(home+'/Desktop/backupdir/*%s*'%filename)
for element in list_of_back:
if element.find(file) != -1:
date_components = element.split('-')[-4:-1]
date = str(date_components[0]) + ":" + str(date_components[1]) +
":" + str(date_components[2])
time = element.split('-')[-1]
yield (date, time)
print listAllbackups('fstab')
I ran it in the terminal and I got:

<generator object at 0x81ed58c>

Why does it do that and not just print out all the values of (date,
time)

Because the generator object returned by the function has to be used in
an iterative context to produce its values. Which is why the list
comprehension produces a list!

If you don't really want a generator then you could just rewrite the
function to return a list in the first place. You did specifically ask
"how do I use the yield statement ...". If you are going to iterate o
ver the list then a generator is typically more memory-efficient
(because it only produces one element at a time), and can do things that
a list can't (like deal with a potentially infinite sequence of results).

regards
Steve
--
Steve Holden [phone] [phone]
Holden Web LLC/Ltd http://www.holdenweb.com
Skype: holdenweb http://del.icio.us/steve.holden

Sorry, the dog ate my .sigline

[Marc 'BlackJack' Rintsch]

On Thu, 27 Sep 2007 08:16:59 -0400, Steve Holden wrote:

Shriphani wrote:

>Hello,
Would that mean that if I wanted to append all the (date, time) tuples
to a list, I should do something like:

for file in list_of_backup_files:
some_list.append(file)

That would be one way to do it (assuming you started with some_list as
an empty list). But a faster way would be to use a list comprehension
and say

some_list = [f for f in list_of_backup_files]

Or:

some_list = list(list_of_backup_files)

If `some_list` is not empty:

some_list.extend(list_of_backup_files)

Ciao,
Marc 'BlackJack' Rintsch

[Gabriel Genellina]

En Thu, 27 Sep 2007 07:44:08 -0300, Shriphani <sh********@gmail.com>
escribi�:

def listAllbackups(filename):
list_of_backups = glob(home+'/Desktop/backupdir/*%s*'%filename)
for element in list_of_back:
if element.find(file) != -1:
date_components = element.split('-')[-4:-1]
date = str(date_components[0]) + ":" + str(date_components[1]) +
":" + str(date_components[2])
time = element.split('-')[-1]
yield (date, time)
print listAllbackups('fstab')
I ran it in the terminal and I got:

<generator object at 0x81ed58c>

Why does it do that and not just print out all the values of (date,
time)

Because listAllbackups is a generator function: each time someone asks it
the next value, it runs until the "yield" statement, and pauses. Next time
execution continues from the same point.
You have to iterate over it. Try:

for date,time in listAllbackups('fstab'):
print date,time

or

items = list(listAllbackups('fstab'))
print items

--
Gabriel Genellina