Writing to TextBox from Seperate Clas Redux

J

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.Collections;

using System.ComponentModel;

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.ComponentModel.Container components = null;

public Form1()

{

//

// Required for Windows Form Designer support

//

InitializeComponent();

//

// TODO: Add any constructor code after InitializeComponent call

//

}



public void addstatus(string status)

{


this.textBox1.AppendText(status + "\r\n\r\n");

}

/// <summary>

/// Clean up any resources being used.

/// </summary>

protected override void Dispose( bool disposing )

{

if( disposing )

{

if (components != null)

{

components.Dispose();

}

}

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 InitializeComponent()

{

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

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

this.SuspendLayout();

//

// textBox1

//

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

this.textBox1.Multiline = true;

this.textBox1.Name = "textBox1";

this.textBox1.ReadOnly = true;

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

this.textBox1.TabIndex = 0;

this.textBox1.Text = "";

//

// button1

//

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

this.button1.Name = "button1";

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

this.button1.TabIndex = 1;

this.button1.Text = "button1";

this.button1.Click += new System.EventHandler(this.button1_Click);

//

// Form1

//

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

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

this.Controls.Add(this.button1);

this.Controls.Add(this.textBox1);

this.Name = "Form1";

this.Text = "Form1";

this.ResumeLayout(false);

}

#endregion

/// <summary>

/// The main entry point for the application.

/// </summary>

[STAThread]

static void Main()

{

Application.Run(new Form1());

}

private void button1_Click(object sender, System.EventArgs e)

{

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

Class1 cls1 = new Class1();

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

}

public string ValueForTextBox

{

set

{

this.textBox1.Text = value;


}

get

{

return this.textBox1.Text;

}

}



}

}





file CLASS1.CS:

using System;

namespace simpleapp

{

/// <summary>

/// Summary description for Class1.

/// </summary>

public class Class1

{

public Class1()

{

//

// TODO: Add constructor logic here

//


addstatustoo("part two, I don't work \r\n\r\n");




}

public void addstatustoo(string status)

{

//this doesn't work.

Form1 obj = new Form1();

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

obj.addstatus(status);

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

}

}

}
 
J

Jon Skeet [C# MVP]

JP said:
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.
 
J

JP

Well, if I make it a static method:

public static void addstatus(string status)

{

Form1 obj = new Form1();

obj.textBox1.AppendText(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.
 
J

JP

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

private static System.Windows.Forms.TextBox textBox1;
 
J

Jon Skeet [C# MVP]

JP said:
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.
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Top