My JScrollpane wont scroll horizontally?

Question

hmmmm...
As the title says, my JSCrollPane wont scroll horizontally. I've passed a customer "Drawer" object to it.
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.util.ArrayList;
public class gui extends JFrame{
     Container c;
     JPanel p;
     Drawer d;
     JComboBox box;
     JLabel lab;
     JScrollPane sp=new JScrollPane(p);
     String options[]={"Terminal","Non-Terminal","Repeat","Or"};
     ArrayList list=new ArrayList();
     String lastTerminal="";
     String lastNonTerminal="";
     int x=0;
     int y=350;
     boolean stupid=true;
     int pixelsize=0;
     String orString;
     ArrayList orStringList=new ArrayList();
     ArrayList orList=new ArrayList();
     boolean done=false;
     int count=0;
     int lastx=0;
     int lasty=0;
     public gui(){
          super("Something");
          setSize(900,700);
          c=getContentPane();
          c.setLayout(new BorderLayout(5,5));
          c.setBackground(Color.white);          
          p=new JPanel();
         d=new Drawer();
         box=new JComboBox(options);
         lab=new JLabel("Add:");
        box.addActionListener(new BoxListener());
        box.setOpaque(false);
         p.add(lab);
          p.add(box);
          sp=new JScrollPane(d);
          c.add(sp,BorderLayout.CENTER);
          c.add(p,BorderLayout.SOUTH);
          show();
     public class BoxListener implements ActionListener{
      public void actionPerformed(ActionEvent evt) {
           //if(!stupid){
            if (box.getSelectedIndex()==0)
                 lastTerminal=JOptionPane.showInputDialog("Enter your Terminal name");
                 list.add(new Terminal(lastTerminal,x,y,true));
                 Terminal tt=(Terminal)list.get(list.size()-1);
                 x=x+tt.getStringWidth()+45;
                 repaint();
            if (box.getSelectedIndex()==1)
                   lastNonTerminal=JOptionPane.showInputDialog("Enter your Non-Terminal name");
                 list.add(new Terminal(lastNonTerminal,x,y,false));
                 Terminal tt=(Terminal)list.get(list.size()-1);
                 x=x+tt.getStringWidth()+45;
                 repaint();
         if (box.getSelectedIndex()==3)
                   orStringList.clear();
                   orString=JOptionPane.showInputDialog("Enter your Or, seperate each Or with a |")+"|";
                   StringBuffer sb=new StringBuffer(orString);
                   while(sb.indexOf("|")!=-1)
                        orStringList.add(sb.substring(0,sb.indexOf("|")));
                        sb.delete(0,sb.indexOf("|")+1);
                   int eveny=y;
                    int oddy=y;
                    for(int i=1;i<orStringList.size()+1;i++)
                             if((i%2)==0)
                                       eveny=eveny+25;
                                       orList.add(new OrTerm((String)orStringList.get(i-1),x+35,eveny,false,x,y,x+130,y));
                              else
                                        oddy=oddy-25;
                                       orList.add(new OrTerm((String)orStringList.get(i-1),x+35,oddy,false,x,y,x+130,y));
                   OrTerm tt=(OrTerm)orList.get(orList.size()-1);
                 x=x+tt.getStringWidth()+100;
                   repaint();
     public static void main (String args[]) {
          gui app = new gui();
          app.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     public class Drawer extends JPanel
          public Drawer()
               this.setBackground(Color.ORANGE);
     public void paint(Graphics g2)
         super.paint(g2);
         Graphics2D g=(Graphics2D)g2;
          g.setFont(new Font(null,Font.BOLD,15));
          Terminal temp=null;
          OrTerm temp2=null;
          for(int i=0;i<orList.size();i++)
                    temp2=(OrTerm)orList.get(i);
                    pixelsize=SwingUtilities.computeStringWidth(g.getFontMetrics(),temp2.getName());
                    g.drawString(temp2.getName(),temp2.getx(),temp2.gety());
                    g.drawRect(temp2.getx()-4,temp2.gety()-16,pixelsize+7,25);
                    g.drawLine(temp2.getx()-2,temp2.gety()-2,temp2.getx2()-5,temp2.gety2()-2);
                    g.drawLine(temp2.getx()+pixelsize+7,temp2.gety()-2,temp2.getx3(),temp2.gety3());
          for(int i=0;i<list.size();i++)
               temp=(Terminal)list.get(i);
               g.drawString(temp.getName(),temp.getx(),temp.gety());
               pixelsize=SwingUtilities.computeStringWidth(g.getFontMetrics(),temp.getName());
               if(temp.isTerminal()==true)
                 g.drawOval(temp.getx()-4,temp.gety()-16,pixelsize+7,25);
               else
                 g.drawRect(temp.getx()-4,temp.gety()-16,pixelsize+7,25);
               lastx=temp.getx()+pixelsize+7+30;
               lasty=y-2;
               //drawing line
               g.drawLine(temp.getx()+pixelsize+7,y-2,temp.getx()+pixelsize+7+30,y-2); 
             //drawing little array >
             g.drawLine(temp.getx()+pixelsize+7+30,y-2,temp.getx()+pixelsize+7+25,y-7);
              g.drawLine(temp.getx()+pixelsize+7+30,y-2,temp.getx()+pixelsize+7+25,y+5);
class Terminal {
     private String name;
     private int x2;
     private int y2;
     private boolean terminal;
     public Terminal(String name, int x, int y,boolean boo)
          this.name=name;
          this.x2=x;
          this.y2=y;
          this.terminal=boo;
     public String getName(){return this.name;}
     public int getx(){return this.x2;}
     public int gety(){return this.y2;}
     public int getStringWidth()
          return SwingUtilities.computeStringWidth(Toolkit.getDefaultToolkit().getFontMetrics(new Font(null,Font.BOLD,15)),this.name);
     public boolean isTerminal(){return this.terminal;}
class OrTerm {
     private String name;
     private int x;
     private int y;
     private int x2;
     private int y2;
     private int x3;
     private int y3;
     private boolean terminal;
     public OrTerm(String name, int x, int y,boolean boo,int x2,int y2,int x3,int y3)
          this.name=name;
          this.x=x;
          this.y=y;
          this.terminal=boo;
          this.x2=x2;
          this.y2=y2;
          this.x3=x3;
          this.y3=y3;          
     public String getName(){return this.name;}
     public int getx(){return this.x;}
     public int gety(){return this.y;}
     public int getx2(){return this.x2;}
     public int gety2(){return this.y2;}
     public int getx3(){return this.x3;}
     public int gety3(){return this.y3;}
     public int getStringWidth()
          return SwingUtilities.computeStringWidth(Toolkit.getDefaultToolkit().getFontMetrics(new Font(null,Font.BOLD,15)),this.name);
     public boolean isTerminal(){return this.terminal;}
     import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.util.ArrayList;
public class gui extends JFrame{
     Container c;
     JPanel p;
     Drawer d;
     JComboBox box;
     JLabel lab;
     JScrollPane sp=new JScrollPane(p);
     String options[]={"Terminal","Non-Terminal","Repeat","Or"};
     ArrayList list=new ArrayList();
     String lastTerminal="";
     String lastNonTerminal="";
     int x=0;
     int y=350;
     boolean stupid=true;
     int pixelsize=0;
     String orString;
     ArrayList orStringList=new ArrayList();
     ArrayList orList=new ArrayList();
     boolean done=false;
     int count=0;
     int lastx=0;
     int lasty=0;
     public gui(){
          super("Something");
          setSize(900,700);
          c=getContentPane();
          c.setLayout(new BorderLayout(5,5));
          c.setBackground(Color.white);          
          p=new JPanel();
     d=new Drawer();
     box=new JComboBox(options);
     lab=new JLabel("Add:");
box.addActionListener(new BoxListener());
box.setOpaque(false);
     p.add(lab);
          p.add(box);
          sp=new JScrollPane(d);
          c.add(sp,BorderLayout.CENTER);
          c.add(p,BorderLayout.SOUTH);
          show();
     public class BoxListener implements ActionListener{
     public void actionPerformed(ActionEvent evt) {
          //if(!stupid){
          if (box.getSelectedIndex()==0)
     lastTerminal=JOptionPane.showInputDialog("Enter your Terminal name");
     list.add(new Terminal(lastTerminal,x,y,true));
     Terminal tt=(Terminal)list.get(list.size()-1);
     x=x+tt.getStringWidth()+45;
     repaint();
          if (box.getSelectedIndex()==1)
     lastNonTerminal=JOptionPane.showInputDialog("Enter your Non-Terminal name");
     list.add(new Terminal(lastNonTerminal,x,y,false));
     Terminal tt=(Terminal)list.get(list.size()-1);
     x=x+tt.getStringWidth()+45;
     repaint();
if (box.getSelectedIndex()==3)
          orStringList.clear();
          orString=JOptionPane.showInputDialog("Enter your Or, seperate each Or with a |")+"|";
          StringBuffer sb=new StringBuffer(orString);
          while(sb.indexOf("|")!=-1)
               orStringList.add(sb.substring(0,sb.indexOf("|")));
               sb.delete(0,sb.indexOf("|")+1);
          int eveny=y;
                    int oddy=y;
                    for(int i=1;i<orStringList.size()+1;i++)
                    if((i%2)==0)
                              eveny=eveny+25;
                              orList.add(new OrTerm((String)orStringList.get(i-1),x+35,eveny,false,x,y,x+130,y));
                              else
                                        oddy=oddy-25;
                              orList.add(new OrTerm((String)orStringList.get(i-1),x+35,oddy,false,x,y,x+130,y));
          OrTerm tt=(OrTerm)orList.get(orList.size()-1);
     x=x+tt.getStringWidth()+100;
          repaint();
     public static void main (String args[]) {
          gui app = new gui();
          app.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     public class Drawer extends JPanel
          public Drawer()
               this.setBackground(Color.ORANGE);
public void paint(Graphics g2)
     super.paint(g2);
     Graphics2D g=(Graphics2D)g2;
          g.setFont(new Font(null,Font.BOLD,15));
     Terminal temp=null;
     OrTerm temp2=null;
          for(int i=0;i<orList.size();i++)
               temp2=(OrTerm)orList.get(i);
               pixelsize=SwingUtilities.computeStringWidth(g.getFontMetrics(),temp2.getName());
               g.drawString(temp2.getName(),temp2.getx(),temp2.gety());
               g.drawRect(temp2.getx()-4,temp2.gety()-16,pixelsize+7,25);
               g.drawLine(temp2.getx()-2,temp2.gety()-2,temp2.getx2()-5,temp2.gety2()-2);
               g.drawLine(temp2.getx()+pixelsize+7,temp2.gety()-2,temp2.getx3(),temp2.gety3());
     for(int i=0;i<list.size();i++)
          temp=(Terminal)list.get(i);
          g.drawString(temp.getName(),temp.getx(),temp.gety());
          pixelsize=SwingUtilities.computeStringWidth(g.getFontMetrics(),temp.getName());
          if(temp.isTerminal()==true)
          g.drawOval(temp.getx()-4,temp.gety()-16,pixelsize+7,25);
          else
          g.drawRect(temp.getx()-4,temp.gety()-16,pixelsize+7,25);
          lastx=temp.getx()+pixelsize+7+30;
          lasty=y-2;
          //drawing line
          g.drawLine(temp.getx()+pixelsize+7,y-2,temp.getx()+pixelsize+7+30,y-2);
     //drawing little array >
     g.drawLine(temp.getx()+pixelsize+7+30,y-2,temp.getx()+pixelsize+7+25,y-7);
     g.drawLine(temp.getx()+pixelsize+7+30,y-2,temp.getx()+pixelsize+7+25,y+5);
class Terminal {
     private String name;
     private int x2;
     private int y2;
     private boolean terminal;
     public Terminal(String name, int x, int y,boolean boo)
          this.name=name;
          this.x2=x;
          this.y2=y;
          this.terminal=boo;
     public String getName(){return this.name;}
     public int getx(){return this.x2;}
     public int gety(){return this.y2;}
     public int getStringWidth()
          return SwingUtilities.computeStringWidth(Toolkit.getDefaultToolkit().getFontMetrics(new Font(null,Font.BOLD,15)),this.name);
     public boolean isTerminal(){return this.terminal;}
class OrTerm {
     private String name;
     private int x;
     private int y;
     private int x2;
     private int y2;
     private int x3;
     private int y3;
     private boolean terminal;
     public OrTerm(String name, int x, int y,boolean boo,int x2,int y2,int x3,int y3)
          this.name=name;
          this.x=x;
          this.y=y;
          this.terminal=boo;
          this.x2=x2;
          this.y2=y2;
          this.x3=x3;
          this.y3=y3;          
     public String getName(){return this.name;}
     public int getx(){return this.x;}
     public int gety(){return this.y;}
     public int getx2(){return this.x2;}
     public int gety2(){return this.y2;}
     public int getx3(){return this.x3;}
     public int gety3(){return this.y3;}
     public int getStringWidth()
          return SwingUtilities.computeStringWidth(Toolkit.getDefaultToolkit().getFontMetrics(new Font(null,Font.BOLD,15)),this.name);
     public boolean isTerminal(){return this.terminal;}
     

Answer 1

Scrollbars will appear when the preferred size of the component is greater than the size of the scroll pane. If you are adding a custom drawn component to the scrollpane then you must set the preferredSize.