编程崩溃原因未知

Question

所以我有一个用 C# 编写的基本应用程序。它基本上写入一个库存文件。它只会在创建文件的过程中停止。我对这里发生的事情感到非常困惑，因为如果我在 IDE 中运行它，它就会停止工作。该文件在文件中的不同停止处停止，因此这不是单一事件。如果这有什么不同的话，我正在使用线程池。我有一个循环，它遍历一个文件并读取该文件并提示一个新线程。如果没有错误的话，调试起来真的很困难。

  static void Main(string[] args)
    {
        //string asins;
        Readfile r = new Readfile();
        r.read();

        Console.WriteLine("Press any key to exit.");
        System.Console.ReadKey();

        //Thread.Sleep(60000);

        //createoutward c = new createoutward();
       // c.read();

        //p.print(s.scrap(r.read()));

    }

我制作线程

public string[] read()
    {
        ThreadPool.SetMaxThreads(10, 100);
        string[] asins;

        string[] lines = System.IO.File.ReadAllLines(@"C:\Users\Joe T\Desktop\AmazonAsins.csv");

        using (System.IO.StreamWriter file = new System.IO.StreamWriter(@"\\WIN-UWPWZ7Z3RKX\wwwroot\repricer\WriteLines2a.txt"))
            file.WriteLine("Product-ID\tPrice1\tPrice2\tRank\tAFN\t" + DateTime.Now);
        prices = new string[lines.Length, 2];
        int i = 0;
        asins = new string[lines.Length];

        foreach (string line in lines)
        {
            scraping s = new scraping();


            char[] tabs = { '\t' };
            string asin;
            string[] words = line.Split(tabs);
            asin = words[1];
            asins[i] = asin;

            Thread.Sleep(1000);
            ThreadPool.QueueUserWorkItem(new WaitCallback(s.scraping1), asin);

            ++i;


        }


        return asins;
    }

抓取类的方法

     public void scraping1(object a)
    {
        string AFN = "N";

        string asin = (string)a;

        double price, price2;
        string sprice;
        string context;
        string page = "*****" + asin;
        try
        {
            WebZinc WebZincProduct = new WebZinc();
            WebZincProduct.OpenPage(page);

            context = WebZincProduct.CurrentPage.Text;
        }
        catch
        {
            scraping1(a);
            return;
        }

        Regex regex11 = new Regex("****\r\n((.|\n)*?)****",
            RegexOptions.IgnoreCase);
        Match oP1 = regex11.Match(context);

        if (oP1.Value.Contains("*******"))
        {
            AFN = "Y";
        }
        Regex reg = new Regex(@"[0-9]+\.[0-9]+");
        MatchCollection mc = reg.Matches(oP1.Value);


        double cost = 0.0;
        double cost2 = 0.0;
        double shipping2 = 0.0;
        double shipping = 0.0;
        int j = 0;
        int j3 = 0;

        foreach (Match m in mc)
        {
            if (j == 0) cost = Convert.ToDouble(m.Value);
            if (j == 1) shipping = Convert.ToDouble(m.Value);
            Console.WriteLine("{0}", m.Value);
            ++j;
        }


        Regex regex4 = new Regex("****\r\n\r\n((.|\n)*?)****",
            RegexOptions.IgnoreCase);
        Match oP4 = regex4.Match(context);

        MatchCollection mc4 = reg.Matches(oP4.Value);

        foreach (Match m in mc4)
        {
            if (j3 == 0) cost2 = Convert.ToDouble(m.Value);
            if (j3 == 1) shipping2 = Convert.ToDouble(m.Value);
            Console.WriteLine("{0}", m.Value);
            ++j3;
        }



        price2 = cost2 + shipping2;
        price = cost + shipping;
        if (price == 0.0 && i != 5)
        {
            scraping1(a);
        }



        string rank = rankAFN(asin);
        lock (Program._locker)
        {

                using (System.IO.StreamWriter file = new System.IO.StreamWriter(@"\\WIN-UWPWZ7Z3RKX\wwwroot\repricer\WriteLines2a.txt", true))
                    file.WriteLine(asin + "\t" + price + "\t" + price2 + "\t" + rank + "\t" + AFN);

    }
}

Answer 1

这是我的最佳猜测，基于这样的事实：这里有很多额外的代码，如果没有看到整个代码，我们就不可能解释这些代码（这就是为什么最好发布最小的示例来重现问题，如 Jon Skeet 如此精彩）在他的博客文章 写出完美的问题）。

但这是我的猜测。我猜测，并且感觉非常强烈，你在这里有失控的递归。您的方法 scrapping1() 在出现异常和未从参数解释的某些条件时对其自身进行递归调用。

因为这些递归调用取决于局部变量或操作而不是参数，所以很难安全地控制递归将执行的操作，并且在这种情况下您可能不应该进行递归调用。

    catch
    {
        // recursion here passing the SAME arg, what is to stop this?
        scraping1(a);
        return;
    }

你可以做的另一

    // WHERE does this 'i' come from?  I don't see where it's incrementing!
    // possible unsafe recursion...
    if (price == 0.0 && i != 5)
    {
        // recursion here passing the SAME arg, no stop condition?
        scraping1(a);
    }

件事是用 try/catch 包围 scraping1() 方法的主体，这样你至少可以在屏幕上看到异常并知道它发生在哪一行。

 public void scraping1(object a)
 {
     try 
     {
          // your method logic
     }
     catch (Exception ex)
     {  
          Console.WriteLine(ex);
          Console.Out.Flush();
     }
}

但是，如果它是递归，则会导致 StackOverflowException ，CLR 将终止您的进程。在这种情况下，请注释掉那些递归调用并考虑您真正想要做什么。我认为在这种情况下你真的不想进行递归。您只是想“重试”吗？

Answer 2

排队到线程池的任务中发生的任何未处理的异常都将导致您的应用程序关闭。将 scraping1 方法的整个主体放入 try/catch 块中，在 catch 的顶部设置一个断点，您至少应该能够看到抛出了什么异常。