With, mon meilleur allié pour la lisibilité de code

08-08-2025 221 mots 2 minutes

Contenu

Comme on passe plus de temps à lire du code qu’à l’écrire, il faut trouver des paterns qui améliore la lisibilité. with c’est mon statement préféré pour gérer des suites de conditions.

Il y a quelque années je faisais:

    socket =
      with {:ok, user} <- {:user, get_user_by_email(email)},
           true <- {:check_user, user_can_be_added_to_organization?(user, org_id)} do
        ...
        socket
        |> put_flash(:info, gettext("%{email} added", email: user.email))
      else
        {:user, nil} ->
          ...
          put_flash(socket, :error, gettext("user not found"))
        {:check_user, false} ->
          ...
          put_flash(socket, :error, gettext("user cannot be admin"))
      end

Avec cette structure on perd en lisibilité sur:

l’enchainement des actions du with
la gestion des erreurs

Si on lit plus qu’on écrit, donc je dois orienter mon écriture pour que ce soit plus simple à lire. Quel est mon besoin ?

Comprendre rapidement les fonctions appelé
Quand je dois debug, je veux savoir ou je dois aller. Pas le besoin de voir toute la gestion d’erreur, ça ne me sert à rien.

    socket =
      with {:ok, user} <- get_user_by_email(socket, email),
           true <- user_can_be_added_to_organization?(socket, user, org_id) do
        ...
        socket
        |> put_flash(:info, gettext("%{email} added", email: user.email))
      end
...

defp get_user_by_email(socket, email) do
  case User.get_by_email(email) do
    nil -> put_flash(socket, :error, gettext("user not found"))
      user -> {:ok, user}
  end
end

defp user_can_be_added_to_organization?(socket, user, org_id) do
  case user_can_be_added_to_organization?(user, org_id) do
    false -> put_flash(socket, :error, gettext("user cannot be admin"))
    true -> true
  end
end