Python TCP Verbindung einmal senden dann schließen

[Sven (Gast)]

Hallo,
ich habe hier folgenden Code:

import socket
import sys

#Globale Variablen
port_TCP = 5000


#Main function
print('Program is started')

s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
print('Socket created')
s.bind(('', port_TCP))
s.listen(1)
try:
    while True:
        komm, addr = s.accept()
        while True:
            command = komm.recv(1024)
            if not command:
                komm.close
                break
            print 'Nachricht [%s]: %s' % (addr[0], command)
            komm.send('Befehl erhalten: ' + command)
            if command == 'NULL':
                komm.close()
        
finally:
    s.close()

print(command)                  
print ('123')
raw_input('Eingabe...')

Ich möchte einmal einen Befehl per TCP an diesen Server senden (Mache 
ich zwekcs Testzwekce per Packet Sender) und danach soll er die 
Verbindung beenden und im Programmcode fortfahren.
Wenn er den Befehl (Vier Buchstaben) erhált wird dieser Befehl richtig 
zurückgesendet, dann wird aber nichts mehr gemacht. Ich kann danach 
einfach weitere Befehle senden. Das will ich aber nicht mehr. Er soll 
dann die Ausgabe unten machen.

[Rainer U. (r-u)]

Hmm, ich kenn zwar python nicht, aber

- wofür benutzt Du 2 verschachtelte Schleifen? / was bedeutet "while 
True" für Dich?

- Kommt nach einem Try nicht ein "catch" / except ?

[rk (Gast)]

Soll er für weitere Verbindungen zur Verfügung stehen, oder nie wieder 
antworten?

[rk (Gast)]

Wenn er immer wieder reagieren soll, könnte es inj etwa so aussehen:

#Main function
print('Program is started')

s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
print('Socket created')
s.bind(('', port_TCP))
s.listen(1)
try:
    while True:
        komm, addr = s.accept()
        command = komm.recv(1024)
        if command is not None and len(command) > 0:
            print 'Nachricht [%s]: %s' % (addr[0], command)
            komm.send('Befehl erhalten: ' + command)
            komm.close()

            print(command)                  
            print ('123')
            raw_input('Eingabe...')

except:
    print "Fehler aufgetreten..."
        
finally:
    s.close()

[Sheeva P. (sheevaplug)]

Sven schrieb:
> Ich möchte einmal einen Befehl per TCP an diesen Server senden (Mache
> ich zwekcs Testzwekce per Packet Sender) und danach soll er die
> Verbindung beenden und im Programmcode fortfahren.

Das ist jetzt schon die zweite Frage, in der Du irgendwelche komischen 
Sachen mit Deinem Netzwerk machst. Magst Du uns nicht vielleicht einmal 
erzählen, was Du eigentlich vorhast?

Was den Code angeht, so ist er an zwei Stellen kaputt. Auf den ersten 
Fehler hat "Rainer Unsinn" Dich bereits hingewiesen, nämlich: was 
bedeutet "while True" für Dich? Und zweitens: was ist "komm.close"?

[Sven (Gast)]

Sheeva P. schrieb:
> Sven schrieb:
>> Ich möchte einmal einen Befehl per TCP an diesen Server senden (Mache
>> ich zwekcs Testzwekce per Packet Sender) und danach soll er die
>> Verbindung beenden und im Programmcode fortfahren.
>
> Das ist jetzt schon die zweite Frage, in der Du irgendwelche komischen
> Sachen mit Deinem Netzwerk machst. Magst Du uns nicht vielleicht einmal
> erzählen, was Du eigentlich vorhast?
>
> Was den Code angeht, so ist er an zwei Stellen kaputt. Auf den ersten
> Fehler hat "Rainer Unsinn" Dich bereits hingewiesen, nämlich: was
> bedeutet "while True" für Dich? Und zweitens: was ist "komm.close"?

Ich erhalte einen Befehl per TCP. Diesen muss ich auswerten und 
anschlißend entsprechnd einen Befehl per UDP weitersenden. Deshalb nur 
einen Befehl erhalten per TCP. Das hab ich jetzt auch soweit 
hinbekommen. Diesen Befehl muss ich jetzt auswerten. Problem dabei gerad 
ist das ich zum Beispiel

CAEE

bekomme. Wenn ich jetzt folgende Abfrage machen will

if (command[0] == 'C')

kommt immer die Meldung: IndexError: string index out of range

Was muss ich tun damit das tut. Bin leider absoluter Neuling in Python.

[Bernhard R. (bernhard_r28)]

Zu 1:
Wenn du den Socket schließen möchtest, solltest du nach dem 
erfolgreichem Empfangen deines Strings die while Schleife verlassen. 
(Stichwort break)

Zu 2:
Du übergibst deiner if Abfrage gerade einen leeren String, und das mag 
sie nicht...

Hierzu ein kleines Beispiel (Python 3.x)

def testing(str):
    print("Testing: " + str)
    if str[0] == 'A':
        print("got it!")


testing('AFFE')
testing('DOOF')
testing('')

Viel Erfolg,

Bernhard

[MaWin (Gast)]

Anhand dieses Codefetzens wird dir wahrscheinlich kaum jemand
helfen können. An welcher Stelle hast du diese Zeile eingefügt?
Ganzen Code zeigen.

PS: Die Bedingung brauchst du nicht in Klammern einzuschließen.

[Sven (Gast)]

Bernhard R. schrieb:
> Zu 1:
> Wenn du den Socket schließen möchtest, solltest du nach dem
> erfolgreichem Empfangen deines Strings die while Schleife verlassen.
> (Stichwort break)
>
> Zu 2:
> Du übergibst deiner if Abfrage gerade einen leeren String, und das mag
> sie nicht...
>
> Hierzu ein kleines Beispiel (Python 3.x)def testing(str):
>     print("Testing: " + str)
>     if str[0] == 'A':
>         print("got it!")
>
> testing('AFFE')
> testing('DOOF')
> testing('')
>
> Viel Erfolg,
>
> Bernhard

Ich verwende Python 2.7. Funktioniert das dann trotzdem so wie in deinem 
Beispiel gezeigt.

Hier nochmal der ganze Code.

receive = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
print('Socket created')
receive.bind(('', port_TCP))
receive.listen(1)

komm, addr = receive.accept()
print('Connection adress is: [%s]' % (addr[0]))
while 1:
        command = (komm.recv(1024))
        if not command:
                komm.close
                break
        print 'Command [%s]: %s' % (addr[0], command)
        komm.send('Befehl erhalten: ' + command)
komm.close() #TCP-Verbindung beenden


if (command[0] == 'C'):
        print('Konfiguration')
elif (command[0] == 'R'):
        print('Daten lesen')
elif (command[0] == 'W'):
        print('Daten schreiben')
else:
        print('Wrong command')

Grüsse

[Sven (Gast)]

Sorry für den Doppelpost.

@Bernhard: Hab dein Beispiel kurz getestet. Das versteh ich soweit. Aber 
ich erhalte doch per TCP keinen leeren String sondern in der Variable 
command steht doch etwas drinne.


Grüsse

[Bernhard R. (bernhard_r28)]

Ob du wirklich keinen leeren string bekommst würde ich mal testen:

if command == '':
    print('empty string!!!')
else:
    #break the while loop
    break

Ansonsten tut das Code Snippet bei mir ...

[Sven (Gast)]

Bernhard R. schrieb:
> Ob du wirklich keinen leeren string bekommst würde ich mal
> testen:if command == '':
>     print('empty string!!!')
> else:
>     #break the while loop
>     break
>
> Ansonsten tut das Code Snippet bei mir ...

Bernhard R. schrieb:
> Ob du wirklich keinen leeren string bekommst würde ich mal
> testen:if command == '':
>     print('empty string!!!')
> else:
>     #break the while loop
>     break
>
> Ansonsten tut das Code Snippet bei mir ...

Also ich erhalte tatsächlich einen leeren String. Dann ist klar das die 
Fehlermeldung kommt. Aber weiter oben im Code gebe ich ja das command 
aus, also den Inhalt welchen ich erhalten habe. Dann kann der ja 
eigentlich gar nicht leer sein. Außer das er nicht übergeben wird. 
Vielleicht pack ich das ganze mal in eine Funktion und übergebe am ende 
das command mit "return command"
Oder hast du noch eine andere Lösung bzw. den Fehler

[Tek (Gast)]

Was passiert wenn Du deine While Schleife hier abbrichst...

if not command:
   komm.close
   break

und dann hierhin kommst?

if (command[0] == 'C'):
      print('Konfiguration')
elif (command[0] == 'R'):
      print('Daten lesen')
elif (command[0] == 'W'):
      print('Daten schreiben')
else:
      print('Wrong command')

[Bernhard R. (bernhard_r28)]

Ich habs gerade nochmal mit netcat versucht.
Ein leerer String sendet mir trotzdem ein 0x0a (line feed) mit.
Teste das mal mit print(command.__len__()).
Bei mir kam 1 raus...

[MaWin (Gast)]

Sven schrieb:
> while 1:
>         command = (komm.recv(1024))
>         if not command:
>                 komm.close
>                 break

Du springst hier aus der while-Schleife, wenn "command" leer ist!
Das dürfte das Gegenteil von dem sein, was du eigentlich willst.

[Sheeva P. (sheevaplug)]

Sven schrieb:
> Ich erhalte einen Befehl per TCP. Diesen muss ich auswerten und
> anschlißend entsprechnd einen Befehl per UDP weitersenden.

Das hatte ich gestern schon verstanden. ;-)

Nein, im Ernst: Dein per TCP erhaltener Befehl kommt ja irgendwo her, 
also vermutlich von einem anderen Netzwerkgerät. Die Auswertung dieses 
Befehls hat vermutlich einen Sinn wie etwa eine Filterung oder 
Konvertierung. Und das Weitersenden des Befehls dürfte doch vermutlich 
irgendein Ziel haben, möglicherweise ein anderes Netzwerkgerät. Sind 
meine Vermutungen soweit richtig? Wenn ja: gut, wenn nicht: bitte 
korrigiere mich.

Um Dir jetzt einen vernünftigen Rat zu geben, wäre es durchaus sinnvoll, 
etwas mehr über den Hintergrund Deiner Fragestellung zu erfahren. Also 
beispielsweise, um welche Art von Netzwerkgeräten es bei dem TCP-Client 
und dem UDP-Server geht, wie die Auswertung aussehen soll, und welchen 
tieferen Sinn die ganze Veranstaltung haben soll. Anders gesagt: mit ein 
wenig mehr Hintergrundinformationen könnte man Dir wesentlich schneller 
und zweifellos auch besser helfen.

> Deshalb nur einen Befehl erhalten per TCP.

Siehst Du, da geht (für mich) die Unsicherheit schon los: was genau 
meinst Du damit? Willst Du genau einen Befehl per TCP erhalten, diesen 
auswerten, und dann den TCP-Server beenden? Oder soll der TCP-Server 
nach Empfang und Auswertung des Befehls auf weitere Befehle lauschen?

Dabei stellt sich eine Reihe weiterer Fragen, zum Beispiel, ob es 
möglich ist, daß der TCP-Server mehrere Befehle gleichzeitig, oder 
sonstwie einen zweiten Befehl erhält, bevor der erste fertig ausgewertet 
und verarbeitet ist? Ob der UDP-Server, an den Du Deine Weiterleitung 
schickst, Dir eine Bestätigung zurückschickt, daß er Deinen 
weitergeleiteten Befehl empfangen hat? Warum Du wegen im Prinzip 
derselben Frage noch einen zweiten Thread eröffnest, obwohl Du Dich im 
ersten Thread nicht mehr gemeldet hast?

> Das hab ich jetzt auch soweit hinbekommen.

Sei mir nicht böse, aber Dein Code sieht für mich nicht danach aus, als 
ob Du das zuverlässig hinbekommen hättest. ;-)

> Was muss ich tun damit das tut. Bin leider absoluter Neuling in Python.

Ich bin mir ehrlich gesagt nicht ganz sicher, ob Netzwerkprogrammierung 
wirklich der beste Weg ist, um eine Programmiersprache zu lernen.

[Sheeva P. (sheevaplug)]

Tek schrieb:
> Was passiert wenn Du deine While Schleife hier abbrichst...
>
>

> if not command:
>    komm.close
>    break
>

Leute! Bin ich wirklich der Einzige, dem "komm.close" auffällt?

[Sheeva P. (sheevaplug)]

Bernhard R. schrieb:
> Ich habs gerade nochmal mit netcat versucht.
> Ein leerer String sendet mir trotzdem ein 0x0a (line feed) mit.
> Teste das mal mit print(command.__len__()).

Es gilt als sehr schlechter Stil, Variablen, Funktionen und Operatoren 
zu benutzen, die per Konvention "private" sind. Daher lieber nicht

command.__len__()

sondern stattdessen bitte

len(command)

benutzen.

[Sven (Gast)]

Sheeva P. schrieb:
> Sven schrieb:
>
> Nein, im Ernst: Dein per TCP erhaltener Befehl kommt ja irgendwo her,
> also vermutlich von einem anderen Netzwerkgerät. Die Auswertung dieses
> Befehls hat vermutlich einen Sinn wie etwa eine Filterung oder
> Konvertierung. Und das Weitersenden des Befehls dürfte doch vermutlich
> irgendein Ziel haben, möglicherweise ein anderes Netzwerkgerät. Sind
> meine Vermutungen soweit richtig? Wenn ja: gut, wenn nicht: bitte
> korrigiere mich.
>
> Um Dir jetzt einen vernünftigen Rat zu geben, wäre es durchaus sinnvoll,
> etwas mehr über den Hintergrund Deiner Fragestellung zu erfahren. Also
> beispielsweise, um welche Art von Netzwerkgeräten es bei dem TCP-Client
> und dem UDP-Server geht, wie die Auswertung aussehen soll, und welchen
> tieferen Sinn die ganze Veranstaltung haben soll. Anders gesagt: mit ein
> wenig mehr Hintergrundinformationen könnte man Dir wesentlich schneller
> und zweifellos auch besser helfen.
>
>> Deshalb nur einen Befehl erhalten per TCP.
>
> Siehst Du, da geht (für mich) die Unsicherheit schon los: was genau
> meinst Du damit? Willst Du genau einen Befehl per TCP erhalten, diesen
> auswerten, und dann den TCP-Server beenden? Oder soll der TCP-Server
> nach Empfang und Auswertung des Befehls auf weitere Befehle lauschen?
>
> Dabei stellt sich eine Reihe weiterer Fragen, zum Beispiel, ob es
> möglich ist, daß der TCP-Server mehrere Befehle gleichzeitig, oder
> sonstwie einen zweiten Befehl erhält, bevor der erste fertig ausgewertet
> und verarbeitet ist? Ob der UDP-Server, an den Du Deine Weiterleitung
> schickst, Dir eine Bestätigung zurückschickt, daß er Deinen
> weitergeleiteten Befehl empfangen hat? Warum Du wegen im Prinzip
> derselben Frage noch einen zweiten Thread eröffnest, obwohl Du Dich im
> ersten Thread nicht mehr gemeldet hast?


Hallo,

also nochmal etwas genauer vielleicht. Der TCP Client ist momentan noch 
nicht ausgereift, dient aber nur dazu Befehle zu senden bzw. wie du 
schon gesagt hast am Ende eine quittierung zu erhalten. Deshalb kann ich 
hier im ersten Schritt mit dem Programm Packet Sender arbeiten.
Es ist so das ich eben einen Befehl sende (z.B. CAII) welcher für 
Konfiguration all steht. Dann soll dieser Befehl von meinem TCP Server 
empfangen werden. Außerdem muss ich über diesen TCP Server vermutlich zu 
aller erst eine IP Adresse senden, welches Gerät konfiguriert werden 
muss. Das ganze befindet sich später an einem Computer alles, also nicht 
übers Local-Netz hinaus soweit ich das verstanden habe.
So der Befehl bzw. die IP Adresse werden dann verwendet um einen 
definierten Befehl (den weiß ich selbst noch nicht so genau) im nächsten 
schritt per UDP an einen zweiten Client so senden. Hier handelt es sich 
um einen Sensor der angesprochen und konfiguriert wird bzw. auch 
ausgelesen werden kann. Ist der befehl dort angekommen wird automatisch 
der befehl vom Sensor quittiert und per UDP wieder empfangen. Diese 
Quittierung soll dann wieder an den TCP Client gesendet werdern so das 
dieser weiß, der Befehl wurde ausgeführt.
Ic hoffe ich konnte damit etwas licht ins dunkle bringen. Habe leider 
immer noch keine Lösung für mein Problem mit der variable "command".


Grüße

[Sven (Gast)]

Habe jetzt einfach mal mit Funktionen gearbeitet welche ich in die TCP 
Verbindung eingebaut habe:

def funktion_1():
        print('Das ist C')
        return

def funktion_2():
        print('Das ist A')
        return


#Main function
print('Program started')
ip_adress = raw_input('Enter IP-Adress from BNI: ') #IP Adresse BNI-Master

while 1:
        #Start TCP-Connection
        receive = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
        print('Socket created\nWaiting for Communication to Client')        
        receive.bind(('', port_TCP))
        receive.listen(1)

        komm, addr = receive.accept()
        print('Connection adress is: [%s]' % (addr[0]))
        while 1:
                command = (komm.recv(1024))
                print 'Command: %s' % (command)
                komm.send('Befehl erhalten: ' + command)
                print 'Command: %s' % (command)
                if command[0] == 'C':
                        funktion_1()
                        break
                elif command[0] == 'A':
                        funktion_2()
                        break
                else:
                        print('wrong')
                        break

So erkennt er jetzt ohne Probleme ob zum Beispiel ein C oder ein A vorne 
steht. Das bedeutet doch das ich in die Funktion_1 dann meine weiteren 
Aufgaben schreiben kann. Oder gibt es einen besseren weg? Durch das 
break wird ja anschließend die while Schleife beendet. Aber meine TCP 
Verbindung ja eigentlich nicht? Macht es sinn diese zu beenden und beim 
senden der quittierung die verbindung neu aufzubauen oder kann ich diese 
problemlos offen halten und trotzdem eine UDP verbindung aufmachen und 
senden? Quasi am Ende dann meine Verbindungen erst alle schließen?


Grüße

[imonbln (Gast)]

Warum nimmst du nicht zumindest für die TCP Verbindung einen 
SocketServer, dieser sollte die das TCP Handling abnehmen und du kannst 
dich um das erkennen der Befehle und das UDP handling kümmern. Außerdem 
finde  ich dein if command[0] == 'C': sieht sehr nach basteln aus :)

ich würde dir hier vorschlagen ein dict zu verwenden key ist der gültige 
Befehl und value ist die dann zu rufende funktion.  denn Funktionen sind 
in Python auch nur Objekte d.h. du kannst sie auch so verwenden.

import SocketServer


class MyTCPHandler(SocketServer.BaseRequestHandler):
    def handle(self):
        command = { 'A': function1,
                    'B': function2}
        exit = False
        try:
            while not exit:
                self.data = self.request.recv(1024).strip()
                if 'exit' in self.data.lower():
                    exit = True
                elif len(self.data) < 1:
                    continue
                else:
                    try:
                        # detect command
                        cmd = command[self.data]
                    except KeyError:
                        print 'Unkown Command %s' % self.data
                    else:
                        # call associated function if cmd was found.
                        cmd()
                        # tell client execution is done
                        self.request.sendall('Execution done')
        except Exception as e:
            print('Error : {}'.format(e))


def function1():
    print 'A is here'

def function2():
    print 'B is here'

if __name__ == "__main__":
    HOST, PORT = "", 9999

    SocketServer.TCPServer.allow_reuse_address = True
    # Create the server, binding to localhost on port 9999
    server = SocketServer.TCPServer((HOST, PORT), MyTCPHandler)

    # Activate the server; this will keep running until you
    # interrupt the program with Ctrl-C
    server.serve_forever()

das ist eigentlich nur das SocketServer example mit ein wenig 
Anpassungen im handle auf deine Bedürfnisse.

[imonbln (Gast)]

Hier ist der handle nochmal ein wenig umgebaut, und entspricht etwas 
mehr dem was dein Code macht. Der Server macht nach jeden Befehl die 
Verbindung zu und kann mit Befehlen, wie dem oben erwähnten 'CAEE' 
umgehen. Wobei die for-schleife davon ausgeht das CAEE bedeutet führe C 
dann A, E, E aus.

    def handle(self):
        command = { 'A': function1,
                    'B': function2}
        exit = False
        try:
            self.data = self.request.recv(1024).strip()
            if len(self.data) > 1:
                for cur  in self.data:
                    try:
                        cmd = command[cur]
                    except KeyError:
                        print 'Unkown Command %s' % cur 
                    else:
                        cmd()
                        self.request.sendall('Execution done')
        except Exception as e:
            print('Error : {}'.format(e))