Tråde textbox1

JEg vil gerne skrive til textbox1 fra Modem klassen

Prøv at ændre din UpdateText metode til dette:


private void UpdateText(String logdata)//
{
    this.Invoke(new MethodInvoker(delegate()
        {
            textBox1.Text = logdata;
        }));

}

Invoke metoden sætter en delegate til at køre i formens egen tråd.

ok super

Men hvordan skriver jeg til metoden fra modem klassen??

Jeg tror ikke det jeg har skrevet i Modem klassen er rigtigt?

For at kunne opdatere formen fra din Modem klasse, bliver du nød til at oprette en reference til formen i Modem klassen. Dette kan for eksempel være en property som bliver sat fra konstruktøren.

Class Modem
{
  public Form1 OwnerForm{get; private set;}

  public Modem(Form1 ownerForm)
  {
    this.OwnerForm = ownerForm;

    // nu kan du sætte labelen fra Modem klassen, hvis Label1 er sat til at være public.
    this.OwnerForm.Label1.Text = "test fra modem";
  }
}

Dette er dog meget dårligt design, da du gør din Modem klasse afhængig af denne form.

Et bedre design vil være at lade Modem klassen være uafhængig. Formen kan da bruge Modem klassen, og opdatere sit brugergrænsesnit alt efter dens state.

Hvis du forklare lidt mere om hvad du prøver at lave, så skal jeg prøve at hjælpe.

Jeg prøver at lave en "log" i en textbox1
I min modem klasse ønsker jeg at string variablen data, bliver printet i textbox1.

Jeg kunne godt tænke mig at jeg i min modem klasse kunne skrive

UpdateText(data);

Hvorpå min textbox1 vil vise indeholdet af data.

Håber det er til at forstå :o)

Undskyld jeg var lidt langsom om at svare, men jeg havde lidt travlt :)

Her er en god måde at gøre det på.

Her er en Modem klassen:
public class Modem
{
    private Timer testTimer;

    public Modem()
    {
        // Dette er testkode, for at simulere at der kommer nye logbeskeder en gang i sekundet.
        testTimer = new Timer(1000);
        testTimer.Elapsed += new ElapsedEventHandler(testTimer_Elapsed);
        testTimer.Start();
    }

    private string _logData;
    public string LogData
    {
        get { return _logData; }
        set
        {
            _logData = value;
            this.OnLogDataChanged();
        }
    }

    public event EventHandler LogDataChanged;

    protected void OnLogDataChanged()
    {
        if (this.LogDataChanged != null)
            this.LogDataChanged(this, EventArgs.Empty);
    }

    //testkode
    void testTimer_Elapsed(object sender, ElapsedEventArgs e)
    {
        this.LogData += string.Format("log entry, tid: {0}\r\n\", DateTime.Now.ToString());
    }
}

Det som har med timeren at gøre, er bare testkode for at simulere at der kommer nye logbeskeder en gang i sekundet. Den som sker er, at når LogData propertien bliver ændret, kastet LogDataChanged eventet. Dette event skal du lytte på i formen. Herunder er koden til min form:

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

    private Modem modem = new Modem();

    protected override void OnLoad(EventArgs e)
    {
        base.OnLoad(e);

        this.modem.LogDataChanged += new EventHandler(modem_LogDataChanged);
    }

    void modem_LogDataChanged(object sender, EventArgs e)
    {
        if (this.InvokeRequired)
        {
            this.Invoke(new MethodInvoker(delegate()
                {
                    this.textBox1.Text = this.modem.LogData;
                }));
        }
        else
        {
            this.textBox1.Text = this.modem.LogData;
        }
    }
}

Bare spørg, hvis der er noget du ikke forstår :)

KAn jeg så bruge this.LogData += string.Format("log entry, tid: {0}\r\n", DateTime.Now.ToString());  som en "funktion" jeg kan indsætte forskellige steder i min kode??

JEg ville gerne hvis jeg kunne gøre sådan

data=sp.ReadLine();
UpdateText(data); //"log" skriver til textbox1

Du kan jo godt oprette en UpdateText(string data) metode i Modem klassen:

publiv void UpdateText(string data)
{
  this.LogData += data + Environment.NewLine;
}

Hvis du abonnere på eventet i formen, vil dette skrive til tekstboksen.

hehe så mistede du mig lige igen.

Hvis du abonnere på eventet i formen, vil dette skrive til tekstboksen.

hvad vil det betyde hvis jeg bruger din kode i Form1??

Ok, lidt bedre forklaring :)

Hvis du har gjort, som jeg gjorde i mit eksempel, på din form. Dvs først abonnerer du på eventet:

this.modem.LogDataChanged += new EventHandler(modem_LogDataChanged);

og så laver du en handler til eventet:
void modem_LogDataChanged(object sender, EventArgs e)
{
....
}

Hvis du har gjort dette. Så kan du tilføje følgende metode til Modem klassen fra mit tidligere eksempel:

publiv void UpdateText(string data)
{
  this.LogData += data + Environment.NewLine;
}

Så vil der blive skrevet til tekstboksen, hver gang du kalder UpdateText(data);

Jeg kan bare ikke få det til at virke

Modem klassen: - flere stedet har jeg skrevet UpdataText(data);

        private string _logData;
        public string LogData
        {
            get { return _logData; }
            set
        {
            _logData = value;
            this.OnLogDataChanged();
        }
    }

    public event EventHandler LogDataChanged;

    protected void OnLogDataChanged()
    {
        if (this.LogDataChanged != null)
            this.LogDataChanged(this, EventArgs.Empty);
    }
public void UpdateText(string data)
{
  this.LogData += data + Environment.NewLine;
}

Klassen Form1

        private MultiModemSetup modem = new MultiModemSetup();

        protected override void OnLoad(EventArgs e)
        {
            base.OnLoad(e);

            this.modem.LogDataChanged += new EventHandler(modem_LogDataChanged);
        }

        void modem_LogDataChanged(object sender, EventArgs e)
        {
            if (this.InvokeRequired)
            {
                this.Invoke(new MethodInvoker(delegate()
                {
                    this.textBox1.Text = this.modem.LogData;
                }));
            }
            else
            {
                this.textBox1.Text = this.modem.LogData;
            }
        }

Nu prøvede jeg at copy paste koden ind, præcist som du har skrevet den. Og det virker helt fint hos mig.

Hvilken fejl får du?

Jeg får ingen fejl, men der sker bare intet i textbox1

Prøv lige igen tak :o)

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

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

        private void Form1_Load(object sender, EventArgs e)
        {
            modem.start();
        }
        private Modem modem = new Modem();

        protected override void OnLoad(EventArgs e)
        {
            base.OnLoad(e);

            this.modem.LogDataChanged += new EventHandler(modem_LogDataChanged);
        }

        void modem_LogDataChanged(object sender, EventArgs e)
        {
            if (this.InvokeRequired)
            {
                this.Invoke(new MethodInvoker(delegate()
                {
                    this.textBox1.Text = this.modem.LogData;
                }));
            }
            else
            {
                this.textBox1.Text = this.modem.LogData;
            }
        }

    }
}



using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Timers;

namespace Traade
{
    class Modem
    {

    public Modem()
    {
       
    }

    private string _logData;
    public string LogData
    {
        get { return _logData; }
        set
        {
            _logData = value;
            this.OnLogDataChanged();
        }
    }

    public event EventHandler LogDataChanged;

    protected void OnLogDataChanged()
    {
        if (this.LogDataChanged != null)
            this.LogDataChanged(this, EventArgs.Empty);
    }

    public void UpdateText(string data)
    {
        this.LogData += data + Environment.NewLine;
    }
  public void start()
    {
        String data = "hej";
        UpdateText(data);
    }
    }


    }

Det er fordi du kalder modem.Start() før eventhandleren bliver tilføjet. Prøv at flytte model.Start ned i denne metode:

rotected override void OnLoad(EventArgs e)

derefter kan du bare slette Form1_load metoden. Brug OnLoad i stedet.

Nu ser OnLoad metoden altså sådan ud:

Ok

Jeg tror det der går galt er at jeg kun ønsker at skrive i textbox1 når jeg selv vil, ikke noget med at den skal køre og opdater heletiden.

UpdateText("hej");
.....
....
...
UpdateText("hej");
.....
....
...
UpdateText("hej");

Den forstod jeg ikke?

Det er vel det der sker nu! Når du kalder UpdateText, skriver du til tekstboksen!

Eller misforstod jeg lige hvad du mente?

Når jeg prøver det virker det kun når jeg loader programmet, så kommer data til at stå i textbox1.

Jeg vil gerne kunne skrive til textbox1 under kørsel af mit program.

Altså når programmet køre og der skal en handling så bliver det skrevet ud i textbox1.

Ja, og det gør den også nu. Er det en multiline tekstboks du skriver til? Hvis det er en single line tekstboks, vil du altid kun se den øverste linje. Sådan som den virker nu, lægger den en ny linje til, hver gang du kalder UpdateText.

Hvis du vil have den til at overskrive den gamle tekst, skal du ændre din UpdateText metode, sådan:

Her ved mig virker det bare som om at det ikke kommer i samme tråd som form1

jeg har sendt dig en mail herinde.

hmm får du min mail?? den ligger i min indbakke virkerligt mærkeligt

Nu har jeg set på din kode. Jeg tror aldrig at UpdateText() metoden bliver kaldt. Du kalder den fra SendSMS metoden, men jeg kan ikke se at du kalder sendSMS() nogen steder fra i din form!

Prøv at sætte et breakpoint inde i UpdateText metoden, for at se om den breaker.

Hvis du er i tvivl om, hvordan man sætter breakpoints og bruger dem, så prøv at læse denne korte artikkel:
http://msdn.microsoft.com/en-us/library/ms173083%28v=vs.80%29.aspx (Se på "Debugging" afsnittet).

Det du kan se er kun en lille del af den samlede kode, men den kommer igennem SendSMS, for jeg modtager både SMS og jeg får skrevet til min log txt fil.

Hmm, mærkeligt.

Blokerer du tråden? Dette kan ske hvis koden suser rundt i en while løkke eller lignende? Et symptom på dette, vil være at formen går i "svarer ikke" mode. Hvis dette sker, vil formen ikke blive opdateret.

Men, jeg skal prøve at se lidt mere på det senere i dag. Lige nu har jeg lidt travlt :)

Ja jeg kunne jo have prøvet at steppe igennem programmet sorry havde jeg ikke lige tænkt på.

Når UpdateText() bliver kaldt, hopper den i methoden, og derefter i LogData(), og OnLogDataChanged(), så køre den
if(this.LogDataChanged !=null), og her hopper den over handlingen i if sætningen og køre videre i mit program.

Som jeg forstår dig, så er kalder den ikke eventet? Altså at (this.LogDataChanged != null) bliver false?

Hvis dette sker, er det fordi at denne linje ikke har været kaldt:
this.modem.LogDataChanged += new EventHandler(modem_LogDataChanged);

eventuelt at du kalder this.modem = new MultiModemSetup() efter at du abonere på eventet.

nej det virker heller ikke, den kommer stadig ud som false

Vil det fra MultiModemSetup klassen komme i samme tråd som Form1??

Med mindre du gør noget aktivt for at køre noget kode i en anden tråd, vil det køre i samme tråd som det der kalder koden. Hvis du er i tvivl om du har flere tråde, så har du højest sandsynligt bare en. Med mindre selvfølgelig, at du bruger et eller andet tredjeparts værktøj som opretter nye tråde.


hvis den springer over denne linje: if(this.LogDataChanged !=null), så er det fordi der ikke er tilføjet en event-handler. Det kan kun skyldes 2 ting. Enten har du ikke tilføjet event-handleren, ellers har du oprettet event-handleren på forkert instans af Modem klassen. Event-handlers er aktive, uanset om de bliver kaldt fra andre tråde, så det har ikke noget med det at gøre.

Prøv at brug lidt mere tid til at steppe igennem koden. Kontroller at du tilføjer eventhandleren før du kalder UpdateText, og dobbeltjek at det er rigtig instans du tilføjer eventhandleren på.

Det er denne linje der tilføjer en event-handler.
this.modem.LogDataChanged += new EventHandler(modem_LogDataChanged);

Og den tilføjer eventhandleren til this.modem instansen, og ikke andre eventuelt oprettede instanser af klassen.

Hej

Jeg fandt ud af det. Jeg oprettede 2 instancer af multimodem klassen og det var efter eventhandleren.

Men 1000 tak for hjælpen, det har været meget lærerigt.

Jeg lægger et svar :)