Tester si une chaîne de caractères est un guide sans lancer d'exceptions ?

Question

Je veux essayer de convertir une chaîne de caractères en Guid, mais je ne veux pas compter sur la capture des exceptions (

• pour des raisons de performance - les exceptions sont coûteuses
• pour des raisons de convivialité, le débogueur apparaît.
• pour des raisons de conception - l'attendu n'est pas exceptionnel

En d'autres termes, le code :

public static Boolean TryStrToGuid(String s, out Guid value)
{
    try
    {
        value = new Guid(s);
        return true;
    }
    catch (FormatException)
    {
        value = Guid.Empty;
        return false;
    }
}

ne convient pas.

J'essaierais bien d'utiliser RegEx, mais comme le guidon peut être enveloppé de parenthèses, d'accolades ou non, c'est difficile.

De plus, je pensais que certaines valeurs Guid sont invalides ( ?).

---

Mise à jour 1

ChristianK a eu la bonne idée d'attraper seulement FormatException plutôt que tous. Modification de l'exemple de code de la question pour inclure la suggestion.

---

Mise à jour 2

Pourquoi s'inquiéter des exceptions levées ? Est-ce que je m'attends vraiment à des GUIDs invalides si souvent ?

La réponse est oui . C'est pourquoi j'utilise TryStrToGuid. Je suis s'attendant à de mauvaises données.

Exemple 1 Les extensions d'espace de nommage peuvent être spécifiées en ajoutant un GUID au nom d'un dossier. . Je pourrais analyser les noms de dossiers, vérifier si le texte après la finale . est un GUID.

c:\Program Files
c:\Program Files.old
c:\Users
c:\Users.old
c:\UserManager.{CE7F5AA5-6832-43FE-BAE1-80D14CD8F666}
c:\Windows
c:\Windows.old

Exemple 2 Il se peut qu'un serveur web très utilisé veuille vérifier la validité de certaines données renvoyées. Je ne veux pas que des données non valides accaparent des ressources supérieures de 2 ou 3 ordres de grandeur à ce qu'elles devraient être.

Exemple 3 Je pourrais analyser une expression de recherche entrée par un utilisateur.

S'ils saisissent des GUID, je veux les traiter spécialement (par exemple en recherchant spécifiquement cet objet, ou en mettant en évidence et en formatant ce terme de recherche spécifique dans le texte de la réponse).

---

Mise à jour 3 - Critères de performance

Test de conversion de 10 000 bons Guids, et 10 000 mauvais Guids.

Catch FormatException:
   10,000 good:     63,668 ticks
   10,000 bad:   6,435,609 ticks

Regex Pre-Screen with try-catch:
   10,000 good:    637,633 ticks
   10,000 bad:     717,894 ticks

COM Interop CLSIDFromString
   10,000 good:    126,120 ticks
   10,000 bad:      23,134 ticks

p.s. Je ne devrais pas avoir à justifier une question.

Answer 1

Critères de performance

Catch exception:
   10,000 good:    63,668 ticks
   10,000 bad:  6,435,609 ticks

Regex Pre-Screen:
   10,000 good:   637,633 ticks
   10,000 bad:    717,894 ticks

COM Interop CLSIDFromString
   10,000 good:   126,120 ticks
   10,000 bad:     23,134 ticks

COM Intertop (le plus rapide) Réponse :

/// <summary>
/// Attempts to convert a string to a guid.
/// </summary>
/// <param name="s">The string to try to convert</param>
/// <param name="value">Upon return will contain the Guid</param>
/// <returns>Returns true if successful, otherwise false</returns>
public static Boolean TryStrToGuid(String s, out Guid value)
{
   //ClsidFromString returns the empty guid for null strings   
   if ((s == null) || (s == ""))   
   {      
      value = Guid.Empty;      
      return false;   
   }

   int hresult = PInvoke.ObjBase.CLSIDFromString(s, out value);
   if (hresult >= 0)
   {
      return true;
   }
   else
   {
      value = Guid.Empty;
      return false;
   }
}

namespace PInvoke
{
    class ObjBase
    {
        /// <summary>
        /// This function converts a string generated by the StringFromCLSID function back into the original class identifier.
        /// </summary>
        /// <param name="sz">String that represents the class identifier</param>
        /// <param name="clsid">On return will contain the class identifier</param>
        /// <returns>
        /// Positive or zero if class identifier was obtained successfully
        /// Negative if the call failed
        /// </returns>
        [DllImport("ole32.dll", CharSet = CharSet.Unicode, ExactSpelling = true, PreserveSig = true)]
        public static extern int CLSIDFromString(string sz, out Guid clsid);
    }
}

---

En résumé : Si vous avez besoin de vérifier si une chaîne est un guid et que vous vous souciez des performances, utilisez COM Interop.

Si vous avez besoin de convertir un guid en représentation String en un Guid, utilisez

new Guid(someString);

Answer 2

Une fois que .net 4.0 sera disponible, vous pourrez utiliser Guid.TryParse() .

Answer 3

Vous n'allez pas aimer ça, mais qu'est-ce qui vous fait penser que la capture de l'exception va être plus lente ?

Combien de tentatives d'analyse d'un GUID échouent par rapport aux tentatives réussies ?

Mon conseil est d'utiliser la fonction que vous venez de créer et de profiler votre code. Si vous trouvez que cette fonction est vraiment un point chaud puis le réparer, mais pas avant.

Answer 4

En .NET 4.0, vous pouvez écrire comme suit :

public static bool IsValidGuid(string str)
{
    Guid guid;
    return Guid.TryParse(str, out guid);
}

Answer 5

Je le réécrirais au moins comme suit :

try
{
  value = new Guid(s);
  return true;
}
catch (FormatException)
{
  value = Guid.Empty;
  return false;
}

Vous ne voulez pas dire "GUID invalide" lors d'une SEHException, d'une ThreadAbortException ou d'autres choses fatales ou non liées.

Mise à jour : À partir de .NET 4.0, un nouvel ensemble de méthodes est disponible pour Guid :

• Guid.TryParse
• Guid.TryParseExact

Il faut vraiment les utiliser (ne serait-ce que pour le fait qu'elles ne sont pas implémentées "naïvement" en utilisant try-catch en interne).