Try not to cast so much and your compiler should give you important information about what's going wrong; you certainly shouldn't need to cast a string to a string as you have done for the third parameter of your MessageBox call.

CProgramming FAQ
Caution: this person may be a carrier of the misinformation virus.