View Source Credo.Check.Refactor.FilterFilter (Credo v1.7.2-rc.4)

This check has a base priority of 0 and works with any version of Elixir.

Explanation

One Enum.filter/2 is more efficient than Enum.filter/2 |> Enum.filter/2.

This should be refactored:

["a", "b", "c"]
|> Enum.filter(&String.contains?(&1, "x"))
|> Enum.filter(&String.contains?(&1, "a"))

to look like this:

Enum.filter(["a", "b", "c"], fn letter ->
  String.contains?(letter, "x") && String.contains?(letter, "a")
end)

The reason for this is performance, because the two separate calls to Enum.filter/2 require two iterations whereas doing the functions in the single Enum.filter/2 only requires one.

Check-Specific Parameters

There are no specific parameters for this check.

General Parameters

Like with all checks, general params can be applied.

Parameters can be configured via the .credo.exs config file.