2016-06-27 58 views
1

文字列を出力するCプログラム(クライアント)に文字列を送信するPythonソケットサーバーがあります。このサーバーは、それを印刷するPythonプログラムに送り返します。通信は機能しますが、長い文字列、短い文字列を送信すると、現在のデータと以前のデータがマージされます。ここでは例です:CとPythonコードのソケット通信

出力:(ちょうどループ内で呼び出される)

5.0standing0.0    //That is how it should look like 
5.0walk0ing0.055111512313e-17 //After the long string 

のpythonコード:

import socket 
import sys 

# Create a TCP/IP socket 
sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM) 

# Bind the socket to the port 
server_address = ('localhost', 8888) 
print >>sys.stderr, '\nstarting up on %s port %s\n' % server_address 
sock.bind(server_address) 

# Listen for incoming connection 
sock.listen(1) 

def send_control(state,xdiff,xactual): 
    connection, client_address = sock.accept() 
    connection.sendall(str(xactual)+str(state)+str(xdiff)) 
    data = connection.recv(2000) 
    print data 
    data = '\0' 

Cコード(あまりにもループ内で呼び出されます):

void comm_func() 
{ 
    int sock; 
    int i=0; 
    struct sockaddr_in server; 
    char message[1024] , server_reply[2000]; 

    //Create socket 
    sock = socket(AF_INET , SOCK_STREAM , 0); 
    if (sock == -1) 
    { 
     printf("Could not create socket"); 
    } 
    // puts("Socket created"); 

    server.sin_addr.s_addr = inet_addr("127.0.0.1"); 
    server.sin_family = AF_INET; 
    server.sin_port = htons(8888); 

    //Connect to remote server 
    if (connect(sock , (struct sockaddr *)&server , sizeof(server)) < 0) 
    { 
     perror("connect failed. Error"); 
    } 

    // Receive a reply from the server 
    if(recv(sock , server_reply , 2000 , 0) < 0) 
    { 
     puts("recv failed"); 
    } 
    puts(server_reply); 
    write(sock,server_reply,50); 
    for (i=0; i< strlen(server_reply); i++) 
    { 
    server_reply[i] = '\0'; 
    } 
    close(sock); 
} 

誰かが私を助けることができますか?

+2

受信した文字列はnullで終了していないので、[UB](https://en.wikipedia.org/wiki/Undefined_behavior)です。簡単な修正 'char server_reply [2000] = {0}' – LPs

+2

'recv()'が返されることを決して期待しないでください。これが戻り値であるかどうかを確認してください。 '0'についてもテストしてください。また、TCPはストリーム指向であることも考慮に入れてください。 N個の送信は、N> MまたはN alk

+0

文書を暗黙的に呼び出すことでこれを閉じるための投票。 – alk

答えて

1

変更

// Receive a reply from the server 
    if(recv(sock , server_reply , 2000 , 0) < 0) 
    { 
     puts("recv failed"); 
    } 

の中へ:コメント欄で@LPと@ack言及両方として

ssize_t len; 
    // Receive a reply from the server 
    if((len = recv(sock , server_reply , 2000 , 0)) < 0) 
    { 
     puts("recv failed"); 
     close(sock); 
     return -1; 
    } 
    server_reply[len] = 0; 

、問題は、ショートメッセージはNULでの通常である、終了していないということですソケット通信ここで、0は送信する有効なバイトです。

関連する問題