c# - una - Forma más eficiente para probar el tipo de objeto

procesando dinámicamente fórmulas creadas por el usuario c#

Question

Tengo valores almacenados como cadenas en una DataTable donde cada valor podría representar realmente un int , double o string (todos se convirtieron en cadenas durante un proceso de importación desde una fuente de datos externa). Necesito probar y ver qué tipo es cada valor realmente.

¿Qué es más eficiente para la aplicación (o no hay diferencia práctica)?

1. Intenta convertir a int (y luego double ). Si la conversión funciona, la devolución es true . Si se lanza una excepción, devuelve false .
2. Expresiones regulares diseñadas para coincidir con el patrón de un int o double
3. Algún otro método?

Answer 1

El problema que tienes es que podría haber situaciones en las que la respuesta podría ser de los tres tipos.

3 podría ser un int, un doble o una cadena!

Depende de lo que trates de hacer y cuán importante es que sean de un tipo particular. Lo mejor sería dejarlos tal como están, siempre que sea posible o, alternativamente, usar un método para marcar cada uno (si tiene control de la fuente de la cadena original).

Answer 2

Obtendrá diferentes resultados para los diferentes métodos según compile con optimizaciones. Básicamente tienes algunas opciones:

object o;
//checking with is
o is int
//check type
o.GetType() != typeof( int )
//cast and catch exception
try{ int j = (int) o; }
catch {}
//use the tryparse
int.TryParse( Convert.ToString( o ), out j )

Puede configurar fácilmente una aplicación de consola que intente cada una de estas 10.000 veces y devuelva la duración de cada una (pruebe cuando o es un int y cuando se trata de otra cosa).

El método try-catch es el más rápido si el objeto contiene un int, y de lejos el más lento si no lo hace (incluso más lento que GetType ). int.TryParse es bastante rápido si tienes una cadena, pero si tienes un objeto desconocido es más lento.

Curiosamente, con .Net 3.5 y las optimizaciones activadas, la comprobación de o is int toma el mismo tiempo que try-catch cuando o en realidad es un int. o is int es solo un poco más lento si o realmente es otra cosa.

Curiosamente, FxCop lanzará advertencias si haces algo como:

if( o is int )
int j = (int) o;

Pero creo que es un error en FxCop, no sabe que int es un tipo de valor y te recomienda utilizar o as int lugar.

Si su entrada es siempre una cadena int.TryParse es lo mejor; de lo contrario, el operador es el más rápido.

Como tiene una cadena, miraría si necesita saber que es un int, en lugar de un doble. Si int.TryParse pasa, entonces se double.TryParse . double.TryParse para que pueda obtener la mitad del número de cheques - devuelva el doble o el string y el piso el doble cuando espere un int.

Answer 3

Utilizaría double.TryParse, tiene beneficios de rendimiento.

Answer 4

Yo diría que no te preocupes tanto por ese micro rendimiento. Es mucho mejor hacer que algo funcione, y luego hacerlo lo más claro y conciso y fácil de leer posible. Lo peor que puedes hacer es sacrificar la legibilidad por una cantidad insignificante de rendimiento.

Al final, la mejor forma de lidiar con los problemas de rendimiento es guardarlos cuando tenga datos que indiquen que hay un problema de rendimiento real ... de lo contrario perderá mucho tiempo micro-optimizando y de hecho causará mayores costos de mantenimiento para mas tarde.

Si encuentra que esta situación de análisis es realmente el cuello de botella en su aplicación, ENTONCES es el momento de intentar averiguar cuál es la forma más rápida de resolver el problema. Creo que Jeff (y muchos otros) han escrito mucho sobre este tipo de cosas.

Answer 5

Yo personalmente usaría int.tryparse, luego double.tryparse. El rendimiento en esos métodos es bastante rápido. Ambos devuelven un booleano. Si ambas fallas, entonces tienes una cadena, por la forma en que definiste tus datos.