Require usage of sample.int(n, m, ...) over sample(1:n, m, ...)

sample.int() is preferable to sample() for the case of sampling numbers between 1 and n. sample calls sample.int() "under the hood".

Usage

sample_int_linter()

See also

linters for a complete list of linters available in lintr.

Tags

efficiency, readability, robustness

Examples

# will produce lints
lint(
  text = "sample(1:10, 2)",
  linters = sample_int_linter()
)
#> ::warning file=<text>,line=1,col=1::file=<text>,line=1,col=1,[sample_int_linter] sample.int(n, m, ...) is preferable to sample(1:n, m, ...).

lint(
  text = "sample(seq(4), 2)",
  linters = sample_int_linter()
)
#> ::warning file=<text>,line=1,col=1::file=<text>,line=1,col=1,[sample_int_linter] sample.int(n, m, ...) is preferable to sample(seq(n), m, ...).

lint(
  text = "sample(seq_len(8), 2)",
  linters = sample_int_linter()
)
#> ::warning file=<text>,line=1,col=1::file=<text>,line=1,col=1,[sample_int_linter] sample.int(n, m, ...) is preferable to sample(seq_len(n), m, ...).

# okay
lint(
  text = "sample(seq(1, 5, by = 2), 2)",
  linters = sample_int_linter()
)

lint(
  text = "sample(letters, 2)",
  linters = sample_int_linter()
)