套接字消息客户端/服务器：客户端不发送消息

Question

我正在尝试使用 Python 和 Socket 创建一个简单的聊天程序。 服务器处理两个客户端之间的连接。 每当我运行该程序时，对于其中一个客户端，第一个消息之后的每条消息都不会发送。 消息甚至没有到达服务器。

这是在 Windows 上，Python3。 我已经集成了线程，因此可以同时发送和接收消息； 然而，问题仍然存在。

服务器.py

conns = [] # list that stores both connections


def accept(sk):
    while True:
        conn, addr = sk.accept()
        conns.append(conn)
        print(addr, "has connected")

    t = threading.Thread(target=accept, args=(sk,))
    t.start() #start accept thread



def send(conn):
    while True:
        message = conn.recv(2048)
        print(message)
        for conn in conns:
            conn.send(message) #sends message to every connection
            print("Sent message")


t = threading.Thread(target=send, args=(conn,))
t.start() #start threading for send

客户端.py

def recvMessages(s):
    while True:
        message = s.recv(2048)
        print(message)
        message = message.decode()
        messages.append(message)
        os.system("cls")

        for message in messages:
            print(message)

def sendMessage(s):
    while True:
        message = input()
        message = message.encode()
        s.send(message)

s = socket.socket()
host = socket.gethostname()
port = 8080
s.connect((host, port))
messages = []

print("Connected")
connected = True




threading.Thread(target=sendMessage, args=(s,)).start()
threading.Thread(target=recvMessages, args=(s,)).start()

所有的消息都应该从两个客户端发送，但一个客户端永远不能发送多条消息，另一个工作正常。

Answer 1

您的服务器代码缺少它的套接字，并且没有在您的示例中运行接受，正如詹姆斯指出的那样，您的缩进也无效，下次提供一个最小的可重现示例。

我还清理了您的文件，因为您遵循了一些不好的做法，特别是将消息广播给实际接收的所有客户端“def send”，避免混淆命名:)

在您的服务器代码中，您也仅发送到一个连接（在您的示例中不存在），它应该运行接收并在每次收到新消息时发送

服务器.py

import socket
import threading

conns = []  # list that stores both connections


def accept(sk):
    while True:
        conn, addr = sk.accept()
        conns.append(conn)
        print(addr, "has connected")

        # receive from new client
        receive_t = threading.Thread(target=receive, args=(conn,))

        receive_t.start()  # start threading for send


def send(s, msg):

        # change message ..
        s.send(msg)  


def broadcast(msg):
    for conn in conns:
        send(conn, msg)


def receive(conn):
    try:
        while True:
            message = conn.recv(2048)

            if not message: # detects if socket is dead, by testing message
                print("client sent Nothing, removing") 
                conns.remove(conn)
                break
            broadcast(message)  # send to all clients
    except ConnectionResetError  as e:
        print('Could not send must be disconnected ')
        conns.remove(conn) # remove dead socket



# added missing socket
sock = socket.socket()
sock.bind(('127.0.0.1', 8080))
sock.listen(1)  # needs to  bind and listen
t = threading.Thread(target=accept, args=(sock,))
t.start()  # start accept thread

客户端.py

import os
import socket
import threading

messages = []


def recvMessages(s):
    while True:
        message = s.recv(2048)

        message = message.decode()
        print('new message= ', message)
        messages.append(message)
        os.system("cls")

        for message in messages:
            print(message)


def send_message(s):
    while True:
        message = input()
        message = message.encode()
        s.send(message)


s = socket.socket()
host = '127.0.0.1'
port = 8080
s.connect((host, port))

print("Connected")
connected = True
# added missing receive

receive = threading.Thread(target=recvMessages, args=(s,)) # missed receive thread in example
receive.start()
send_message(s)