Skip to content

Commit

Permalink
corrige bug #38
Browse files Browse the repository at this point in the history
  • Loading branch information
dhersz committed Jan 13, 2025
1 parent 79fda4e commit 2792735
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 7 deletions.
9 changes: 6 additions & 3 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,12 @@

## Correção de bugs

- Corrigido bug na `padronizar_numeros()` em que zeros após o separador de milhares
eram suprimidos. Por exemplo, "1.028" virava "1.28". Relacionado ao [issue
#37](https://github.com/ipeaGIT/enderecobr/issues/37).
- Corrigido bug na `padronizar_numeros()` em que zeros após o separador de
milhares eram suprimidos. Por exemplo, "1.028" virava "1.28". Relacionado ao
[issue #37](https://github.com/ipeaGIT/enderecobr/issues/37).
- Corrigido bug na `padronizar_numeros()` em que zeros de vetores numéricos não
eram adequadamente transformados em "S/N". Relacionado ao [issue
#38](https://github.com/ipeaGIT/enderecobr/issues/38).

## Novas funcionalidades

Expand Down
7 changes: 3 additions & 4 deletions R/padronizar_numeros.R
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,9 @@ padronizar_numeros <- function(numeros) {
)

if (is.numeric(numeros)) {
numeros_na <- which(is.na(numeros))

numeros_padrao <- formatC(numeros, format = "d")
numeros_padrao[numeros_na] <- "S/N"
numeros_padrao <- data.table::fifelse(numeros == 0, NA_integer_, numeros)
numeros_padrao <- formatC(numeros_padrao, format = "d")
numeros_padrao[numeros_padrao == "NA"] <- "S/N"

return(numeros_padrao)
}
Expand Down
1 change: 1 addition & 0 deletions tests/testthat/test-padronizar_numeros.R
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ test_that("lida com vetores vazios corretamente", {
test_that("padroniza corretamente - numero", {
gabarito <- tibble::tribble(
~original, ~padronizado_esperado,
0, "S/N", # issue #38 (https://github.com/ipeaGIT/enderecobr/issues/38)
1, "1",
1.1, "1",
NA_integer_, "S/N",
Expand Down

0 comments on commit 2792735

Please sign in to comment.