Das deutsche Python-Forum

Code: Alles auswählen

def suchen(adressen, gesucht):
    """Einzelne Adresse suchen"""
    suche = []
    for adresse in adressen:
        if gesucht.lower() in [x.lower() for x in adresse.values()]:
            suche.extend(adresse)
    return suche

Code: Alles auswählen

/usr/local/bin/python3.3m /Users/martin/Documents/Projekte/_stuff/Adressbuch_Python/Adressbuch.py
Traceback (most recent call last):
  File "/Users/martin/Documents/Projekte/_stuff/Adressbuch_Python/Adressbuch.py", line 104, in <module>
    main()
  File "/Users/martin/Documents/Projekte/_stuff/Adressbuch_Python/Adressbuch.py", line 97, in main
    adresse_anzeigen(adressen, suchen(adressen, "Kins"))
  File "/Users/martin/Documents/Projekte/_stuff/Adressbuch_Python/Adressbuch.py", line 70, in adresse_anzeigen
    adresse = suchen(adressen, name)
  File "/Users/martin/Documents/Projekte/_stuff/Adressbuch_Python/Adressbuch.py", line 56, in suchen
    if gesucht.lower() in [x.lower() for x in adresse.values()]:
AttributeError: 'list' object has no attribute 'lower'

Code: Alles auswählen

def adressen_anzeigen(adressen):
    """Alle Adressen werden formatiert ausgegeben"""
    for adresse in adressen:
        adresse_anzeigen(adressen, suchen(adressen, adresse["Name"]))
        print()


def adresse_anzeigen(adressen, adresse):
    """Eine Adresse wird formatiert ausgegeben"""
    for key in adresse:
        print("{}: {}".format(key, adresse[key]))

Code: Alles auswählen

def adressen_anzeigen(adressen):
    """Alle Adressen werden formatiert ausgegeben"""
    for adresse in adressen:
        adresse_anzeigen(adresse)
        print()
 
 
def adresse_anzeigen(adresse):
    """Eine Adresse wird formatiert ausgegeben"""
    for key, value in adresse.items():
        print("{}: {}".format(key, value))

Code: Alles auswählen

def tue_etwas_fuer_ein_objekt(ein_objekt):
    pass

def tue_etwas_fuer_viele_objekte(objekte):
    for ein_objekt in objekte:
        tue_etwas_fuer_ein_objekt(ein_objekt)

Code: Alles auswählen

{"Name": ...., "ID": ...}

Code: Alles auswählen

{{"Name": ...}: "ID}

Code: Alles auswählen

>>> {{"Name": "Python"}: "ID"}
Traceback (most recent call last):
  File "<input>", line 1, in <module>
TypeError: unhashable type: 'dict'

Code: Alles auswählen

while True:
    auswahl = input("Welche Adresse haben Sie gesucht? >> ")
    try:
        auswahl = int(auswahl)
    except (ValueError, IndexError) as e:
        print(e)
    else:
        break

Code: Alles auswählen

KEYS[1]

Code: Alles auswählen

NAME = "name"
VORNAME = "vorname"
...
# und zur Verwendung
adresse[NAME]

Code: Alles auswählen

def suchen(adressen, gesucht):
    """Adresse wird gesucht. Nutzer wählt aus Ergebnissen Suche Eintrag."""
    # ich würde ``suche`` eher ``such_ergbenisse`` oder nur ``ergebnisse`` oder
    # ``gefunden`` nennen.
    suche = []
    for adresse in adressen:
        if gesucht.lower() in [x.lower() for x in adresse.values()]:
            suche.append(adresse)
    # hier müsste eigentlich Schluss sein! Gib das Ergebnis zurück und gut ist!

    # wieso nutzt Du nicht die Funktion für die Ausgabe von *mehreren* Adressen?
    # nichts anderes ist doch in ``suche`` enthalten.
    # eine wichtige Fähigkeit beim Programmieren ist es, wiederkehrende Aufgaben
    # zu kapseln und darauf zurückzugreifen.
    for idx, ergebnis in enumerate(suche, 1):
        print("{}: {}".format(idx, ergebnis))

    # Benutzereingaben / Entscheidungen gehören nicht in eine Funktion,
    # die schlicht "suchen" soll. Du kannst hier gar nicht wissen, ob
    # evtl. auch etwas mit *allen* Resultaten gemacht werden soll!
    auswahl = input("Welche Adresse haben Sie gesucht? >> ")
    try:
        auswahl = int(auswahl)
    except (ValueError, IndexError) as e:
        print(e)
        suchen(adressen, gesucht)
    else:
        return suche[auswahl - 1]