Funktionen richtig formuliert ?

[kaytec]

Hallo birkenfeld !

Ich glaube du hast Probleme. Helfe dir selbst sonst hilft die ein Pädagoge.
Ich bin einer.

gruss frank

[birkenfeld]

Ich glaube eher, du hast keinen Humor oder die falschen Erwartungen an ein Forum. Hier sitzen nicht Leute, die bezahlt werden, deine Fragen zu beantworten, sondern Leute, die in ihrer Freizeit hier an einer Community teilnehmen. Dazu gehört auch, Witze zu machen und zu erkennen und nicht bierernst am Topic zu kleben.

Und was meine Probleme angeht, woher weißt du von den Stimmen in meinem Kopf?

[kaytec]

Ich habe Humor und mache dies auch in meiner Freizeit. Euren Einsatz in diesem Forum ist auch echt nett für alle die von der Materie keine Ahnung haben. Hättest es mir auch erklären können und danach Witze machen - hätten wir zusammen lachen können .
"Erzähle es mir und ich vergesse, zeige es mir und ich erinnere, lass es mich tun und ich verstehe." Konfuzius 554 v. Chr. - ist der einzigste den ich kenne

gruss frank

[birkenfeld]

Abgemacht

[kaytec]

Damit man sehen kann was am Ende rauskommt(Müll:-))
Das steht in der Datei adressbuch.txt:
448.0
Max Mustermann
Musterstrasse 1
10000 Musterhausen
012345678
012345678
max@muster.de
muster
Maxi Muster
Maxstrasse 1
2000 Maxhausen
0123123
0123123
maxi@muster.de
muster

Aus der oberen Zahl berechnet es den letzten Eintrag (für neue Einträge)
und daraus werden auch die Einträge nach Name, Wohnort etc. ausgelesen. Immer in Sprüngen von 32 (deswegen immer diese 32)

Die Anmerkungen von BlackJack werde ich mal versuche zu berücksichtigen - und später mal neu posten - noch mehr Augenkrebs !

gruss frank

[kaytec]

So nun die erste Änderung:

Code: Alles auswählen

liste1=('Name:','Strasse:','Wohnort:',
            'Festnetz:','Handy:','email:','Bemerkung:')
suchk=[]

for i in xrange(len(liste1)):
    print '(%d) %s' % (i + 1, liste1[i]) 

prompt = '-(%s)->:' % ')-('.join('%d'%(i+1) for i in xrange(len(liste1))) 

try:
    keingabe = int(raw_input(prompt))
      
except ValueError:
    return

if 1<= keingabe <len(liste1)+1:

    suchk=[raw_input(liste1[keingabe-1]),keingabe]
    suche_editiere(suchk)

Ich benötige 2 Werte und deswegen erzeuge ich die liste suchk - Wäre es besser zwei einzelne Werte zu übergeben?

danke frank

[rolgal_reloaded]

Warum eigentlich für das Menü überhaupt eine Funktion?

Ist das nicht sinnvoller diesen Code quasi als Hauptprogramm zu schreiben?

LG

rolgal_reloaded

[kaytec]

Habe mal so weit versucht, es richtig zu formulieren.

http://paste.pocoo.org/show/1410/

gruss und dank frank

[kaytec]

Habe einen Fehler in der Suche gefunden

Code: Alles auswählen

def suchen(spkrit ,strlng):

    suchk=[]

    f = file('adressbuch.txt','r')
    
    print
    print 'Suchkriterien:'
            
    for i in range(len(spkrit)):
        
        print '(%d) %s' % (i + 1, spkrit[i]) 

        prompt = '-(%s)->:' % ')-('.join('%d'%(i+1) for i in xrange(len(spkrit)))

    try:
            
        keingabe = int(raw_input(prompt))-1
            
    except ValueError:
        return

    if 0 <= keingabe < len(spkrit):
        
        suchk=[raw_input(spkrit[keingabe]),keingabe]
        
        lpos = strlng + strlng * suchk[1]
        
        zaehler = 0                           
            
        for x in range (int(float(f.read(strlng - 2)) / (strlng * len(spkrit)))):
        
            f.seek(lpos)
            suche=f.read(strlng - 2).upper()
            s=suche.find(suchk[0].upper())
            
            if s >= 0:
                        
                zaehler = zaehler + 1
                lpos = lpos - strlng * suchk[1]            
                ausgabe(lpos, spkrit, strlng, zaehler)
                lpos = lpos + strlng * suchk[1]
                              
            lpos = lpos + strlng * len(spkrit)
                  
    print
    print '=====!!!Suche abgeschlossen!!!====='
            

def editieren(lpos, spkrit, strlng):

    neueint = []

    f = file('adressbuch.txt','r')

    lpos = lpos - strlng * len(spkrit)
       
    for x in range(len(spkrit)):

        f.seek(lpos)
        print'____________________________________'
        print spkrit[x]
        print 'alter Eintrag:',f.read(strlng - 2)
        print
            
        edt=raw_input('neuer Eintrag:')
            
        if edt !='':     
            neueint.insert(x, edt)
                
            for x1 in range(strlng - 2 - len(neueint[x])):
                
                neueint[x] = neueint[x] + ' '
            neueint[x] = neueint[x] + '\n'

        if edt =='':
            
            f.seek(lpos)
            neueint.insert(x, f.read(strlng-2))
            neueint[x] = neueint[x] + '\n'
        f.seek(lpos + strlng)
            
    print '============!!!Fertig!!!============'

    schreiben (spkrit, neueint, strlng, lpos)

[kaytec]

Habe noch eine alphabetische Sortierung eingebaut, doch ob es so Sinn macht? Wird beim Beenden sortiert und steht beim Neustand zu Verfügung.

Namen der Listen sind auch nicht so schön - ist nur zum Testen

Code: Alles auswählen

def beenden(spkrit, strlng):

    f = file('adressbuch.txt','r+')
    fbak = file('sichadressbuch.txt','r+')
    lpos = strlng
    
    liste = []
    a=1
    liste1 = []
    liste2 = []

    f.seek(0)

    for x in range (int(float(f.read(strlng - 2)) / (strlng * len(spkrit)))):
    
        f.seek(lpos)
        liste.insert(x, f.read(strlng - 2) + '#' + str(lpos))
        lpos = lpos + strlng * len(spkrit)

    liste.sort()

    for x in range (len(liste)):

        liste1 = liste1 + liste[x].split('#')
        liste2.insert(x, liste1[a])
        a=a+2   

    f.seek(0)
    fbak.write(f.read(strlng))
    f.seek(0)
    spos = strlng
    
    for x in range (int(float(f.read(strlng - 2)) / (strlng * len(spkrit)))):
        
        lpos = int(liste2[x])

        for x1 in range (len(spkrit)):
            
            f.seek(lpos)
            fbak.seek(spos)
            fbak.write(f.read(strlng))
            lpos = lpos + strlng
            spos = spos + strlng

    fbak.seek(0)
    f.write(fbak.read(strlng))
    fbak.seek(0)
    spos = strlng
    
    for x in range (int(float(fbak.read(strlng - 2)) / strlng )):
         
            fbak.seek(spos)
            f.seek(spos)
            f.write(fbak.read(strlng))
            spos = spos + strlng

    fbak.close()
    f.close()

danke frank

[EyDu]

Da kann man aber noch eine ganze Menge zusammenfassen. Ich sehe hier vier Blöcke, die nahezu den selben Aufbau haben:

Code: Alles auswählen

fbak.seek(spos)
f.seek(spos)
f.write(fbak.read(strlng))
spos = spos + strlng

Und dann noch zwei identische Ausdrücke:

Code: Alles auswählen

range (int(float(f.read(strlng - 2)) / (strlng * len(spkrit))))

Und wegen dem alphabetischen Sortieren solltest du noch mal in die Doku schauen (Sortieren bezgl. Gross- und Kleinschreibung).

[BlackJack]

Und die Namen sind alle ein wenig kryptisch. `neueint` habe ich erst als `neue_int` gelesen ('_' für die Stelle wo ich in Gedanken die Worte getrennt habe). Bei `neuer_eintrag` gäbe es solche Missverständnisse nicht. Und `spkrit`!? Klingt witzig, aber selbst für den der's ursprünglich mal geschrieben hat, ist das nach einem Jahr oder so, vielleicht nicht mehr so klar.

[kaytec]

Hallo EyDu

Das mit den vielen gleichen Schleifen bzw. f.seek etc. ensteht dadurch, dass ich die Daten ein und auslesen muss und ich kann doch erst nach dem erstellen einer Textdatei auf diese zugreifen. Oder sollte ich noch eine dritte Datei erzeugen?

Welche Doku ? Gross - und Kleinschreibung?

Vielen Dank EyDu

Hallo BlackJack

Du magst meine schönen Namen - wie mache ich es richtig ? Verständliche Namen ist schon klar - nur bei meinen "schöpferischen Fähigkeiten" werden die nur noch schlimmer!

Vielen Dank BlackJack

[BlackJack]

Also IMHO ist das Programm einfach furchtbar. Ich weiss das klingt ziemlich hart, aber alleine die Sortierfunktion ist dermassen komisch und umständlich, das man sie am besten von Grund auf neu schreiben sollte.

Zum Beispiel ist beim Anlegen einer Kopie von einer Datei die Struktur derselben doch völlig egal. Wenn man es schon "selbst" macht, dann ist die Grösse der Blöcke in denen kopiert wird unwichtig. Anderseits muss man das nicht selber machen, wenn es in der Standardbibliothek Funktionen wie `shutil.copy()` gibt.

Das gesamte Programm würde davon profitieren wenn der Dateizugriff Datensatzweise und nur in zwei dafür vorgesehenen Funktionen, zum lesen und schreiben, stattfinden würde. Dann würden an allen möglichen Stellen die immer wiederkehrenden Positionsberechnungen und der Aufruf von `file.seek()` wegfallen.

Letztendlich sehe ich den Sinn der Übung nicht ganz. Python ist wesentlich ausdrucksstärker und aktuelle Rechner sind leistungsstärker als die Sprache und das System für das dieses Programm offensichtlich geschrieben ist. Eine weniger umständliche Lösung würde die Daten am Programmanfang einlesen und am Programmende oder auf Wunsch abspeichern und Datensätze schon sortiert einfügen.

[Zap]

wie mache ich es richtig ? Verständliche Namen ist schon klar - nur bei meinen "schöpferischen Fähigkeiten" werden die nur noch schlimmer!

An der Stelle hat jeder in gewisser Art und Weise seinen eigenen Stil
aber wie Blackjack schon sagt. Einzelne Wörter die du zu einem Variablennamen zusammenbaust sollten entweder mit einem '_' oder mit einem Großbuchstaben optisch aufgeteilt werden.

Beispiele:
neuer_eintrag
neuerEintrag (my favorite)

Ich würde dir (wie viele andere auch ) einmal die PEP8
und zweitens die Codebeispiele von DiveIntoPython empfehlen.
Vorallem bei dive into Python lernst du einige Tricks und Kniffe wie man in Python
ordentlich mit den mitgelieferten Batterien arbeitet.

[EyDu]

Beispiele:
neuer_eintrag
neuerEintrag (my favorite)

Aber nicht nur PEP8 lesen sondern auch befolgen :

Function names should be lowercase, with words separated by underscores
as necessary to improve readability.

mixedCase is allowed only in contexts where that's already the
prevailing style (e.g. threading.py), to retain backwards compatibility.

[Zap]

Hm, ich hatte noch dran gedacht was dazu zu schreiben aber es dann doch vergessen

Ich muss ganz ehrlich sagen das es meiner Meinung nach echt sch** aussieht alles mit Underscores zu bauen.
Spätestens wenn es mehr als 3 Einzelwörter sind.
Ich hatte mit Dive Into Python meinen Einstieg in Python gehabt und bin seit dem Anhänger der Camelcase-Syntax
Beispiel zu meiner Verteidigung

Aber hast recht, man sollte meine schlechten Gewohnheiten nicht weitergeben

[kaytec]

Danke für die vielen Antworten !

Ich brauche diese Prgramm nicht wirklich - hatte eine Idee und wollte sie mit meinen Wissen umsetzen. Das ist noch sehr gering, doch erst alles zu lesen und dann etwas versuchen ist nicht wirklich spannend. Es gibt bestimmt tolle Fuktionen in der Standartbibliothek, doch ich kenne sie halt nicht. Was nicht bedeutet, dass ich es nicht weiter versuchen möchte. Was ist eigentlich IMHO ?

Na ok - Wie sollte ich einen Neuanfang machen ?

Gruss und Dank Frank

[CM]

IMHO

[kaytec]

Danke CM !

Ich vertstehe evt. ja noch die einzelnen Wörter - was es aber bedeutet ? Ich war vor 20 Jahren mal auf der Hauptschule. Ich sollte mal einen Fremdsprachkurs besuchen !

Na gut - gehe jetzt mal arbeiten.

gruss und dank frank