Python

Python RSS

Sorting a File

Thread Solved

•

Join Date: Oct 2006

Posts: 34

Reputation: babutche is an unknown quantity at this point

Solved Threads: 0

babutche

Offline

Light Poster

Sorting a File

Jan 6th, 2007

Hi,

I am trying to extend my "Student with the best GPA" program so that it allows the user to sort a file of students based on gpa, name, or credits. The program needs to prompt for the input and output files and also the field to sort on (gpa, name or credits).

The input file consists of students name, credits(hours) and quality points (gpa = quality points / hours)

I keep getting an error that global name "gpa" is not defined and I don't understand why. Can anyone steer me in the right direction as to where I am going wrong? Thank you!

This is what I have so far on my program:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
from p2gpa import Student, makeStudent
def readStudents(filename):
    infile = open(filename, 'r')
    students = []
    for line in infile:
        students.append(makeStudent(line))
    infile.close()
    return students
 
def writeStudents(students, filename):
    outfile = open(filename, 'w')
    for s in students:
        outfile.write("%s\t%f\t%f\n" %
                      (s.getName(), s.getHours(), s.getQPoints()))
    outfile.close()
def cmpgpa(s1, s2):
    return cmp(s1.gpa(), s2.gpa())
def cmpname(s1, s2):
    return cmp("s1.name()", "s2.name()")
def cmpcredits(s1, s2):
    return cmp(s1.credits(), s2.credits())
 
def main():
    print "This program sorts student grade information"
    filename = raw_input("Enter the name of the data file: ")
    dfield = raw_input("Enter gpa, name, or credits to sort: ")
    filename = raw_input("Enter a name for the output file: ")
    data = readStudents(filename)
    if dfield == gpa:
        data.sort(cmpgpa)
    elif dfield == name:
        data.sort(cmpname)
    else:
        data.sort(cmpcredits)
    filename = raw_input("Enter a name for the output file: ")    
    writeStudents(data, filename)
    print "The data has been written to", filename
if __name__ == '__main__':
    main()
from p2gpa import Student, makeStudent
def readStudents(filename):
    infile = open(filename, 'r')
    students = []
    for line in infile:
        students.append(makeStudent(line))
    infile.close()
    return students
 
def writeStudents(students, filename):
    outfile = open(filename, 'w')
    for s in students:
        outfile.write("%s\t%f\t%f\n" %
                      (s.getName(), s.getHours(), s.getQPoints()))
    outfile.close()
def cmpgpa(s1, s2):
    return cmp(s1.gpa(), s2.gpa())
def cmpname(s1, s2):
    return cmp("s1.name()", "s2.name()")
def cmpcredits(s1, s2):
    return cmp(s1.credits(), s2.credits())
 
def main():
    print "This program sorts student grade information"
    filename = raw_input("Enter the name of the data file: ")
    dfield = raw_input("Enter gpa, name, or credits to sort: ")
    filename = raw_input("Enter a name for the output file: ")
    data = readStudents(filename)
    if dfield == gpa:
        data.sort(cmpgpa)
    elif dfield == name:
        data.sort(cmpname)
    else:
        data.sort(cmpcredits)
    filename = raw_input("Enter a name for the output file: ")    
    writeStudents(data, filename)
    print "The data has been written to", filename
if __name__ == '__main__':
    main()

This is the p2gpa program that I imported:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
import string
import math
class Student:
    def __init__(self, name, hours, qpoints):
        self.name = name
        self.hours = float(hours)
        self.qpoints = float(qpoints)
    def getName(self):
        return self.name
    def getHours(self):
        return self.hours
    def getQPoints(self):
        return self.qpoints
    def gpa(self):
        return self.qpoints/self.hours
    def name(self):
        return self.name
def makeStudent(infoStr):
    name, hours, qpoints = string.split(infoStr,"\t")
    return Student(name, hours, qpoints)
def main():
    filename = raw_input("Enter name the grade file: ")
    infile = open(filename, 'r')
    best = makeStudent(infile.readline())
    for line in infile:
        s = makeStudent(line)
        if s.gpa() > best.gpa():
            best = s
    infile.close()
    print "The best student is:", best.getName()
    print "hours:", best.getHours()
    print "GPA:", best.gpa()
if __name__ == '__main__':
    main()
import string
import math
class Student:
    def __init__(self, name, hours, qpoints):
        self.name = name
        self.hours = float(hours)
        self.qpoints = float(qpoints)
    def getName(self):
        return self.name
    def getHours(self):
        return self.hours
    def getQPoints(self):
        return self.qpoints
    def gpa(self):
        return self.qpoints/self.hours
    def name(self):
        return self.name
def makeStudent(infoStr):
    name, hours, qpoints = string.split(infoStr,"\t")
    return Student(name, hours, qpoints)
def main():
    filename = raw_input("Enter name the grade file: ")
    infile = open(filename, 'r')
    best = makeStudent(infile.readline())
    for line in infile:
        s = makeStudent(line)
        if s.gpa() > best.gpa():
            best = s
    infile.close()
    print "The best student is:", best.getName()
    print "hours:", best.getHours()
    print "GPA:", best.gpa()
if __name__ == '__main__':
    main()

This is the input file I want to sort (p2sortin.py):

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
Computewell, Susan 100 400
DibbleBit, Denny 18 41.5
Jones, Jim 48.5 155
Smith, Frank 37 125.33
Adams, Henry 127 228
Computewell, Susan 100 400
DibbleBit, Denny 18 41.5
Jones, Jim 48.5 155
Smith, Frank 37 125.33
Adams, Henry 127 228

This is the error that I keep getting:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
This program sorts student grade information
Enter the name of the data file: p2sortin.py
Enter gpa, name, or credits to sort: gpa
Enter a name for the output file: p2sortout.py
Traceback (most recent call last):
  File "C:\Python23\p2sort.py", line 50, in ?
    main()
  File "C:\Python23\p2sort.py", line 40, in main
    if dfield == gpa:
NameError: global name 'gpa' is not defined
>>>
This program sorts student grade information
Enter the name of the data file: p2sortin.py
Enter gpa, name, or credits to sort: gpa
Enter a name for the output file: p2sortout.py
Traceback (most recent call last):
  File "C:\Python23\p2sort.py", line 50, in ?
    main()
  File "C:\Python23\p2sort.py", line 40, in main
    if dfield == gpa:
NameError: global name 'gpa' is not defined
>>>

Thanks!

•

Join Date: Aug 2005

Posts: 1,523

Reputation: Ene Uran has a spectacular aura about

Solved Threads: 169

Ene Uran

Offline

Posting Virtuoso

Re: Sorting a File

Jan 6th, 2007

Your initial problem is right here:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
def main():
    print "This program sorts student grade information"
    filename = raw_input("Enter the name of the data file: ")
    dfield = raw_input("Enter gpa, name, or credits to sort: ")
    filename = raw_input("Enter a name for the output file: ")
    data = readStudents(filename)
    ...
def main():
    print "This program sorts student grade information"
    filename = raw_input("Enter the name of the data file: ")
    dfield = raw_input("Enter gpa, name, or credits to sort: ")
    filename = raw_input("Enter a name for the output file: ")
    data = readStudents(filename)
    ...

You are assigning the same variable 'filename' to both the input and output file.

Your next problem is here:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
    if dfield == gpa:
        data.sort(cmpgpa)
    if dfield == gpa:
        data.sort(cmpgpa)

The input function raw_input() returns a string, so this ought to be:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
    if dfield == 'gpa':
        data.sort(cmpgpa)
    if dfield == 'gpa':
        data.sort(cmpgpa)

Give your data file a name like in.dat, don't use .py (it is not Python code!).
Your next problem is here ( I added a test print):

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
def makeStudent(infoStr):
    print string.split(infoStr,"\t")  # test --> ['Computewell, Susan 100 400\n']
    name, hours, qpoints = string.split(infoStr,"\t")
    return Student(name, hours, qpoints)
def makeStudent(infoStr):
    print string.split(infoStr,"\t")  # test --> ['Computewell, Susan 100 400\n']
    name, hours, qpoints = string.split(infoStr,"\t")
    return Student(name, hours, qpoints)

The result of splitting a string is a list, not a tuple! Also the data file needs tabs at the right places to make this work!

Last edited by Ene Uran; Jan 6th, 2007 at 3:44 pm.

drink her pretty

•

Join Date: Dec 2006

Posts: 1,008

Reputation: woooee is a jewel in the rough

Solved Threads: 285

woooee

Offline

Veteran Poster

Re: Sorting a File

Jan 6th, 2007

•

File "C:\Python23\p2sort.py", line 40, in main
if dfield == gpa:
NameError: global name 'gpa' is not defined

I think you want "gpa" in quotes.
if dfield == "gpa":
A simple menu would be easier for the user, with 1=gpa, etc. In any case, you should have a list of correct responses, or be able to add something like 0 < number < 4, to check for a valid input.

Last edited by woooee; Jan 6th, 2007 at 9:29 pm.

•

Join Date: Oct 2004

Posts: 3,982

Reputation: vegaseat is just really nice

Solved Threads: 925

vegaseat

Offline

DaniWeb's Hypocrite

Re: Sorting a File

Jan 7th, 2007

I played with the code for a while and made these observations and corrections ...

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
# save as p2sort.py
from p2gpa import Student, makeStudent
def readStudents(filename):
    infile = open(filename, 'r')
    students = []
    for line in infile:
        students.append(makeStudent(line))
    infile.close()
    return students
 
def writeStudents(students, filename):
    outfile = open(filename, 'w')
    for s in students:
        print "%-20s %0.2f %0.2f" % (s.getName(), s.gpa(), s.getHours())  # test
        outfile.write("%s\t%0.2f\t%0.2f\n" %
                      (s.getName(), s.getHours(), s.getQPoints()))
    outfile.close()
 
# for these compares s1 and s2 are class instances ...
def cmpgpa(s1, s2):
    # low to high
    #return cmp(s1.gpa(), s2.gpa())
    # high to low
    return cmp(s2.gpa(), s1.gpa())
def cmpname(s1, s2):
    #return cmp(s1.name(), s2.name())  # problems!!!?
    return cmp(s1.getName(), s2.getName())
def cmpcredits(s1, s2):
    # low to high
    #return cmp(s1.getHours(), s2.getHours())
    # high to low
    return cmp(s2.getHours(), s1.getHours())
 
def main():
    print "This program sorts student grade information"
    # file has line format -->  name tab hours tab qpoints
    file_in = raw_input("Enter the name of the data file: ")
    dfield = raw_input("Enter gpa, name, or credits to sort: ")
    # data is a list of Student class instances
    data = readStudents(file_in)
    if dfield == 'gpa':
        data.sort(cmpgpa)
    elif dfield == 'name':
        data.sort(cmpname)
    else:
        data.sort(cmpcredits)
    file_out = raw_input("Enter a name for the output file: ")
    writeStudents(data, file_out)
    print "The data has been written to", file_out
 
if __name__ == '__main__':
    main()
# save as p2sort.py
from p2gpa import Student, makeStudent
def readStudents(filename):
    infile = open(filename, 'r')
    students = []
    for line in infile:
        students.append(makeStudent(line))
    infile.close()
    return students
 
def writeStudents(students, filename):
    outfile = open(filename, 'w')
    for s in students:
        print "%-20s %0.2f %0.2f" % (s.getName(), s.gpa(), s.getHours())  # test
        outfile.write("%s\t%0.2f\t%0.2f\n" %
                      (s.getName(), s.getHours(), s.getQPoints()))
    outfile.close()
 
# for these compares s1 and s2 are class instances ...
def cmpgpa(s1, s2):
    # low to high
    #return cmp(s1.gpa(), s2.gpa())
    # high to low
    return cmp(s2.gpa(), s1.gpa())
def cmpname(s1, s2):
    #return cmp(s1.name(), s2.name())  # problems!!!?
    return cmp(s1.getName(), s2.getName())
def cmpcredits(s1, s2):
    # low to high
    #return cmp(s1.getHours(), s2.getHours())
    # high to low
    return cmp(s2.getHours(), s1.getHours())
 
def main():
    print "This program sorts student grade information"
    # file has line format -->  name tab hours tab qpoints
    file_in = raw_input("Enter the name of the data file: ")
    dfield = raw_input("Enter gpa, name, or credits to sort: ")
    # data is a list of Student class instances
    data = readStudents(file_in)
    if dfield == 'gpa':
        data.sort(cmpgpa)
    elif dfield == 'name':
        data.sort(cmpname)
    else:
        data.sort(cmpcredits)
    file_out = raw_input("Enter a name for the output file: ")
    writeStudents(data, file_out)
    print "The data has been written to", file_out
 
if __name__ == '__main__':
    main()

Here is the module file ...

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
# save this module file as p2gpa.py
import string
#import math
class Student:
    def __init__(self, name, hours, qpoints):
        self.name = name
        self.hours = float(hours)
        self.qpoints = float(qpoints)
    def getName(self):
        return self.name
    def getHours(self):
        return self.hours
    def getQPoints(self):
        return self.qpoints
    def gpa(self):
        return self.qpoints/self.hours
    def name(self):
        return self.name
 
def makeStudent(infoStr):
    if '\t' in infoStr:
        #print string.split(infoStr,"\t")  # test --> ['Computewell, Susan ', '100 ', '400\n']
        name, hours, qpoints = string.split(infoStr,"\t")
    return Student(name, hours, qpoints)
def main():
    filename = raw_input("Enter name the grade file: ")
    infile = open(filename, 'r')
    best = makeStudent(infile.readline())
    for line in infile:
        s = makeStudent(line)
        if s.gpa() > best.gpa():
            best = s
    infile.close()
    print "The best student is:", best.getName()
    print "hours:", best.getHours()
    print "GPA:", best.gpa()
 
# test the module ...
if __name__ == '__main__':
    main()
    #raw_input("Press Enter to go on ... ")
# save this module file as p2gpa.py
import string
#import math
class Student:
    def __init__(self, name, hours, qpoints):
        self.name = name
        self.hours = float(hours)
        self.qpoints = float(qpoints)
    def getName(self):
        return self.name
    def getHours(self):
        return self.hours
    def getQPoints(self):
        return self.qpoints
    def gpa(self):
        return self.qpoints/self.hours
    def name(self):
        return self.name
 
def makeStudent(infoStr):
    if '\t' in infoStr:
        #print string.split(infoStr,"\t")  # test --> ['Computewell, Susan ', '100 ', '400\n']
        name, hours, qpoints = string.split(infoStr,"\t")
    return Student(name, hours, qpoints)
def main():
    filename = raw_input("Enter name the grade file: ")
    infile = open(filename, 'r')
    best = makeStudent(infile.readline())
    for line in infile:
        s = makeStudent(line)
        if s.gpa() > best.gpa():
            best = s
    infile.close()
    print "The best student is:", best.getName()
    print "hours:", best.getHours()
    print "GPA:", best.gpa()
 
# test the module ...
if __name__ == '__main__':
    main()
    #raw_input("Press Enter to go on ... ")

I also would recommend to use extension .dat or .txt for your data files, and make sure the text editor you use leaves the tab separator intact. Some editors replace tabs with spaces!

This little code will produce a correct sample data file called 'in.txt' ...

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
# this will create a sample data file with tab separators
data = [['Computewell, Susan', 100, 400],
['DibbleBit, Denny', 18, 41.5],
['Jones, Jim', 48.5, 155],
['Smith, Frank', 37, 125.33],
['Adams, Henry', 127, 228]]
fout = open("in.txt", "w")
for line in data:
    text = "%s\t%s\t%s\n" % (line[0], line[1], line[2])
    print text,
    fout.write(text)
fout.close()
# this will create a sample data file with tab separators
data = [['Computewell, Susan', 100, 400],
['DibbleBit, Denny', 18, 41.5],
['Jones, Jim', 48.5, 155],
['Smith, Frank', 37, 125.33],
['Adams, Henry', 127, 228]]
fout = open("in.txt", "w")
for line in data:
    text = "%s\t%s\t%s\n" % (line[0], line[1], line[2])
    print text,
    fout.write(text)
fout.close()

Last edited by vegaseat; Jan 7th, 2007 at 3:56 pm. Reason: tab/space goof

May 'the Google' be with you!

•

Join Date: Oct 2006

Posts: 34

Reputation: babutche is an unknown quantity at this point

Solved Threads: 0

babutche

Offline

Light Poster

Re: Sorting a File

Jan 7th, 2007

Thank you all very much for your input!! I will have to study this and see if I can figure out exactly what is going on (so I know what I am doing).

Now I see that one of my major problems was that I was trying to return:

return cmp(s1.name, etc..) instead of return cmp(s1.getName, etc...)
(same thing for Hours)

I have also fixed the other problem where I did not have my quotation marks in the

proper places.

Now I just have to fix a few more things in my files and I should be able to finish!!

Thank You For Helping!!

Last edited by babutche; Jan 7th, 2007 at 4:30 pm. Reason: Added Comments