addNA avec une option string about questionr

[juba]

Personnellement je pense que ça pourrait tout à fait intégrer questionr (sous réserve de l'accord de @briatte). La question serait de savoir si on fait une surcharge de addNA ou si on ajoute une fonction supplémentaire.

En tous cas si on surcharge addNA avec un argument as.string, il faut que ce dernier soit à FALSE par défaut pour ne pas modifier le comportement de code existant après chargement de questionr.

[briatte]

Le problème étant bien documenté et le coût de la surcharge minimal, je vote pour 👍 !

[larmarange]

Il me semble plus pertinent de surcharger addNA en passant as.string à FALSE pour compatibilité. Par ailleurs, il est facile de permettre de personnaliser l'étiquette en passant directement une valeur texte à as.string ainsi :

addNA <- function (x, ifany = FALSE, as.string = FALSE)
{
    if (!is.factor(x)) 
        x <- factor(x)
    if (ifany & !any(is.na(x))) 
        return(x)
    ll <- levels(x)
    if (!any(is.na(ll))) 
        ll <- c(ll, NA)
    x <- factor(x, levels = ll, exclude = NULL)
    if (is.character(as.string)) {
         s <- as.string
         as.string <- TRUE
    } else
        s <- "NA"
    if(as.string) levels(x)[is.na(levels(x))] <- s
    x
}

[larmarange]

Pull request proposée

[juba]

After some thoughts, I'm not totally happy with the current function implementation. Please see Issue #18 for further discussion.

[briatte]

Would the issue go away if you called the function na.level or na.char for instance?

[larmarange]

I would prefer a name as addNAstr

Joseph Larmarange
Le 7 nov. 2013 17:36, "François" [email protected] a écrit :

Would the issue go away if you called the function na.level or na.charfor instance?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-27981724
.