Alle Komponenten eines JPanels

[Vxnc]

Hallo!
Ich möchte auf meinem JPanel alle Komponenten verschieben. Unabhängig davon wie viele es sind. Für einen Einblick habe ich mir das ungefähr so vorgestellt:

int size = screen.ground.getComponentCount();
       
        for(int i = 0; i <= size; i++){
           
            int x = (int) screen.ground.getComponent(i).getLocation().getX();
            int y = (int) screen.ground.getComponent(i).getLocation().getY();
           
            screen.ground.getComponent(i).setLocation(x+20,y);
           
        }

leider funktioniert diese Version nicht...(NullPointerException)
Ich habe bei dem Test genau eine Komponente auf dem JPanel "ground" ind der Klasse "screen"..

Ich würde mich über Hilfe freuen

 

[Harry Kane]

leider funktioniert diese Version nicht...(NullPointerException)

Dann ist entweder screen null oder screen.ground null. Warum das so ist, ist ohne weiteren Code unmöglich zu sagen.

 

[Vxnc]

Wie null?

Beides ist vorhanden und funktioniert ansonsten

 

[Vxnc]

package pack;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.*;

public class screen extends JFrame implements KeyListener{

    static JFrame frame;
    static JPanel ground;
   
    Pixel p1;
   
    Color blockscol;
   
    public screen() throws InterruptedException{
       
        Color blockscol = new Color(139,69,19);
       
        JFrame frame = new JFrame("JumpN'Run");
            frame.setExtendedState(JFrame.MAXIMIZED_BOTH);
            frame.setDefaultCloseOperation(EXIT_ON_CLOSE);
            frame.setVisible(true);
           
        JPanel ground = new JPanel();
            frame.add(ground);
            ground.setMaximumSize(ground.getMaximumSize());
            ground.setBackground(Color.WHITE);
            ground.setLayout(new BorderLayout());
            ground.setVisible(true);
           
        Pixel p1 = new Pixel(50, 50, 500, 500);
            ground.add(p1);
            p1.setBackground(blockscol);
           
           
        ground.setLayout(null);   
       
       
        System.out.println(ground.getComponentCount());
       
        Thread.sleep(1000);
        p1.setLocation(500, 500);
        p1.setSize(50, 50);
       
       
    }

    public static void main (String[] args) throws InterruptedException{
        screen wind = new screen();
        reloader rel = new reloader();
        rel.move();
    }
   
    @Override
    public void keyTyped(KeyEvent e) {
        // TODO Auto-generated method stub
       
    }

    @Override
    public void keyPressed(KeyEvent e) {
        // TODO Auto-generated method stub
       
    }

    @Override
    public void keyReleased(KeyEvent e) {
        // TODO Auto-generated method stub
       
    }
   
   
   
}

[\code]

 

[Vxnc]

Das ist meine Hauptklasse

 

[Vxnc]

package pack;

import java.awt.Component;

import javax.swing.JComponent;
import javax.swing.JPanel;

public class reloader {

    public static void main(String[] args) throws InterruptedException{
       
        while (true){
            Thread.sleep(500);
            move();
            System.out.println("test");
        }
       
       
       
    }
   
    static void move(){
       
       
       
        /*int size = screen.ground.getComponentCount();
       
        for(int i = 0; i <= size; i++){
           
            int x = (int) screen.ground.getComponent(i).getLocation().getX();
            int y = (int) screen.ground.getComponent(i).getLocation().getY();
           
            screen.ground.getComponent(i).setLocation(x+20,y);
           
        }
        */
       
        for(int i=0;i<screen.frame.getContentPane().getComponents().length;i++){
            System.out.println(screen.frame.getContentPane().getComponents()[i]);
        }
       
    }
   
}

[\code]

das die klasse woraus der code oben ist

 

[JStein52]

Und in welcher Zeile wirft er nun die NPE

 

[JStein52]

Und wieso hast du in beiden Klassen eine main-Methode ? Welche wird denn nun aufgerufen ?

Edit: wenn er die zweite aufruft ist ja so gut wie alles null ! Da bist du mit einer einzigen NPE echt gut bedient

 

[Harry Kane]

Eine Klasse, die JFrame erweitert, und die gleichzeitig eine static Variable vom Typ JFrame hat, sorgt nur für Ärger und Verwirrung.
Und den Aufruf von new reloader() kannst du dir sparen, weil reloader nur statische Methode hat. Das Erzeugen einer Instanz bringt dir gar nix.
Wieviel Programmierfahrung hast du in Java?

 

[JStein52]

Eine Klasse, die JFrame erweitert, und die gleichzeitig eine static Variable vom Typ JFrame hat

Und die ausserdem im konstruktor Variablen mit gleichem Namen nochmal lokal anlegt.

 

[JStein52]

Beides ist vorhanden und funktioniert ansonsten

Du weisst zwar "ground" ein JPanel zu. Aber es ist nicht das ground was du glaubst dass es ist. Also wird screen.ground auf jeden Fall null sein

 

[Vxnc]

Eine Klasse, die JFrame erweitert, und die gleichzeitig eine static Variable vom Typ JFrame hat, sorgt nur für Ärger und Verwirrung.
Und den Aufruf von new reloader() kannst du dir sparen, weil reloader nur statische Methode hat. Das Erzeugen einer Instanz bringt dir gar nix.
Wieviel Programmierfahrung hast du in Java?

Naja...Habe schon recht viel gemacht (seit 1 Jahr). Aber ich glaube man merkt, dass ich halt alles immer selber mir beigebracht habe und hier im Forum nachgelesen habe usw. ...Mir fehlen (wahrscheinlich) viele Grundprinzipien und vor allem, wie man seine Projekte aufbaut usw... Ich gucke mir immer wieder etwas an aber vor allem suche ich, wenn ich etwas suche danach und mach dann weiter einfach.. Halt alles ziemlich selbst beigebracht

 

[JStein52]

Dann wirf in einem ersten Schritt mal die main-Methode aus reloader raus und benutze im Konstruktor von screen die attribute der Klasse screen, nämlich frame und ground statt dort neue loakle Variablen anzulegen. Dann könnte es schon mal das tun was du willst. Dann ist es aber immer noch nicht schön. Dazu müsstest du die ganzen static-Anweisungen (bis auf main natürlich) entfernen und deine Klasse Screen nennen damit man sie nicht mit einer Variablen verwechselt (Klassennamen sollten immer mit Grossbuchstaben beginnen)

 

[Vxnc]

benutze im Konstruktor von screen die attribute der Klasse screen, nämlich frame und ground statt dort neue loakle Variablen anzulegen.

Wie meinen sie das?

 

[Harry Kane]

Wie meinen sie das?

Mal an einem vereinfachten Codebeispiel erklärt:
Du hast folgendes

public class screen{
    static JFrame frame;
    public screen(){
        JFrame frame = new JFrame();
    }
}

und sollst da folgendes draus machen:

public class screen{
    static JFrame frame;
    public screen(){
        frame = new JFrame();
    }
}

Ist dir klar, warum im oberen Beispiel screen.frame auch nach Aufrufen des Konstruktors von screen immer noch null ist?