[Java] Waar compileert ie niet?

[Darius]

Ben bezig met het volgende spelletje onder java met Borland jbuilder. elke keer dat ik hem compile krijg ik volgende error :

java.lang.NullPointerException
at spel.Applet2.init(Applet2.java:20)

Weet iemand waar het aan kan liggen of wat ik moet veranderen om dit vervelend bericht weg te krijgen ? Source is als volgt :

====================

code:

package spel;

import java.awt.*;
import java.applet.*;

public class Applet2 extends Applet
{
  private Achtergrond achtergrond;
  private Bal bal;
  private Mannetje man;
  private int steps=5, appBreedte, appHoogte;
  private Image afbeelding, ball, buffer;
  private Graphics gBuffer;

  public void init()
  {
    appBreedte=getSize().width;
    appHoogte=getSize().height;
    buffer=createImage(appBreedte,appHoogte);
    gBuffer=buffer.getGraphics();

    achtergrond = new Achtergrond( getGraphics(), gBuffer, appBreedte, appHoogte);
    requestFocus();
    afbeelding= getImage( getDocumentBase(),"mallow.gif");
    ball=getImage(getDocumentBase(),"ball.gif");
  }

  public void start()
  {
    if(bal==null)
    {
    //bal = new Bal(gBuffer, 10, ball, this);
    //bal.start();
    }
    if(man==null)
    {
    man = new Mannetje(50,150,20,20, afbeelding, this);
    man.start();
    }
  }

  public void stop()
  {
    if(bal!=null)
    {
    bal.stop();
    bal=null;
    }
    if(man!=null)
    {
    man.stop();
    man=null;
    }
  }

  public boolean keyDown(Event e, int key)
    {
    switch(key)
    {
      case Event.DOWN:  if(man.y()+man.lengte()<appHoogte || man.y()-man.lengte() > achtergrond.ygrens1())
                    man.sety(steps);
                  break;
      case Event.UP:    man.sety(-steps);
                  break;
      case Event.RIGHT:  man.setx(steps);
                  break;
      case Event.LEFT:   man.setx(-steps);
                  break;
    }
    return true;
    }

  public void paint(Graphics g)
  {
    achtergrond.teken();
    gBuffer.drawImage(afbeelding, 20, 20, 100, 100, this);
    man.teken();
    //bal.teken();
    g.drawImage(buffer, 0, 0, this);
  }

  public void update(Graphics g) {
    //achtergrond.teken();
//    man.teken();
    gBuffer.drawImage(afbeelding, 20, 20, 1, 1, this);
    paint(g);
  }

  public void destroy() {
    gBuffer.dispose();
  }
}

class Achtergrond
{
  private Graphics g, gBuffer;
  private int breedte, hoogte, xgrens1, xgrens2, ygrens1, ygrens2;

  public Achtergrond(Graphics g, Graphics gBuffer, int breedte, int hoogte)
  {
    this.g=g;
    this.gBuffer=gBuffer;
    this.breedte=breedte;
    this.hoogte=hoogte;
  }

  public int ygrens1() {
    ygrens1=hoogte-230+40;
    return ygrens1;
  }
  public void teken()
  {
    gBuffer.setColor(Color.blue);
    gBuffer.fillRect(0,0,breedte,150); // De Lucht
    gBuffer.setColor(Color.yellow);
    gBuffer.fillRect(0,hoogte-190,breedte,150); // Het zand
    gBuffer.setColor(Color.orange);
    gBuffer.fillRect(0,hoogte-230,breedte,40);
    gBuffer.fillRect(0,hoogte-40,breedte,40); // Rand
    gBuffer.setColor(Color.white);
    gBuffer.fillRect( (breedte/2)-5,hoogte-230,10,230); // Paal
  }
}

class Bal extends Thread
{
  private Graphics g;
  private double y, dy=1, x, dx=1;
  private int grootte;
  private Image ball;
  private Applet theApplet;

  public Bal(Graphics g, int grootte, Image ball, Applet theApplet)
  {
    this.g=g;
    this.grootte=grootte;
    this.ball=ball;
    this.theApplet=theApplet;
  }

  public void teken()
  {
    //g.drawImage(ball, (int)x, (int)Math.round(y), grootte, grootte, theApplet);
    g.drawImage(ball, 10, 10, 50, 50, theApplet);
  }

  public void run()
  {
    x=10;
    y=40;
    //sleep(1000);

    while (true)
    {
    x=x+dx;
    y=y+dy;
    if (y >= 300)
      dy--;
    //theApplet.repaint();
    }
  }
}

  class Mannetje extends Thread
  {
    private Graphics g, gBuffer;
    private int x, y, lengte, breedte;
    private Image afbeelding;
    Applet theApplet;

    public Mannetje(int x, int y, int breedte, int lengte, Image afbeelding, Applet theApplet)
    {
    this.x=x;
    this.y=y;
    this.breedte=breedte;
    this.lengte=lengte;
    this.afbeelding=afbeelding;
    this.theApplet=theApplet;
    }

    public void run()
    {
    while(true)
    {
        theApplet.repaint();
        sleep(10);
    }
    }

    public void teken()
    {
    gBuffer.drawImage(afbeelding, x, y, lengte, breedte, theApplet);
    }

    public void setx(int temp)
    {
    x +=temp;
    }

    public void sety(int temp)
    {
    y +=temp;
    }

    public void sleep(int millisec)
    {
    try{ Thread.sleep(millisec); }
    catch( InterruptedException e) {}
    }

    public int lengte() {
    return lengte;
    }

    public int breedte() {
    return breedte;
    }

    public int x() {
    return x;
    }

    public int y() {
    return y;
    }
  }

[MisterData]

NullPointerException betekent dat je een variabele wel hebt gemaakt, maar nog niet hebt geinitialiseerd. Met andere woorden : als je :

code:

Button knopOK;

doet en dan :

code:

knopOK.setText("blaat");

voordat je :

code:

knopOK = new Button();

doet dan geeft ie een fout omdat je nog geen knop hebt geconstrueerd

Zoek je code daarop es goed door. Ik zal ook ff kijken , maar je leert meer als je het zelf doet

[Janoz]

Een exception tijdens het compileren??
haal je compileren en uitvoeren niet een beetje door de war?

Daarnaast is sourcecode leesbaarder waneer je het tussen [code] en [/code] zet.

[Darius]

Ok. Dat is wat duidelijker. Thanks Hij zegt dat de error in regel 20 zit (gBuffer=buffer.getGraphics();) maar deze buffer vul ik later in de code echt wel in (int paint() ). Wat kan ik anders doen zodat hij stopt met klagen?

[Darius]

Yeah. Dat bedoel ik. Bedankt voor de tip.

[MisterData]

Beetje inspringen in je code is trouwens ook wat netter

[Jelmer]

[edit] was al gezegd

[MisterData]

Neej dat ging over de code tags
Maarja boeiiu

[Darius]

Heb ergens anders gelezen dat Java dit niet pikt en dat ik addnotify moet overriden. Heb ik gedaan. Maar geeft nog steeds dezelfde error. Code ziet er nu als volgt uit :

code:

package spel;

import java.awt.*;
import java.applet.*;

public class Applet2 extends Applet
{
  private Achtergrond achtergrond;
  private Bal bal;
  private Mannetje man;
  private int steps=5, appBreedte, appHoogte;
  private Image afbeelding, ball, buffer;
  private Graphics gBuffer;

  public void addNotify()
  {
    super.addNotify();
    appBreedte=getSize().width;
    appHoogte=getSize().height;
    buffer=createImage(600,400);
    //gBuffer=buffer.getGraphics();
  }

  public void init()
  {
    //buffer=createImage(appBreedte,appHoogte);
    gBuffer=buffer.getGraphics();

    achtergrond = new Achtergrond( getGraphics(), gBuffer, appBreedte, appHoogte);
    requestFocus();
    afbeelding= getImage( getDocumentBase(),"mallow.gif");
    ball=getImage(getDocumentBase(),"ball.gif");
  }

  public void start()
  {
    if(bal==null)
    {
    //bal = new Bal(gBuffer, 10, ball, this);
    //bal.start();
    }
    if(man==null)
    {
    man = new Mannetje(50,150,20,20, afbeelding, this);
    man.start();
    }
  }

  public void stop()
  {
    if(bal!=null)
    {
    bal.stop();
    bal=null;
    }
    if(man!=null)
    {
    man.stop();
    man=null;
    }
  }

  public boolean keyDown(Event e, int key)
    {
    switch(key)
    {
      case Event.DOWN:  //if(man.y()+man.lengte()<appHoogte || man.y()-man.lengte() > achtergrond.ygrens1())
                    man.sety(steps);
                  break;
      case Event.UP:    man.sety(-steps);
                  break;
      case Event.RIGHT:  man.setx(steps);
                  break;
      case Event.LEFT:   man.setx(-steps);
                  break;
    }
    return true;
    }

  public void paint(Graphics g)
  {
    achtergrond.teken();
    gBuffer.drawImage(afbeelding, 20, 20, 100, 100, this);
    man.teken();
    //bal.teken();
    g.drawImage(buffer, 0, 0, this);
  }

  public void update(Graphics g) {
    //achtergrond.teken();
//    man.teken();
    gBuffer.drawImage(afbeelding, 20, 20, 1, 1, this);
    paint(g);
  }

  public void destroy() {
    gBuffer.dispose();
  }
}

class Achtergrond
{
  private Graphics g, gBuffer;
  private int breedte, hoogte, xgrens1, xgrens2, ygrens1, ygrens2;

  public Achtergrond(Graphics g, Graphics gBuffer, int breedte, int hoogte)
  {
    this.g=g;
    this.gBuffer=gBuffer;
    this.breedte=breedte;
    this.hoogte=hoogte;
  }

  public int ygrens1() {
    ygrens1=hoogte-230+40;
    return ygrens1;
  }
  public void teken()
  {
    gBuffer.setColor(Color.blue);
    gBuffer.fillRect(0,0,breedte,150); // De Lucht
    gBuffer.setColor(Color.yellow);
    gBuffer.fillRect(0,hoogte-190,breedte,150); // Het zand
    gBuffer.setColor(Color.orange);
    gBuffer.fillRect(0,hoogte-230,breedte,40);
    gBuffer.fillRect(0,hoogte-40,breedte,40); // Rand
    gBuffer.setColor(Color.white);
    gBuffer.fillRect( (breedte/2)-5,hoogte-230,10,230); // Paal
  }
}

class Bal extends Thread
{
  private Graphics g;
  private double y, dy=1, x, dx=1;
  private int grootte;
  private Image ball;
  private Applet theApplet;

  public Bal(Graphics g, int grootte, Image ball, Applet theApplet)
  {
    this.g=g;
    this.grootte=grootte;
    this.ball=ball;
    this.theApplet=theApplet;
  }

  public void teken()
  {
    //g.drawImage(ball, (int)x, (int)Math.round(y), grootte, grootte, theApplet);
    g.drawImage(ball, 10, 10, 50, 50, theApplet);
  }

  public void run()
  {
    x=10;
    y=40;
    //sleep(1000);

    while (true)
    {
    x=x+dx;
    y=y+dy;
    if (y >= 300)
      dy--;
    //theApplet.repaint();
    }
  }
}

  class Mannetje extends Thread
  {
    private Graphics g, gBuffer;
    private int x, y, lengte, breedte;
    private Image afbeelding;
    Applet theApplet;

    public Mannetje(int x, int y, int breedte, int lengte, Image afbeelding, Applet theApplet)
    {
    this.x=x;
    this.y=y;
    this.breedte=breedte;
    this.lengte=lengte;
    this.afbeelding=afbeelding;
    this.theApplet=theApplet;
    }

    public void run()
    {
    while(true)
    {
        theApplet.repaint();
        sleep(10);
    }
    }

    public void teken()
    {
    gBuffer.drawImage(afbeelding, x, y, lengte, breedte, theApplet);
    }

    public void setx(int temp)
    {
    x +=temp;
    }

    public void sety(int temp)
    {
    y +=temp;
    }

    public void sleep(int millisec)
    {
    try{ Thread.sleep(millisec); }
    catch( InterruptedException e) {}
    }

    public int lengte() {
    return lengte;
    }

    public int breedte() {
    return breedte;
    }

    public int x() {
    return x;
    }

    public int y() {
    return y;
    }
  }

Weet iemand waar dit aan kan liggen. Word er nu een beetje gek van

[Verwijderd]

Ik zeg maar wat hoor, maarre moet je niet eerst voordat je

code:

buffer=createImage(600,400);

neeerzet, eerst

code:

buffer=new Image();

neerzetten? Ik weet het niet hoor, tis lang geleden dat ik iets met Java deed...

[The - DDD]

Nee, dat hoeft niet. Je vangt met dat ding de return op...

Wat dus een instantie is van de juiste klasse.

[Verwijderd]

code:

gBuffer=buffer.getGraphics();

Genereert idd de NullPointerException. Dit komt omdat de variabele buffer in het vorige statement NIET geinitialiseerd is.

code:

createImage(appBreedte,appHoogte);

retourneert dus NULL. Ik denk dat je eens moet kijken naar de API-docs van die functie, om te kijken wanneer deze NULL retourneert.

edit:
Als je ff zegt in welke klasse createImage() zit, kan ik wat verder zoeken, ik weet het nl niet uit mijn hoofd

[Darius]

Weet het zelf ook niet. Heb geprobeerd om AddNotify te overriden :

code:

/*public void addNotify()
  {
    super.addNotify();
    //appBreedte=getSize().width;
    //appHoogte=getSize().height;
    buffer=createImage(appBreedte,appHoogte);
    //gBuffer=buffer.getGraphics();
  }*/

Maar dat werkte niet. Heb vele combinaties van uitgecomment code gebruikt. Ik moet het classes man in init initialiseren, dus zie geen manier om die nullpointer te omzeilen. Heb de code versimpeld. Het is nu als volgt:

code:

package spel;

import java.awt.*;
import java.applet.*;

public class Applet1 extends Applet {
  private Image speler1,buffer;
  //private Mannetje man;
  private Graphics gBuffer;

  public void init() {
    speler1=getImage( getDocumentBase(),"mallow.gif");
    //man = new Mannetje(gBuffer,speler1,50,150,20,20, this);
  }

  public void addNotify() {
    super.addNotify();
    initGraphics();
  }

  public void initGraphics()
  {
    buffer=createImage(400,300);
    gBuffer=buffer.getGraphics();
  }

  public boolean keyDown(Event e, int key) {
    switch(key) {
    case Event.DOWN:  man.sety(steps);
                break;
    case Event.UP:    man.sety(-steps);
                break;
    case Event.RIGHT: man.setx(steps);
                break;
    case Event.LEFT:  man.setx(-steps);
                break;
    }
    return true;
  }

  public void paint(Graphics g) {
    g.drawImage(speler1, 50, 150, 20, 20, this);
  }
}

class Mannetje implements Runnable {
  private Graphics g;
  private Image speler;
  int x, y, breedte, lengte;
  private Thread thread;
  private Applet theApplet;

  public Mannetje(Graphics g,Image speler, int x, int y, int breedte, int lengte, Applet theApplet) {
    this.g=g;
    this.speler=speler;
    this.x=x;
    this.y=y;
    this.breedte=breedte;
    this.lengte=lengte;
  }

  public void start() {
    if (thread==null) {
    thread=new Thread(this);
    thread.start();
    }
  }

    public void run() {
    teken();
    while(true) {
      theApplet.repaint();
      sleep(10);
    }
    }

    public void stop() {
    if(thread !=null) {
      thread.stop();
      thread=null;
    }
    }

    public void teken()
    {
    //g.drawImage(speler, x, y, lengte, breedte, theApplet);
    g.setColor(Color.black);
    g.fillOval(100,50,40,40);
    }

    public void sleep(int millisec)
    {
    try{ Thread.sleep(millisec); }
    catch( InterruptedException e) {}
    }
  }

Hij doet het nog steeds niet . Weet iemand hier raad mee?

[Alarmnummer]

dit viel me wel op.

code:

public Mannetje(Graphics g,Image speler, int x, int y, int breedte, int lengte, Applet theApplet) {
    this.g=g;
    this.speler=speler;
    this.x=x;
    this.y=y;
    this.breedte=breedte;
    this.lengte=lengte;
  }

je vergeet theApplet in te stellen..

[Alarmnummer]

en waarom die je zo ingewikkeld met die thread?

code:

AlarmThread extends Thread
{
  public AlarmThread()
  {
    ...constructor code
  }

  publiv void run()//deze override ik van thread
  {
     //en hier mooie thread code.
  }
}

//en starten met..
AlarmThread alarmThread = new Thread();
alarmThread.start();

[edit] heb de rest van je code even doorgekeken en daar zie ik staan dat je het eerst met een Thread had, dus dat zit wel snor. En die fout van theApplet in te stellen is er pas later ingekropen.

[Darius]

in the applet vul ik gewoon this in als ik het initialiseer. Dat is het probleem niet. Hij gaat klagen bij getGraphics().

[Verwijderd]

getsize geeft een double terug. Je zou dus eerst kunnen checken of de hoogte en breedte goed terug gegeven word na je getsize, voordat je een image gaat maken.

Verders moet je natuurlijk altijd!! checken of je buffer dus niet null is.

createImage
public Image createImage(int width,
int height)
Creates an off-screen drawable image to be used for double buffering.
Parameters:
width - the specified width.
height - the specified height.
Returns:
an off-screen drawable image, which can be used for double buffering.