大家好，以下是我的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;
}
}