Python Credit Card Validation

Ich bin ein Anfänger-Python-Anfänger, und ich bin derzeit auf Luhn-Algorithmus zum prüfen der credit card validation. Ich schrieb die meisten der code, aber ich bin stecken mit 2 Fehler, die ich bekommen 1. ist num verwiesen wird vor der Zuweisung. 2. ich bin immer object ist vom Typ '_io.TextIOWrapper' has no len(). Weitere Hilfe/Beratung wird sehr geschätzt.

Diese Schritte sind für Luhn-Algorithmus (Mod10 Check)

Doppel-jedes zweite Ziffer von rechts nach Links. Wenn diese "Verdoppelung" Ergebnisse in eine zwei-stellige Zahl, fügen Sie die zwei-stellige
Zahl um eine einzelne Ziffer.
Nun fügen Sie alle einstelligen zahlen von Schritt 1.
Fügen Sie alle Ziffern in den ungeraden Plätze von rechts nach Links in die Nummer der Kreditkarte.
Summe der Ergebnisse aus den Schritten 2 & 3.
Wenn das Ergebnis von Schritt 4 ist durch 10 teilbar, wird die Kartennummer gültig ist; andernfalls ist er ungültig.

Hier ist, was meine Ausgabe soll

Card Number         Valid / Invalid
--------------------------------------
3710293             Invalid
5190990281925290    Invalid
3716820019271998    Valid
37168200192719989   Invalid
8102966371298364    Invalid
6823119834248189    Valid

Und hier ist der code.

def checkSecondDigits(num):
    length = len(num)
    sum =  0
    for i in range(length-2,-1,-2):
      number = eval(num[i])
      number = number * 2
      if number > 9:
          strNumber = str(number)
          number = eval(strNumber[0]) + eval(strNumber[1])
          sum += number
      return sum

def odd_digits(num):
    length = len(num)
    sumOdd = 0
    for i in range(length-1,-1,-2):
        num += eval(num[i])
    return sumOdd

def c_length(num):
    length = len(num)
    if num >= 13 and num <= 16:
    if num [0] == "4" or num [0] == "5" or num [0] == "6" or (num [0] == "3" and num [1] == "7"):
        return True
    else:
        return False


def main():
    filename = input("What is the name of your input file? ")
    infile= open(filename,"r")
    cc = (infile.readline().strip())
    print(format("Card Number", "20s"), ("Valid /Invalid"))
    print("------------------------------------")
    while cc!= "EXIT":
        even = checkSecondDigits(num)
        odd = odd_digits(num)
        c_len = c_length(num)
        tot = even + odd

        if c_len == True and tot % 10 == 0:
            print(format(cc, "20s"), format("Valid", "20s"))
        else:
            print(format(cc, "20s"), format("Invalid", "20s"))
        num = (infile.readline().strip())

main()

Sollten Sie die traceback -, nicht nur die Fehlermeldung
even = checkSecondDigits(num)... Aktivieren Sie das Kontrollkästchen. Was ist Zahl? Das ist Ihr Erster Fehler
Traceback (most recent call last): Zeile 58, in <module> main () - Zeile 48, die im wesentlichen auch = checkSecondDigits(num) UnboundLocalError: lokale variable 'num' referenced before assignment
Wenn ich den code bekomme ich diesen Fehler
überprüfen Sie Ihre while, wo ist dieser num variable kommen?
eval(num[i]) <-- Gute Herr, WARUM?!? Mit eval hier ist mit einer Atombombe zu swat Fliegen; es funktioniert, aber es ist eine wirklich schlechte Idee. Sie wissen, sollten die Werte werden integer-Ziffern, verwenden Sie einfach int(num[i]); es ist schneller, sicherer und nicht aufrufen Cthulhu. In anderen Fällen, z.B. strNumber = str(number), number = eval(strNumber[0]) + eval(strNumber[1]) Sie tun können, Mathematik und vermeiden Sie die Konvertierung bei allen, z.B. number -= 9 (für 10-18, die Summe der zahlen ist die gleiche wie einfach nur zu subtrahieren 9) oder weniger geschickt number = number // 10 + number % 10.
Gut, dass die ganze Funktion gegeben wurde, um uns alle von unserem professor, so idk lol

InformationsquelleAutor Apurva Patel | 2016-11-19

Du nur vergessen zu initialisieren num

def main():
    filename = input("What is the name of your input file? ")
    infile= open(filename,"r")
    # initialize num here
    num = cc = (infile.readline().strip())
    print(format("Card Number", "20s"), ("Valid /Invalid"))
    print("------------------------------------")
    while cc!= "EXIT":
        even = checkSecondDigits(num)
        odd = odd_digits(num)
        c_len = c_length(num)
        tot = even + odd

        if c_len == True and tot % 10 == 0:
            print(format(cc, "20s"), format("Valid", "20s"))
        else:
            print(format(cc, "20s"), format("Invalid", "20s"))
        num = cc = (infile.readline().strip())

Er tatsächlich initialisiert es unter dem anderen.
Du bist ein Lebensretter bro. Ich danke Ihnen sehr
Könnte einfach umbenennen cc num

InformationsquelleAutor 2ps

Dies ist eine sehr vereinfachte version von code ist es, basierend auf lunh ' s Algorithmus

def validator(n):

    validatelist=[]

    for i in n:
        validatelist.append(int(i))


    for i in range(0,len(n),2):


        validatelist[i] = validatelist[i]*2

        if validatelist[i] >= 10:

            validatelist[i] = validatelist[i]//10 + validatelist[i]%10


    if sum(validatelist)%10 == 0:
        print('This a valid credit card') 

    else:
        print('This is not valid credit card')

def cardnumber():

    result=''
    while True:
        try:
            result = input('Please enter the 16 digit credit card number : ')

            if not (len(result) == 16) or not type(int(result) == int) :
                raise Exception

        except Exception:    
            print('That is not a proper credit card number. \nMake sure you are entering digits not characters and all the 16 digits.')
            continue

        else:
            break


    return result

def goagain():
    return input('Do you want to check again? (Yes/No) : ').lower()[0] == 'y'

def main():

    while True:

        result = cardnumber()
        validator(result)


        if not goagain():
            break

if __name__ == '__main__':
    main()

InformationsquelleAutor Refah Mahaboob Shaik

0

Alten thread, aber die Antwort die mir sorgen... und das eigentliche Problem nicht erkannt.
Tatsächlich, der Fehler ist, dass Sie verwendet haben, die id (num) für die parameter bei der Definition checkSecondDigits als die id/der name des Arguments beim Aufruf der Funktion in der mainline. Die Funktion sollte in main() aufgerufen durch
auch = checkSecondDigits(cc), also der Wert im cc (das ist das argument) übergeben wird in num (als parameter) für die Verwendung innerhalb der Funktion.
Das gleiche rookie Fehler gemacht mit odd_digits und cc_length.

Diese Frage (und die zunächst vorgeschlagene Antwort) zeugt von einem grundsätzlichen mis-Verständnis von übergeben von Argumenten an Parameter...
Die vorgeschlagene 'Deklaration' der num-nur blendet diesen Fehler/Missverständnis und auch verbirgt die lokalen und globalen Gültigkeitsbereiche von num (soll nur lokal sein) und cc (die Globale) während also der Vorschlag funktioniert in diesem Fall, funktioniert es für den falschen Grund und ist schlechter Stil und schlechte Programmierung.
- Dies gilt nicht bieten eine Antwort auf die Frage. Sobald Sie ausreichend Ruf Sie werden in der Lage sein Kommentar auf jedem post; stattdessen Antworten, die nicht erfordern die Klärung von asker. - Nach Bewertung
- Naja, eigentlich ist es NICHT DIE Antwort. Es besagt GENAU das, was der original-poster den Fehler und beschreibt GENAU, was zu tun ist, um es zu beheben. Schade, die Mitwirkenden, die scheinbar ausreichende reputation kann die FALSCHE Antwort geben! Klar Ruf != wissen, unbedingt. Posting-code, der falsch diagnostiziert das problem und schlägt dann eine FALSCHE Antwort, muss genannt werden. Daumen nach unten auf stackoverflow reputation, wenn dies allgemein üblich ist...
InformationsquelleAutor zedman350

Weiter,
num nicht erscheinen sollte irgendwo in main (), wie es sein sollte lokale (nur erscheinen in der) die Funktionen aufgerufen...
Die Letzte Zeile in diesem code sollte die gleiche wie die erste, aber die Letzte Zeile weist fälschlicherweise die Daten zu num anstelle von cc

cc = (infile.readline().strip())
print(format("Card Number", "20s"), ("Valid /Invalid"))
print("------------------------------------")
while cc!= "EXIT":
    even = checkSecondDigits(num)
    odd = odd_digits(num)
    c_len = c_length(num)
    tot = even + odd

    if c_len == True and tot % 10 == 0:
        print(format(cc, "20s"), format("Valid", "20s"))
    else:
        print(format(cc, "20s"), format("Invalid", "20s"))
    num = (infile.readline().strip())

InformationsquelleAutor zedman350

können Sie meinen code für card validation es ist 100% dynamisch, weil Sie die card-Struktur gespeichert ist, in der CSV-Datei, so ist es einfach zu aktualisieren, hier ist der code, der auf GitHub Profil, python-Datei link, code-Erläuterung-Datei link und CSV-Format für die Datendatei link

python-code:

# -*- coding: utf-8 -*-
"""
Created on Tue Sep 10 20:55:30 2019
@author: [email protected]
"""


import csv
import numpy as np

#csv file imported and storf in reader
reader = csv.DictReader(open("card_data.csv"))

#input card number
card_number = input("Enter the card No: ")

#global variable declaration
min_digits=0
max_digits=0
card_number_list = list(card_number)
card_number_list_reverse=card_number_list[::-1]
card_number_length=len(card_number_list)
first_digit = int(card_number_list[0])

#global variable for final output
card_provider_list_number = 0
result_found = False
card_number_digits = 0
mit_name=""

#list
start=[]
end=[]
name=[]
c_d=[]
number_length=[]
min_max_digits_list=[]

#append the list from csv
for raw in reader:
    start.append(raw['start'])
    end.append(raw['end'])
    name.append(raw['name'])
    c_d.append(raw['c_d'])
    number_length.append(raw['number_length'])

#initialize the value of min_digits & max_digits
def min_max_digits():
    global min_digits
    global max_digits
    for i in range(len(start)):
        available_length=number_length[i].split(',')
        for j in range(len(available_length)):
            min_max_digits_list.append(available_length[j])
    min_max_digits_array = np.array(min_max_digits_list) 
    np.unique(min_max_digits_array)
    min_digits=int(min(min_max_digits_array))
    max_digits=int(max(min_max_digits_array))

#list to int
def list_to_int(noofdigits): 
    str1 = ""
    return int(str1.join(noofdigits))

#card validation
def iin_identifier():
    first_six_digit = list_to_int(card_number_list[0:6])
    for i in range(len(start)):
        if(first_six_digit >= int(start[i]) and first_six_digit <= int(end[i])):
            available_length=number_length[i].split(',')
            for j in range(len(available_length)):
                if(card_number_length == int(available_length[j])):
                    global card_provider_list_number
                    card_provider_list_number = i
                    global card_number_digits
                    card_number_digits = available_length[j]
                    global result_found
                    result_found = True

#Major Industry Identifier (MII) identification
def mit_identifier():
    global first_digit
    global mit_name
    switcher = { 
         1: "Airlines",
         2: "Airlines",
         3: "Travel and Entertainment",
         4: "Banking and Financial Services",
         5: "Banking and Financial Services",
         6: "Merchandising and Banking",
         7: "Petroleum",
         8: "Health care, Telecommunications",
         9: "National Assignment"
    }
    mit_name=switcher.get(first_digit, "MIT Identifier Not Found") 

#Luhn Algorithm or modulo-10 Algorithm
def luhn_algorithm():
    for i in range(card_number_length):
        if(i%2!=0 and i!=0):
            card_number_list_reverse[i]=int(card_number_list_reverse[i])*2
            #print(str(i)+" "+ str(card_number_list_reverse[i]))
            if(len(str(card_number_list_reverse[i]))==2):
                even_number_2=list(str(card_number_list_reverse[i]))
                card_number_list_reverse[i] = int(even_number_2[0])+int(even_number_2[1])
                #print("\tsubsum "+str(i)+" "+str(card_number_list_reverse[i]))
        else:
            card_number_list_reverse[i]=int(card_number_list_reverse[i])
    division_int = int(sum(card_number_list_reverse)/10)
    division_float=sum(card_number_list_reverse)/10
    if(division_int-division_float==0):
        return True

#initial level number length validation
def card_number_validation():
    min_max_digits()
    if(card_number_length>= min_digits and card_number_length <= max_digits and first_digit != 0):
        iin_identifier()
        mit_identifier()
        if(result_found and luhn_algorithm()):
            print("\nEntered Details are Correct\n")
            print("\nHere are the some details we know about you card")
            print("\nNo: "+card_number)
            print("\nIssuing Network: "+name[card_provider_list_number])
            print("\nType: "+c_d[card_provider_list_number]+" Card")
            print("\nCategory of the entity which issued the Card: "+mit_name)
        else:
            print("\nCard Number is Invalid\nPlease renter the number!\n")
    else:
        print("\nCard Number is Invalid\n")

#method called to run program
card_number_validation()

InformationsquelleAutor Preyash Patel

-1

Ersten, vielleicht sollten Sie entfernen Sie die zusätzlichen Zeichen:

def format_card(card_num):
    """
    Formats card numbers to remove any spaces, unnecessary characters, etc
    Input: Card number, integer or string
    Output: Correctly formatted card number, string
    """
    import re
    card_num = str(card_num)
    # Regex to remove any nondigit characters
    return re.sub(r"\D", "", card_num)

Nach überprüfen, ob die Kreditkarte gültig ist, mit dem Luhn-Algorithmus:

def validate_card(formated_card_num):
    """
    Input: Card number, integer or string
    Output: Valid?, boolean
    """
    double = 0
    total = 0

    digits = str(card_num)

    for i in range(len(digits) - 1, -1, -1):
        for c in str((double + 1) * int(digits[i])):
            total += int(c)
        double = (double + 1) % 2

    return (total % 10) == 0

Dies ist eine Implementierung der Luhn-Algorithmus, ja, aber OP war um Hilfe zu bitten, herauszufinden, Ihre Fehler, nicht Fragen für eine Implementierung von it

InformationsquelleAutor Vadim Kuzmyk

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.