Code: Alles auswählen
Norbert 1234
Herbert 5678
usw.Code: Alles auswählen
while True:
try:
a = int(input("Geben die die Zahlen ein"))
b = open("pin.txt").read()
if str(a) in b:
print("Hallo")
else:
print("Falscher PIN")
except:
print("Fehler")Code: Alles auswählen
def ist_4_stellig(zahl):
return 999<zahl<10000
print ist_4_stellig( 567)
print ist_4_stellig( 1567)
print ist_4_stellig(11567)Code: Alles auswählen
import math
def ist_4_stellig(zahl):
return int(math.log10(zahl))==3
print ist_4_stellig( 567)
print ist_4_stellig( 1567)
print ist_4_stellig(11567)
Eben, denn dann wird das Validieren kinderleicht:BlackJack hat geschrieben:Man könnte auch einfach die unsinnige Umwandlung in eine Zahl sein lassen.
Code: Alles auswählen
def is_pin_valid(pin):
return len(pin) == 4 and all(c.isdigit() for c in pin)
def test_is_pin_valid():
assert is_pin_valid("1234"), "'1234' ist ein gültiger PIN"
assert not is_pin_valid("123"), "'123' hat nur drei Stellen und ist damit zu kurz!"
assert not is_pin_valid("12345"), "'12345' hat fünf Stellen und ist damit zu lang!"
assert not is_pin_valid("a234"), "'a234' enthält andere Zeichen als Ziffern!"
) 
Code: Alles auswählen
def is_pin_valid(pin):
return len(pin) == 4 and pin.isdigit()
Verbuchen wir obiges also unter dem Gesichtspunkt "Refactoring" des TDD Ansatzes 
Code: Alles auswählen
from time import sleep
def is_pin_valid(PIN):
return len(PIN) == 4 and PIN.isdigit()
while True:
korrekt = 0
for i in range(1,6):
try:
PIN = input("Bitte geben sie Ihren Pin ein: ")
except:
print("Es dürfen nur Zahlen verwendet werden!")
continue
data = open("pin.txt").read()
if PIN in data and is_pin_valid(PIN):
print("Der PIN war korrekt")
korrekt = 1
sleep(5)
continue
else:
print("Dieser Pin ist nicht korrekt")
if korrekt != 1:
print("5 Fehlversuche!")
sleep(5)Code: Alles auswählen
from time import sleep
def is_pin_valid(pin):
return len(pin) == 4 and pin.isdigit()
speicher = [0,0,0,0,0,0]
gefunden = 0
while True:
for i in range(1,6):
pin = input("Bitte geben sie Ihren Pin ein: ")
data = open("pin.txt").read()
if pin in data and is_pin_valid(pin):
print("Der PIN war korrekt")
for k in range(6):
if speicher[k] == pin:
speicher[k] = 0
gefunden = True
print(speicher)
break
for j in range(6):
if speicher[j] == 0 and not gefunden:
speicher[j] = pin
print(speicher)
break
gefunden = False
sleep(0.5)
break
else:
print(i)
print("Dieser Pin ist nicht korrekt")
if i > 4 :
print("5 Fehlversuche!")
sleep(0.5)
Also wirklich *eine* PIN-Datei für das gesamte System? Wozu das dann? Ich kapiere da immer noch nicht den Sinn! Wenn das nur ein Benutzer bedienen kann, wozu dann überhaupt ein PIN?doggy hat geschrieben: Name PIN
Also
Herbert Schuhmacher 2323 z.B.
Aber wieso so kompliziert? Und wieso wird einmal der PIN reingeschrieben und *direkt* danach wieder entfernt?doggy hat geschrieben: Die for-Schleifen sollen eine Liste beschreiben. Ich habe überlegt, diese Liste später zum zuweisen einer Kabine zu nutzen.
Ich empfinde es zwar merkwürdig, dass dann PIN zu nennen, aber ok. Du suchst hier nun einfach ein Dictionary! Damit kannst Du das viel einfacher handhaben, da es die Abbildung "PIN → Kabine" *direkt* unterstützt und Du nicht implizit über einen Index eine Verbindung schaffen musst! Da brauchst Du nicht mehr iterieren, sondern kannst ganz einfach über die PIN direkt herausfinden, welche Kabine jemand hat. (Wie auch immer eine Kabine datentechnisch aussieht!)doggy hat geschrieben: Das soll dazu dienen, dass derselbe Benutzer bei erneuter eingabe seines PINs auch wieder seine Kabine und nicht irgendeine andere öffnet.
Das heißt die erste Zeile der Liste steht für die Kabine 1, die zweite für Kabine 2 und so weiter.