Mon ScrollPanel fonctionne à l'envers

**petole** · 22/09/2010, 13h53

Bonjour, bonjour,

Je suis entrain de faire une application pour mes cours et j'ai un petit problème.
J'ai fait une classe "Help" qui ouvre un fichier *.txt dans un JTextField et j'ai un scrollpanel.

Mon problème:
quand j'ouvre ma page, il ouvre depuis le bas (mon scroll vertical est tout en bas).

Je vous mets mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
 
public class Help extends JFrame {
 
 
	private static final long serialVersionUID = 1L;
	private JPanel helpPanel = new JPanel();
	private JTextArea textCommandes = new JTextArea();
	private String filename = new String("commandes.txt");
	private JScrollPane scrollPane = new JScrollPane(helpPanel);
	private JMenuBar menu = new JMenuBar();
	private JMenu menu1 = new JMenu("File");
	private JMenuItem menuQuit = new JMenuItem("Quit");
 
 
	/**
         * This is the default constructor
         */
	public Help() {
 
		  setTitle("Help");
		  setSize(600, 600);
		  setDefaultCloseOperation(DISPOSE_ON_CLOSE);
		  helpPanel.setLayout(null);
 
		  menuQuit.addActionListener(new MenuQuitListener());
 
		  try {
			commandeText();
		} catch (IOException e) {
			e.printStackTrace();
		}
 
		menu.add(menu1);
		menu1.add(menuQuit);
 
 
 
		helpPanel.setBounds(0, 0, 600, 1700);
 
		textCommandes.setBounds(0, 20, 600, 1700);
		textCommandes.setEditable(false);
		helpPanel.add(textCommandes);
 
		helpPanel.setPreferredSize(new Dimension(600, 1700));
 
		helpPanel.add(menu);
		menu.setBounds(0, 0, 600, 20);
 
		add(scrollPane);
 
	}
 
	class MenuQuitListener implements ActionListener{
 
		public void actionPerformed(ActionEvent e){
			quit(e);
		}
	}
 
	private void quit(ActionEvent e){
		dispose();
	}
 
	public void commandeText() throws IOException{
 
		  try {
		         File file = new File(filename);
		         BufferedReader br = new BufferedReader(new FileReader(file));
		         String s = ""; 
		         int c=0;
 
		         while((c=br.read())!=-1){
		        	 s+=(char)c;
		         }
		         br.close(); 
 
		         textCommandes.setText(s);
 
 
		      } catch (Exception e) {
 
		         JOptionPane.showMessageDialog(this,e.getMessage(),
		         "File error",JOptionPane.ERROR_MESSAGE);}
 
	}
}

Je vous remercie d'avance!

**wax78** · 22/09/2010, 14h10

Tu as fait de grosse erreur dans ton interface je pense. (Menu a l'interieur du scrollpane etc...).

Voici une correction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
package Main;
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
 
public class Help extends JFrame {
 
 
	private static final long serialVersionUID = 1L;
//	private JPanel helpPanel = new JPanel();
	private JTextArea textCommandes = new JTextArea();
	private String filename = new String("rapports\\index.html");
	private JScrollPane scrollPane = new JScrollPane();
	private JMenuBar menu = new JMenuBar();
	private JMenu menu1 = new JMenu("File");
	private JMenuItem menuQuit = new JMenuItem("Quit");
 
 
	/**
         * This is the default constructor
         */
	public Help() {
 
		  setTitle("Help");
		  setSize(600, 600);
		  setDefaultCloseOperation(DISPOSE_ON_CLOSE);
		  menuQuit.addActionListener(new MenuQuitListener());
 
		  try {
			commandeText();
		} catch (IOException e) {
			e.printStackTrace();
		}
 
		menu.add(menu1);
		menu1.add(menuQuit);
		textCommandes.setAutoscrolls(false);
		textCommandes.setEditable(false);
		scrollPane.setViewportView(textCommandes);
		this.setJMenuBar(menu);
		this.add(scrollPane, BorderLayout.CENTER);
 
	}
 
	class MenuQuitListener implements ActionListener{
 
		public void actionPerformed(ActionEvent e){
			quit(e);
		}
	}
 
	private void quit(ActionEvent e){
		dispose();
	}
 
	public void commandeText() throws IOException{
 
		  try {
		         File file = new File(filename);
		         BufferedReader br = new BufferedReader(new FileReader(file));
		         String s = ""; 
		         int c=0;
 
		         while((c=br.read())!=-1){
		        	 s+=(char)c;
		         }
		         br.close(); 
 
		         textCommandes.setText(s);
		//         setCarretPosition(s.length() ); // A la fin;
		         textCommandes.setCaretPosition(0 ); // Au debut;
 
		      } catch (Exception e) {
 
		         JOptionPane.showMessageDialog(this,e.getMessage(),
		         "File error",JOptionPane.ERROR_MESSAGE);}
 
	}
}

Ensuite pour le probleme du positionnement il suffit de :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

textCommandes.setCaretPosition(0 ); // Au debut;

**petole** · 22/09/2010, 14h15

Ah ouais merci!
J'ai changé.. J'ai mis le menu directement sur mon Frame..
Mais ça ne change rien au problème du ScrollPanel.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
 
public class Help extends JFrame {
 
 
    private static final long serialVersionUID = 1L;
    private JPanel helpPanel = new JPanel();
    private JTextArea textCommandes = new JTextArea();
    private String filename = new String("commandes.txt");
    private JScrollPane scrollPane = new JScrollPane(helpPanel);
    private JMenuBar menu = new JMenuBar();
    private JMenu menu1 = new JMenu("File");
    private JMenuItem menuQuit = new JMenuItem("Quit");
 
 
    /**
     * This is the default constructor
     */
    public Help() {
 
          setTitle("Help");
          setSize(600, 600);
          setDefaultCloseOperation(DISPOSE_ON_CLOSE);
          helpPanel.setLayout(null);
 
          menuQuit.addActionListener(new MenuQuitListener());
 
          try {
            commandeText();
        } catch (IOException e) {
            e.printStackTrace();
        }
 
        menu.add(menu1);
        menu1.add(menuQuit);
 
 
 
        helpPanel.setBounds(0, 0, 600, 1700);
 
        textCommandes.setBounds(0, 20, 600, 1700);
        textCommandes.setEditable(false);
        helpPanel.add(textCommandes);
 
        helpPanel.setPreferredSize(new Dimension(600, 1700));
 
        add(menu);
        menu.setBounds(0, 0, 600, 20);
 
        add(scrollPane);
 
    }
 
    class MenuQuitListener implements ActionListener{
 
        public void actionPerformed(ActionEvent e){
            quit(e);
        }
    }
 
    private void quit(ActionEvent e){
        dispose();
    }
 
    public void commandeText() throws IOException{
 
          try {
                 File file = new File(filename);
                 BufferedReader br = new BufferedReader(new FileReader(file));
                 String s = ""; 
                 int c=0;
 
                 while((c=br.read())!=-1){
                     s+=(char)c;
                 }
                 br.close(); 
 
                 textCommandes.setText(s);
 
 
              } catch (Exception e) {
 
                 JOptionPane.showMessageDialog(this,e.getMessage(),
                 "File error",JOptionPane.ERROR_MESSAGE);}
 
    }
}

Désolé j'avais pas vu tout ton message...
Est-ce que mes changements sont juste ou est-ce que tu en as fait d'autres?
Le problème est toujours présent si jamais.
Merci beaucoup de ton aide

**adiGuba** · 22/09/2010, 14h18

Salut,

Je n'arrive pas à reproduire exactement ton problème, mais il y a plein de petit soucis dans ton code.

Tu places ton menu dans le panel, alors que tu devrais plutôt le mettre sur ta JFrame avec setJMenuBar().
Tu utilises un layout null avec plein de setBounds()
Non seulement ce n'est pas du tout pratique mais c'est source d'erreur !
En fait ton helpPanel ne sert à rien : tu pourrais directement mettre ton JTextArea dans ton JScrollPane...

Ta lecture du fichier est calamiteuse : tu utilises bien un buffer pour la lecture mais pas pour la création de la chaine que tu fait caractère par caractère. En gros si le fichier fait 5000 caractères tu vas créer 5000 objets temporaire

Il faut passer par un StringBuilder :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    public void commandeText() throws IOException {
        try {
            Reader br = new FileReader(filename);
            try {
                StringBuilder sb = new StringBuilder();
                char[] cbuf = new char[8196];
                int len;
 
                while ( (len=br.read(cbuf)) >= 0 ) {
                    sb.append(cbuf, 0, len);
                }
                textCommandes.setText(sb.toString());
            } finally {
                br.close();
            }
        } catch (Exception e) {
            JOptionPane.showMessageDialog(this, e.getMessage(), "File error",
                    JOptionPane.ERROR_MESSAGE);
        }
    }

a++

**wax78** · 22/09/2010, 14h19

Forcement j'avais pas fini d'editer mon message désolé ^^. Relis a la fin de mon post précedent.

Je pense avoir corrigé les point qu'adi souligne en vitesse. Et c'est vrai que la lecture est bcp trop lente

**petole** · 22/09/2010, 14h25

Oula!!

merci beaucoup pour vos conseil mais je suis pas tout^^

Alors voilà mon code maintenant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.FileReader;
import java.io.IOException;
import java.io.Reader;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
 
public class Help extends JFrame {
 
 
	private static final long serialVersionUID = 1L;
	private JPanel helpPanel = new JPanel();
	private JTextArea textCommandes = new JTextArea();
	private String filename = new String("commandes.txt");
	private JScrollPane scrollPane = new JScrollPane(helpPanel);
	private JMenuBar menu = new JMenuBar();
	private JMenu menu1 = new JMenu("File");
	private JMenuItem menuQuit = new JMenuItem("Quit");
 
 
	/**
         * This is the default constructor
         */
	public Help() {
 
		  setTitle("Help");
		  setSize(600, 600);
		  setDefaultCloseOperation(DISPOSE_ON_CLOSE);
 
		  menuQuit.addActionListener(new MenuQuitListener());
 
		  try {
			commandeText();
		} catch (IOException e) {
			e.printStackTrace();
		}
 
		menu.add(menu1);
		menu1.add(menuQuit);
 
		textCommandes.setEditable(false);
 
		helpPanel.add(textCommandes);
 
		setJMenuBar(menu);
		add(scrollPane);
 
	}
 
	class MenuQuitListener implements ActionListener{
 
		public void actionPerformed(ActionEvent e){
			quit(e);
		}
	}
 
	private void quit(ActionEvent e){
		dispose();
	}
 
    public void commandeText() throws IOException {
        try {
            Reader br = new FileReader(filename);
            try {
                StringBuilder sb = new StringBuilder();
                char[] cbuf = new char[8196];
                int len;
 
                while ( (len=br.read(cbuf)) >= 0 ) {
                    sb.append(cbuf, 0, len);
                }
                textCommandes.setText(sb.toString());
            } finally {
                br.close();
            }
        } catch (Exception e) {
            JOptionPane.showMessageDialog(this, e.getMessage(), "File error",
                    JOptionPane.ERROR_MESSAGE);
        }
    }
}

J'ai essayer de mettre mon JTextField directement sur mon JScrollPanel comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

scrollPane.add(textCommandes);

Mais ça ne marche pas.
Avec ceci, mon fichier texte ne s'affiche plus.

Merci beaucoup pour le code de lecture de mon fichier texte...
Je suis vraiment débutante et j'ai tenté de faire quelque chose^^

**wax78** · 22/09/2010, 14h35

Supprime HelpPanel il ne te sert a rien.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
public Help() {
 
		  setTitle("Help");
		  setSize(600, 600);
		  setDefaultCloseOperation(DISPOSE_ON_CLOSE);
 
		  menuQuit.addActionListener(new MenuQuitListener());
 
		  try {
			commandeText();
		} catch (IOException e) {
			e.printStackTrace();
		}
 
		menu.add(menu1);
		menu1.add(menuQuit);
 
		textCommandes.setEditable(false);
 
		scrollPane.setViewportView(textCommandes);
 
		setJMenuBar(menu);
		add(scrollPane);
 
	}

**petole** · 22/09/2010, 14h39

J'ai fait ce que tu m'as dit.
J'ai supprimé mon helpPanel mais ma scrollbar part toujours d'en-bas

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.FileReader;
import java.io.IOException;
import java.io.Reader;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
 
public class Help extends JFrame {
 
 
	private static final long serialVersionUID = 1L;
	private JTextArea textCommandes = new JTextArea();
	private String filename = new String("commandes.txt");
	private JScrollPane scrollPane = new JScrollPane();
	private JMenuBar menu = new JMenuBar();
	private JMenu menu1 = new JMenu("File");
	private JMenuItem menuQuit = new JMenuItem("Quit");
 
 
	/**
         * This is the default constructor
         */
	public Help() {
 
		  setTitle("Help");
		  setSize(600, 600);
		  setDefaultCloseOperation(DISPOSE_ON_CLOSE);
 
		  menuQuit.addActionListener(new MenuQuitListener());
 
		  try {
			commandeText();
		} catch (IOException e) {
			e.printStackTrace();
		}
 
		menu.add(menu1);
		menu1.add(menuQuit);
 
		textCommandes.setEditable(false);
 
		scrollPane.setViewportView(textCommandes);
 
		setJMenuBar(menu);
		add(scrollPane);
 
 
	}
 
	class MenuQuitListener implements ActionListener{
 
		public void actionPerformed(ActionEvent e){
			quit(e);
		}
	}
 
	private void quit(ActionEvent e){
		dispose();
	}
 
    public void commandeText() throws IOException {
        try {
            Reader br = new FileReader(filename);
            try {
                StringBuilder sb = new StringBuilder();
                char[] cbuf = new char[8196];
                int len;
 
                while ( (len=br.read(cbuf)) >= 0 ) {
                    sb.append(cbuf, 0, len);
                }
                textCommandes.setText(sb.toString());
            } finally {
                br.close();
            }
        } catch (Exception e) {
            JOptionPane.showMessageDialog(this, e.getMessage(), "File error",
                    JOptionPane.ERROR_MESSAGE);
        }
    }
}

**wax78** · 22/09/2010, 14h47

Apres avoir remplis ton TextArea, tu dois faire ceci je penses :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

textCommandes.setCaretPosition(0 ); // Au debut;

**petole** · 22/09/2010, 14h53

ça marche!!!

Merci beaucoup pour les coups de main!

Mon ScrollPanel fonctionne à l'envers

Java

Discussions similaires

Partager

Partager