Проверьте, является ли строка идентификатором 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] атрибут, чтобы убедиться, что отладчик не ломается, даже если вы установили его на разрыв при броске.

а то и правда, что использование ошибок дороже, большинство людей считают, что большинство их 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

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

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

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

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

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

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

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

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

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

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

TheRage3K

Если ctype для вызова typeof(аргумент myVar,объект) - это идентификатор 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);
}