Python selection sort

Question

Question: The code is supposed to take a file (that contains one integer value per line), print the (unsorted) integer values, sort them, and then print the sorted values.

Is there anything that doesn't look right? I know I could test it and I did test the selectionSort, which worked fine. But I don't really know how I could test whether it successfully takes the file and does what its supposed to do.

Thank you

filename=input('Enter file path:')
file = open(filename, 'r')
alist = [int(line) for line in file.readlines()]
print(alist)

def selectionSort(alist):
    for index in range(0, len(alist)):
        ismall = index
        for i in range(index,len(alist)):
            if alist[ismall] > alist[i]:
                ismall = i
        alist[index], alist[ismall] = alist[ismall], alist[index]
    return alist

To test it, why don't you just create a file with numbers and run your script on it? — user861537
– user861537, Commented Mar 1, 2014 at 22:58

Community · Accepted Answer · 2017-05-23 10:31:57Z

1

Your selection sort seems to be correct but the part before it has issues:

(I am assuming this is Python 2.X, if it isn't ignore my answer)

The corrected code:

filename=raw_input('Enter file path:')
file = open(filename, 'r')
alist = [int(line.strip()) for line in file.readlines()]
print(alist)

edited May 23, 2017 at 10:31

CommunityBot

11 silver badge

answered Mar 1, 2014 at 22:57

Nitish

7,4661 gold badge18 silver badges15 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Seeker · Accepted Answer · 2014-03-01 23:00:42Z

0

change your 3rd line to

alist = [int(line.strip()) for line in file.readlines()]

From

alist = [int(line) for line in file.readlines()]

answered Mar 1, 2014 at 23:00

Seeker

5097 silver badges19 bronze badges

Collectives™ on Stack Overflow

Python selection sort

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related