There must be some basic concept about classes that I dont understand.

PokerJFrame.java i have:



In the hand class i have an array of player objects called players:



in the player class i have:



Hoping for a helping hand

Sorry the problem was in the JFrame not in the above

add logging and debug in the future.

thank you for the advise

I have a bonus quistion.

Which of the two following ways are the best (fastest execute/best programming practice)

JFrame needs to display the playernames in the hand.

1:


or

2:

players are named after numbers?

use three layers. your UI should call a controller and ask for the name, then your controller (which tracks app state) should look into the player array and ask the player for its name

Yes the players know which seat they are sitting at as does the hand.

Need some advise about the controller which u mention. Sounds like a good idea. But how does the controller keep track of the state of the JFrame.

At this point my hand class keeps track of the hand, poker rules, players, stacks etc.
The JFrame gives actions via "fold" button, "new hand" button etc.

But I need a way to let the JFrame know certain things. for example: When the players are all in. the hand class need to tell the JFrame to turn one card at a time and pause for a sec.
Alternatively this could be done by a controller. but it still needs to tell the JFrame to update the cards icons.

Dont know if I have made it clear what my problem is. But any constructive advise is valued.

This post was edited by tigeranden on Aug 23 2012 03:42pm

most ui frames need to have a reloadUi function or similar, which is what you call when the backend representation has changed and you want the UI to update.

the reloadUi function ought to more or less be the same as the function you call at the very beginning when you load the UI for the first time. of course, you'll only want to reload the pieces that can actually change and not the ones that are static (i.e. menu bars etc)

i think in java, that function is even built into the frame's class - you just need to properly implement or override it.

This post was edited by irimi on Aug 23 2012 03:48pm

Ty Irimi. Ill be looking at my code again tomorrow.

I have a method called drawboard() which takes the state of the board from the hand class and redraws board. But I can't figure out how to call it from the hand class.

Like the idea that the hand class just keeps track of the logical/mathematical side of the game, while other classes takes care of the graphics etc.

your backend code should NEVER call any UI code.

your UI code should be able to keep track of the internal state and refresh whenever is necessary.

a controller doesn't need to keep track of "the state of the JFrame". UI code should be stateless - all it should ever do is reflect the state of the backend.

This post was edited by irimi on Aug 23 2012 04:44pm