I don't thinkl that this class is not needed. You could either move this method into the controller as a private method.
However if you want to keep this class, I suggest renaming the class to something more meaningful and removing the "autorun" package. Also this method needs to be more explicit in name as it's actually generating the report, creating the attachment and then emailing the result. The method name is misleading as it's doing more than just emailReport.