Finestra bianca all'apertura, se ridimensiono appare tutto

**ixol** · 21-04-2016, 17:14

Le uniche differenze dal tuo codice sono che ho creato una nuova classe per il main (ho letto in giro su internet che conviene farlo) e che non uso il metodo pack() perchè ho un gridlayout e avrei una finestra priva di dimensioni.

In ogni caso posto il codice:

codice:

import java.awt.Color;import java.awt.Container;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.BorderFactory;
import javax.swing.ButtonGroup;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JRadioButton;
import javax.swing.JTextField;
import java.awt.Dimension;
import java.awt.FlowLayout;
import javax.swing.border.LineBorder;
import javax.swing.SwingConstants;


public class Finestra extends JFrame{
    Container mainContainer;
    JTextField mazzo, carte, percentuale;
    JRadioButton primo, secondo;


    public Finestra() {
        JFrame window = new JFrame("Calcolo Probabilità");
        window.setSize(347, 228);
        window.setResizable(false);
        window.setLocationRelativeTo(null);
        window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);


        mainContainer = window.getContentPane();
        mainContainer.setLayout(new GridLayout(5, 2));
        
        JPanel panelMazzo1 = new JPanel();
        mainContainer.add(panelMazzo1);
        JLabel labelMazzo = new JLabel("Numero di carte nel mazzo:");
        panelMazzo1.add(labelMazzo);
        
        JPanel panelMazzo2 = new JPanel();
        mainContainer.add(panelMazzo2);
        mazzo = new JTextField();
        mazzo.setPreferredSize(new Dimension(6, 24));
        mazzo.setColumns(5);
        mazzo.setBorder(BorderFactory.createLineBorder(Color.BLACK));
        panelMazzo2.add(mazzo);
        
        JPanel panelCarte1 = new JPanel();
        mainContainer.add(panelCarte1);
        JLabel labelCarte = new JLabel("<html>Numero di carte di cui<br>calcolare la probabilità:</html>");
        panelCarte1.add(labelCarte);
        
        JPanel panelCarte2 = new JPanel();
        mainContainer.add(panelCarte2);
        carte = new JTextField();
        carte.setPreferredSize(new Dimension(6, 24));
        carte.setColumns(5);
        carte.setBorder(BorderFactory.createLineBorder(Color.BLACK));
        panelCarte2.add(carte);
        
        JPanel panelPartenza1 = new JPanel();
        mainContainer.add(panelPartenza1);
        JLabel labelPartenza = new JLabel("Partenza:");
        panelPartenza1.add(labelPartenza);
        
        JPanel panelPartenza2 = new JPanel();
        mainContainer.add(panelPartenza2);
        ButtonGroup group = new ButtonGroup();
        primo = new JRadioButton("Primo", true);
        secondo = new JRadioButton("Secondo");
        group.add(primo);
        group.add(secondo);
        panelPartenza2.add(primo);
        panelPartenza2.add(secondo);
        
        JPanel panelBottoni1 = new JPanel();
        mainContainer.add(panelBottoni1);
        JButton calcola = new JButton("Calcola");
        calcola.addActionListener(new clickBottone());
        panelBottoni1.add(calcola);
        
        JPanel panelBottoni2 = new JPanel();
        mainContainer.add(panelBottoni2);
        JButton reset = new JButton("Reset");
        panelBottoni2.add(reset);
        reset.addActionListener(new resetBottone());
        
        JPanel panelPercentuale1 = new JPanel();
        mainContainer.add(panelPercentuale1);
        panelPercentuale1.setLayout(new FlowLayout(FlowLayout.CENTER, 5, 5));
        JLabel labelPercentuale = new JLabel("Percentuale:");
        panelPercentuale1.add(labelPercentuale);
        
        JPanel panelPercentuale2 = new JPanel();
        mainContainer.add(panelPercentuale2);
        percentuale = new JTextField();
        percentuale.setHorizontalAlignment(SwingConstants.CENTER);
        percentuale.setPreferredSize(new Dimension(6, 24));
        percentuale.setEditable(false);
        percentuale.setColumns(4);
        panelPercentuale2.add(percentuale);
        percentuale.setBorder(new LineBorder(new Color(0, 0, 0), 0));
        
        //window.setVisible(true);
    }


    private class clickBottone implements ActionListener {
        public void actionPerformed(ActionEvent e){
            int p=0,n,c;
            float val=1;
            try{
                n=Integer.valueOf(mazzo.getText() );
                c=Integer.valueOf(carte.getText() );
                if(n<=60 && c<=n && c>=0){
                    if(secondo.isSelected()) p=1;
                    for(int i=0;i<5+p;i++){
                        val=val*(n-c)/n;
                        n--;
                    }
                    val=Math.round((1-val)*1000);
                    val/=10;
                    String str=val+"%";
                    percentuale.setText(str);
                    percentuale.setBackground(Color.red);
                }else{
                    JOptionPane.showMessageDialog(null, "Valori inseriti errati");
                }
            }catch(Exception exc){
                JOptionPane.showMessageDialog(null, "Valori inseriti errati");        
            }
            
        }
    }


    private class resetBottone implements ActionListener {
        public void actionPerformed(ActionEvent e) {
            mazzo.setText("");
            carte.setText("");
            primo.setSelected(true);
            percentuale.setBackground(Color.white);
            percentuale.setText("");
        }
    }


}

codice:


public class Principale {
public static void main(String[] args) {
    Finestra finestra1 = new Finestra();
    finestra1.setVisible(true);
}
}

Già che ci sei se noti qualcosa che faccio che però conviene fare diversamente fammelo notare, vorrei imparare anche lo "stile di scrittura" di un codice.

**Ansharja** · 21-04-2016, 17:45

Originariamente inviata da ixol

Le uniche differenze dal tuo codice sono che ho creato una nuova classe per il main (ho letto in giro su internet che conviene farlo) e che non uso il metodo pack() perchè ho un gridlayout e avrei una finestra priva di dimensioni.

La classe per il Main va benino, se guardi l'esempio che ho fatto io l'invocazione del frame viene fatta da SwingUtilities.invokeLater(), il perché lo puoi trovare facilmente online, riguarda quello che accennavo prima sull'EDT (in realtà in questa parte sono anch'io abbastanza alle prime armi).

Invece il metodo pack() devi usarlo, altrimenti è chiaro che il tuo frame non avrà dimensioni.Puoi anche specificarle tu manualmente, richiamando sul JFrame il metodo setSize(int w,int h), ma è molto meglio lasciare che il sia il frame stesso a calcolare le proprie dimensioni.

Il motivo per cui tu non vedi nulla è un errore abbastanza semplice: hai esteso correttamente JFrame nella classe Finestra, quindi l'oggetto costruito dal costruttore è effettivamente un JFrame, che poi rendi visibile dalla classe principale.

Ma all'interno del costruttore invece che aggiungere tutti i pannelli e settare le proprietà direttamente sull'oggetto che stai istanziando, aggiungi tutto a una variabile window, che poi non rendi mai visibile !

Come dicevo non ha alcun senso mantenere una variabile di tipo JFrame all'interno della tua classe, è proprio l'oggetto Finestra il tuo frame! Quindi ad esempio le righe:

codice:

JFrame window = new JFrame("Calcolo Probabilità Yugioh");
window.setSize(347, 228);
window.setResizable(false);
window.setLocationRelativeTo(null);
window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

vanno modificate in :

codice:

super("Calcolo Probabilità Yugioh"); // stai richiamando il costruttore della superclasse JFrame direttamente sul tuo oggetto !!
setSize(347, 228);
setResizable(false);
setLocationRelativeTo(null);
setDefaultCloseOperation(EXIT_ON_CLOSE);

Questa parte è cruciale e senza comprendere questo non vai molto avanti.Nella prima riga, come scritto nel commento, richiami il costruttore di JFrame, e da quel momento tutte le caratteristiche vengono applicate direttamente alla tua Finestra (con i metodi setSize(),setResizable() ecc.).

Quindi la tua variabile window non serve assolutamente a niente, va tutto applicato all'oggetto che stai costruendo.

Vedrai che se modifichi questo aspetto e inserisci come ultima riga nel costruttore il metodo pack(), tutto quello che hai creato e aggiunto all'oggetto viene correttamente visualizzato.

Altri consigli: i metodi pack() e setLocationRelativeTo(null) vanno richiamati dopo aver costruito tutto il frame.
Il motivo è semplice: pack() richiude il contenitore con tutto quello che è inserito, mentre setLocationRelativeTo(null) è un modo per centrare il frame nello schermo, e sarebbe meglio farlo quando esso è già correttamente dimensionato.

Per il resto vedo che usi molti (troppi a mio parere) pannelli per un'interfaccia abbastanza semplice, ma qui si tratta di usare bene i layout e il design non è un aspetto così immediato e facile da padroneggiare.
Per ora modifica quelle cose, al massimo poi si riparlerà dello "stile di scrittura", tieni comunque conto che ognuno ha il proprio stile e io non sono certo un professionista, tanto per dirne una comunque io odio a morte il GridLayout e non lo utilizzo mai

[EDIT]: accidenti battuto dal veloce schumy di poco , comunque non abbiamo scritto cose così diverse

Discussione: Finestra bianca all'apertura, se ridimensiono appare tutto

Strumenti discussione

Ricerca discussione

Visualizza

Hybrid View

Permessi di invio