Когда JPanel изменяется, элементы старого JPanel остаются позади - PullRequest
Новая
       40

Когда JPanel изменяется, элементы старого JPanel остаются позади

1 голос
/ 11 ноября 2011

Я использую JTabbedPane, и на каждой вкладке есть панель, которая меняется, когда пользователь делает такие вещи, как нажатие кнопок. Проблема, которую я получаю, заключается в том, что элементы предыдущей панели остались позади. Обычно вы не можете видеть их, пока не наведете на них мышь, но иногда вы не можете видеть элементы новой панели, пока не наведете указатель мыши на то место, где они должны быть . Так что довольно часто сначала видно только немного, затем:

при наведении мышки на другие предметы, они становятся видимыми

Затем, по какой-то причине, когда вы нажимаете «множественный выбор», который должен создать четыре новые кнопки, все становится отлично видимым .

У меня есть repaint(); почти столько же, сколько и в любой другой строке, и прежде чем что-то изменить в графическом интерфейсе, сначала я делаю removeAll();, но все возвращается! Какие-либо предложения? Код этой панели приведен ниже, если это может помочь ... 

package com.GC01.gui;

import java.awt.Color;

public class PracticeQuizPanel extends JPanel implements MouseListener {
    /**
     * This panel will allow the user to practice quiz questions.
     */
private static final long serialVersionUID = 1L;

private User user = new User("misha");

boolean isMultipleChoice=false;
boolean usedClue=false;
boolean isStarted=false;

Calendar calendar = Calendar.getInstance();
AnswerAnalysis aA = new AnswerAnalysis();   
private Quiz qz = new Quiz( Quiz.getQuestionsFromDisk(), 4, TabbedQuiz.getUser().getNumLogins() );

private JTextArea questionArea;
private JTextArea clueArea;
private JTextArea answerArea;

private JButton clueButton;
private JButton multiButton;
private JButton answerButton;
private JButton startButton; 

private int index=0;
private Calendar startCalendar, endCalendar;

private JPanel backPanel;

public PracticeQuizPanel(){
    add(new StartButtonPanel());
}

PracticeQuizPanel(int index) {
    createVisualElements(index);
}

public void offerAnswer(){
    endCalendar = Calendar.getInstance();
    aA.setTimeSpent((int) (endCalendar.getTimeInMillis()-startCalendar.getTimeInMillis()));
    aA.setRight(answerArea.getText());
    JOptionPane.showMessageDialog(null, aA.toString());
    answerArea.setEditable(false);
    qz.setAnswersAnalysis(index, aA);
    index++;
    removeAll(); 
    if( index<qz.getLength() ) createVisualElements(index);
    else {
        removeAll();
        JOptionPane.showMessageDialog(null, qz.toFriendlyString());
        addQuizResultsToUserProgress();
        JOptionPane.showMessageDialog(null, qz.toFriendlyString());
        UserProgress uP = new UserProgress(user);
        System.out.println(uP.toString());
    }
    repaint();
    startCalendar = Calendar.getInstance();
    //JOptionPane.showMessageDialog(null, isStarted);
}

public void addQuizResultsToUserProgress(){
    UserProgress userProgress = new UserProgress(user);
    ArrayList<AnswerAnalysis> asA = userProgress.getAnswersAnalysis();
    for (int i=0; i<qz.getLength(); i++){
        asA.add( qz.getAnswersAnalysis()[i]);
    }
    userProgress.setAnswersAnalysis(asA);
    userProgress.saveProgress();
}



/**
 * This method creates/recreates all the text boxes, buttons etc. without resetting the quiz and
 * the objects in memory.
 */
private void createVisualElements(int index){
    if (TabbedQuiz.getUser().getNumLogins()<0) 
        JOptionPane.showMessageDialog(null, "There was an error. You may have done this quiz before.");
    removeAll();
    repaint();
    startCalendar = Calendar.getInstance();
    this.index=index;
    setBackground(new Color(112, 128, 144)); 
    setBounds(0,0,728,380);
    setLayout(null);

    questionArea = new JTextArea();
    questionArea.setFont(TabbedQuiz.getDefaultFont().deriveFont(20));//new Font("Courier New", 0, 20));
    questionArea.setEditable(false);
    questionArea.setLineWrap(true);
    questionArea.setWrapStyleWord(true);
    questionArea.setBounds(295, 11, 423, 74);
    add(questionArea);
    //int index=0;
    Question q = qz.getQuestions().get(index);
    aA = new AnswerAnalysis(q.getQuestionID());
    questionArea.setText(q.getQuestionText() );

    clueArea = new JTextArea();
    clueArea.setFont(TabbedQuiz.getDefaultFont().deriveFont(20));
    clueArea.setEditable(false);
    clueArea.setLineWrap(true);
    clueArea.setWrapStyleWord(true);
    clueArea.setBounds(295, 104, 423, 55);
    add(clueArea);

    JLabel lblQuestion = new JLabel("QUESTION:");
    lblQuestion.setFont(TabbedQuiz.getDefaultFont().deriveFont(40));
    lblQuestion.setBounds(43, 11, 216, 61);
    add(lblQuestion);

    answerArea = new JTextArea();//index+"");
    answerArea.setFont(TabbedQuiz.getDefaultFont().deriveFont(20));
    answerArea.setLineWrap(true);
    answerArea.setWrapStyleWord(true);
    answerArea.setBounds(295, 301, 423, 50);
    answerArea.addKeyListener(new KeyAdapter() {
        @Override
        public void keyPressed(KeyEvent e) {
            if(e.getKeyCode() == KeyEvent.VK_ENTER) offerAnswer();
        }
    });

    add(answerArea);
    answerArea.setFocusable(true);
    answerArea.requestFocusInWindow();

    clueButton = new JButton("CLUE?");
    clueButton.setFont(TabbedQuiz.getDefaultFont().deriveFont(40));
    clueButton.addMouseListener(this);
    clueButton.setBounds(15, 104, 244, 55);
    add(clueButton);

    multiButton = new JButton("MULTIPLE CHOICE?");
    multiButton.setFont(TabbedQuiz.getDefaultFont().deriveFont(20));
    multiButton.addMouseListener(this);
    multiButton.setBounds(15, 195, 244, 55);
    add(multiButton);

    answerButton = new JButton("ANSWER!");
    answerButton.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent arg0) {
        }
    });
    answerButton.setFont(TabbedQuiz.getDefaultFont().deriveFont(40));
    answerButton.setBounds(15, 301, 244, 55);
    answerButton.addMouseListener(this);
    add(answerButton);

    backPanel = new JPanel();
    backPanel.setBounds(0, 0, 728, 380);
    //add(backPanel);

    this.setVisible(true);
    repaint();
}

@Override
public void mouseClicked(MouseEvent e) {
    if      ( e.getSource().equals(startButton)  ) {
        remove(startButton); repaint(); isStarted=true;
        startCalendar = Calendar.getInstance();
    }
    else if ( e.getSource().equals(answerButton) ) offerAnswer();
    else if ( e.getSource().equals(clueButton )  ) {
        clueArea.setText(clueArea.getText() + qz.getQuestions().get(index).getClueText() + "\n");
        aA.setUsedClue(true);
        //JOptionPane.showMessageDialog(null, "hi");
        }
    else if ( e.getSource().equals(multiButton)  ) {            
        String[] answerOptions = qz.getQuestions().get(index).getAnswerOptions(3);
        JButton[] optionsButtons = new JButton[4];
        for(int j=0;j<4;j++){
            optionsButtons[j]=new JButton(answerOptions[j]);
            if(optionsButtons[j].getText().length()>13) 
                optionsButtons[j].setFont(TabbedQuiz.getDefaultFont().deriveFont(10));
            else optionsButtons[j].setFont(TabbedQuiz.getDefaultFont().deriveFont(15));
            if(j<2) optionsButtons[j].setBounds(295+211*j       , 170, 211, 55);
            else    optionsButtons[j].setBounds(295+211*(j-2)   , 226, 211, 55);
            optionsButtons[j].addMouseListener(this);
            optionsButtons[j].setName("optionsButton"+"["+j+"]");
            add(optionsButtons[j]);
            repaint();
        }
        aA.setMultipleChoice(true);
    }
    else if ( ( (JButton) e.getSource() ).getName().startsWith("optionsButton") ) {
        String answerOffered = ( (JButton) e.getSource() ).getText();
        answerArea.setText(answerOffered);
        offerAnswer();
    }
}

@Override
public void mouseEntered(MouseEvent e) {
    if(e.getSource()!=startButton && e.getSource().getClass().equals( answerButton.getClass()) ){
        ( (JButton) e.getSource() ).setBackground(Color.green);
    }if(index>0 && e.getSource()==startButton) remove(startButton);
}

@Override
public void mouseExited(MouseEvent e) {
    if(e.getSource()!=startButton && e.getSource().getClass().equals( answerButton.getClass()) ){
        ( (JButton) e.getSource() ).setBackground(UIManager.getColor("control"));
    }
}

@Override
public void mousePressed(MouseEvent e) {
}

@Override
public void mouseReleased(MouseEvent e) {
}
}

Ответы [ 3 ]

3 голосов
/ 11 ноября 2011

и на каждой вкладке есть панель, которая меняется, когда пользователь делает такие вещи, как кнопки нажатия.

и перед тем, как что-либо изменить в графическом интерфейсе, я делаю removeAll ();

Всякий раз, когда я вижу такие комментарии, мне кажется, что вы должны использовать Макет карты , чтобы вам не приходилось беспокоиться обо всех этих проблемах.

2 голосов
/ 11 ноября 2011

Вам необходимо вызвать revalidate после удаления или добавления компонентов в контейнер при использовании менеджеров компоновки.Затем после этого вы звоните repaint.

Вы опубликовали много кода, и большая часть его полностью не связана с рассматриваемой проблемой, но, с другой стороны, опубликованный код является неполным, не компилируется и, следовательно, не является кодомчто мы можем проверить.В следующий раз, пожалуйста, постарайтесь не публиковать много кода, который не имеет отношения к проблеме.Попытайтесь изолировать проблему, постоянно вырезая код, а затем, если он все еще застрял, опубликуйте минимальную совместимую программу, которую мы можем изучать, тестировать, запускать и модифицировать сами, sscce .

Я просмотрел некоторые из ваших кодов GUI (ради всего святого, не все!), И там есть большая проблема, когда вы наступаете на рисование Swing, такое как этот код из ProgressPanel.java:

   public void paintComponent(Graphics g) {
      super.paintComponent(g);
      Graphics2D g2d = (Graphics2D) g;
      createAndShowGUI(g2d);
      // drawDifficultyChart(g2d, fontComboBox.getSelectedIndex());
   }

   private void createAndShowGUI(Graphics2D g2d) {
      JButton showChartButton = new JButton("Show new chart!");
      user = TabbedQuiz.getUser();
      showChartButton.addActionListener(new ActionListener() {
         public void actionPerformed(ActionEvent e) {
            try {
               if (categoryComboBox.getSelectedIndex() == 0
                     && difficultyComboBox.getSelectedIndex() != 0) {
                  drawDifficultyChart((Graphics2D) getGraphics(),
                        difficultyComboBox.getSelectedIndex());
               } else if (difficultyComboBox.getSelectedIndex() == 0
                     && categoryComboBox.getSelectedIndex() != 0) {
                  drawCategoryChart((Graphics2D) getGraphics(),
                        (String) categoryComboBox.getSelectedItem());
               } else
                  drawGeneralChart((Graphics2D) getGraphics(),
                        (String) categoryComboBox.getSelectedItem(),
                        difficultyComboBox.getSelectedIndex());
            } catch (NullPointerException e1) {
               JOptionPane.showMessageDialog(null, "Sign in first.");
            }
         }
      });
      showChartButton.setBounds(10, 90, 96, 26);
      showChartButton.setFont(font);
      add(showChartButton);
   }

Проблема заключается в том, что вы создаете и добавляете компоненты в контейнер из метода paint / paintComponent, что вы никогда не должны делать, и что может испортить графику вашей программы.

0 голосов
/ 14 ноября 2011

Для кого-то еще с этой проблемой: я в конечном счете решил проблему. setVisible(false); setVisible(true);. Положите это ВЕЗДЕ.

...