Python string comparison -noobie

[zynif]

Καλησπέρα έχω ένα αρχείο του οποίου κάθε γραμμή είναι όπως φαίνεται παρακάτω.

 

low,low,5more,more,med,high,acc
low,low,5more,more,big,low,unacc
low,low,5more,more,big,med,good
low,low,5more,more,big,high,vgood

 

Προσπαθώ να το διαβάσω με το παρακάτω python script

class Instance:
    Vals=[]
    Sign=0;
    
        
    
if __name__ == '__main__':
   #read file
    for line in open('/tmp/car.data','r'):
        #print(line)
        ins=Instance()
        ins.Vals=line.split(',')
        print(ins.Vals[-1])
        if 'vgood'==ins.Vals[-1]:
            ins.Sign=4
        if 'good'==ins.Vals[-1]:
            ins.Sign=3
        if 'acc'==ins.Vals[-1]:
            ins.Sign=2
        if 'unacc'==ins.Vals[-1]:
            ins.Sign=1
        ins.Vals.pop()
        print(ins.Sign)
            
            

To θέμα είναι όμως ότι δεν γίνεται σωστά η σύγκριση.Δηλαδή το ins.Sign είναι πάντα μηδέν ενώ θα έπρεπε να πάρει τιμές από 1 έως 4.Τι κάνω λάθος ;

[b_real]

Αρχικά έχεις δηλώσει λάθος την κλάση σου.

Κοίτα εδώ http://stackoverflow.com/questions/10004850/python-classes-and-oop-basics

Επισης δεν χρειαζεσαι class για να κανεις αυτο που βλεπω, 2 μεταβλητες (vals, sign) αρκουν.

Τελος θες if... elif ...elif... και έτσι οπως το κανεις θα παιζει αλλα ειναι λαθος στην λογικη.

[pmav99]

with open("/path/to/filename", "r") as f:

    lines = f.readlines()



for line in lines:

    last = line.split(",")

    if last == "vgood":

        sign = 4

    elif last == "good":

        sign = 3

    elif ...

[b_real]

Το last ηθελε [-1]

 

with open("/path/to/filename", "r") as f:
    lines = f.readlines()

for line in lines:
    last = line.split(",")[-1]
    if last == "vgood":
        sign = 4
    elif last == "good":
        sign = 3
    elif ...

 

[pmav99]

Σωστός. Στον browser το έγραψα κατευθείαν, τέτοια ώρα που ήταν και όλας, πάλι καλά.

[iceblade]

Το πολλαπλό if καλύτερα με dictionary κατά τη γνώμη μου

[pmav99]

@iceblade

Συμφωνούμε, αλλά αν δεις το αρχικό post δεν είναι αυτό το πρόβλημα του OP