would be better here, because this way it gets instantiated for a particular File , one time. 
Adding this to getAuthor method would result in new author entries within a partcular file. Whereas Author can be only one, for one file entry. Each call to getAuthor, would end up creating new Author. 
Same goes for modifier.