錯誤捕獲最佳方法

Question

嗨我想獲得所有故障單的列表，但它說並非所有的代碼路徑都返回一個值。是否有人沒有我在做什麼錯誤謝謝

public List<TroubleTicket> GetAllTroubleTickets() 
{ 
    try 
    { 

     List<TroubleTicket> tickets = new List<TroubleTicket>(); 


     var q = _supportDeskEntities.TroubleTickets.ToList(); 
     return q; 
    } 

    catch (Exception ex) 
    { 


    } 


} 

Answer 1

您需要從catch語句返回一個值（或拋出另一個異常）。否則，該函數將返回null

你可以試試這個：

public List<TroubleTicket> GetAllTroubleTickets() 
{ 
    try 
    { 

     List<TroubleTicket> tickets = new List<TroubleTicket>(); 


     var q = _supportDeskEntities.TroubleTickets.ToList(); 
     return q; 
    } 

    catch (Exception ex) 
    { 
     return new List<TroubleTicket>(); // This is just in case you want to ignore any exceptions 

    } 


} 

或

catch (Exception ex) 
    { 
     throw new Exception("There was an error getting tickets"); // Probably not as good of a way as you lose the exception details 

    } 

Answer 2

如果你發現一個異常的函數返回值。因此，改變這種：

catch (Exception ex) 
{ 
} 

要這樣：

catch (Exception ex) 
{ 
    return null; 
} 

或者當一個例外是捕獲，那麼你可以做到這一點，如果你希望它返回一個空列表：

catch (Exception ex) 
{ 
    return new List<TroubleTicket>(0); 
} 

如果我們正在採取最佳做法，然後我會說你應該記錄異常，然後重新拋出它。像這樣：

catch (Exception ex) 
{ 
    //write to log 
    throw ex; 
} 

Answer 3

如果沒有拋出異常，則只返回一個值。您必須從catch塊內返回，或者返回try/catch結構之外。

所以，你可以返回在兩個不同的地方：

public List<TroubleTicket> GetAllTroubleTickets() 
{ 
    try 
    { 
     var q = _supportDeskEntities.TroubleTickets.ToList(); 
     return q; 
    } 
    catch (Exception ex) 
    { 
     // You can also return "new List<TroubleTicket>()" if null is an unacceptable return value 
     return null; 
    } 
} 

或保持一個返回值的變量，並將其設置在兩個不同的位置，並在一個位置返回它：

public List<TroubleTicket> GetAllTroubleTickets() 
{ 
    List<TroubleTicket> tickets; 
    try 
    { 
     tickets = _supportDeskEntities.TroubleTickets.ToList(); 
    } 
    catch (Exception ex) 
    { 
     // You can also use "new List<TroubleTicket>()" if null is an unacceptable return value 
     tickets = null; 
    } 
    return tickets; 
} 

Answer 4

如果發生異常，您不會返回任何內容，並且您不會將列表分配給List<TroubleTicket> tickets，而是分配給其他變量。你可以這樣做：

public List<TroubleTicket> GetAllTroubleTickets() 
{ 
    List<TroubleTicket> tickets = null; 
    bool gotTickets = true; 
    try{ 
     tickets = _supportDeskEntities.TroubleTickets.ToList(); 
    } 
    catch (SpecificException ex){ 
     gotTickets = false; 
    } 
    catch (Exception ex){ // catches all other "unexpected" exceptions 
     // log and/or... 
     throw; 
    } 
    return gotTickets ? tickets : null; 
} 

但是，你不應該捕捉所有類型的例外，但只有你期望的特定類型。您還應該記錄所有其他類型的異常。

Answer 5

你的catch塊沒有返回任何東西。

public List<TroubleTicket> GetAllTroubleTickets() 
{ 
    List<TroubleTicket> tickets =null; 
    try 
    { 
     tickets = new List<TroubleTicket>(); 
     var q = _supportDeskEntities.TroubleTickets.ToList(); 
     return q; 
    } 

    catch (Exception ex) 
    { 
     //Log or handle your error 
    } 

    return tickets; 
}