using message box is a better way