FoodOrdering | takeaway ordering app includes the basic ordering process

I think your setup is incorrect.

You use quantity to "guess" the index. If quantity becomes 1, you add a new row to the table, if it is not 1, then you try to modify a row in the table. This has a few faults:

• If there are more than one spinner, click each, and finally click any one of them a second time, which row should be modified?
• If you click on a spinner twice, and then click on its - button to decrease and it becomes 1 once again, a new row would be added...

To address the above problems, there are a few options.
The most trivial one would be to implement your own TableModel not based on DefaultTableModel...
Another approach could be adding a column to your existing DefaultTableModel to be used for searching, and making it invisible. This column could be EG the JSpinner instance.

So try something like this:

The JTextfield objects are so narrow because of a very old "bug" in GridBagLayout which causes it to ignore the preferred size of its contents.

There are several possible ways to work-around this bug:

1. Create a class called PreferredGridBagLayout as explained in the link and use that instead of GridBagLayout.
2. Set the minimum size of each of your qtyField instances with qtyField[i].setMinimumSize(qtyField[i].getPreferredSize()).
3. Create subclasses of JTextField which override the method getMinimumSize() to return the same value as getPreferredSize(), or some other reasonable minimum size.

Because this problem is so common when using GridBagLayout, solution #1 is the easiest in the long term.

Afterwards, you'll need to make your tabbedPane object a little wider, or switch to a layout manager in the main panel that automatically determines the size of the tabbed pane.

There are multiple things that could be improved in your code. Creating good layouts in Swing is not easy, and you will need much more work to make a pretty layout. But this will solve your problem with collapsing text fields.

I don't have images to test, there's no valid MCVE in the question, however from what I see in it you could try (in order of preference)

• Using GridBagLayout instead of FlowLayout that you're currently using, this will help placing your images in a grid and align them in it.

• Using GridLayout which will make your images and JTextFields the same size and placing them in a grid of the same size, each component.

• Resizing your images to be all the same size, before getting their getScaledInstance(...)

BTW, this tabbedPane.setBounds(10, 11, 400, 450); suggests that in some point of your code, you're using null layout, you should know that null layout is evil and thus it's use is frowned upon. If you're not using null layout anywhere (we can't know because we don't have that code in the question), then, that line is needed.

Also, you could improve your code by wrapping all the same functionality like this:

As mentioned by @HassanAhmed, this is what happens when inputs are not properly handled; you're seeing the same issue as if someone was using SQL injection against you. The quick fix is to wrap the fields in quotation marks: