C#} For Loop-Probleme

Question

Hallo, also meine Schule hat Aufzeichnungen, die ich zugreifen musste, aber ich habe vergessen, meine Benutzer-ID, so machte ich ein Programm, das jede mögliche PDF-Datei herunterladen und sie nach meinem Namen suchen würde. Wenn es meine gefunden hat, würde es mich jedoch warnen, wenn das Programm heruntergefahren war und sagte: "Wir haben eine Datei gefunden, die Sean Dateiname enthält: xxxxxx.pdf" Der tatsächliche Zahlenteil wäre ausgeschaltet, egal wie stark die for-Schleife inkrementiert wird war auf einer separaten Methode! BTW Schulname wird hervorgehoben!

* D ie Art und Weise! In Search() verwenden I i-10 als temporäre Lösung, aber ich war ihr Denken muss ein besserer Weg geben *

---

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 
using System.Threading.Tasks; 
using System.Net; 
using BitMiracle.Docotic.Pdf; // pdf parser/viewer 
using System.IO; 
using System.Threading; 

namespace XXXXXXX_TEMP 
{ 
    class Program 
{ 
    static void Main(string[] args) 
    { 
     Program p1 = new Program(); 
     p1.Start(); 
    } 

    private void Start() 
    { 
     WebClient wc = new WebClient(); 
     for (int i = 200000; i < 999999999; i = i + 10) 
     { 
      try 
      { 
       wc.DownloadFile("XXXXXXXXXXXXXXXXXXXXXX/" + i + ".pdf", i + ".pdf"); 
      } 
      catch (WebException) { 
       continue; 
      } 
      PdfDocument pdf = new PdfDocument(i + ".pdf"); 
      Thread a = new Thread(() => Search(i, pdf)); 
      a.Start(); 

      if (i == 999999) { 
       Console.ReadLine(); 
      } 
     } 
    } 

    private void Search(int i, PdfDocument pdf) 
    { 
     i = i - 10; 
     String html = pdf.GetText(); 
     if (html.ToLower().Contains("sean")) 
     { 
      Console.WriteLine("Found File Containing Sean! File Name Is : " + i + ".pdf\n"); 
      Console.WriteLine("PDF Text = " + html); 
     } 
    } 
    } 
} 

Answer 1

Was Sie ausgeführt haben in ist closing over the loop variable genannt. Sie sollten die verknüpfte Antwort sowie den Blogeintrag, auf den sie verweist, sorgfältig lesen.

Die sofortige Lösung für Ihr Problem besteht darin, eine Kopie von i an den Thread zu übergeben. Das heißt:

int index = i; 
Thread a = new Thread(() => Search(index, pdf)); 

Allerdings könnten Sie feststellen, dass alle Threads beginnen ernsthaft Ihr System versinkt, und verlassen alle diese Fäden hängen möglicherweise auf Ressourcen essen konnte und zu einem der Speicher Ausnahme führen. Ein sauberer Weg, es zu tun ist ThreadPool.QueueUserWorkItem zu nennen, wie folgt aus:

ThreadPool.QueueUserWorkItem(o => Search(index, pdf)); 

Oder in neueren Versionen von .NET:

Task.Run(() => Search(index, pdf)); 

Simplest way to do a fire and forget method in C#? finden Sie weitere Informationen über Fire-and-Forget-Threads.