Hallo!

Ich versuche mich gerade in Threads einzuarbeiten. Da das noch nicht so richtig klappt, habe ich mal eine Beispielanwendung geschrieben, die nur einen simplen Thread enthält.
Könnte sich das irgendwer schnell ansehen, und mir sagen, was man da anders machen sollte und so?

Hier der Code:

markieren

Code:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;
using System.Threading;

namespace Thread_Test
{
  public partial class Form1 : Form
  {
    public Form1()
    {
      InitializeComponent();
    }

    private void button1_Click(object sender, EventArgs e)
    {
      MyThread MyInstance = new MyThread();
      label1.Text = "Gestartet";
      while (MyInstance.Thread.IsAlive)
      {
        textBox1.Text = MyInstance.Output;
        Application.DoEvents();
      }
      MyInstance.Thread.Join();
      label1.Text = "Beendet";
      textBox1.Text = MyInstance.Output;
    }
  }
  public class MyThread
  {
    public Thread Thread;
    public string Output;

    public MyThread()
    {
      this.Thread = new Thread(new ThreadStart(Execute));
      this.Thread.Start();
    }
    void Execute()
    {
      for (int i = 1; i <= 5e6; i++)
      {
        Output = i.ToString();
      }
    }
  }
}

Das Formular besteht aus: label1, textBox1, button1.

Danke schon mal.

Grüße
Faux

Heißt das, das Niemand antwortet daran, dass keiner versteht was ich will, ich keinen fehlergemacht habe, oder keiner Lust zum Antworten hat?

Grüße
Faux

Nun, was ist denn das Problem? Läuft der Thread erst gar nicht, bricht er mit fehlermeldung ab, ... ?

Es gibt kein Problem. Alles läuft besten, nur wollte ich wissen, ob die Klasse so passt wie sie ist, oder ob ich vielleicht statt public ein private oä. machen soll (nur ein Bsp.).
Denn wenn das so passt, mache ich das jetzt in mehrere Programme rein, wenn mir dann aber in einem Monat wer sagt, dass das und das anders ginge, wär das viel Arbeit zum Ändern..

Faux

markieren

Code:

public Thread Thread;
public string Output;

Das sind ja Public-Felder Für solche Sachen nimmt man private Felder und ReadOnly-Properties

Zitat von Dax:

markieren

Code:

public Thread Thread;
public string Output;

Das sind ja Public-Felder Für solche Sachen nimmt man private Felder und ReadOnly-Properties

Definitiv nicht das Thema!

Allerdings ändere mal den namen deines Threads, von mir aus in thread!

Zitat von Speedmaster:

Definitiv nicht das Thema!

Doch! *g*
GENAU das, wollte ich wissen.

@Dax: Einspruch: Wie könnte ich den Thread dann Stoppen und/oder das Ergebnis auslesen???

Zitat von Speedmaster:

Allerdings ändere mal den namen deines Threads, von mir aus in thread!

Was passt an Thread Testanwendung nicht? Steckt ja auch sogar das Wort Thread drinnen...

Zitat von faux:

@Dax: Einspruch: Wie könnte ich den Thread dann Stoppen und/oder das Ergebnis auslesen???

Das Ergebnis kannst du doch auslesen, eben weil es eine Read-Only-Property ist . Und für Referenzvariablen reicht Read-Only, um auf alle Member zugreifen zu können. Oder willst du der Property andere Instanzen zuweisen?

[add]Ich denke, Speedmaster meint den Namen der Public-Variable. [/add]

Aber Dax meine, dass ich sie private UND readonly machen soll. Und als Private kann ich dann von außen nix machen.
Nein ich will dem Thread keine andere Instanz zuweisen.

Zitat von Dax:

[...] private Felder und ReadOnly-Properties

Das Feld soll private sein, die Property ist natürlich public .