我是一名擁有 4 個月經驗的學徒，我的任務是使用 Microsoft Graph 中的數據構建假期請求應用程序。應用程序的功能之一是查找用戶的經理并將其顯示在儀表板上。一切都很順利，直到我的老板登錄。運行 Microsoft Graph Query 后，為了找出當前的用戶管理器，Graph Api 返回錯誤(Request_ResourceNotFound)并破壞整個應用程序，而不是返回 null 或 0。我不知道如何處理該錯誤。null如果結果是，我嘗試返回null，但這沒有做任何事情。這是我的控制器所期望的：var allUsersConnectedToCurrentManagerDisplayName = graphHelper.GetManagerForCurrentUser(userIdToCheck).DisplayName;var allUsersConnectedToCurrentManagerEmail = graphHelper.GetManagerForCurrentUser(userIdToCheck).UserPrincipalName;var allUsersConnectedToCurrentManagerId = graphHelper.GetManagerForCurrentUser(userIdToCheck).Id;微軟圖形助手：User GetDirectManagerForUser(GraphServiceClient _graphServiceClient, string managerId){ using(var task = Task.Run(async() => await _graphServiceClient.Users[managerId].Manager.Request().GetAsync())) { while (!task.IsCompleted) Thread.Sleep(200); var manager = task.Result as Microsoft.Graph.User; return manager; }}我期望它返回 null 并且只是不為用戶顯示沒有任何人在他之上的直接經理。
2 回答
絕地無雙
TA貢獻1946條經驗 獲得超4個贊
所以這里發生了一些事情。
第一個也是最明顯的問題是您的代碼連續三次從圖表請求相同的用戶記錄。您撥打的每個電話都會GetDirectManagerForUser下載整個用戶配置文件。你想避免這樣做：
var manager = await graphHelper.GetManagerForCurrentUser(userIdToCheck);
var allUsersConnectedToCurrentManagerDisplayName = manager.DisplayName;
var allUsersConnectedToCurrentManagerEmail = manager.UserPrincipalName;
var allUsersConnectedToCurrentManagerId = manager.Id;
要避免的第二個問題是將您的請求包裝在這樣的任務中。它增加了代碼的復雜性，使調試變得非常困難，而且沒有必要。只需在方法級別添加async Task<>，然后讓編譯器為您處理連接：
async Task<User> GetDirectManagerForUser(GraphServiceClient _graphServiceClient, string managerId)
第三，您投射結果但沒有捕獲任何異常（即404您得到的）。您想要捕獲這些并返回一個空用戶：
var manager = await graphHelper.GetManagerForCurrentUser(userIdToCheck);
var allUsersConnectedToCurrentManagerDisplayName = manager.DisplayName;
var allUsersConnectedToCurrentManagerEmail = manager.UserPrincipalName;
var allUsersConnectedToCurrentManagerId = manager.Id;
async Task<User> GetDirectManagerForUser(GraphServiceClient _graphServiceClient, string managerId)
{
try
{
// Normal path
return await _graphServiceClient
.Users[managerId]
.Manager
.Request()
.GetAsync();
}
catch (Exception)
{
// Something went wrong or no manager exists
var emptyUser = new User();
}
}
慕婉清6462132
TA貢獻1804條經驗 獲得超2個贊
您必須捕獲異常才能返回 null。我會這樣寫這個函數：
? ? public User GetDirectManagerForUser(GraphServiceClient _graphServiceClient, string managerId)
? ? {
? ? ? ? //.Result, because this function in synchronious
? ? ? ? try?
? ? ? ? {? ? ? ?
? ? ? ? ? ? var manager = await _graphServiceClient.Users[managerId].Manager.Request().GetAsync().Result;
? ? ? ? ? ? return manager;
? ? ? ? }
? ? ? ? catch(Exception)
? ? ? ? {
? ? ? ? ? ? return null;
? ? ? ? }
? ? }?
您還可以像這樣使函數異步：
? ? public async Task<User> GetDirectManagerForUser(GraphServiceClient _graphServiceClient, string managerId)
? ? {
? ? ? ? try?
? ? ? ? {? ? ? ?
? ? ? ? ? ? var manager = await _graphServiceClient.Users[managerId].Manager.Request().GetAsync();
? ? ? ? ? ? return manager;
? ? ? ? }
? ? ? ? catch(Exception)
? ? ? ? {
? ? ? ? ? ? return null;
? ? ? ? }
? ? }?
為什么沒有指定可訪問性級別？
- 2 回答
- 0 關注
- 108 瀏覽
添加回答
舉報
0/150
提交
取消