La forma más rápida de contar el número de caracteres en mayúsculas en C#

Question

¿Alguna idea sobre la eficacia de esto? ...

CommentText.ToCharArray().Where(c => c >= 'A' && c <= 'Z').Count() 

Answer 1

bien, sólo golpeó un poco de código para programar su método contra esta:

int count = 0; 
for (int i = 0; i < s.Length; i++) 
{ 
    if (char.IsUpper(s[i])) count++; 
} 

El resultado:

saluda: 19737 garrapatas

minas: 118 garrapatas

diferencia bastante grande! A veces, la forma más directa es la más eficiente.

Editar

Solo por curiosidad, esto:

int count = s.Count(c => char.IsUpper(c)); 

está en el puesto en alrededor de 2.500 garrapatas. Entonces, para un trazador de líneas "Linqy" es bastante rápido.

Answer 2

Sólo estás contando ASCII estándar y no ADE etc.

¿Qué tal

CommentText.ToCharArray().Where(c => Char.IsUpper(c)).Count() 

Answer 3

Sin ni siquiera las pruebas diría

int count = 0; 
foreach (char c in commentText) 
{ 
    if (Char.IsUpper(c)) 
     count++; 
} 

es más rápido, ahora fuera de probarlo.

Answer 4

Primero, no hay ninguna razón por la que deba llamar al ToCharArray() ya que, suponiendo que CommentText es una cadena, ya es un IEnumerable<char>. En segundo lugar, probablemente debería llamar al char.IsUpper en lugar de asumir que solo está tratando con valores ASCII. El código realmente debe ser similar,

CommentText.Count(char.IsUpper) 

En tercer lugar, si usted está preocupado acerca de la velocidad que no hay mucho que puede vencer a la edad de bucle,

int count = 0; 
for (int i = 0; i < CommentText.Length; i++) 
    if (char.IsUpper(CommentText[i]) count++; 

En general, llamar a cualquier método va ser más lento que escribir el código, pero este tipo de optimización solo debe realizarse si está absolutamente seguro de que este es el cuello de botella en su código.

Answer 5

Lo que está haciendo con ese código es crear una colección con los caracteres, luego crear una nueva colección que contenga solo los caracteres en mayúsculas, luego recorra esa colección solo para descubrir cuántos hay.

Esto se obtienen mejores resultados (pero todavía no es tan bueno como un bucle sin formato), ya que no crea las colecciones intermedias:

CommentText.Count(c => Char.IsUpper(c)) 

Editar: Se ha eliminado la llamada ToCharArray también, como Matt sugirió.

Answer 6

Tengo este

Regex x = new Regex("[A-Z]{1}", 
    RegexOptions.Compiled | RegexOptions.CultureInvariant); 
int c = x.Matches(s).Count; 

pero no sé si es particularmente rápida. No va a caracteres especiales, ya sea, yo -Supongo

EDIT:

Comparativa rápida de la respuesta de esta pregunta. Depurar en vshost, 10000 iteraciones con la cadena:
aBcDeFGHi1287jKK6437628asghwHllmTbynerA

• La respuesta: 20-30 ms
• La solución de expresiones regulares: 140-170 ms