refactor session controller

Author: SophieDeBenedetto

lib/pointing_party/user.ex

@@ -1,11 +1,22 @@
 defmodule PointingParty.User do
   use Ecto.Schema
   import Ecto.Changeset
+  alias PointingParty.User
 
   schema "users" do
     field :username, :string
   end
 
+  def create(attrs) do
+    changeset = changeset(%User{}, attrs)
+    if changeset.valid? do
+      user =  apply_changes(changeset)
+      {:ok, user}
+    else
+      {:error, %{changeset | action: :insert}}
+    end
+  end
+
   @doc false
   def changeset(user, attrs \\ %{}) do
     user

lib/pointing_party_web/controllers/session_controller.ex

@@ -8,14 +8,12 @@ defmodule PointingPartyWeb.SessionController do
   end
 
   def create(conn, params) do
-    changeset = User.changeset(%User{}, params["user"])
-    if changeset.valid? do
-      user =  Ecto.Changeset.apply_changes(changeset)
-      put_session(conn, :username, user.username)
-      |> redirect(to: "/cards") |> halt()
-    else
-      changeset = %{changeset | action: :insert}
-      render(conn, "new.html", changeset: changeset)
+    case User.create(params["user"]) do
+      {:ok, user} ->
+        put_session(conn, :username, user.username)
+        |> redirect(to: "/cards") |> halt()
+      {:error, changeset} ->
+        render(conn, "new.html", changeset: changeset)
     end
   end