Link to home
Start Free TrialLog in
Avatar of ctjoumas
ctjoumas

asked on

Login Question

I have an application that I need a user to first log into an application before actually launching the application.  So, I want to provide a login window, which provides the ability to login to the server, or create a new account to log into the server.  Both the login and create account windows are separate windows.

Previously, I had both windows implemented as JDialogs - so, this way, I coudl have a "while !loggedIn" loop, since execution would stop until the dialogs returned.  But, I need to change these into JFrames now.  The problem is, of course, execution no longer waits for the modal JDialog to return, so I would get an infinite loop.  The way I got around this was to have two separate methods in my main application class that would open each different window whenever needed.  So, my final code in the constructor was:

doLogin();

// launch application - at this point we are gauranteed to have been connected to the server

my doLogin() would simple call the method to open the login window - the login window has a button that allows the user to close the login window and open the create account window.  If the user cancels creating an account, the login window will be reopened.  Of course, the problem here is that the "doLogin();" call would be passed by since it is not waiting for anything and the application will begin launching as the login screen is open.

My ugly hack around this for now is to set a flag in the main class - called loggedIn - that is initially false.  If the login or create account windows logs the user in, the flag is updated and the application can then be launched.  So, my code looks like:

doLogin();

// here comes the nasty part!
while ( !loggedIn ) { }

// launch application

What is a better way of getting around this?  Sorry for the long explanation!!!
Avatar of expertmb
expertmb

>>If the user cancels creating an account, the login window will be reopened.  Of course, the problem here is that the "doLogin();" call would be passed by since it is not waiting for anything and the application will begin launching as the login screen is open.

when user cancels the account creation then do System.exit(0);

Avatar of CEHJ
Personally i'd be more inclined to use the same window with say a CardLayout. One you've logged in, it moves to the next 'screen'
Avatar of ctjoumas

ASKER

I think that makes sense for the login and create account windows, but I'm not so sure about the main application.  This is a chat application, so using a CardLayout doesn't seem that it would work too well - from what I gather, all panels in a CardLayout will be the same size.  Once the user logs in, I am displaying a separate "tree" window...much like AIM.
ASKER CERTIFIED SOLUTION
Avatar of cjjclifford
cjjclifford

Link to home
membership
This solution is only available to members.
To access this solution, you must be a member of Experts Exchange.
Start Free Trial
Oh, and another reason I don't want to keep the tree visible is because the things being displayed on the tree are retrieved from the server.  So, I cannot setup the tree until a connection to the server has been made.

I somehow need to wait for the logging in process to finish before proceeding (other than using the ugly hack I have).
cjjclifford:  I will take a look at that.  Sorry, I was in the middle of posting then got tied up for a few hours and finished posting before checking if anyone had responded.  I'll let you know what happens :)
cjjclifford: Hmm..that works.  I'm not quite sure what is really going on here though, can you explain how that works?

the "wait()" method basically suspends the current thread, until the "notify()" method is called on the object. When the "notify()" method is called, the suspended thread is woken up and the "wait()" call returns.
The "synchronized(obj)" code is used to become "the owner of this object's monitor" - see JavaDoc for "java.lang.Object" "wait" and "notify" for more details - not that if there is a possibility that multiple calls to wait() can be made for a single lock object, the "notifyAll()" method should be used instead (notify() only awakens a single wait() while notifyAll() awakens all wait() methods... generally for safety, notifyAll() can be used...

Hope this is description is helpful - its late Friday evening where I am, and I'm heading off for the weekend!
Thanks a lot cjjclifford.  Your descrition is helpful...I read up on java.lang.Object wait and notify stuff earlier.  It made sense to me.  Your clarification of what synchronized does (becoming the owner of the object's monitor) explained the rest of it.  Everything is working...Thanks!

CEHJ - thanks for the tip about CardLayout.  I am using that for my login and create account screens and it is much simpler and more clean.  I haven't used that before and I'm glad you brought that to my attention :)