lundi 5 décembre 2016

How can I improve this code and make it more short and efficient?

print("Hello")
decision=int(input("Enter \"1\" to calculate check digit or enter \"2\" to validate GTIN 8 code:"))
if decision==1:
    GTIN=(input("Enter a 7 digit GTIN number: "))
    length=len(GTIN)
    if(length==7):
        P1=int(GTIN[0])
        P2=int(GTIN[1])
        P3=int(GTIN[2])
        P4=int(GTIN[3])
        P5=int(GTIN[4])
        P6=int(GTIN[5])
        P7=int(GTIN[6])
        GTINT=int(P1*3+P2+P3*3+P4+P5*3+P6+P7*3)
        roundup=round(GTINT, -1)
        GTIN8=int(roundup-GTINT)%10
        print("Your full GTIN-8 code is: "+str(GTIN)+str(GTIN8))
    elif(length!=7):
        print("You have entered something wrong")
elif decision==2:
    GTIN=input("Enter 8 digit GTIN 8 number for validation:")
    length=len(GTIN)
    if(length==8):
        P1=int(GTIN[0])
        P2=int(GTIN[1])
        P3=int(GTIN[2])
        P4=int(GTIN[3])
        P5=int(GTIN[4])
        P6=int(GTIN[5])
        P7=int(GTIN[6])
        P8=int(GTIN[7])
        GTINT=int(P1*3+P2+P3*3+P4+P5*3+P6+P7*3+P8)
        GTINT %=10
        if GTINT==0:
            print("This code is valid")
        elif GTINT!=0:
             print("This code is invalid")
    elif length!=8:
        print("There is something incorrect with what you have entered")
else:
    print("You have entered something wrong")

Someone please help me improve this code. I want to make it more short and concise.

Thanks in advance.

cmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm

Aucun commentaire:

Enregistrer un commentaire