C# Forms Help

[mobile_fan_2k5]

After the user inputs idnum and password. If thats correct the loginwindow is suppose to minimize and the mainwindow is suppose to come up. The main window comes up but the login window wont go away.

login window



main window


my problem

if (userName_login_window.Equals("1") && pass_login_window.Equals("2"))
          {
             
              MainWindow window = new MainWindow();
              window.ShowDialog();//Brings up the main window
              Application.Exit();//suppose to close login window but doesnt work.
              
              
              
          }

HELP Greatly needed.

[Skele Drew]

Did you try setting the login window's Visible property to False?

[mobile_fan_2k5]

I placed the following code in the if statement after the main windows is supposed to come up, but it still didnt work.

LoginWindow login = new LoginWindow();
login.Visible = false;

Remember the login window should be initially visible and running, then after the user inputs idnum and password, then the main window comes up. After the main comes up the login window would be sitting there taking up space. I want to hide. Since i just found out it cant be close because its the default form.

[Skele Drew]

Ah yes. I'm definitely seeing your foul up now. If the login window is your main form, then when you close it, the app closes (don't see why hiding it doesn't work though). You put the main window as a child of the login window, so the Application.Exit(); statement won't be executed until you close the main window. It should be the other way around, then use the OnLoad event (or similar) to load that login window.

EDIT:

LoginWindow login = new LoginWindow();
login.Visible = false;

This won't work the way how you expected it. For one, you just created the object, you didn't even ShowDialog() it in the first place, yet you're hiding it. Also, the Visible = False; isn't needed once you validate the password and the login window closes.

[mobile_fan_2k5]

the answer was staring in front of me all this time, its so simply.

if (userName_login_window.Equals("1") && pass_login_window.Equals("2"))
{
MainWindow window = new MainWindow();
this.Hide();
window.ShowDialog();
}

the this.Hide(); would be executed before the window.showDialog(); and just like that what seemed so hard just flow.

[Skele Drew]

That will work. But I still don't get why you're still using the login as the main window and creating the main window as a child of the login window. Seems like a minor resource mismanagement since you're only using the login window once (it's still taking up memory when it's hidden). This style can also lead to more expansion problems later...

[mobile_fan_2k5]

I figured out that much, but i making the app first then fixing problems later. Kinda like microsoft.

[seanbee20]

Why are u hiding the login window? That's a waste of resource for a form that isn't gonna be used anymore.

Change your code to this:

if (userName_login_window.Equals("1") && pass_login_window.Equals("2"))
{
MainWindow window = new MainWindow();
window.Show(); // no need to use ShowDialog()
this.dispose(); //results in the CLR executing the necessary garbage collection to reclaim resources
}

[mobile_fan_2k5]

@seanbee20, thanks for the advice. I'll have to take resource management into consideration, not everyone have 4gb of ram to pour kbs or mbs of space to run apps.