[Rd] R CMD check returns NOTE about package data set as global variable

[Henrik Bengtsson]

On Fri, Apr 6, 2012 at 1:33 PM, peter dalgaard <pdalgd at gmail.com> wrote:
>
> On Apr 6, 2012, at 22:23 , Hervé Pagès wrote:
>
>> On 04/06/2012 12:33 PM, Brad McNeney wrote:
>>> OK, thanks for the tip on good coding practice. I'm still getting the NOTE though when I make the suggested change.
>>
>> Because when you do return(RutgersMapB36[,1]), the code checker has no
>> way to know that the RutgersMapB36 variable is actually defined.
>>
>> Try this:
>>
>> test<-function() {
>>   RutgersMapB36 <- NULL
>>   data(RutgersMapB36)
>>   return(RutgersMapB36[,1])
>> }
>>
>
> That might remove the NOTE, but as far as I can see, it also breaks the code...

For data() per se, which by default clutter up the global environment,
you can do:

test<-function() {
  env <- new.env()
  data("RutgersMapB36", envir=env)
  env$RutgersMapB36[,1]
}

That is more explicit, and I do believe you won't get a NOTE about it.

Other than that, one can also use the following style (which still
seems to do the trick) for data(), attach(), load() et al., iff have
to use them:

test<-function() {
  # To avoid NOTEs by R CMD check
  RutgersMapB36 <- NULL; rm(RutgersMapB36);

  data(RutgersMapB36)
  return(RutgersMapB36[,1])
}


/Henrik

>
> --
> Peter Dalgaard, Professor,
> Center for Statistics, Copenhagen Business School
> Solbjerg Plads 3, 2000 Frederiksberg, Denmark
> Phone: (+45)38153501
> Email: pd.mes at cbs.dk  Priv: PDalgd at gmail.com
>
> ______________________________________________
> R-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel