用侦听过程构造Elixir GenServer的正确方法

时间:2018-09-10 11:47:58

标签: elixir otp dialyzer

我正在使用elixir-socket库作为将后端应用程序连接到外部websocket的一种方式。我需要管理此过程(如果出现故障,请重新启动;如果无法连接,则按指数退缩,等等)。

当前,我创建了一个管理GenServer进程,该进程在给定的时间(以下简化)之后生成了一个循环套接字。我有一个主管管理SocketManager(并因此链接了Socket)流程:

socket_manager.ex

defmodule MyApp.SocketManager do
  def init(_) do
    Process.flag(:trap_exit, true)
    state = %{socket: nil}
    {:ok, state, {:continue, :init}}
  end

  def handle_continue(:init, state) do
    Task.start_link(fn ->
      Socket.connect!()
    end)
    {:noreply, state}
  end
end

socket.ex

defmodule MyApp.Socket do
  def connect! do
    socket = Socket.Web.connect!("xx.xx.com", secure: true, path: "/api/xxx")
    SocketManager.socket_connected(socket) # save the socket in the SocketManager state
    listen(socket)
  end

  defp listen(socket) do
    case socket |> Socket.Web.recv!() do
      {:text, data} ->
        # handle message
      {:close, :abnormal, _} ->
        Process.exit(self(), :kill)
      {:pong, _} ->
        nil
    end
    listen(socket)
  end
end

以上方法效果很好,但是我不确定这是否是构造此方法的最佳方法。据我了解,Task仅适用于具有确定寿命的任务,而不适用于永久性过程。另外,在运行mix dialyzer时,我得到以下输出(参考Task.spawn_link中的SocketManager行):

lib/myapp/socket_manager.ex:40:no_return
The created fun has no local return.

有人可以帮助我提出其他建议,以及如何使Dialyzer满意吗?

谢谢!

1 个答案:

答案 0 :(得分:0)

如果其他人有兴趣,这就是我最后的目的。我认为这是一个稍微更好的结构,尽管可能有更好/更惯用的方式。它使用DynamicSupervisor来监督套接字进程。进程

时,它也不再尝试连接

socket_manager.ex

defmodule MyApp.SocketManager do
  def start_link(_) do
    GenServer.start_link(__MODULE__, [], name: __MODULE__)
  end

  def connect do
    GenServer.cast(__MODULE__, {:connect})
  end

  def handle_cast({:connect}, state) do
    spec = %{
      id: LiveSocket,
      start: {MyApp.Socket, :connect, []},
      type: :worker
    }
    {:ok, pid} = DynamicSupervisor.start_child(MyApp.DynamicSupervisor, spec)
    Process.link(pid)
    {:noreply, state}
  end
end