Python Script tut nicht auf der Console des rasberry pi aber in Thonny Python IDE

BaumKL · Jun 27th 2020

ich habe auf meinem pc ein Python Programm geschriebene und das hat auch alles funktioniert dann habe ich es auf den rasberrypi geladen wo es nur über den Editor funktioniert versuche ich das Sript über die Console auszuführen bekomme ich einen Fehlermeldung

Code

Traceback (most recent call last): File "/var/www/html/Zeiterfassung/Main.py", line 46, in <module> In ( ) File "/var/www/html/Zeiterfassung/Main.py", line 33, in In i = input ( "Wie heist du ? " ) File "<string>", line 1, in <module> NameError: name 'USchi' is not defined

das ist mein Cod

Code

import datetime
import Converter

St = 1
Stop = ['Stop']

A = ['Uschi']
VA = 0
B = ['Helga']
VB = 0
C= ['Mete']
VC=0
D= ['Andreas']
VD=0
E= ['Maria']
VE=0
F= ['Thomas']
VF=0
G= ['Konrad']
VG=0
H= ['Amed']
VH=0
J= ['Zusatz1']
VJ=0
K= ['Zusatz2']
VK=0
L=['Zusatz3']
VL=0


def In():
    global i
    i = input ( "Wie ist dein Name? " )



def Drucken():
    print ( datetime.datetime.now ( ).strftime ( "%A  %d.%m.%Y - %H:%M" ) )
    file = open ("output/"+ i + ".txt", "a" )
    file.write ( datetime.datetime.now ( ).strftime ( "%A  %d.%m.%Y - %H:%M" ) )
    file.write ( "        " )
    file.close ( )


while St == 1:

    In ( )
    
    if i in Stop :
        St = 0

    if i in 1:
        Drucken()
        VA = VA + 1
        if VA == 2 :
            VA = 0
            Converter.CUschi()
   
    if i in B:
        Drucken()
        VB = VB + 1
        if VB == 2:
            VB = 0
            Converter.CHelga()
        
    if i in C:
        Drucken()
        VC = VC + 1
        if VC == 2:
            VC = 0
            Converter.CMete()
        
    if i in D:
        Drucken()
        VD = VD + 1
        if VD == 2:
            VD = 0
            Converter.CAndreas()
        
    if i in E:
        Drucken()
        VE = VE + 1
        if VE == 2:
            VE = 0
            Converter.CMaria()

    if i in F:
        Drucken()
        VF = VF + 1
        if VF == 2:
            VF = 0
            Converter.CThomas()

    if i in G:
        Drucken()
        VG = VG + 1
        if VG == 2:
            VG = 0
            Converter.CKonrad()

    if i in H:
        Drucken()
        VH = VH + 1
        if VH == 2:
            VH = 0
            Converter.CAmed()
        
    if i in J:
        Drucken()
        VJ = VJ + 1
        if VJ == 2:
            VJ = 0
            Converter.CZusatz1()
        
    if i in K:
        Drucken()
        VK = VK + 1
        if VK == 2:
            VK = 0
            Converter.CZusatz2()
        
    if i in L:
        Drucken()
        VL = VL + 1
        if VL == 2:
            VL = 0
            Converter.CZusatz3()

Display More

Vielen Dank führ ihre Antworte

hyle · Jun 27th 2020

Quote from BaumKL

USchi

Was passiert, wenn Du die Uschi richtig schreibst? Denn die USchi ist nicht definiert, aber das steht ja in der Fehlermeldung.

MistyFlower59469 · Jun 27th 2020

Das ist ein Versionsproblem. Du musst das Programm mit Python 3!!!!!! ausfuehren. Du benutzt aber Python 2, und da funktioniert das input() leider anders.

Und dann ist das Programm wirklich sehr schlecht geschrieben. Diese ganzen schlecht bennanten Variablen, die dann einfach nur die Namen beinhalten - das ist unuebersichtlich und von hinten durch die Brust ins Auge. Leider zeigst du nich twie der Converter programmiert ist, ich bin sicher, auch den koennte man besser programmieren. Doch wenn man das als gegeben annimmt, kann man das einfach so machen:

Python

import datetime
from collections import defaultdict
#import Converter

# Das hier ist einfach nur ein Fake
# des eigentlichen Converter Moduls
# das wir ja nicht kennen
class Converter:

    def __init__(self):
        self._known_users = [
                             'Uschi',
                             'Helga',
                             'Mete',
                             'Andreas',
                             'Maria',
                             'Thomas',
                             'Konrad',
                             'Amed',
                             'Zusatz1',
                             'Zusatz2',
                             'Zusatz3',
                             ]

    def __getattr__(self, name):
        # name hat die Form "C<User>"
        if name.startswith("C") and name[1:] in self._known_users:
            return lambda: print(f"Converter tut was fuer {name}")
        raise AttributeError

Converter = Converter()


def Drucken(name):
    zeitpunkt = datetime.datetime.now().strftime("%A  %d.%m.%Y - %H:%M")
    print(zeitpunkt)
    # ACHTUNG, GEAENDERT!
    with open("/tmp/{}.txt".format(name), "a") as outf:
        outf.write(zeitpunkt)
        outf.write("        ")


def main():
    schliessvorgaenge = defaultdict(int)
    while True:
        command = input( "Wie ist dein Name? ")
        if command == "Stop":
            break
        try:
            action = getattr(Converter, "C{}".format(command))
        except AttributeError:
            pass
        else:
            Drucken(command)
            # wir haben einen echten Schliessvorgang
            schliessvorgaenge[command] += 1
            if schliessvorgaenge[command] % 2 == 0:
                action() # wirklich die Aktion nur bei jedem zweiten mal ausfuehren


if __name__ == '__main__':
    main()

Display More

BaumKL · Jun 27th 2020

Danke für die Hilfe

ich verstehe nicht die variablen

__init__

__getattr__

und den Schließvorgang weil ich möchte doch gar nicht das Programm schlissen das soll doch einen Dauerschleife sein .

wehre mega nett wen du mir nochmal helfen könntest ich bin noch nicht so gut im Programmiren .

MistyFlower59469 · Jun 27th 2020

Das sind keine Variablen, sondern Methoden einer Klasse. Und ich habe doch einen grossen, dicken Kommentar darueber geschrieben, das es sich dabei um ein gefaktes Converter-Modul handelt. Weil du das Converter-Modul nicht gezeigt hast.

Und das Programm laeuft in einer Dauerschleife. Hast du es wenigestens mal ausprobiert? Dann haettest du das selbst gemerkt. Den Begriffe Schliessvorgang habe ich mir nur ausgedacht, weil ich das fuer eine Art Schliessanlage oder aehnliches gehalten haben, aber am Ende ist das doch egal.

BaumKL · Jun 27th 2020

Ja an das sohlte eine Schließanlage werden die die zeit in drei Dokumenten abspeichert wann wer das haus betreten hatt oder verlassen .

der Conveyer mint die Datei mit der Ankunfts zeit und der Abgangs zeit und fügt sie an zwei neue Tabellen an die ein ist die Wochen übersiecht die nach einer Woche gelehrt wird un die andre die totale . Nach diesem vornag wird die Datei bei der die ankomm und geh Zeiten eintragen werden wieder gelöscht diese Datei wird also nur als zwischen speicher verwendet .

Code

def CUschi() :
    file1 = open ( "output/Uschi.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Uschiprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open("send/Uschisend.txt","a")
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Uschi.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CHelga() :
    file1 = open ( "output/Helga.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Helgaprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Helgasend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Helga.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CMete() :
    file1 = open ( "output/Mete.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Meteprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Metesend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Mete.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CAndreas() :
    file1 = open ( "output/Andreas.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Andreasprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Andreassend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Andreas.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CMaria() :
    file1 = open ( "output/Maria.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Mariaprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Mariasend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Maria.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CThomas() :
    file1 = open ( "output/Thomas.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Thomasprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Thomassend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Thomas.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CKonrad() :
    file1 = open ( "output/Konrad.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Konradprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Konradsend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Konrad.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CAmed() :
    file1 = open ( "output/Amed.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Amedprint.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Amedsend.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Amed.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CZusatz1() :
    file1 = open ( "output/Zusatz1.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Zusatz1print.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Zusatz1send.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Zusatz1.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CZusatz2() :
    file1 = open ( "output/Zusatz2.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Zusatz2print.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Zuastz2send.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Zusatz2.txt","w")
    filereset.write(" ")
    filereset.close()
    
def CZusatz3() :
    file1 = open ( "output/Zusatz3.txt"   , "r" )
    a = file1.read ( )
    b = a[0:30]
    c = a[34:67]
    file2 = open ( "print/Zusatz3print.txt", "a" )
    file2.write ( b + "     " )
    file2.write ( c + "\n" )
    file2.close()
    file3 = open ( "send/Zusatz3send.txt", "a" )
    file3.write ( b + "     " )
    file3.write ( c + "\n" )
    file3.close ( )
    filereset = open("output/Zusatz3.txt","w")
    filereset.write(" ")
    filereset.close()

Display More

MistyFlower59469 · Jun 27th 2020

Das Ding macht doch im Grunde nichts anderes, als deine anderes Programm. Naemlich Dateien schreiben. Warum machst du das nicht gleich da?

Und man schreibt nicht dutzende male das gleiche hin. Man schreibt eine Funktion, die einen Parameter "namen" bekommt, und dann schreibt man den Code so, dass er den Parameter benutzt. So wie ich das vorgemacht habe.

Zu guter Letzt: diese Dateien alle so zu schreiben ist eh unnoetig, es reicht EINE Datei, die schon in dem Hauptprogramm geschrieben wird. Wenn man daraus eine Wochenansicht erzeugen will, dann liest man die gesamten Daten ein zB in Excel, und erstellt einen aggregierten Report.