At 8:19 AM -0600 12/11/09, Mark Na wrote:
Hello,

I am learning how to use functions, but I'm running into a roadblock.

I would like my function to do two things: 1) convert an object to a
dataframe, 2) and then subset the dataframe. Both of these commands work
fine outside the function, but I would like to wrap them in a function so I
can apply the code iteratively to many such objects.

Here's what I wrote, but it doesn't work:

convert<-function(d) {
 d<-data.frame(d); #convert object to dataframe
 d<-subset(d,select=c(time,coords.x1,coords.x2)) #select some columns
}
convert(data) #the problem is that "data" is the same as it was before
running the function


Objects (variables, data frames, etc) inside a function are in a different environment than those outside the function, and changes inside do not affect the outside unless you explicitly make it happen. In other words, you changed d inside the function, but did nothing to cause it to change data outside the function.

Change your function like this

convert<-function(d) {
 d<-data.frame(d); #convert object to dataframe
  ## no need to assign to d in the next line
 subset(d,select=c(time,coords.x1,coords.x2)) #select some columns
}

Then do:

  data <- convert(data)


Or the function could be

convert<-function(d) {
 d<-data.frame(d); #convert object to dataframe
 d <-  subset(d,select=c(time,coords.x1,coords.x2)) #select some columns
 d
}


The use of return() as suggested in another response is unnecessary. Functions return the value of their last expression, though possibly invisibly if the last expression is an assignment. For example;

 ick <- function(x) {
  x^2
 }

 ick(4)
[1] 16
 ick(3)
[1] 9


 ick <- function(x) {
  y <- x^2
 }

 foo <- ick(4)
 foo
[1] 16

The invisible return as in the second case is something I hadn't noticed before, and may deserve some reading of documentation.

-Don




The objects being processed through my function are SpatialPointsDataFrames
but I'm quite sure that's not my problem, as I can process these outside of
the function (using the above code) ... it's when I try to wrap the code in
a function that it doesn't work.

Thanks, Mark

        [[alternative HTML version deleted]]

______________________________________________
R-help@r-project.org mailing list
https://*stat.ethz.ch/mailman/listinfo/r-help
PLEASE do read the posting guide http://*www.*R-project.org/posting-guide.html
and provide commented, minimal, self-contained, reproducible code.


--
--------------------------------------
Don MacQueen
Environmental Protection Department
Lawrence Livermore National Laboratory
Livermore, CA, USA
925-423-1062

______________________________________________
R-help@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-help
PLEASE do read the posting guide http://www.R-project.org/posting-guide.html
and provide commented, minimal, self-contained, reproducible code.

Reply via email to