[Java] Nullpointer getGraphics

[Wes46]

Ik wil een programma maken dat lijkt op een voorbeeld dat ik heb uit een boek. Dit voorbeeld tekent een balletje zodra je op de startknop drukt. Nu wil ik die startknop eruit hebben en gewoon gelijk dat balletje zien.

Echter wanneer ik de class startknophandler verwijder en er een andere class van maak om die vervolgens in de constructor aan te roepen, geeft hij aan dat getGraphics () een nullpointer is. Hier de code:

Java:

public class lift extends JFrame {
  public static void main( String args[] ) {
    JFrame frame = new lift();
    frame.setSize( 1440, 900 );
    frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
    frame.setTitle( "Voorbeeld 1308  Thread met pauzemogelijkheid" );
    frame.setContentPane( new Liftpaneel() );
    frame.setVisible( true );
  }
}


class Liftpaneel extends JPanel {
  private Bal bal;
  private JButton startknop, pauzeknop, eindePauzeknop;

  public Liftpaneel() {
    startknop = new JButton( "Start" );
    pauzeknop = new JButton( "Pauze" );
    eindePauzeknop = new JButton( "Einde pauze" );
    startknop.addActionListener( new StartknopHandler() );
    pauzeknop.addActionListener( new PauzeknopHandler() );
    eindePauzeknop.addActionListener( new EindePauzeknopHandler() );
    add( startknop );
    add( pauzeknop );
    add( eindePauzeknop );
  }

class StartknopHandler implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
      if( bal == null ) {
        bal = new Bal( getGraphics(), 10, getBackground());
        bal.start();
      }
    }
  }

Wie kan mij hiermee helpen?

[xzaz]

getGraphics() van wat? Van welke control moet hij zijn Graphics ophalen?

[Robtimus]

xzaz schreef op zaterdag 17 oktober 2009 @ 17:17:
getGraphics() van wat? Van welke control moet hij zijn Graphics ophalen?

StartknopHandler is een inner class van Liftpaneel, en die inherit getGraphics() weer van JPanel.

[xzaz]

IceManX schreef op zaterdag 17 oktober 2009 @ 17:22:
[...]

StartknopHandler is een inner class van Liftpaneel, en die inherit getGraphics() weer van JPanel.

En toch is die null?

[Raynman]

xzaz schreef op zaterdag 17 oktober 2009 @ 17:33:
[...]

En toch is die null?

Hij is null nadat TS in de code heeft zitten rommelen.

Inhoud van die actionPerformed() methode binnen de constructor van Liftpaneel zetten en dan de knop zelf weghalen zou toch voldoende moeten zijn?

Maar als je op de plaats van StartknopHandler een andere klasse zet die in zijn eigen constructor of zomaar in een methode getGraphics() aanroept, zou dat ook moeten werken omdat dat dan ook weer een inner class is.

[Wes46]

Raynman schreef op zaterdag 17 oktober 2009 @ 18:02:
[...]

Hij is null nadat TS in de code heeft zitten rommelen.

Inhoud van die actionPerformed() methode binnen de constructor van Liftpaneel zetten en dan de knop zelf weghalen zou toch voldoende moeten zijn?

Maar als je op de plaats van StartknopHandler een andere klasse zet die in zijn eigen constructor of zomaar in een methode getGraphics() aanroept, zou dat ook moeten werken omdat dat dan ook weer een inner class is.

precies dat heb ik dus gedaan maar dan wordt hij juist null. Dat vind ik ook zo vreemd

[BalusC]

Dus met andere woorden: getGraphics() retourneert null? De manier waarop je het probleem beschreef suggereert namelijk dat het aanroepen van getGraphics() een NullPointerException veroorzaakt. In het laatste geval is al de code die je hebt gepost irrelevant en zou je de code van getGraphics() moeten tonen, samen met een stacktrace dat de schuldige regel aanwijst.

[ Voor 8% gewijzigd door BalusC op 18-10-2009 00:33 ]

[Wes46]

Wanneer ik er een regel als dit toevoeg: System.out.printf ("%s\n", getGraphics ());
dan krijg ik null in de situatie die ik wil hebben, dus zonder de startknophandler.
Met startknophandler krijg ik echter wel gewoon de inhoud van getGraphics dus getGraphics returnt null zonder de handler.

[Raynman]

Tweede resultaat voor "jpanel getgraphics" zegt dat getGraphics() null retourneert totdat er iets zichtbaar is. Dan kun je dus pas iets nadat regel 8 uitgevoerd is. Je kunt een eigen paint()[paintComponent() voor Swing] methode in Liftpaneel definiëren, die precies doet wat StartknopHandler.actionPerformed nu doet.

[ Voor 5% gewijzigd door Raynman op 18-10-2009 11:23 ]

[Wes46]

Op dit moment heb ik dit. Volgens mijn output via het System.out.printf doet hij precies wat hij nu zou moeten doen. Het gekke alleen is dat ik geen rood balletje zie die hij zou moeten tekenen, ik heb het idee dat ik iets over het hoofd zie

Java:

 
public class lift extends JFrame {
  public static void main( String args[] ) {
    JFrame frame = new lift();
    frame.setSize( 1440, 900 );
    frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
    frame.setTitle( "Voorbeeld 1308  Thread met pauzemogelijkheid" );
    Liftpaneel paneel = new Liftpaneel ();
    frame.setContentPane(paneel);
    frame.setVisible( true );
  }
}


class Liftpaneel extends JPanel {
  private Bal bal;
  private JButton pauzeknop, eindePauzeknop;

  public Liftpaneel() {
   // startknop = new JButton( "Start" );
    pauzeknop = new JButton( "Pauze" );
    eindePauzeknop = new JButton( "Einde pauze" );
  //  startknop.addActionListener( new StartknopHandler() );
    pauzeknop.addActionListener( new PauzeknopHandler() );
    eindePauzeknop.addActionListener( new EindePauzeknopHandler() );
    //add( startknop );
    add( pauzeknop );
    add( eindePauzeknop );
  }

    public void paintComponent(Graphics g) {
      super.paintComponent (g);
    if( bal == null ) {
        System.out.printf ("test\n");
        bal = new Bal(g, 10, getBackground());
        System.out.printf ("graphics = %s \n", g);
        bal.start();
      }
  }


  class PauzeknopHandler implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
      if( bal != null )
        bal.pauzeer();
    }
  }

  class EindePauzeknopHandler implements ActionListener {
    public void actionPerformed( ActionEvent e ) {
      if( bal != null )
        bal.pauzeVoorbij();
    }
  }
}


class Bal extends Thread {
  private Graphics pen;
  private int x, y, grootte;
  private Color wiskleur;
  private int dy;
  private boolean doorgaan, pauze;


  public Bal( Graphics g, int grootte, Color wiskleur ) {
    this.pen = g;
    System.out.printf ("%s \n", g);
    this.grootte = grootte;
    this.wiskleur = wiskleur;
    x = 200;
    y = 300;
    dy = 1;
    doorgaan = true;
    pauze = true;
  }

  public void run() {
    while( doorgaan ) {
      teken( Color.red );
      slaap( 10 );
      teken( wiskleur );
      verplaats();
      System.out.printf ("nu in bal.teken\n");
    }
  }

  public void teken( Color kleur ) {
      System.out.printf ("%s \n", pen);
    pen.setColor( kleur );
    pen.fillOval( x, y, grootte, grootte );
          System.out.printf ("%s \n", pen);
  }

  public void nuStoppen() {
    doorgaan = false;
  }

  public void pauzeer() {
    pauze = true;
  }

  public synchronized void pauzeVoorbij() {
    pauze = false;
    notify();
  }

  private synchronized void controleerPauze() throws
                                      InterruptedException {
    while( pauze ) {
      wait();
    }
  }

  private void verplaats() {
    if( y + dy > 300 || y + dy < 100 ) {
      dy = -dy;
    }
    y += dy;
  }

  private void slaap( int millisec ) {
    try {
      Thread.sleep( millisec );
      controleerPauze();
    }
    catch( InterruptedException e ) {}
  }
}

[Wes46]

anyone?

[Janoz]

De denkfout die je maakt gaat over het Graphics object. In tegenstelling tot wat jij denkt is dat graphics object alleen maar van toepassing waneer het panel gepaint wordt. Het is niet een handler die voor altijd aan het scherm verbonden blijft waarop je kunt tekenen.

Ik zou nu wel kunnen gaan uitleggen hoe het painten in Java precies werkt, maar het lijkt me handiger wanneer je de volgende tutorial even doorneemt om een beetje 'feeling' te krijgen:

http://java.sun.com/docs/...swing/painting/index.html

[Wes46]

Ok bedankt, ik zal het even doornemen