pokestop | Find nearby Pokéstops | iOS library

Am i using the wrong layout or my coding are having problems?

Well, yes, a little, but it has more to do with the size of your JFrame, never call setSize() on them, instead call pack(). However if you change it for pack(), the default orientation of the FlowLayout is FlowLayout.HORIZONTAL and thus all your components will go to the right of each other.

Then, you have 2 options:

1. Keep the JFrame's layout as FlowLayout but make two JPanels, one for the image and one for the data, the image might have the default FlowLayout as well but the other one might have GridLayout.

2. Use GridBagLayout for all your components, letting the image JLabel to have a height of 7 cells and all the others a height of 1.

For the following example I will be using the GridBagLayout option, and I recommend you to try to make the multipanel option (option 1).

However, before starting I must mention your other problems in your code:

• You're not following the Java naming conventions: firstWordLowerCaseVariable, firstWordLowerCaseMethods(), FirstWordUpperCaseClasses, ALL_WORDS_UPPER_CASE_CONSTANTS, this will make your code easier to read and understand for you and for us.

• You're not changing the behavior of the JFrame so, there's no need to inherit from it: See Extends JFrame vs. creating it inside the program for more information.

• Related to the above point, it's wise to build your GUI towards JPanels instead of JFrames, see: The Use of Multiple JFrames: Good or Bad Practice? as well, as I think you might end up in similar problems in the future.

• You're not placing your program on the Event Dispatch Thread (EDT), see this related answer to see how to solve this problem.

• You're calling setSize(...) method, and as I said before, it's always better to call pack() method instead, leaving the calculations for the GUI size to the layout manager.

• You're calling setVisible before adding all your components to the GUI, this could lead you to blank or black screens, it should be the last line in your program.

• Related to the above point, I always like to start doing the GUI from inside out and adding the items that way, instead of outside in.

• Why are you using JTextAreas instead of JTextFields for the user information?

Now, the code that follows the above recommendations using the 2nd option I gave you is this: