[Iera Namhorun]

I tried to display the string into the richtextbox, but always got the same or duplicate string after the second time it was run as in example of displaying a second string.

Example of displaying the first string:

[21:55:07] - [-] Starting scanning, please wait. . .

[21:55:09] - [-] Finished.

Example of displaying a second string:

[21:55:07] - [-] Starting scanning, please wait. . .

[21:55:09] - [-] Finished.

[21:55:07] - [-] Starting scanning, please wait. . .

[21:55:09] - [-] Finished.

code :
Spoiler (Click to View)

Code

   private void Display(string text, bool newLine = true)
   {
       DisplayTextBox.AppendText(text);
       if (newLine)
       {
           DisplayTextBox.AppendText(Environment.NewLine);
           DisplayTextBox.ScrollToCaret();
       }
   }

       private void startScan()
       {
           Display("...........................................................................................", true);
           Display("[ " + DateTime.Now.ToLongTimeString() + " ] - [ # ] Starting scanning, please wait . . . . . .", true);
           Display("...........................................................................................", true);
           Display(Environment.NewLine, true);
           Thread.Sleep(100);
           this.DirButton.Enabled = false;
           this.ScanButton.Enabled = false;
           scanned = 0;
           skipped = 0;
           worker.WorkerReportsProgress = true;
           worker.DoWork += Scan;
           worker.ProgressChanged += this.ProgressChanged;
           worker.RunWorkerCompleted += this.RunWorkerCompleted;
           worker.RunWorkerAsync();
       }

       private static void Scan(object sender, DoWorkEventArgs e)
       {
           startScan(e.Argument.ToString());
       }

       private void ProgressChanged(object sender, ProgressChangedEventArgs e)
       {
           Display(e.UserState.ToString(), true);  
       }

       private void RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
       {
           if (e.Error != null)
           {
               this.Display("[ " + DateTime.Now.ToLongTimeString() + " ] - [ ! ] Exception: " + e.Error.ToString(), true);
           }
           if (!e.Cancelled)
           {
               string text = "[ " + DateTime.Now.ToLongTimeString() + " ] - Scanned [ " + scanned + " ] files ";
               string text2 = "[ " + DateTime.Now.ToLongTimeString() + " ] - Skipped [ " + skipped + " ] files ";
               Display(Environment.NewLine,true);
               Display("--------------------------------------------------------------------------------------", true);
               Display("[ " + DateTime.Now.ToLongTimeString() + " ] - [ √ ] Finished.", true);
               Display("--------------------------------------------------------------------------------------", true);
               Thread.Sleep(100);
               if (Form1.skipped > 0)
               {
                   Display(text2,true);
               }

               else if (Form1.skipped == 0)
               {
                   Display(text2,true);
               }

               this.Display(text,true);
               Display("--------------------------------------------------------------------------------------", true);
               Thread.Sleep(100);
               DirButton.Enabled = true;
               ScanButton.Enabled = true;
               return;
           }
           Display("[ " + DateTime.Now.ToLongTimeString() + " ] - [ x ] Scanning cancelled", true);
           DirButton.Enabled = true;
           ScanButton.Enabled = true;
       }

   private static int scanned = 0;
   private static int skipped = 0;
   private static BackgroundWorker worker = new BackgroundWorker();

I have tried adding

Code

if (DisplayTextBox.Text ! = null)
   {
      DisplayTextBox.Clear();
    }

but still there is no change, the second time the scan button is run it will bring up the same two strings.

How do I prevent it from displaying duplicate strings like in the example second string?
I want to display strings like in the example of the first string display.

---

[FIXED]
I try to use Application.Restart(); to restart Form into its original state as well to prevent displaying the same string again.
And log all the strings to text file using Streamwriter

[NYANxCAT]

are you using check for cross-thread before calling the backgroundworker? ??
because you can't use the richtextbox without invoking it to UI thread.

if you don't want to invoke it, then add "CheckForIllegalCrossThreadCalls = false;" inside your Form1_Load.

[Washi]

are you using check for cross-thread before calling the backgroundworker? ??
because you can't use the richtextbox without invoking it to UI thread.

if you don't want to invoke it, then add "CheckForIllegalCrossThreadCalls = false;"  inside your Form1_Load.

Unless very specific circumstances are applicable (great emphasis on the word "very" here), nobody should ever use the CheckForIllegalCrossThreadCalls property, ever.

[NYANxCAT]

Unless very specific circumstances are applicable (great emphasis on the word "very" here), nobody should ever use the CheckForIllegalCrossThreadCalls property, ever.

I agree with you.

[Iera Namhorun]

are you using check for cross-thread before calling the backgroundworker? ??
because you can't use the richtextbox without invoking it to UI thread.

if you don't want to invoke it, then add "CheckForIllegalCrossThreadCalls = false;"  inside your Form1_Load.

I don't use it, thank you for the information and I will try to fix it again 😉