Null Pointer Exception

[Caldera90]

Hallo zusammen,

ich will in einer ComboBox das aktuelle Jahr eintragen. Das funktioniert auch.
Wenn ich jetzt den Monat ändere, zb. von Dezember 2017 auf Januar 2018 bekomme ich eine NullPointerException. In der Variable steht aber etwas drin, habe es mit System.out getestet.
Was mache ich falsch???

Hier der Code:

package timetable;

import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.GraphicsEnvironment;
import java.awt.Rectangle;
import java.lang.reflect.Array;
import java.util.Calendar;
import java.util.Date;
import java.util.EventObject;
import java.util.GregorianCalendar;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.border.EmptyBorder;
import javax.swing.table.DefaultTableModel;

import timetable.Kalender.tblCalendarRenderer;

import javax.swing.JLabel;
import javax.swing.JComboBox;
import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import javax.swing.JTextField;
import javax.swing.DropMode;
import javax.swing.UIManager;
import java.beans.PropertyChangeListener;
import java.beans.PropertyChangeEvent;


public class TimetableMain extends JFrame {

    /**
     *
     */
    private static final long serialVersionUID = 1L;
    private static JPanel contentPane;
    private static JTable tblCal;
  
    static int realDay, realMonth, realYear, currentMonth;
    public static int monat;
    static int currentYear;

    private static JComboBox cmbJahr;
    private static JLabel tfMonat;
    private static JButton btnVor;
    private static JButton btnZurueck;
    private static JTextField tfJahr;
  
 
  
  
  
    public static void main(String[] args) {
    EventQueue.invokeLater(new Runnable() {
       public void run() {
        try {
           TimetableMain frame = new TimetableMain();
           frame.setVisible(true);
          
        } catch (Exception e) {
           e.printStackTrace();
        }
       }
  

  
  
    });
  
    } 

    
  
    public TimetableMain() {
    setTitle("Timetable");
          
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setBounds(100, 100, 827, 470);
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    this.setContentPane(contentPane);
    contentPane.setLayout(null);
  
//Label Monat beim Start mit aktuellem Monat vorbelegen
    String[] monate = {"Januar", "Februar", "März", "April", "Mai", "Juni", "Juli", "August", "September", "Oktober", "November", "Dezember"};
  
  
    JLabel lblDatum = new JLabel("Datum:");
    lblDatum.setBounds(30, 213, 46, 14);
    contentPane.add(lblDatum);
  
    JLabel lblMonat = new JLabel("");
    lblMonat.setBounds(236, 150, 89, 31);
    contentPane.add(lblMonat);
  
  
    GregorianCalendar calendar = new GregorianCalendar(); //Create calendar
          realMonth = calendar.get(GregorianCalendar.MONTH); //Get month
          realYear = calendar.get(GregorianCalendar.YEAR); //Get year
          currentYear = realYear;
          monat = realMonth;      

  
  
      
    JButton btnVor = new JButton(">>");
    btnVor.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
              
           if (monat == 11) { //Forward one year
               monat = 0;
               currentYear++;
            
              
            }
            else { //Forward one month
               monat++;
                            
            }
          
           refreshCalendar(monat, currentYear);
          
          
          
           //Aktuellen Monat ausgeben
          
            if (monat == 0){
               lblMonat.setText("Januar");
            
              
            }
          
            else if(monat == 1){
               lblMonat.setText("Februar");
              
            }

            else if(monat == 2){
               lblMonat.setText("März");
              
            }
          
            else if(monat == 3){
               lblMonat.setText("April");
            
            }
          
            else if(monat == 4){
               lblMonat.setText("Mai");
              
            }
          
            else if(monat == 5){
               lblMonat.setText("Juni");
              
            }
          
            else if(monat == 6){
               lblMonat.setText("Juli");
              
            }
          
            else if(monat == 7){
               lblMonat.setText("August");
              
            }
          
            else if(monat == 8){
               lblMonat.setText("September");
              
            }
          
            else if(monat == 9){
               lblMonat.setText("Oktober");
            
            }
          
            else if(monat == 10){
               lblMonat.setText("November");
              
            }
          
            else  if(monat == 11){
               lblMonat.setText("Dezember");
              
            }
          
          
          
            
          
          
          
        }     
    });
    btnVor.setBounds(130, 163, 89, 23);
    contentPane.add(btnVor);
  
      
    btnZurueck = new JButton("<<");
    btnZurueck.addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
      
           if (monat == 0){ //Back one year
               monat = 11;
               currentYear--;
              
            }
            else { //Back one month
               monat--;
            }
          
        
           refreshCalendar(monat, currentYear);
         //Aktuellen Monat ausgeben
          
            if (monat == 0){
               lblMonat.setText("Januar");
            
              
            }
          
            else if(monat == 1){
               lblMonat.setText("Februar");
              
            }

            else if(monat == 2){
               lblMonat.setText("März");
              
            }
          
            else if(monat == 3){
               lblMonat.setText("April");
            
            }
          
            else if(monat == 4){
               lblMonat.setText("Mai");
              
            }
          
            else if(monat == 5){
               lblMonat.setText("Juni");
              
            }
          
            else if(monat == 6){
               lblMonat.setText("Juli");
              
            }
          
            else if(monat == 7){
               lblMonat.setText("August");
              
            }
          
            else if(monat == 8){
               lblMonat.setText("September");
              
            }
          
            else if(monat == 9){
               lblMonat.setText("Oktober");
            
            }
          
            else if(monat == 10){
               lblMonat.setText("November");
              
            }
          
            else  if(monat == 11){
               lblMonat.setText("Dezember");
              
            }
          
          
         }
      
  
    });
    btnZurueck.setBounds(30, 163, 89, 23);
    contentPane.add(btnZurueck);
  
    JComboBox cmbJahr = new JComboBox();
    cmbJahr.setEditable(true);
    cmbJahr.setBounds(120, 119, 133, 20);
    contentPane.add(cmbJahr);
  
  
  
  

  
  
//Label Monat beim Start mit aktuellem Monat vorbelegen
        if (lblMonat.getText().length() == 0) {
          lblMonat.setText(monate[monat]);
        
        }  
      
//Erzeugen der Jahre in der ComboBox
    for (int i = realYear-100; i <= realYear+100; i++) {//Anzahl der Jahre vor und zurück
       cmbJahr.addItem(String.valueOf(i));      
    }
  
//Das aktuelle Jahr in der ComboBox belegen
    cmbJahr.setSelectedItem(String.valueOf(realYear));
  
  
  
  

      
  
//Das aktuell in der ComboBox gesetzte Jahr
    //cmbJahr.getSelectedItem(String.valueOf(Jahre));
  
    GraphicsEnvironment graphicsEnvironment=GraphicsEnvironment.getLocalGraphicsEnvironment();
    Rectangle maximumWindowBounds=graphicsEnvironment.getMaximumWindowBounds();
    setBounds(maximumWindowBounds);  
      
  
    refreshCalendar(monat, currentYear);  
    }


public static void refreshCalendar(int month, int year) { 
      
    cmbJahr.setSelectedItem(String.valueOf(year));
  
  
  
//Variables
          String[] monate = {"Januar", "Februar", "März", "April", "Mai", "Juni", "Juli", "August", "September", "Oktober", "November", "Dezember"};
          int nod, som; //Number of Days, Start of Month
  
//Get real month/year
          //GregorianCalendar calendar = new GregorianCalendar(); //Create calendar
          //realMonth = calendar.get(GregorianCalendar.MONTH); //Get month
          //realYear = calendar.get(GregorianCalendar.YEAR); //Get year
          //currentYear = realYear;
          //monat = realMonth;
            
            
          tblCal = new JTable();
    tblCal.setBounds(0, 233, 121, 589);
    tblCal.setModel(new DefaultTableModel(
        new Object[][] {
          
              {"01." + " " + String.valueOf(monate[monat])},
            {"02." + " " + String.valueOf(monate[monat])},
            {"03." + " " + String.valueOf(monate[monat])},
            {"04." + " " + String.valueOf(monate[monat])},
            {"05." + " " + String.valueOf(monate[monat])},
            {"06." + " " + String.valueOf(monate[monat])},
            {"07." + " " + String.valueOf(monate[monat])},
            {"08." + " " + String.valueOf(monate[monat])},
            {"09." + " " + String.valueOf(monate[monat])},
            {"10." + " " + String.valueOf(monate[monat])},
            {"11." + " " + String.valueOf(monate[monat])},
            {"12." + " " + String.valueOf(monate[monat])},
            {"13." + " " + String.valueOf(monate[monat])},
            {"14." + " " + String.valueOf(monate[monat])},
            {"15." + " " + String.valueOf(monate[monat])},
            {"16." + " " + String.valueOf(monate[monat])},
            {"17." + " " + String.valueOf(monate[monat])},
            {"18." + " " + String.valueOf(monate[monat])},
            {"19." + " " + String.valueOf(monate[monat])},
            {"20." + " " + String.valueOf(monate[monat])},
            {"21." + " " + String.valueOf(monate[monat])},
            {"22." + " " + String.valueOf(monate[monat])},
            {"23." + " " + String.valueOf(monate[monat])},
            {"24." + " " + String.valueOf(monate[monat])},
            {"25." + " " + String.valueOf(monate[monat])},
            {"26." + " " + String.valueOf(monate[monat])},
            {"27." + " " + String.valueOf(monate[monat])},
            {"28." + " " + String.valueOf(monate[monat])},
            {null},
            {null},
            {null},
          
        },
        new String[] {
            "Datum"
        }
      
      
    ));
    contentPane.add(tblCal);
    tblCal.setRowHeight(19);  
  
    tfJahr = new JTextField();
    tfJahr.setEditable(false);
    tfJahr.setBounds(281, 119, 86, 20);
    contentPane.add(tfJahr);
    tfJahr.setColumns(10);

          //lblMonth.setBounds(160-lblMonth.getPreferredSize().width/2, 25, 180, 25);//Re-align label with calendar
  
  
  
  
  
        
        
/*       
          //Clear table
          for (int i = 0; i < 6; i++) {
             for (int j = 0; j <7; j++) {
              mtblCalendar.setValueAt(null, i, j);
             }
          }
 
*/
/*       
  //Get first day of month and number of days
          GregorianCalendar cal = new GregorianCalendar(year, month, 1);
          nod = cal.getActualMaximum(GregorianCalendar.DAY_OF_MONTH);
          som = cal.get(GregorianCalendar.DAY_OF_WEEK);
        
*/       
        
  //Draw calendar
/*         for (int i = 1; i < nod; i++) {
             int row = new Integer ((i+som-2)/7);
             int column = (i+som-2)%7;
             mtblCalendar.setValueAt(i, row, column);
        
          }

*/       
      //Apply renderers
          //tblCalendar.setDefaultRenderer(tblCalendar.getColumnClass(0), new tblCalendarRenderer());
      
}
}

 

[Dompteur]

Du verwendest einige Instanz- und lokale Variablen mit dem gleichen Namen.
Ich bin mir fast sicher, dass du da durcheinander gekommen bist.
Bereinige das einmal.

In welcher Zeile passiert die NullPointerException ?

 

[Caldera90]

Sobald ich refreshCalendar aufrufe und in die ComboBox etwas hineinschreiben will:


public static void refreshCalendar(int month, int year) {

cmbJahr.setSelectedItem(String.valueOf(year));
...

 

[Dompteur]

Ja, wie vermutet hast du "cmbJahr" 2x deklariert. Einmal als Instanz-Variable, die immer null ist und einmal als lokale Variable beim Erzeugen der Combobox.
In der Methode "refreshCalendar" greifst du auf die Instanzvariable zu. Die ist null.

 

[Caldera90]

Hat funktioniert. Vielen Dank

 

[hellobook]

Bei mir gibt es auch eine Null Pointer exception an folgener stelle, wenn sie vom 2. Knoten durchlaufen wird:

 n = n + knoten[i].BezeichnungGeben();

der dazugehörige Quelltext ist dieser

public class ZUSATZ
{

    Knoten[] knoten;

    public ZUSATZ(int anzahl)
    {
        knoten = new Knoten[anzahl];
    }

    public String abkürzung (String name, int anzahl)
    {
        char[] wort  = name.toCharArray();
        String  nameNeu= "";
        boolean gleich;
        if(anzahl < 2)
        {
            nameNeu = nameNeu + wort[0];
            return nameNeu;
        }
        else
        {
            for(int y = 0; y < name.length(); y++)
            {
                for(int i=1; i<anzahl;i++)
                {
                    String n = "";
                    n = n + knoten[i].BezeichnungGeben();
                    char[] vergleich =n.toCharArray();
                    if(wort[y]==vergleich[y])
                    {
                        gleich = true;
                    }
                }
                if(gleich = false)
                {
                    nameNeu = nameNeu + wort[y];
                    break;
                }
                else
                {
                    nameNeu = nameNeu + wort[y];
                }
            }
            return nameNeu;
        }
    }
}

Ich finde einfach den Grund dafür nicht.

 

[hellobook]

Die Klasse Knoten sieht so aus:

public class Knoten
{
    String bezeichnung;
    public Knoten(String name)
    {
        bezeichnung = name;
    }

    public String BezeichnungGeben()
    {
       return bezeichnung;
    }
   
    public String BezFormatGeben(int nummer)
    {
        String a = BezeichnungGeben();
        for(int i = bezeichnung.length();i<=nummer;i++)
        {
            a=a+" ";
        }
        return a;
    }
}

 

[JStein52]

for(int i=1; i<anzahl;i++)
{
String n = "";
n = n + knoten[i].BezeichnungGeben();

Und du hast schon daran gedacht dass Indizes bei 0 beginnen ?

 

[Dompteur]

Bei mir gibt es auch eine Null Pointer exception an folgener stelle, wenn sie vom 2. Knoten durchlaufen wird:

 n = n + knoten[i].BezeichnungGeben();

Das liegt daran, dass alle knoten[?] null sind, da du keine Knoten-Objekte erzeugt bzw. zugewiesen hast.

 

[hellobook]

Aso ich hab vergessen die 3. Klasse reinzustellen:

public class GRAPH_MATRIX
{
    int anzahlKnoten;
    Knoten[]knoten;
    int[][] matrix;
    int breite = 0;
    int feldlaenge;
    boolean [] besucht;
    ZUSATZ zusatz;
    public GRAPH_MATRIX(int anzahl)
    {
        matrix = new int [anzahl][anzahl];
        besucht = new boolean [anzahl];
        knoten = new Knoten [anzahl];
        feldlaenge = anzahl;
        zusatz = new ZUSATZ(anzahl);
    }

    public void KnotenEinfuegen(String name)
    {

        if(anzahlKnoten<feldlaenge)
        {
            if(KnotenNummer(name)==-1)
            {
                knoten[anzahlKnoten] = new Knoten(name);
                matrix[anzahlKnoten][anzahlKnoten] = 0;
                for(int i=0; i<anzahlKnoten; i++)
                {
                    matrix[anzahlKnoten][i]=0;
                    matrix[i][anzahlKnoten]=0;
                }
                anzahlKnoten = anzahlKnoten +1;
                String nameKurz = zusatz.abkürzung(name, anzahlKnoten);
                if(breite < nameKurz.length())
                {
                    breite = nameKurz.length();
                }
            }
            else
            {
                System.out.println(name +" ist bereits vorhanden.");
            }
        }

    }

    public void KanteEinfuegen(String von, String nach, int gewichtung)
    {
        int vonNummer,nachNummer;
        vonNummer =KnotenNummer(von);
        nachNummer = KnotenNummer(nach);
        if(vonNummer !=-1 && nachNummer != -1 && vonNummer != nachNummer)
        {
            matrix[vonNummer][nachNummer] = gewichtung;
            matrix[nachNummer][vonNummer] = gewichtung;
        }

    }

    public int KnotenNummer(String Name)
    {
        int knotennummer = -1;
        for(int i=0; i<anzahlKnoten;i++)
        {
            if(knoten[i].BezeichnungGeben().equals(Name) && knoten[i]!= null)
            {
                knotennummer = i;
            }                                                                        
        }
        return knotennummer;
    }

    public void Ausgeben()
    {
        String b ="";
        for(int t = 0;t<=breite;t++)
        {
            b=b+" ";
        }
        String h = "";
        for(int i =0;i<anzahlKnoten;i++)
        {
            b=b+knoten[i].BezFormatGeben(breite)+" ";
        }
        System.out.println(b);
        String f="";
        for(int y=0;y<anzahlKnoten;y++)
        {
            h=knoten[y].BezFormatGeben(breite)+" ";
            for(int g =0;g<anzahlKnoten;g++)
            {
                String str = Integer.toString(matrix[y][g]);
                f=str;
                for(int k = str.length();k<=breite;k++)
                {
                    f=f+" ";
                }
                h=h+f+" ";
                f="";
            }
            System.out.println(h);
            h="";
        }
    }

    public void tiefenAusgabe(int nummer, int tiefe)
    {
        String abstand = " ";
        String abstandNeu = "";
        for(int i=0; i< breite; i++)
        {
            abstand = abstand + " ";
        }
        for(int i=0; i < tiefe;i++)
        {
            abstandNeu = abstandNeu + abstand;
        }
        System.out.println(abstandNeu +  "----> " + knoten[nummer].BezeichnungGeben());
    }

    public void besuchen(int kntnr)
    {
        int tiefe = 0;
        besucht[kntnr] = true;
        System.out.println(knoten[kntnr].BezeichnungGeben());
        for(int abzweignr=0; abzweignr<anzahlKnoten; abzweignr++)
        {
            if(matrix[kntnr][abzweignr]>0 && besucht[abzweignr] == false)
            {
                tiefe = tiefe +1;
                tiefenAusgabe(abzweignr,tiefe);
                besuchen(abzweignr);
            }
        }
        tiefe = tiefe -1;
        System.out.println(knoten[kntnr].BezeichnungGeben() + "(fertig)");
    }

    public void tiefensuche(String startKnoten)
    {
        int startnr;
        startnr = KnotenNummer(startKnoten);
        if(startnr != -1)
        {
            besuchen(startnr);
        }
        else
        {
            System.out.println("Knoten ist nicht vorhanden.");
        }
    }

    public void WegeSuche(String StartKnoten, String ZielKnoten)
    {   int startNummer = KnotenNummer(StartKnoten);
        int zielNummer = KnotenNummer(ZielKnoten);
        if(startNummer != -1 && zielNummer != -1 && startNummer != zielNummer)
        {
            for( int i=0; i<anzahlKnoten; i++)
            {
                besucht[i] = false;
            }
            Ablaufen(startNummer, zielNummer, StartKnoten, 0);
        }

    }

    public void Ablaufen(int StartKnotenNr, int ZielKnotenNr, String Pfad, int laenge)
    {
        int neueLaenge;
        String neuerPfad;
        besucht[StartKnotenNr] = true;
        if(StartKnotenNr == ZielKnotenNr)
        {
            System.out.println(Pfad+ " ; Länge "+ laenge);
        }
        else
        {
            for(int abzweigeNr =0; abzweigeNr<anzahlKnoten;abzweigeNr++)
            {
                if( matrix[StartKnotenNr][abzweigeNr]>0 && besucht[abzweigeNr] == false)
                {
                    neuerPfad = Pfad + "/"+knoten[abzweigeNr].BezeichnungGeben();
                    neueLaenge = laenge + matrix[StartKnotenNr][abzweigeNr];
                    Ablaufen(abzweigeNr, ZielKnotenNr,neuerPfad, neueLaenge);
                }
            }
        }
        besucht[StartKnotenNr] = false;
    }

    public void Legende()
    {
        String ausgabe = "Ortskürzel: ";
        System.out.println("/n__________________________________________________________________________________");
        System.out.println("Legende:");
        System.out.println("---->: Straße");
        for(int i=0; i<anzahlKnoten; i++)
        { 
            if(i%5 ==0)
            {
                System.out.println(ausgabe);
                ausgabe = knoten[i].BezeichnungGeben()+": " + zusatz.abkürzung(knoten[i].BezeichnungGeben(), anzahlKnoten);
            }
            else
            {
                ausgabe = ausgabe + knoten[i].BezeichnungGeben()+": " + zusatz.abkürzung(knoten[i].BezeichnungGeben(), anzahlKnoten);
            }
        }
    }
}

Die wichtige Klasse dürfte dabei das einfügen sein

public void KnotenEinfuegen(String name)
    {

        if(anzahlKnoten<feldlaenge)
        {
            if(KnotenNummer(name)==-1)
            {
                knoten[anzahlKnoten] = new Knoten(name);
                matrix[anzahlKnoten][anzahlKnoten] = 0;
                for(int i=0; i<anzahlKnoten; i++)
                {
                    matrix[anzahlKnoten][i]=0;
                    matrix[i][anzahlKnoten]=0;
                }
                anzahlKnoten = anzahlKnoten +1;
                String nameKurz = zusatz.abkürzung(name, anzahlKnoten);
                if(breite < nameKurz.length())
                {
                    breite = nameKurz.length();
                }
            }
            else
            {
                System.out.println(name +" ist bereits vorhanden.");
            }
        }

    }

 

[hellobook]

Ok ich hab das Problem gelößt indem ich die Methode in die Klasse GRAPH_MATRIX verschoben habe