Python

Python RSS

Optimizing this code

Please support our Python advertiser: Programming Forums - DaniWeb Sister Site

•

Join Date: May 2009

Posts: 70

Reputation: shoemoodoshaloo is an unknown quantity at this point

Solved Threads: 2

shoemoodoshaloo shoemoodoshaloo is offline

Offline

Junior Poster in Training

Optimizing this code

May 26th, 2009

First let me say thanks to everyone who has been responding to my posts and providing valuable insight. I have been trying to add rep whenever possible, and appreciate your help.

My assignment was to write a python code which takes data from an infile, then has the user specify a column out of the infile, and then the program writes only the information from that column to an outfile. Thanks to help from you guys, I've also been able to include a portion of the code which will skip over blank lines instead of crashing. I'd like to optimize/change the code with your suggestions. I feel that I've probably included too many operations and am being redundant in places. Here is my code, with comments which try to convey my level of understanding. Please add suggestions and point out where my comments are fallacious.

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
def columns(infile, outfile):
	f = open(infile,'r')
	o = open(outfile,'w')
 
	col = int(raw_input('Please select a column (starting at 0) from you in file, %s:' % (infile))) 
 
	temp = []   #Store an empty list
 
	for line in f:
		if not line.strip():   #If empty line, strip the new line character and remaining space 
			continue
		else:
			line = line.split() #Split lines so that they can be operated on via list operations
        		temp.append(line[col])	 #Write the user-specified column into the empty list, "temp
 
	o.write('\n'.join(temp))   
 
 
	print "See %s" % (outfile)  
 
	f.close()
	o.close()
def columns(infile, outfile):
	f = open(infile,'r')
	o = open(outfile,'w')

	col = int(raw_input('Please select a column (starting at 0) from you in file, %s:' % (infile))) 

	temp = []   #Store an empty list
 
	for line in f:
		if not line.strip():   #If empty line, strip the new line character and remaining space 
			continue
		else:
			line = line.split() #Split lines so that they can be operated on via list operations
        		temp.append(line[col])	 #Write the user-specified column into the empty list, "temp
		       
	o.write('\n'.join(temp))   
					

	print "See %s" % (outfile)  
	
	f.close()
	o.close()

Last edited by shoemoodoshaloo; May 26th, 2009 at 11:23 am.

•

Join Date: May 2009

Posts: 74

Reputation: billymcguffin is an unknown quantity at this point

Solved Threads: 1

billymcguffin billymcguffin is offline

Offline

Junior Poster in Training

Re: Optimizing this code

May 26th, 2009

this isnt that big of a deal, but you could add col -= 1 after this:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
	col = int(raw_input('Please select a column (starting at 0) from you in file, %s:' % (infile)))
	col = int(raw_input('Please select a column (starting at 0) from you in file, %s:' % (infile)))

this way, you can eliminate the "starting at 0" part and they will enter "1" for the first column.

Last edited by billymcguffin; May 26th, 2009 at 11:28 am.

•

Join Date: May 2009

Posts: 70

Reputation: shoemoodoshaloo is an unknown quantity at this point

Solved Threads: 2

shoemoodoshaloo shoemoodoshaloo is offline

Offline

Junior Poster in Training

Re: Optimizing this code

May 26th, 2009

Cool suggestion, added.

•

Join Date: May 2009

Posts: 74

Reputation: billymcguffin is an unknown quantity at this point

Solved Threads: 1

billymcguffin billymcguffin is offline

Offline

Junior Poster in Training

Re: Optimizing this code

May 26th, 2009

sure

•

Join Date: Jul 2008

Posts: 1,054

Reputation: jlm699 is a jewel in the rough

Solved Threads: 265

jlm699

Offline

Knows where his Towel is

Re: Optimizing this code

May 26th, 2009

Some of your comments may be overkill. Here's how I'd revise your function:

* Added Billy's suggestion from above
* I like comments on separate lines unless they're extremely short
* Tabs are an absolute no-no in my book
* No need to say if something: continue ... when there's no other action in the loop, simply omit the check for something

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
def columns(infile, outfile):
    f = open(infile,'r')
    o = open(outfile,'w')
 
    col = raw_input('Please select a column from your input file, %s:' % (infile))
    col = int(col) - 1
 
    temp = []
    for line in f:
        line = line.strip()
        if line:
            # Split each line into columns
            line = line.split()
            # Write contents of user-specified columns to temp
            temp.append(line[col])
    o.write('\n'.join(temp))   
 
 
    print "See %s" % (outfile)  
    f.close()
    o.close()
def columns(infile, outfile):
    f = open(infile,'r')
    o = open(outfile,'w')

    col = raw_input('Please select a column from your input file, %s:' % (infile))
    col = int(col) - 1

    temp = []
    for line in f:
        line = line.strip()
        if line:
            # Split each line into columns
            line = line.split()
            # Write contents of user-specified columns to temp
            temp.append(line[col])
    o.write('\n'.join(temp))   
                    

    print "See %s" % (outfile)  
    f.close()
    o.close()

1. Use Code Tags.
2. Homework? Show Effort.
3. Keep discussions on the forum: no PMs

•

Join Date: Jun 2008

Posts: 127

Reputation: slate is an unknown quantity at this point

Solved Threads: 31

slate

Offline

Junior Poster

Re: Optimizing this code

May 27th, 2009

You unnecessary use the list.

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
 
def columns(infile, outfile):
    f = open(infile,'r')
    o = open(outfile,'w')
 
    col = raw_input('Please select a column from your input file, %s:' % (infile))
    col = int(col) - 1
    for line in f:
        line = line.strip()
        if line:
            # Split each line into columns
            line = line.split()
            # Write contents of user-specified columns to temp
            o.write(line[col])
            o.write("\n")
    f.close()
    o.close() # this is the actual flushing
    print "See %s" % (outfile)
 
def columns(infile, outfile):
    f = open(infile,'r')
    o = open(outfile,'w')

    col = raw_input('Please select a column from your input file, %s:' % (infile))
    col = int(col) - 1
    for line in f:
        line = line.strip()
        if line:
            # Split each line into columns
            line = line.split()
            # Write contents of user-specified columns to temp
            o.write(line[col])
            o.write("\n")
    f.close()
    o.close() # this is the actual flushing
    print "See %s" % (outfile)

•

Join Date: Jul 2008

Posts: 1,054

Reputation: jlm699 is a jewel in the rough

Solved Threads: 265

jlm699

Offline

Knows where his Towel is

Re: Optimizing this code

May 28th, 2009

If you're really looking to get fancy use some list comprehension:

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
 
def columns(infile, outfile):
    f = open(infile,'r')
    o = open(outfile,'w')
    #
    col = raw_input('Please select a column from your input file, %s:' % (infile))
    col = int(col) - 1
    o.write('\n'.join([line.strip().split()[col] for line in f if line != '\n']))
    #
    f.close()
    o.close() # this is the actual flushing
    print "See %s" % (outfile)
 
def columns(infile, outfile):
    f = open(infile,'r')
    o = open(outfile,'w')
    #
    col = raw_input('Please select a column from your input file, %s:' % (infile))
    col = int(col) - 1
    o.write('\n'.join([line.strip().split()[col] for line in f if line != '\n']))
    #
    f.close()
    o.close() # this is the actual flushing
    print "See %s" % (outfile)