Methode

[KurzeFrage]

public void Punktezähler(){
    final JLabel Punkte = new JLabel();
    Punkte.setBounds(100,100,100,100);
    Punkte.setText(Integer.toString((punkte)));	
    f.add(Punkte);
}

Ich bekomme ein Error wenn ich mein Programm inklusive dieser Methode aufrufe.
Kann mir jemand sagen warum?

 

[Gast2]

Wenn du uns noch verrätst was für nen Error kommt, dann bestimmt

 

[KurzeFrage]

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Spiel.Punktezähler(Spiel.java:221)
at Spiel.paintComponent(Spiel.java:99)
at javax.swing.JComponent.paint(Unknown Source)
at javax.swing.JComponent.paintToOffscreen(Unknown Source)
at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(Unknown Source)
at javax.swing.RepaintManager$PaintManager.paint(Unknown Source)
at javax.swing.RepaintManager.paint(Unknown Source)
at javax.swing.JComponent._paintImmediately(Unknown Source)
at javax.swing.JComponent.paintImmediately(Unknown Source)
at javax.swing.RepaintManager.paintDirtyRegions(Unknown Source)
at javax.swing.RepaintManager.paintDirtyRegions(Unknown Source)
at javax.swing.RepaintManager.seqPaintDirtyRegions(Unknown Source)
at javax.swing.SystemEventQueueUtilities$ComponentWorkRequest.run(Unknown Source)
at java.awt.event.InvocationEvent.dispatch(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)


Das wird am laufenden Band wiederholt.

 

[Karl Neidhart]

public void Punktezähler(){
    final JLabel Punkte = new JLabel();
    Punkte.setBounds(100,100,100,100);
    Punkte.setText(Integer.toString((punkte)));	
    f.add(Punkte);
}

Ich bekomme ein Error wenn ich mein Programm inklusive dieser Methode aufrufe.
Kann mir jemand sagen warum?

Was ist "punkte", was ist "f"? Möglicherweise liegt das Problem ja in der Deklaration von diesen beiden Unbekannten (vermutlich ein int und ein JFrame, aber wer weiß?).

 

[Schandro]

poste deinen gesamten Code, wir sind keine Wahrsager...

 

[Gast2]

Eine Variable in Zeile 221 (Spiel.java) ist null.
Welche Zeile ist das in deinem Post?

 

[KurzeFrage]

    int punkte=0;
    JLabel Punkte;
    JFrame f;


   public static void main(String args[])
    {
        final JFrame f = new JFrame();
        Spiel s = new Spiel();
        f.add(s);
        f.setVisible(true);
        f.setDefaultCloseOperation(3);
        f.setSize(800, 600);
}


        if(Münze){
        punkte=punkte+10;
}


public void Punktezähler(){
    final JLabel Punkte = new JLabel();
    Punkte.setBounds(100,100,100,100);
    Punkte.setText(Integer.toString((punkte)));	
    //Zeile 221  f.add(Punkte);
}

 

[Gast2]

final JFrame f = new JFrame();

Damit überdeckst du dir deine Klassenvariable, die dann null bleibt und später die Exception auslöst.
Ändere die Zeile mal zu:

f = new JFrame();

 

[KurzeFrage]

Dann kommt dieser Error: "Cannot make a static reference to the non-static field f"

 

[Gast2]

Stimmt, hatte ich übersehen, du rufst das ganze ja aus der main auf. Du könntest f natürlich static machen, aber ich denke so kommen wir nicht weiter.
Vielleicht postest du mal deinen kompletten Quellcode und beschreibst mal was du vorhast. In deinem jetzigen Quellcode steht z.b. auch noch eine if-Abfrage mitten im nirgendwo.

 

[Karl Neidhart]

(oops, war zu langsam)

 

[KurzeFrage]

import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Image;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.Timer;

public class Spiel extends JPanel
    implements ActionListener, KeyListener
{
	

    Timer t;
    int x;
    int velx;
    int Höhe;
    int Schnelligkeit;
    boolean left;
    boolean right;
    boolean spiel =true;
    boolean Geldsack = true;
    Graphics g;
    int sacknummer;
    int punkte=0;
    JLabel Punkte;
    JFrame f;
    int leben=3;

    

    public Spiel()
    {
        t = new Timer(15, this);
        x = 0;
        velx = 0;
        t.start();
        addKeyListener(this);
        setFocusable(true);
        setFocusTraversalKeysEnabled(true);
    	

        
    }

    public static void main(String args[])
    {
    		

        final JFrame f = new JFrame();
        Spiel s = new Spiel();
        f.add(s);
        f.setVisible(true);
        f.setDefaultCloseOperation(3);
        f.setSize(800, 600);
        
        
        
        

 

    }


    public void paintComponent(Graphics g)
    {
    	
    	
    	super.paintComponent(g);

    	
    	
    	String filename2 = "hintergrund.jpg";
        Image img2 = Toolkit.getDefaultToolkit().getImage(filename2);
        ((Graphics2D)g).drawImage(img2, (int)0, (int)0, this);
    	
    	String filename = "Figur1.gif";
        Image img1 = Toolkit.getDefaultToolkit().getImage(filename);
        ((Graphics2D)g).drawImage(img1, (int)x, (int)462, this);
    	

        if(sacknummer < 1 ){
        	Geldsack(300,g,-100);
        }
        if(sacknummer < 2 ){
        	Geldsack(200,g,-400);
        }
        if(sacknummer < 3 ){
        	Geldsack(200,g,-600);
        }
        if(sacknummer < 4 ){
        	Geldsack(600,g,-700);
        }
        
        Punktezähler();

    }



       

    public void actionPerformed(ActionEvent e)
    {
        repaint();
        
       

        x += velx;
    	Höhe += Schnelligkeit;
    	
    	if(spiel){
    	Schnelligkeit=2;

        if(left)
            velx = -5;
    	}

        if(right)
            velx = 5;
        if(!right && !left)
            velx = 0;
        if(x > 720)
            x -= 5;
        if(x < 10)
            x += 5;

        

        

    }

    public void keyPressed(KeyEvent e)
    {
        int code = e.getKeyCode();
        if(code == 37)
        {
            left = true;

        }
        if(code == 39)
        {
            right = true;

        }
    }

    
    

    public void keyTyped(KeyEvent keyevent)
    {
    }

    public void keyReleased(KeyEvent e)
    {
        if(e.getKeyCode() == 37)
            left = false;
        if(e.getKeyCode() == 39)
            right = false;

    }
    
    public void Geldsack(int Position, Graphics g,int y){
    	
    	
    	String filename = "Geldsack.gif";
        Image img = Toolkit.getDefaultToolkit().getImage(filename);
        ((Graphics2D)g).drawImage(img, (int)Position, (int)y+Höhe, this);
    	

        
        if(Position >= x-20 && Position < x+80 && y+Höhe>= 402 && y+Höhe<404 ){
        	sacknummer=sacknummer +1;
        	punkte=punkte+10;


        	

        }
        if(y+Höhe == 480){
        	sacknummer=sacknummer +1;

        }

    }
    

public void Punktezähler(){
    final JLabel Punkte = new JLabel();
    Punkte.setBounds(100,100,100,100);
    Punkte.setText(Integer.toString((punkte)));	
    f.add(Punkte);
}
    
    

	
	 
		        
	    

	  

    
}

Es handelt sich um ein Spiel, in dem man ein Männchen spielt, dass Gegnestände auffangen muss. Bei Geldsäcken bekommt man Punkte, doch diese bekomme ich nicht richtig angezeigt.

 

[Karl Neidhart]

Ok, ich habe gerade getestet, was passiert, wenn man in Zeile 33 "static" einfügt (vor "JFrame f;") und die Zeile 56 formuliert wie folgt:

f = new JFrame();

Dann wird der Code compiliert und ohne Fehler ausgeführt. Es erscheint ein weißes Fenster. Sonst nichts. Was hätte denn passieren sollen? Insbesondere wird kein Bild Geldsack.gif gezeichnet (bzw. das Programm murrt nicht, weil ich das Bild nicht habe).

 

[KurzeFrage]

Also ich kann das Programm mit dieser Lösung auch starten, aber das gewünschte Punktelabel wird nicht angezeigt.
Bei dir erscheint nur ein weißes Fenster, weil das Spiel auf Bilder aufbaut und das Punktelabel eben komischerweise nicht angezeigt wird.

 

[Karl Neidhart]

Ich glaube, ich habe ein Teil der Lösung gefunden.

Deine Klasse Spiel ist im Prinzip ein Luxus-JPanel. Das fügst Du dem JFrame f zu. Das JLabel, mit dem Du die Punkte anzeigen willst, fügst Du auch dem Frame zu. Du müsstest es aber dem JPanel hinzufügen.

Außerdem generierst Du ständig ein neues JLabel in Punktezähler, und zwar, wenn ich das richtig sehe, alle 15 Millisekunden. Zwar werden die alten JLabel vom Besenwagen (garbage collection) geholt, aber günstig für ein Actionspiel ist das bestimmt nicht.

Wenn Du Punktezähler wie folgt formulierst:

public void Punktezähler(){
    Punkte.setText(Integer.toString((punkte))); 
}

Deine main-Methode so:

 public static void main(String args[])
    {
            
 
        f = new JFrame();
        Spiel s = new Spiel();
        f.add(s);
        Punkte.setBounds(100,100,100,100);
        Punkte.setText(Integer.toString((punkte)));
        s.add(Punkte);
        Punkte.setVisible (true);
        f.setVisible(true);
        f.setDefaultCloseOperation(3);
        f.setSize(800, 600);
   
    }

die Deklaration des JLabels Punkte, Deines Frames und der Variablen so:

    static int punkte=0;
    static JLabel Punkte = new JLabel ();
    static JFrame f;

dann funktioniert es. Allerdings sieht der Code ganz schön pfriemelig aus - ich finde persönlich ja Main-Methoden mit mehr als einem Statement immer ein wenig verdächtig, und setDefaultCloseOperation (3) ist auch etwas karg lesbar.

 

[KurzeFrage]

Es funktioniert!
Vielen Dank!!

 

[Nicer]

bitte noch als gelesen makieren wenn das nu geklärt is

der übersichtheits halber