大家好，以下是我的code，code本身是没有error，
但想请问一下这样的写法有哪里不好，是否应该改进?
主要的问题在于黄色那段try catch，
在进入SetUserDataToXml，有可能会出错，例如xml格式不对，
所以会有一个throw，但回到主程式时，
我把这个SetUserDataToXml，包在一个UserDataExist的验证里面，
逻辑上似乎没有问题，但是直觉上这样包起来好像怪怪的，
能给我一点建议吗? 谢谢
ExamData.ExamData ed = new ExamData.ExamData(context);
if (ed.IsUserDataExist())
{
try
{
ed.SetUserDataToXml();
}
catch (Exception)
{
response.Write(ed.ErrorCode);
}
}
else
{
response.Write(ed.ErrorCode);
}
ExamData Class
public void SetUserDataToXml()
{
XmlDocument doc = new XmlDocument();
try
{
doc.LoadXml(UserData);
}
catch (Exception)
{
ErrorCode = "1000";
throw;
}
}