Python repeating CSV file

Question

I'm attempting to load numerical data from CSV files in order to loop through the calculated data of each stock(file) separately and determine if the calculated value is greater than a specific number (731 in this case). However, the method I am using seems to make Python repeat the list as well as add quotation marks around the numbers ('500'), as an example, making them strings. Unfortunately, I think the final "if" statement can't handle this and as a result it doesn't seem to function appropriately. I'm not sure what's going on and why Python what I need to do to get this code running properly.

    import csv
    stocks = ['JPM','PG','GOOG','KO']
    for stock in stocks:
        Data = open("%sMin.csv" % (stock), 'r')
        stockdata = []
        for row in Data:
           stockdata.extend(map(float, row.strip().split(',')))
           stockdata.append(row.strip().split(',')[0])
        if any(x > 731 for x in stockdata):
            print "%s Minimum" % (stock)

Why do you import the library csv, but not user the csv.reader class? — DaveP
– DaveP, Commented Jul 16, 2012 at 6:26
for stock in stocks: means stock would be a str, so it will have no .extend() method. This can't be the code you are running — John La Rooy
– John La Rooy, Commented Jul 16, 2012 at 6:32
Please put the code which we can run and generate your issue in our system. Then and only then we can solve your problem. As mention in above comment stock is string and it has no .extend() method. — NPatel
– NPatel, Commented Jul 16, 2012 at 6:37
As well as the code you're actually running - some sample data would be useful. — Jon Clements
– Jon Clements, Commented Jul 16, 2012 at 6:38

Jon Clements · Accepted Answer · 2012-07-16 06:55:09Z

1

Currently you're adding all columns of each row to a list, then adding to the end of that, the first column of the row again? So are all columns significant, or just the first?

You're also loading all data from the file before the comparison but don't appear to be using it anywhere, so I guess you can shortcut earlier...

If I understand correctly, your code should be this (or amend to only compare first column).

Are you basically writing this?

import csv

STOCKS = ['JPM', 'PG', 'GOOG', 'KO']

for stock in STOCKS:
    with open('{}Min.csv'.format(stock)) as csvin:
        for row in csv.reader(csvin):
            if any(col > 731 for col in map(float, row)):
                print '{} minimum'.format(stock)
                break

answered Jul 16, 2012 at 6:55

Jon Clements

143k34 gold badges254 silver badges288 bronze badges

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

1 Comment

user1526586 Over a year ago

That would be it exactly. Thanks so much. My Python needs some practice to say the least.

Collectives™ on Stack Overflow

Python repeating CSV file

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related