search

ipeaGIT / urbanformbr

3 stars 5 forks source link

Parênteses faltando no caso de muitas condições lógicas #14

Open dhersz opened 3 years ago

dhersz commented 3 years ago

https://github.com/ipeaGIT/urbanformbr/blob/2c0a412f5b029bd254a90708466aecc534d46adc/R/pca_regression_analysis/00_11_censo.R#L240-L264

Oi pessoal. Meio aleatoriamente tava olhando um dos arquivos do projeto e me parece que faltam alguns parênteses pra fazer as condições lógicas funcionarem direitinho.

Me parece que ali no caso das condições pra considerar como indústria é necessário escrever algo como:

(V6471 >= 05000L & V6471 <= 09999L) |   # INDÚSTRIAS EXTRATIVAS 
     (V6471 >= 10000L & V6471 <= 33999L) | # INDÚSTRIAS DE TRANSFORMAÇÃO 
     (V6471 >= 41000L & V6471 <= 43999L)   # CONSTRUÇÃO 
   , "Indústria",

Em vez de como está hoje, sem os parênteses. Procede? Caso positivo, teria que considerar as condições pra considerar a atividade como serviço também, logo abaixo. Abraços.

joaoparga commented 3 years ago

Oi @dhersz. Muito obrigado pelo aviso.

Dei uma checada aqui e parece que não faz diferença incluir os parênteses ou não: (sector é a variável original e sector_teste com os parênteses)

Screen Shot 2021-09-20 at 16 48 17

De qualquer forma, vou deixar com parênteses para ficar mais claro.

Obrigado!