Consiglio su struttura JInternalFrame duplicato

[fermat]

allora, ho cercato si seguire la terza soluzione.
nella super-classe ho impostato il layout di base, e messo alcuni metodi generici come protected.
ad esempio:

    protected void setTextButtonAdd(String text) {
        btnAdd.setText(text);
    }

    protected void setTextButtonUp(String text) {
        btnUp.setText(text);
    }

    protected void setTextButtonDel(String text) {
        btnDel.setText(text);
    }

    protected void riempiCombo(ArrayList<? extends Object> list) {
        comboUp.removeAllItems();
        comboUp.addItem("*");
        list.stream().forEach((strObj) -> {
            comboUp.addItem(strObj);
        });
    }

il metodo riempiCombo, riempiraà la jcombobox o con oggetti Author o con oggetti Editor (a seconda di quello che mi serve).
poi nella sotto-classe:

public class FrameAuthor extends FrameAEBase {

    private ReadJson jsonRead = new ReadJson();
    private ArrayList<? extends Object> list = null;

    public FrameAuthor() {
        super();
        super.setTextButtonAdd("Aggiungi autore");
        super.setTextButtonUp("Modifica autore");
        super.setTextButtonDel("Cancella autore");
        try {
            list = jsonRead.getAllAuthors();
        } catch (IOException ex) {
            JOptionPane.showMessageDialog(null, ex.getMessage());
        }
        super.riempiCombo(list);
    }
}

a parte il funzionamento (l'ho testato e funziona), può andar bene??

[andbin]

a parte il funzionamento (l'ho testato e funziona), può andar bene??

Grosso modo sì. Ma ci sono alcune cose dubbie.

Innanzitutto cosa fa ReadJson? Non vedo argomenti al suo costruttore e neanche al getAllAuthors. Quindi legge un JSON da cosa/dove e con quale API? Probabilmente è tutto "cablato" nel ReadJson.

Poi non mi piace molto quel ArrayList<? extends Object>. Dovresti cercare di parametrizzare in modo più accurato questa lista. Anche perché in un ArrayList parametrizzato così non puoi inserire nulla (a parte un null letterale). E questo quindi dipende da cosa fa getAllAuthors.