How Do I: Get rid of redundant code? (Put in a userControl?) Part 2:

Sam:
 
I tried the userControl, but could not get it to work. I am sure it was something that I was missing.
 
I am going to ge this working this way. Then I will re-try the UserControl again. I finally have moved the code into the class, but now I have 1 bug left...

Note that in How Do I: Put this routine in a class? I said "Probably the only reason that GetNextControl does not work in another class is that you need to pass the Form." So if you had read that, then you would not have needed to post this question. I also said "When you learn the fundamentals of C#, it will be as clear as air that that is needed and and how to do it.". When you learn the fundamentals of C#, you will instantly know how to answer questions such as this yourself.
 
Also, you said you were going to use a UserControl. When you do, you will not have this problem. Yes, deriving from a form is a good solution, but I did not say much about that since you sais you are using a UserControl.
 
Finally, C++ programmers would not use an #include in the manner you show. C programmers would use a macro, which is close to what you are showing. C++ programmers are less likely to use macros, especially the designer of the C++ language. Macros don't even exist in C#; the #define statement is used to define macros and for C# the define statement is extremely limited in it's use in C#. Good C++ prgrammers and all C# programmers would do something different, such as creating a class or deriving from a class.

The only improvement I can see on Jaish's suggestion for now is to add an extra parameter ctrlType to use in theGetType().Name part.

public ClassUser(Form frm, string DOIT, string ctrlType)
{

then

if (TheControl.GetType().Name == "TextBox")

if (TheControl.GetType().Name == ctrlType)

then the function i snot restricted to TextBoxes, It should work...

Jaish:
 
Interesting... I will read up on it. Sounds a lot cleaner that what I was thinking of doing.
 
 

Hi,
Forgot to suggest a better design like below.
 
//Make custom base class inherited from builtin "Form". All pages can derive from this base class.
//Now you can directly call GetNextControl(), as now you have access to a "Form" class.
public class ClassUser:Form
{
}
 
//Page inherited from custom base class defined above.
class FormICEPack:ClassUser
{
}

Jaish:
 
I did a quick test and I think it will work. Thank you. Now I have something to play with to make it work.
 
Thanks

There is no way to do it like C+ with a #include ???
 
Example: In the main program you add:
        #include "MenuExit.txt";
 
Then the:       MenuExit.txt
 
Will have something like this:
//        ShowMessage("Action Exit - (Generic)");
        ShowMessage("You really want to close... Data verification...");
        Close();
 
So, whereever I put: #include "MenuExit.txt"; the above code will be included.
 

Jaish:
 
I tried that before and I retied it again. Its not working for me. I get the error on the lines with the: GetNextControl.
 
 
This is the code of the class. I took a lot out of it to make it simple and then work from there.
 
namespace ICEPack
{
#region Notes
//
// To Do:
// 1) Remove static. - DONE
// 2) Make Private.
// 3)
// 4)
// 5)
#endregion
//
public class ClassUser
{
//#region Variables
//
//
public ClassUser(string DOIT)
{
MessageBox.Show("IN: ClassUser: " + DOIT);
Control TheControl = GetNextControl(null, true);
while (TheControl != null)
{
if (TheControl.GetType().Name == "TextBox")
{
TheControl.Enter += new EventHandler(Object_Enter);
}
TheControl = GetNextControl(TheControl, true);
}
ClassUser ClassUser = new ClassUser("Hey..." + DOIT);
}

public void Object_Enter(object sender, EventArgs e)
{
((TextBox)sender).BackColor = Color.Red;
}
}
}
 
 
In my main program, this is how I am trying to call it:
 
public FormICEPack()
{
InitializeComponent();
//Control TheControl = GetNextControl(null, true);
//while (TheControl != null)
//{
// if (TheControl.GetType().Name == "TextBox")
// {
// TheControl.Enter += new EventHandler(Object_Enter);
// //TheControl.Leave += new EventHandler(Object_Leave);
// //TheControl.Click += new EventHandler(Object_Click);
// //TheControl.MouseHover += new EventHandler(textBox_MouseHover);
// //TheControl.MouseLeave += new EventHandler(textBox_MouseLeave);
// }
// TheControl = GetNextControl(TheControl, true);
//}
ClassUser ClassUser = new ClassUser("Hey...");
}

Hi,
I didn't tried with smart device. But in WebForms normally creating a BasePage and will define common things inside it, like below
class WebForm1: BasePage
{
   //Page specific things + can call common items inside BasePage
}
class WebForm2 : BasePage
{
   //Page specific things + can call common items inside BasePage
}
 
class BasePage :Page
{
//Common things
}