2013-05-05 162 views
2

我目前正在編寫一個簡單的服務器,它將與客戶端連接,然後與服務器充當中介的彼此交談。Erlang tcp服務器/客戶端發送消息

的設置是:

  • 服務器啓動
  • 2客戶端連接到服務器
  • 客戶端1/2發送一個消息與它的唯一的ID(原子)
  • 服務器連同保存該ID Socket PID
  • Client1發送{send_to_id,client2id,Message}
  • 服務器消息Client2

但這不適用於我,我得到一個function_clause錯誤。

所以基本上我想發送消息給客戶端與tcp_send沒有客戶端是「服務器」,但只有與recv。這可能嗎?

Server代碼:

-export([start/1]). 

-define(TCP_OPTIONS, [binary, {packet, 2}, {active, false}, {reuseaddr, true}]). 
-define(PORT, 8080). 

-spec start(Port) -> pid() when 
     Port::integer(). 
start(Port) -> 
    process_flag(trap_exit, true), 
    ClientDict = dict:new(), 
    GamesDict = dict:new(), 
    HandlerPID = spawn_link(fun() -> handler(ClientDict, GamesDict) end), 
    imup_listener:listen(Port, HandlerPID). 





%%------------------------------ 
% Internal Functions 
%%------------------------------ 

handler(Clients, Games) -> 
    receive 
    {insert_client, Socket, Alias} -> 
     TmpClients = dict:append(Socket, Alias, Clients), 
     TmpClients2 = dict:append(Alias, Socket, TmpClients), 
     handler(TmpClients2, Games); 
    {get_client_id, ReceiverPID, ClientPID} -> 
     {ok , CID} = dict:find(ClientPID, Clients), 
     ReceiverPID ! {id, CID}, 
     handler(Clients, Games); 
    {get_client_pid, ReceiverPID, ClientID} -> 
     {ok, CPID} = dict:find(ClientID, Clients), 
     ReceiverPID ! {pid, CPID}, 
     handler(Clients, Games); 
    {host_game, HostID, GameID} -> 
     TmpGames = dict:append_list(GameID, [HostID], Games), 
     handler(Clients, TmpGames); 
    {add_player, PlayerID, GameID} -> 
     TmpGames = dict:append_list(GameID, [PlayerID], Games), 
     handler(Clients, TmpGames); 
    {get_host, ReceiverPID, GameID} -> 
     {ok, [HID|T]} = dict:find(GameID, Games), 
     {ok, HPID} = dict:find(HID, Clients), 
     ReceiverPID ! {host_is, HID, HPID}, 
     handler(Clients, Games); 

    _ -> 
     {error, "I don't know what you want from me :("} 
    end. 

監聽器代碼:

-export([listen/2]). 

-define(TCP_OPTIONS, [binary, {packet, 2}, {active, false}, {reuseaddr, true}]). 

listen(Port, HandlerPID) -> 
    {ok, LSocket} = gen_tcp:listen(Port, ?TCP_OPTIONS), 
    spawn_link(fun() -> accept(LSocket, HandlerPID) end), 
    LSocket. 




% Wait for incoming connections and spawn a process that will process incoming packets. 
accept(LSocket, HandlerPID) -> 
    {ok, Socket} = gen_tcp:accept(LSocket), 
    Pid = spawn(fun() -> 
      io:format("Connection accepted ~n", []), 
      %%DictPID ! {insert, Socket, Socket}, 
      loop(Socket, HandlerPID) 
     end), 
    gen_tcp:controlling_process(Socket, Pid), 
    accept(LSocket, HandlerPID). 



% Echo back whatever data we receive on Socket 
loop(Sock, HandlerPID) -> 
    inet:setopts(Sock, [{active, once}]), 
    receive 
    {tcp, Socket, Data} -> 
     io:format("Got packet: ~p == ", [Data]), 

     FormatedData = process_data(Socket, Data, HandlerPID), 
     io:format("~p~n", [FormatedData]), 
     convey_message(Socket, FormatedData), 

     loop(Socket, HandlerPID); 
    {tcp_closed, Socket} -> 
     io:format("Socket ~p closed~n", [Socket]); 
    {tcp_error, Socket, Reason} -> 
     io:format("Error on socket ~p reason: ~p~n", [Socket, Reason]) 
    end. 




%%------------------------------ 
% Internal Functions 
%%------------------------------ 


-spec process_data(S, D, P) -> term() when 
     S::port(), 
     D::binary(), 
     P::pid(). 
process_data(Socket, Data, HandlerPID) when is_binary(Data) -> 
    case binary_to_term(Data) of 
    {send_host, GameID, Msg} -> 
     HandlerPID ! {get_host, self(), GameID}, 
     receive 
     {host_is, _HID, HSOCK} -> 
      HSOCK; 
     _ -> 
      {error, nohost} 
     end, 
     Msg; 
    {send_all, GameID, Msg} -> 
     Msg; 
    {send_to_id, ReceiverID, Msg} -> 
     HandlerPID ! {get_client_pid, self(), ReceiverID}, 
     receive 
      {pid, SockPID} -> 
      gen_tcp:send(SockPID, term_to_binary(Msg)); 
     _ -> 
      {error, noid} 
     end, 
     term_to_binary({ok, delivered}); 
    {host_game, GameID} -> 
     GameID; 
    {join_game, GameID} -> 
     GameID; 
    {start_game, GameID} -> 
     GameID; 
    {enter, SenderID} -> 
     HandlerPID ! {insert_client, Socket, SenderID}; 
    Dat -> 
     Dat 
    end; 
process_data(Socket, Data, DictPID) -> 
    Data. 


convey_message(Socket, Data) when is_binary(Data) -> 
    gen_tcp:send(Socket, Data); 
convey_message(Socket, Data) -> 
    gen_tcp:send(Socket, term_to_binary(Data)). 

客戶端代碼:

-export([connect/1, connect/2, disconnect/1, send/2, recv/1]). 

connect(PortNo) -> 
    {ok, Socket} = gen_tcp:connect("localhost", PortNo, [{active, false}, {packet, 2}]), 
    spawn(fun() -> recv(Socket) end), 
    Socket. 

connect(IP, PortNo) -> 
    {ok, Socket} = gen_tcp:connect(IP, PortNo, [{active, false}, {packet, 2}]), 
    spawn(fun() -> recv(Socket) end), 
    Socket. 


send(Socket, Message) -> 
    BinMsg = term_to_binary(Message), 
    gen_tcp:send(Socket, BinMsg). 
%% {ok, A} = gen_tcp:recv(Socket, 0), 
    %%A. 

recv(Socket) -> 
    {ok, A} = gen_tcp:recv(Socket, 0), 
    io:format("Received: ~p~n", [A]), 
    recv(Socket). 


disconnect(Socket) -> 
    gen_tcp:close(Socket). 

你推薦我改寫一切,或者是我的想法有可能? 預先感謝!

編輯:添加了一個testrun。

Eshell V5.8.5 (abort with ^G) 
1> imup_server:start(1234). 
#Port<0.669> 
2> Socket1 = imup_client:connect(1234). 
Connection accepted 
#Port<0.681> 
3> Socket2 = imup_client:connect(1234). 
Connection accepted 
#Port<0.683> 
4> imup_client:send(Socket1, {enter, cOne}). 
ok 
Got packet: <<131,104,2,100,0,5,101,110,116,101,114,100,0,4,99,79,110,101>> == {insert_client,#Port<0.682>,cOne} 
Received: [131,104,3,100,0,13,105,110,115,101,114,116,95,99,108,105,101,110, 
      116,102,100,0,13,110,111,110,111,100,101,64,110,111,104,111,115, 
      116,0,0,2,170,0,100,0,4,99,79,110,101] 
5> imup_client:send(Socket2, {enter, cTwo}). 
ok 
Got packet: <<131,104,2,100,0,5,101,110,116,101,114,100,0,4,99,84,119,111>> == {insert_client,#Port<0.684>,cTwo} 
Received: [131,104,3,100,0,13,105,110,115,101,114,116,95,99,108,105,101,110, 
      116,102,100,0,13,110,111,110,111,100,101,64,110,111,104,111,115, 
      116,0,0,2,172,0,100,0,4,99,84,119,111] 
6> imup_client:send(Socket1, {send_to_id, cTwo, hello}). 
ok 
Got packet: <<131,104,3,100,0,10,115,101,110,100,95,116,111,95,105,100,100,0,4, 
       99,84,119,111,100,0,5,104,101,108,108,111>> == 7> 
=ERROR REPORT==== 5-May-2013::23:25:49 === 
Error in process <0.39.0> with exit value: {function_clause,[{gen_tcp,send,[[#Port<0.684>],<<9 bytes>>]},{imup_listener,process_data,3},{imup_listener,loop,2}]} 


=ERROR REPORT==== 5-May-2013::23:25:49 === 
Error in process <0.40.0> with exit value: {{badmatch,{error,closed}},[{imup_client,recv,1}]} 
+0

爲了進行調試,很高興看到錯誤消息...函數子句,在我的經驗中,經常只是一個錯字或類似的東西。 – 2013-05-05 20:28:10

+0

當然!忘了補充一點,它現在被添加到底部。 – Klesken 2013-05-05 21:28:03

回答

5

所以你調用gen_tcp:send的論據[#Port<0.684>]<<9 bytes>>當得到一個function_clause錯誤。第一個參數是一個包含「端口」(本例中爲套接字)的列表,但它應該只是端口。

如果我正確地閱讀代碼,發生這種情況是因爲您將套接字放入字典dict:append,這會導致字典的值爲列表。除非你真的需要爲每個客戶端存儲多個套接字,反之亦然,否則dict:store會更合適。

+0

謝謝,我一定是盲目的,因爲我甚至測試了字典功能,現在我可以收到消息。 雖然在服務器上接收到二進制文件時出現了另一個問題,但是當客戶端收到它時,它變成了一個列表。我不知道爲什麼會發生這種情況,因爲我在兩次發送中使用了相同的term_to_binary/binary_to_term。任何想法? – Klesken 2013-05-06 13:10:44

+2

如果你想以二進制數據的形式接收消息,你的客戶端應該打開Socket爲二進制。當你在客戶端打開tcp Socket時,添加原子二進制文件到選項列表: gen_tcp:connect(IP,PortNo,[binary,{active,false},{packet,2}]) – Dave 2013-05-06 13:41:33

+0

當然好了:謝謝 ! – Klesken 2013-05-06 13:50:01

相關問題