Exception ohne Ursprung

[michi2]

Hallo,
Wie kann folgende Exception entstehen, sie hat keinen Aufrufer, in dem von mir geschriebenen Programm!

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
        at javax.swing.text.DefaultHighlighter.paint(DefaultHighlighter.java:55)
        at javax.swing.plaf.basic.BasicTextUI.paintSafely(BasicTextUI.java:637)
        at javax.swing.plaf.basic.BasicTextUI.paint(BasicTextUI.java:781)
        at javax.swing.plaf.basic.BasicTextUI.update(BasicTextUI.java:760)
        at javax.swing.JComponent.paintComponent(JComponent.java:742)
        at javax.swing.JComponent.paint(JComponent.java:1005)
        at javax.swing.JComponent.paintWithOffscreenBuffer(JComponent.java:4963)
        at javax.swing.JComponent.paintDoubleBuffered(JComponent.java:4916)
        at javax.swing.JComponent._paintImmediately(JComponent.java:4859)
        at javax.swing.JComponent.paintImmediately(JComponent.java:4666)
        at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:451)
        at javax.swing.SystemEventQueueUtilities$ComponentWorkRequest.run(SystemEventQueueUtilities.java:114)
        at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:461)
        at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

Und die einzigste Stelle, an der ich DefaultHighlighter.paint aufrufe, ist nicht die schuldige!

            new MyHighlighter().highlight(Michis_Editor.e1.textanzeige, suchstring);

Was mach ich dagegen?

Michi

 

[Wildcard]

Hast du vieleicht ausserhalb des EventDispatchThreads in die Oberfläche eingegriffen?

 

[michi2]

???:L Meinst du ob ich einen Threat gestartet habe oder wie?
Nicht direkt, aber ich führe obigen Code im Action-Listener eines JDialogs aus, hat das was damit zu tun?

 

[Wildcard]

Da fehlt absolut der Kontext um irgendwas vernünftiges zu diagnostizieren. Wirf halt selbst den Debugger an :bae:

 

[michi2]

Ich hab noch nie mit nem Debuger gearbeitet
Hast du Links, oder kannst du mir schnell erklären:
1. welchen ich nehmen soll
2. wie das geht?

 

[Wildcard]

Eclipse zB...
Breakpoint setzen und mit Step into (F5) und Step over (F6) durch den Code wandern.
Der Debugger kann noch viel mehr, aber das wirst du mit der Zeit rausfinden...

 

[michi2]

Thx, werd das ausprobieren, aber vermt. morgen, ist schon spät...
Ich hab jetzt den Teil aus dem JDialog ausgegliedert (in die den JDialog aufrufende Methode), ändert aber nichts!
Kann es sein das

new MyHighlighter().highlight(Michis_Editor.e1.textanzeige, suchstring);

Einen Threat startet? wenn ich das auskommentiere, tritt die Exception nicht auf!?

Some more code:

public class MyHighlighter
   {
   // Creates highlights around all occurrences of pattern in textComp
   final public void highlight(JTextComponent textComp, String pattern)
      {
      // First remove all old highlights
      //removeHighlights(textComp);
      
      try
         {
         Highlighter hilite = textComp.getHighlighter();
         Document doc = textComp.getDocument();
         String text = doc.getText(0, doc.getLength());
         int pos = 0;
         
         // Search for pattern
         while ((pos = text.indexOf(pattern, pos)) >= 0)
            {
            // Create highlighter using private painter and apply around pattern
            hilite.addHighlight(pos, pos+pattern.length(), Michis_Editor.e1.myHighlightPainter);
            pos += pattern.length();
            }
         } catch (BadLocationException e) {
      }
   }

aufgerufen wird mit:

   // An instance of the private subclass of the default highlight painter
   Highlighter.HighlightPainter myHighlightPainter;
         if(bcolor);
            {
            myHighlightPainter = new MyHighlightPainter(Color.yellow);
System.out.println("befor");   //Erscheint vor Exception
            new MyHighlighter().highlight(Michis_Editor.e1.textanzeige, suchstring);
System.out.println("danach");   //Erscheint auch [b]vor[/b] Exception
            }

 

[michi2]

:bahnhof:
Ich krig das Programm in Eclipse nicht zum Laufen!
Das Programm liegt in UTF-8 vor, und ich habe in den Einstellungen und in der 'build.xml' auch schon Eingestellt, das er UTF-8 verwenden soll, aber er verwendet nachwievor ein anderes Encoding! Was mus ich machen damit er das neue encoding übernimmt? ???:L

michi

 

[michi2]

Also, das mit Eclipse hab ich geschafft...
Ich hatte auch keine Problemme mit dem Debugger.
Aber dem Problem bin ich immer noch nicht auf den Uhrsprung gekommen! Was ich auch ausprobiert habe der Fehler wird immer merkwürdiger!
Könnte ihr mir bitte helfen? Ich weiß wirklich nicht was ich machen soll! :cry: :bahnhof:
Hier das Ecplips-Projekt: http://www.schoenitzer.de/Michis_Editor_eclipse.tar.gz

Alternativ hier noch mehr Code:

//Datei MyHighlighter.java
package pk;

import java.awt.*;
import javax.swing.text.*;

public class MyHighlighter
   {
   // Creates highlights around all occurrences of pattern in textComp
   final public void highlight(JTextComponent textComp, String pattern)
      {
      // First remove all old highlights
      //removeHighlights(textComp);
      
      try
         {
         Highlighter hilite = textComp.getHighlighter();
         Document doc = textComp.getDocument();
         String text = doc.getText(0, doc.getLength());
         int pos = 0;
         
         // Search for pattern
         while ((pos = text.indexOf(pattern, pos)) >= 0)
            {
            // Create highlighter using private painter and apply around pattern
            hilite.addHighlight(pos, pos+pattern.length(), Michis_Editor.e1.myHighlightPainter);
            pos += pattern.length();
            }
         } catch (BadLocationException e) {
         }
      }

   // Creates highlights around from start to end
   final public void highlight(JTextComponent textComp, int start, int end)
      {
      // First remove all old highlights
      //removeHighlights(textComp);
      
      try
         {
         Highlighter hilite = textComp.getHighlighter();
         Document doc = textComp.getDocument();
         String text = doc.getText(0, doc.getLength());

            // Create highlighter using private painter and apply around pattern
            hilite.addHighlight(start, end, Michis_Editor.e1.myHighlightPainter);
         } catch (BadLocationException e) {
         }
      }
   
   // Removes only our private highlights
   final public void removeHighlights(JTextComponent textComp)
      {
      Highlighter hilite = textComp.getHighlighter();
      Highlighter.Highlight[] hilites = hilite.getHighlights();
      
      for (int i=0; i<hilites.length; i++)
         {
         if (hilites[i].getPainter() instanceof MyHighlightPainter)
            {
            hilite.removeHighlight(hilites[i]);
            }
         }
      }  //End removeHighlights
   
   }//Ende MyHighlighter

// A private subclass of the default highlight painter
class MyHighlightPainter extends DefaultHighlighter.DefaultHighlightPainter
   {
   public MyHighlightPainter(Color color)
      {
      super(color);
      }
   }

package pk;

import java.awt.*;
import javax.swing.*;
import javax.swing.text.*;
import java.util.*;
import java.text.*;

import pk.*;

public class Search
   {
   private String lastSearchTxt;
   private int lastSearchIndex = -1;
   private boolean search_up = false;
   private boolean bgenau = false;
   private boolean bcursor = false;
   private boolean bcolor = false;

   // An instance of the private subclass of the default highlight painter
   Highlighter.HighlightPainter myHighlightPainter;

   
   // Diese Funktion sucht einen String im Text
   // Methode von Michis_Editor
   TextSelection stringSuchen(String searchIn, int Cursorposiston)
      {
      String suchstring;
      FrageDialog frage;
      int Index;
      int Cursorpos = Cursorposiston;	//Cursorposition
      TextSelection foundPos = null;
      
      // Ein Textfeld aufmachen, um nach dem Suchstring zu fragen
      frage = new FrageDialog(null,"Suchen");
      frage.setLocation(150,150);
      frage.pack();
      frage.show();
      
      suchstring = frage.suchstring;
      lastSearchTxt = suchstring;
      
      search_up = frage.search_up;
      bgenau       = frage.getgenau();
      bcursor = frage.getcursor();
      bcolor =  frage.tint();
      
      if(suchstring == null)
         return null;

         if(bcolor);
            {
            myHighlightPainter = new MyHighlightPainter(Color.yellow);
            new MyHighlighter().highlight(Michis_Editor.e1.textanzeige, suchstring);
            }
      
      String aktText = searchIn;		
      
      if(bgenau == false)
         {
         suchstring = suchstring.toLowerCase();
         aktText = aktText.toLowerCase();
         }
      
      //Rückwerts?
      if(search_up)  //Rückwerts
         {
         // nun suchen
         if(bcursor == false)
            Index = aktText.lastIndexOf(suchstring);
         else
         Index = aktText.lastIndexOf(suchstring, Cursorpos);
         
         
         if(Index == -1)
            {
            JOptionPane.showMessageDialog(null,"String nicht gefunden",
            "Meldung",JOptionPane.INFORMATION_MESSAGE);
            }
         else
            {
            lastSearchIndex = Index + suchstring.length();
            
            //Gefundene Position speichern
            foundPos = new TextSelection(Index,    Index + suchstring.length());
            }
         }
      
      else  //Vorwerts
         {
         // nun suchen
         if(bcursor == false)
            Index = aktText.indexOf(suchstring);
         else
         Index = aktText.indexOf(suchstring, Cursorpos);
         
         if(Index == -1)
            {
            JOptionPane.showMessageDialog(null,"String nicht gefunden",
            "Meldung",JOptionPane.INFORMATION_MESSAGE);
            }
         else
            {
            lastSearchIndex = Index + suchstring.length();
            
            //Gefundene Position speichern
            foundPos = new TextSelection(Index,    Index + suchstring.length());
            }
         }
      Michis_Editor.e1.menue3.item3_2.setEnabled(true);		//###
      return foundPos;
      }// Ende von 'stringSuchen'
//[...]weitere Methoden

package pk;

import java.util.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.text.*;
import java.text.*;
import pk.*;
import pk.Michis_Editor;

final class TextActionLauscher implements ActionListener
   {
   public void actionPerformed(ActionEvent e)
      {
      String label;
      label = e.getActionCommand();
      
      if(label.equals("Alles markieren"))
         Michis_Editor.e1.textanzeige.selectAll();
      
      else if(label.equals("String suchen"))
         {
         TextSelection ss = Michis_Editor.e1.dokSearcher.stringSuchen(
         	Michis_Editor.e1.textanzeige.getText(),
         	Michis_Editor.e1.textanzeige.getCaretPosition());
         if(ss!=null)
           {
           Michis_Editor.e1.textanzeige.select(ss.getStart(), ss.getEnd());
           }
    	  }
         }

 

[Beni]

Ich kann den Fehler nicht mal reproduzieren... aber einige Javaversionen haben Fehler - gut möglich dass du hier einen gefunden hast.

Was für ein LookAndFeel verwendest du denn? Welche Runtime?

 

[michi2]

Der Fehler tritt auf wenn du das Prog startest, den Text eingiebts, auf Suchen gehst was eintippst, die Option "Farbig markieren" an lässt und suchst...

java version "1.5.0_06"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_06-b05)
Java HotSpot(TM) Client VM (build 1.5.0_06-b05, mixed mode, sharing)
Metal unter Linux (Gnome 2.0.14)

 

[Roar]

Michis_Editor.e1.myHighlightPainter wird wohl null sein, aber die klasse hast du nich gepostet.

btw: lauter static :shock: :bloed:

 

[Beni]

Hab gerade den Debugger angeworfen, und kann Roars Verdacht bestätigen. Ich sags mal so: die etwas chaotische Aufteilung des Programmes hat einen Einfluss auf den Fehler :bae:

 

[michi2]

Welche Vermutung? Und vorallem: was hab ich bei der aufteilung falsch gemacht? Und wie mach ich's besser?

 

[Beni]

Welche Vermutung?

Die hier:

Michis_Editor.e1.myHighlightPainter wird wohl null sein

Mach mal einen Breakpoint in dieser Zeile...

hilite.addHighlight(pos, pos+pattern.length(), Michis_Editor.e1.myHighlightPainter);

... und guck dir den Wert von "myHighlightPainter" an. Vorausgesetzt du machst die Schritte...

Der Fehler tritt auf wenn du das Prog startest, den Text eingiebts, auf Suchen gehst was eintippst, die Option "Farbig markieren" an lässt und suchst...

... wirst du dort ein null sehen.

Und vorallem: was hab ich bei der aufteilung falsch gemacht? Und wie mach ich's besser?

Du greiffst ungeprüft auf innere Variablen von diesem "e1" (ein Michis_Editor-Objekt wenn ich mich recht erinnere) zu.
Da gibt es gleich zwei Probleme:
- "ungeprüft" ist immer gefährlich :wink:
- der Zugriff auf die innere Variable. Eines der wichtigsten Konzepte der Objektorientierung lautet: "Jede Klasse ist für sich, und nur für sich, zuständig". Wie es im inneren einer Klasse aussieht, hat niemanden zu interessieren. Die Kommunikation mit anderen Klassen läuft nur über eine klar definierte, möglichst einfache, möglichst kleine Schnittstelle (ein paar Methoden wie "set.../get.../doSomething...").
Der Vorteil: jede Klasse ist eine kleine Welt für sich. Du kannst immernoch Fehler haben, aber ihre Wirkungen sind meistens lokal begrenzt, und somit wesentlich einfacher zu finden.
Wenn du aber wild in der Gegend rumspringst, sind die Klassen keine kleinen Welten mehr, und jeder Fehler lässt das gesammte Programm stürzen.

Bei diesem Problem: wenn man nur über eine get-Methode an den HighlightPainter kommt, kann diese Methode:
- entweder ein für allemal überprüfen und sicherstellen, dass nicht "null" zurückkommt
- oder definieren dass null erlaubt ist, und der Aufrufer darauf vorbereitet sein muss

P.S. und versuch unbedingt das Problem im Debugger nachzubauen, der Debugger das wichtigste Instrument um Fehler jeder Art zu entdecken. Wenn du gelernt hast mit dem Debugger zu arbeiten, hast du gewonnen

 

[michi2]

1. Fehler erkannt und behoben
2. Sorry, Nachrich von Roar überlesen
3. welche static? Ich hab doch gar keine1
4. Das mit den Klassen design (get...() und set...()) weiß ich schon, nur hab ich im mom zu vile variablen in michis_Editor, werd die zusammenfassen bzw entfernen - dann kann ich das mit get- get set-ern realisieren...
5. Wie sehe ich was Michis_Editor.e1.myHighlightPainter für einen Wert hat?

Und auf jeden Fall: Vielen Dank!

 

[Beni]

1. gut
2. schlecht :bae:
3. Michis_Editor.e1 ist static :wink:
4. gut
5. Schau dir diesen Screenshot an. Ich wüsste nicht wie man direkt "Michis_Editor.e1.myHighlightPainter" abrufen kann, aber man kann in die Methoden "addHighlight" springen. Das letzte Argument dieser Methode ist dein Highlighter, und unten rechts im Bild siehst du, dass "arg2" (eigentlich sollte das p sein, die Sourcen sind wohl nicht richtig eingebungen) null ist.

 

[michi2]

THX!
zu 3: muss doch static sein!? (und eine ist nicht "lauter"!!!)

 

[Beni]

Bis auf die Main-Methode muss garnichts (weder Methode noch Variable) static sein. Du kannst das Editor-Objekt in der Main-Methode erzeugen, und dann herumreichen wie jedes andere Objekt auch (z.B. über set-Methoden).

Ja, "lauter" ist ein bisschen übertrieben, aber wo ein static ist, ist ein zweites nicht fern; nur ein totes static ist ein gutes static... :wink: