come usare il try catch blocchi di un valore di ritorno del metodo?

Ci sono un sacco di modi che si possono utilizzare le eccezioni in metodi che restituiscono i valori:

Luogo l’istruzione return fuori il try-catch Per esempio:

T returnValue = default(T);
try
{
    //My code
}
catch 
{
    //Exception handling code
}
return returnValue;

Messo un ritorno economico all’interno del vostro cattura

try
{
    //My code
}
catch 
{
    //Handle exception
    return default(T);
}

Lanciare un’eccezione

Non hanno per restituire un valore, il metodo deve semplicemente fine (ad esempio, raggiungere un ritorno economico o di un’istruzione throw). A seconda eccezione per la sua non è sempre valido per restituire un valore.

Si dovrebbe riflettere attentamente su quando e come intercettare e gestire le eccezioni:

1. Quale potrebbe fallire?
2. Perché /come possono fallire?
3. Cosa devo fare quando vengono a mancare?

Nel tuo caso:

1. L’unica istruzione che può fallire è string Ext = Path.GetExtension(FileName);, che, secondo la documentazione può fallire se FileName contiene. (Nota che GetExtension non restituisce null, anche se FileName è null).
2. Questo potrebbe accadere se l’utente ha fornito una stringa che contiene questi caratteri non validi.
3. Se questo accade, credo che ci deve restituire false, per indicare che il percorso non è valido (ma questo dipende dall’applicazione).

Così probabilmente sarei gestire le eccezioni, come questa:

public bool CheckFileType(string FileName)
{
    string Ext;
    try
    {
        Ext = Path.GetExtension(FileName);
    }
    catch (ArgumentException ex)
    {
        return false;
    }
    //Switch statement
}

Di notare che abbiamo solo intercettare l’eccezione che ci si aspetta (ArgumentException), e abbiamo solo posto il try dichiarazione circa l’affermazione che, ci aspettiamo che l’eccezione.

In effetti è una buona idea per evitare di lancio e la cattura di eccezioni, dove possibile, non solo di subire una pena di prestazioni (che può causare seri problemi se questo metodo viene chiamato all’interno di un ciclo), ma si potrebbe inavvertitamente la cattura e la gestione di un’eccezione che non hai anticipare, di mascherare un problema più serio.

In questo caso si può evitare di buttare l’eccezione interamente mediante il controllo di noi stessi per vedere se FileName contiene caratteri non validi:

public bool CheckFileType(string FileName)
{
    if (FileName == null)
    {
        return false;
    }
    if (FileName.IndexOfAny(System.IO.Path.GetInvalidPathChars()) >= 0)
    {
        return false;
    }
    //Your original method goes here
}

OriginaleL’autore Justin

Questo dovrebbe funzionare:

public bool CheckFileType(string FileName)
{
    try
    {
        string Ext = Path.GetExtension(FileName).ToLower();
        string[] okExt = ".gif|.jpg|.jpeg|.png|.bmp".Split('|');

        foreach(var item in okExt)
        {
            if(Ext == item)
                return true;
        }
        return false;
    }
    catch(Exception ex)
    {
        throw;
    }
}

E ricordate: mai intercettare le eccezioni non si sta andando a gestire. (o almeno di ri-lanciare loro)

OriginaleL’autore data