ExSlop.Check.Refactor.SortForTopK (ExSlop v0.4.0)

Copy Markdown View Source

Basics

This check is disabled by default.

Learn how to enable it via .credo.exs.

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

Explanation

Sorting the whole collection just to take a single element does extra work.

# bad
list |> Enum.sort() |> Enum.take(1)
list |> Enum.sort() |> hd()

# good — when only one value is needed
Enum.min(list)
Enum.max(list)

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.