Writing to TextBox from Seperate Clas Redux

[JP]

Okay - I have incorporated some of the advise I got on the first post and
have gone ahead and setup a simple test project to demonstrate the trouble I
am having. Any help is GREATLY appreciated:

FILE FORM1.CS:

using System;

using System.Drawing;

using System.Collecti ons;

using System.Componen tModel;

using System.Windows. Forms;

using System.Data;

namespace simpleapp

{

/// <summary>

/// Summary description for Form1.

/// </summary>

public class Form1 : System.Windows. Forms.Form

{

private System.Windows. Forms.TextBox textBox1;

private System.Windows. Forms.Button button1;

/// <summary>

/// Required designer variable.

/// </summary>

private System.Componen tModel.Containe r components = null;

public Form1()

{

//

// Required for Windows Form Designer support

//

InitializeCompo nent();

//

// TODO: Add any constructor code after InitializeCompo nent call

//

}

public void addstatus(strin g status)

{
this.textBox1.A ppendText(statu s + "\r\n\r\n") ;

}

/// <summary>

/// Clean up any resources being used.

/// </summary>

protected override void Dispose( bool disposing )

{

if( disposing )

{

if (components != null)

{

components.Disp ose();

}

}

base.Dispose( disposing );

}

#region Windows Form Designer generated code

/// <summary>

/// Required method for Designer support - do not modify

/// the contents of this method with the code editor.

/// </summary>

private void InitializeCompo nent()

{

this.textBox1 = new System.Windows. Forms.TextBox() ;

this.button1 = new System.Windows. Forms.Button();

this.SuspendLay out();

//

// textBox1

//

this.textBox1.L ocation = new System.Drawing. Point(0, 24);

this.textBox1.M ultiline = true;

this.textBox1.N ame = "textBox1";

this.textBox1.R eadOnly = true;

this.textBox1.S ize = new System.Drawing. Size(296, 240);

this.textBox1.T abIndex = 0;

this.textBox1.T ext = "";

//

// button1

//

this.button1.Lo cation = new System.Drawing. Point(0, 0);

this.button1.Na me = "button1";

this.button1.Si ze = new System.Drawing. Size(296, 23);

this.button1.Ta bIndex = 1;

this.button1.Te xt = "button1";

this.button1.Cl ick += new System.EventHan dler(this.butto n1_Click);

//

// Form1

//

this.AutoScaleB aseSize = new System.Drawing. Size(5, 13);

this.ClientSize = new System.Drawing. Size(292, 266);

this.Controls.A dd(this.button1 );

this.Controls.A dd(this.textBox 1);

this.Name = "Form1";

this.Text = "Form1";

this.ResumeLayo ut(false);

}

#endregion

/// <summary>

/// The main entry point for the application.

/// </summary>

[STAThread]

static void Main()

{

Application.Run (new Form1());

}

private void button1_Click(o bject sender, System.EventArg s e)

{

addstatus("Part one, I work \r\n\r\n");

Class1 cls1 = new Class1();

cls1.addstatust oo("Part three, I don't work \r\n\r\n");

}

public string ValueForTextBox

{

set

{

this.textBox1.T ext = value;
}

get

{

return this.textBox1.T ext;

}

}

}

}

file CLASS1.CS:

using System;

namespace simpleapp

{

/// <summary>

/// Summary description for Class1.

/// </summary>

public class Class1

{

public Class1()

{

//

// TODO: Add constructor logic here

//
addstatustoo("p art two, I don't work \r\n\r\n");


}

public void addstatustoo(st ring status)

{

//this doesn't work.

Form1 obj = new Form1();

obj.addstatus(" hi, I don't work either");

obj.addstatus(s tatus);

obj.ValueForTex tBox = "hi I don't work either!" + status + "\r\n\r\n";

}

}

}

[Jon Skeet [C# MVP]]

JP <Jp******@gmail .com> wrote:

Okay - I have incorporated some of the advise I got on the first post and
have gone ahead and setup a simple test project to demonstrate the trouble I
am having. Any help is GREATLY appreciated:

You're not calling Application.Run in, so your form is never running a
message pump.

In Class1 you're creating a new form, but never displaying it. If you
pass a reference to the current form to the addstatustoo method (as an
extra parameter) and act on that instead, you'll see it works.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[JP]

Well, if I make it a static method:

public static void addstatus(strin g status)

{

Form1 obj = new Form1();

obj.textBox1.Ap pendText(status + "\r\n\r\n") ;

obj.ShowDialog( );

}

This works, but it displays a new window for everytime I write to it. For
my purposes I need to have the class be able to write to the form - without
having the form pass a reference.

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

JP <Jp******@gmail .com> wrote:

Okay - I have incorporated some of the advise I got on the first post and
have gone ahead and setup a simple test project to demonstrate the
trouble I
am having. Any help is GREATLY appreciated:

You're not calling Application.Run in, so your form is never running a
message pump.

In Class1 you're creating a new form, but never displaying it. If you
pass a reference to the current form to the addstatustoo method (as an
extra parameter) and act on that instead, you'll see it works.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[JP]

FYI - got this to work by making the textbox static:

private static System.Windows. Forms.TextBox textBox1;

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

JP <Jp******@gmail .com> wrote:

Okay - I have incorporated some of the advise I got on the first post and
have gone ahead and setup a simple test project to demonstrate the
trouble I
am having. Any help is GREATLY appreciated:

You're not calling Application.Run in, so your form is never running a
message pump.

In Class1 you're creating a new form, but never displaying it. If you
pass a reference to the current form to the addstatustoo method (as an
extra parameter) and act on that instead, you'll see it works.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[Jon Skeet [C# MVP]]

JP <Jp******@gmail .com> wrote:

FYI - got this to work by making the textbox static:

private static System.Windows. Forms.TextBox textBox1;

That's fine so long as there's only ever one textbox you need to write
to, but it's very inflexible.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too