2017-03-03 74 views
0

我嘗試預處理websocket傳入消息。在lobby_channel.ex文件我有:「Elixir + phoenix」過濾websocket消息

defmodule Chatroom.LobbyChannel do 
    use Phoenix.Channel 

    def join("room:lobby", _payload, socket) do 
    {:ok, socket} 
    end 
    def join("room:" <> _private_room_id, _params, _socket) do 
    {:error, %{reason: "unauthorized"}} 
    end 

    def handle_in("new_move", payload, socket) do 
    changeset = Chatroom.Move.changeset(%Chatroom.Move{}, payload) 

    if changeset.valid? do 
     broadcast! socket, "new_move", payload 
     {:noreply, socket} 
    else 
     IO.puts "-----" <> changeset.errors[:message] 
     push socket, "error", %{msg: "Invalid move"} 
     {:noreply, socket} 
    end 
    end 

    #intercept ["new_move"] 
    #def handle_out("new_move", payload, socket) do 
    # IO.puts payload["name"] <> "=======" <> payload["message"] 
    # push socket, "new_move", payload 
    # {:noreply, socket} 
    #end 
end 

模型文件包含驗證功能:

... 
def changeset(struct, params \\ %{}) do 
    struct 
    |> cast(params, [:name, :message]) 
    |> validate_required([:name, :message]) 
    |> validate_length(:name, min: 1) 
    |> validate_length(:message, is: 2) 
    |> is_move_allowed(params) 
end 

defp is_move_allowed(changeset, params) do 
    IO.puts "--zz---" <> params["message"] 

    if params["message"] == "00" do 
    IO.puts "--yy---" <> params["message"] 
    add_error(changeset, params["message"], "'message' cannot be '00'") 
    else 
    IO.puts "--xx---" <> params["message"] 
    [] 
    end 
end 
... 

..the最後一個是我的自定義驗證功能,我不能讓它的工作沒有錯誤。輸出我得到:

--zz---00 
--yy---00 
[error] GenServer #PID<0.445.0> terminating 
** (ArgumentError) argument error 
    :erlang.byte_size(nil) 
    (chatroom) web/channels/lobby_channel.ex:18: Chatroom.LobbyChannel.handle_in/3 
    (phoenix) lib/phoenix/channel/server.ex:226: anonymous fn/4 in Phoenix.Channel.Server.handle_info/2 
    (stdlib) gen_server.erl:601: :gen_server.try_dispatch/4 
    (stdlib) gen_server.erl:667: :gen_server.handle_msg/5 
    (stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3 
Last message: %Phoenix.Socket.Message{event: "new_move", payload: %{"message" => "00", "name" => "a"}, ref: "2", to 
pic: "room:lobby"} 
State: %Phoenix.Socket{assigns: %{}, channel: Chatroom.LobbyChannel, channel_pid: #PID<0.445.0>, endpoint: Chatroom 
.Endpoint, handler: Chatroom.UserSocket, id: nil, joined: true, pubsub_server: Chatroom.PubSub, ref: nil, serialize 
r: Phoenix.Transports.WebSocketSerializer, topic: "room:lobby", transport: Phoenix.Transports.WebSocket, transport_ 
name: :websocket, transport_pid: #PID<0.439.0>} 

據我瞭解,事情正在向邏輯上正確的方式,消息中包含正確的數據,但我怎麼能解決這個錯誤嗎?我錯過了什麼?

+0

嘗試將其他中的'[]'更改爲'changeset'。 – Dogbert

+0

@Dogbert,同樣的錯誤。對於「00」消息和所有其他人 – Alexey

+0

你可以發佈「web/channels/lobby_channel.ex」的完整內容嗎? – Dogbert

回答

0

有在代碼中的三個錯誤:

  1. is_move_allowed必須始終返回一個變更。你在其他地方返回[]

  2. add_error的第二個參數應該是一個原子

  3. changeset.errors[:message]是一個元組,所以不能使用<>與字符串來連接它。 IO.inspect是一個更好的選擇,因爲它會打印任何值。

所以,變化:

if params["message"] == "00" do 
    IO.puts "--yy---" <> params["message"] 
    add_error(changeset, params["message"], "'message' cannot be '00'") 
else 
    IO.puts "--xx---" <> params["message"] 
    [] 
end 

到:

if params["message"] == "00" do 
    IO.puts "--yy---" <> params["message"] 
    add_error(changeset, :message, "'message' cannot be '00'") 
else 
    IO.puts "--xx---" <> params["message"] 
    changeset 
end 

和變化:

IO.puts "-----" <> changeset.errors[:message] 

到:

IO.inspect changeset.errors[:message] 
# or IO.inspect changeset.errors