Question

考虑以下功能：

public enum Operator
{
    EQUAL = 1,
    GREATER_THAN = 2
}

public class checkString
{
    public static bool isValid(string inputString, string checkString, Operator operation)
    {
        switch (operation)
        {
            case Operator.EQUAL:
                if (inputString == checkString)
                    return true;

                break;

            case Operator.GREATER_THAN:

                // Numeric check for greater than
                try
                {
                    double inputDouble, checkDouble;

                    inputDouble = Convert.ToDouble(inputString);
                    checkDouble = Convert.ToDouble(checkString);

                    if (inputDouble > checkDouble)
                        return true;
                }
                catch (Exception) 
                { }

                // Date check for greater than
                try
                {
                    DateTime inputDate, checkDate;

                    inputDate = DateTime.Parse(inputString);
                    checkDate = DateTime.Parse(inputString);

                    if (inputDate. > checkDate)
                        return true;
                }
                catch (Exception)
                { }

                break;
        }
        return false;
    }
}

参数

inputString：我们想要评估的内容
checkString：输入必须评估的标准（值）
运营商：我们想要执行的操作的枚举

其他要知道的事情

如果满足条件，则根据此方法评估文件中的每一行以返回
评估文件中记录的过程逐行检查，在一个实例中它等于条件。它还可以检查同一行是否也大于条件。检查完成后，它将移至下一条记录
除了默认设置之外，没有其他事件侦听器连接起来，我没有将额外数据推送到调试或跟踪日志

问题

在这个过程的任何一点上，我都不知道人们要评估的是什么，但我需要能够检查“某事物”（无论是什么）等于，大于或小于其他东西。当然我检查其他的东西，但我已经大大简化了这个功能。

也就是说，使用EQUAL或NOT_EQUAL可以快速运行，非常快速有效地处理非常大的文件中的记录。一旦我添加了GREATER_THAN逻辑，它的速度很慢......到了需要几分钟才能处理过去需要半分钟时间的20兆字节文件。

据我所知：

各地都有例外情况。无法保证字段将是数字或日期类型。所以我必须尝试转换为这些数据类型以尝试评估条件
当抛出异常时，控制台会在没有指示它的情况下输出输出，它是自动的

是的我在这方面缺乏经验，我希望了解有关异常处理的更多信息以及幕后真正发生的事情，因为当其他80％的记录都不是数字时，那就是很多例外情况。 meg，8万条记录文件。

有没有更好的方法来处理演员本身以提高效率？我见过double.Parse / TryParse并且可以在前面直接演员，但我不确定哪个最有利。

Answer 1

分别使用double.TryParse和DateTime.TryParse代替Convert.ToDouble和DateTime.Parse。

示例：的

double result;

if (double.TryParse(someString, out result))
{
    Console.WriteLine(result);
}
else
{
    // not a valid double
}

Answer 2

您可以对这些数据类型使用TryParse（）。例外情况既麻烦又昂贵。如果在不抛出异常的情况下，TryParse将返回true / false。所以你可以查看通话结果。比异常更有效率。

Convert.ToDouble（）和Double.Parse（）将抛出异常。

试试这段代码。它不是最好的，但它比你现在考虑的更好，因为你不知道它的类型是什么：

public static bool isValid(string inputString, string checkString, Operator operation)     
        {         
            double dblTmp1;
            double dblTmp2;

            if (Double.TryParse(inputString, out dblTmp1) && double.TryParse(checkString, out dblTmp2))
            {
                return Compare<Double>(dblTmp1, dblTmp1, operation);
            }

            DateTime dtTmp1;
            DateTime dtTmp2;
            if (DateTime.TryParse(inputString, out dtTmp1) && DateTime.TryParse(checkString, out dtTmp2))
            {
                return Compare<DateTime>(dtTmp1, dtTmp2, operation);
            }

            throw new InvalidOperationException("Unknown type");

        }

        public static bool Compare<T>(T obj1, T obj2, Operator operation) where T : IComparable      
        {
            switch (operation)
            {
                case Operator.EQUAL:
                    {
                        return obj1.Equals(obj2);
                    }
                case Operator.GREATER_THAN:
                    {
                        return obj1.CompareTo(obj2) > 0;
                    }
                default:
                    {
                        throw new InvalidOperationException("Unknown operation");
                    }
            }
        }

Answer 3

请记住，使用异常会降低程序的速度，因为在后台运行时正在创建异常堆栈，以便在抛出异常时能够展开它。无论你的程序是否抛出，这个堆栈都会被维护，而这种开销会降低你的速度。

Answer 4

在这种情况下，其他答案可能是最佳解决方案，但在一般情况下，您可以通过捕获特定异常（可能是NumberFormatException或ClassCastException来改进解决方案。捕获Exception会导致各种烦人的，难以追踪的问题（因为您没有记录异常）。

有没有更好的方法来处理强制转换异常？

4 个答案: