Проверьте, является ли string guid без исключения?

Как только .net 4.0 доступен, вы можете использовать Guid.TryParse().

вам это не понравится, но что заставляет вас думать, что поймать исключение будет медленнее?

сколько неудачных попыток проанализировать GUID вы ожидаете по сравнению с успешными?

мой совет-используйте только что созданную функцию и профилируйте свой код. Если вы обнаружите, что эта функция действительно является точка затем исправить это, но не раньше.

в .NET 4.0 вы можете написать следующее:

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

Я бы, по крайней мере, переписал его как:

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

вы не хотите говорить "недопустимый идентификатор GUID" в SEHException, ThreadAbortException или других фатальных или не связанных с ними вещах.

обновление: начиная с .NET 4.0, для Guid доступен новый набор методов:

• Guid.TryParse
• Guid.TryParseExact

действительно, они должны использоваться (хотя бы для того, что они не являются "наивно" реализовано с использованием try-catch внутренне).

взаимодействие медленнее, чем просто поймать исключение:

в счастливом пути, с 10 000 Guids:

Exception:    26ms
Interop:   1,201ms

на несчастливом пути:

Exception: 1,150ms
  Interop: 1,201ms

это более последовательно, но это также последовательно медленнее. Мне кажется, вам лучше настроить отладчик только на разрыв необработанных исключений.

Ну, вот регулярное выражение, которое вам понадобится...

^[A-Fa-f0-9]{32}$|^({|\()?[A-Fa-f0-9]{8}-([A-Fa-f0-9]{4}-){3}[A-Fa-f0-9]{12}(}|\))?$|^({)?[0xA-Fa-f0-9]{3,10}(, {0,1}[0xA-Fa-f0-9]{3,6}){2}, {0,1}({)([0xA-Fa-f0-9]{3,4}, {0,1}){7}[0xA-Fa-f0-9]{3,4}(}})$

но это только для начала. Вам также нужно будет убедиться, что различные части, такие как дата/время, находятся в допустимых диапазонах. Я не могу представить, что это будет быстрее, чем метод try/catch, который вы уже описали. Надеюсь, вы не получаете столько недействительных GUID, чтобы гарантировать этот тип проверки!

по соображениям удобства использования-отладчик всплывает

Если вы собираетесь использовать подход try/catch, вы можете добавить [System.Диагностика.Атрибут DebuggerHidden], чтобы убедиться, что отладчик не ломается, даже если вы установили его на разрыв при броске.

а то is правда, что использование ошибок дороже, большинство людей считают, что большинство их GUID будут генерироваться компьютером так TRY-CATCH не слишком дорого, так как он генерирует только стоимость на CATCH. Вы можете доказать это себе с помощью простого теста два (публичное пользователя, без пароля).

здесь вы идете:

using System.Text.RegularExpressions;


 /// <summary>
  /// Validate that a string is a valid GUID
  /// </summary>
  /// <param name="GUIDCheck"></param>
  /// <returns></returns>
  private bool IsValidGUID(string GUIDCheck)
  {
   if (!string.IsNullOrEmpty(GUIDCheck))
   {
    return new Regex(@"^(\{{0,1}([0-9a-fA-F]){8}-([0-9a-fA-F]){4}-([0-9a-fA-F]){4}-([0-9a-fA-F]){4}-([0-9a-fA-F]){12}\}{0,1})$").IsMatch(GUIDCheck);
   }
   return false;
  }

У меня была аналогичная ситуация, и я заметил, что почти никогда не было недопустимой строки длиной 36 символов. Поэтому, основываясь на этом факте, я немного изменил ваш код, чтобы получить лучшую производительность, сохраняя его простым.

public static Boolean TryStrToGuid(String s, out Guid value)
{

     // this is before the overhead of setting up the try/catch block.
     if(value == null || value.Length != 36)
     {  
        value = Guid.Empty;
        return false;
     }

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

насколько я знаю, Нет ничего похожего на Guid.TryParse в mscrolib. Согласно справочному источнику, тип Guid имеет мега-сложный конструктор, который проверяет все виды форматов guid и пытается их проанализировать. Нет никакого вспомогательного метода, который вы можете вызвать, даже через отражение. Я думаю,вам нужно искать сторонние Парсеры Guid или писать свои собственные.

запустите потенциальный идентификатор GUID, хотя регулярное выражение или какой-то пользовательский код, который проверяет здравомыслие, чтобы убедиться, что strig по крайней мере выглядит как GUID и состоит только из допустимых символов (и, возможно, что он подходит для общего формата). Если он не пройдет проверку здравомыслия, верните ошибку - это, вероятно, отсеет подавляющее большинство недействительных строк.

затем преобразуйте строку, как указано выше, все еще ловя исключение для нескольких недопустимых строк, которые проходят через здравомыслие проверять.

Джон Скит сделал анализ для чего-то подобного для разбора Ints (до того, как TryParse был в рамках): проверка, можно ли преобразовать строку в Int32

 bool IsProbablyGuid(string s)
    {
        int hexchars = 0;
        foreach(character c in string s)
        {
           if(IsValidHexChar(c)) 
               hexchars++;          
        }
        return hexchars==32;
    }

• Сделать Отражатель
• копировать-Н-'paste идентификатор GUID .конструктор(строка)
• замените каждое появление " throw new ..."с"return false".

ctor Guid-это в значительной степени скомпилированное регулярное выражение, таким образом, вы получите точно такое же поведение без накладных расходов исключения.

1. означает ли это обратное проектирование? Я думаю, что да, и как таковой может быть незаконным.
2. сломается, если форма GUID изменения.

еще более крутое решение было бы динамически прибором метода, заменив "бросить новое" на лету.

Я голосую за ссылку GuidTryParse, опубликованную выше Йон или аналогичное решение (IsProbablyGuid). Я буду писать такие же для моей библиотеки конверсий.

Я думаю, что это безнадежно, что этот вопрос должен быть настолько сложным. Ключевое слово" is "или" as " было бы просто прекрасно, если Guid может быть нулевым. Но по какой-то причине, хотя SQL Server в порядке с этим, .NET-нет. Почему? Какова ценность Guid.Пусто? Это просто глупая проблема, созданная дизайн .NET, и это действительно меня беспокоит, когда соглашения языка наступают на себя. Наиболее эффективным ответом до сих пор было использование com-взаимодействия, потому что платформа не обрабатывает его изящно? - Эта струна может быть GUID?- на этот вопрос легко ответить.

полагаться на исключение бросается в порядке, пока приложение не выходит в интернет. В этот момент я просто настроил себя на атаку отказа в обслуживании. Даже если меня не "атакуют", я знаю, что некоторые yahoo собираюсь обезьянничать с URL-адресом, или, может быть, мой отдел маркетинга отправит искаженную ссылку, а затем мое приложение должно пострадать от довольно сильного удара по производительности, который может сбить сервер, потому что я не написал свой код, чтобы справиться с проблемой, которая не должна произойти, но мы все знаем, что произойдет.

это размывает строку немного на "исключение" - но итог, даже если проблема нечаста, если это может произойти достаточно раз за короткий промежуток времени, что ваше приложение аварийно завершает работу обслуживание ловит от всего этого, то я думаю, что исключение-это плохая форма.

TheRage3K

Если TypeOf ctype (myvar,Object) является идентификатором Guid .....

Private Function IsGuidWithOptionalBraces(ByRef strValue As String) As Boolean
    If String.IsNullOrEmpty(strValue) Then
        Return False
    End If

    Return System.Text.RegularExpressions.Regex.IsMatch(strValue, "^[\{]?[0-9a-fA-F]{8}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{12}[\}]?$", System.Text.RegularExpressions.RegexOptions.IgnoreCase)
End Function


Private Function IsGuidWithoutBraces(ByRef strValue As String) As Boolean
    If String.IsNullOrEmpty(strValue) Then
        Return False
    End If

    Return System.Text.RegularExpressions.Regex.IsMatch(strValue, "^[0-9a-fA-F]{8}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{12}$", System.Text.RegularExpressions.RegexOptions.IgnoreCase)
End Function


Private Function IsGuidWithBraces(ByRef strValue As String) As Boolean
    If String.IsNullOrEmpty(strValue) Then
        Return False
    End If

    Return System.Text.RegularExpressions.Regex.IsMatch(strValue, "^\{[0-9a-fA-F]{8}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{12}\}$", System.Text.RegularExpressions.RegexOptions.IgnoreCase)
End Function

С методом расширения в C#

public static bool IsGUID(this string text)
{
    return Guid.TryParse(text, out Guid guid);
}