You should have name collision. First, you must have dim a variable named msgbox. (It is already a very bad idea to name a variable "msgbox".) Second, you should have some function or sub named rating. Why you do that, I have no idea! Do you feel a short of choice in naming thing?