There seem to be some unnecessary get_record, get_coursemodule_from_id() etc., it might be better to be more strict in constructor instead and require all info from the place where comments are used