Editor hängt sich auf

[Oger]

Hallo Zusammen !

Ich schreibe grad einen kleinen Editor als Projekt.

Dabei habe ich die Funktion Text suchen und ersetzen eingebaut.

Für den Suchtext habe ich ein JTextField

JTextField SuchText = new JTextField(30);

Dann packe ich den Inhalt in einen String

String search = SuchText.getText();

Das passiert 2 Mal, da ich ja auch einen Text zum ersetzen habe...

Wenn ich meinen Editor nun starte und die Suchen und Ersetzen Funktion aktiviere, dann öffnet sich zwar das kleine Fenster mit den Textfeldern, aber es wird kein Inhalt angezeigt und der Editor hängt sich auf....

Bei einem Textfeld dauert es ein paar Sekunden und dann erscheint es.

Aus welchem Grund hängt sich die ganze Sache auf? Wird irgendwo etwas überladen oder so ?

MFG
Oger

 

[outbreaker]

wir wäre es mal mit etwas mehr Code oder der Fehlermeldung!?
:autsch:

die beiden Codezeilen die da stehen sind richtig ;-)

aber der zusammenhang in Code wäre nicht schlecht

 

[SlaterB]

tja, an den beiden geposteten Code-Zeilen liegt es sicherlich nicht,
was in deinen restlichen 1000 Zeilen schiefläuft kann aber doch niemand erahnen,

gibts keine Exception in der Konsole?
dann kann man noch am ehesten auf irgendeine Endlosschleife tippen,
sich gegenseitig aufrufende Operationen, Listener die auf Intialisierung von GUI-Elementen reagieren usw.,
da hilft eigentlich nur Debugging, Schritt für Schritt den Weg des Programms bis zum Fehler verfolgen, einzelne Komponenten auskommentieren

 

[Oger]

Also hier der gesamte Such Teil. Es gibt keinerlei Exception oder sonst nen Fehler, bis auf die Tatsache, das das Fenster nicht mehr reagiert....

JFrame Search = new JFrame( "Suchen und Ersetzen" ); 
        	Search.setSize( 350, 200 ); 
        	Search.setVisible( true ); 
        	Search.setLocation(350,250) ;
        	
        	JPanel SuEr = new JPanel();
        	Search.add(SuEr);
        	
        	JTextField SuchText = new JTextField(30);
        	SuEr.add (SuchText);
        	
        	JButton suchen = new JButton( "Suchen" ); 
            SuEr.add( suchen ); 
            JButton weiter = new JButton( "Weiter Suchen" ); 
            SuEr.add( weiter ); 
            
            JTextField ErsetzText = new JTextField(30);
        	SuEr.add (ErsetzText);
            
            JButton ersetzen = new JButton( "Ersetzen" ); 
            SuEr.add( ersetzen ); 
    		
             	
            String text = Editor.getTextFromtArea();
            int index;
            int start = 0;
            String search = SuchText.getText();
            int len = search.length();
            String replace = ErsetzText.getText();
     
            StringBuffer buff = new StringBuffer(text.length()+30);
        
        
	        while ( (index = text.indexOf(search, start)) >=0 ) {
	            if (index > start) {
	                // Kopiere, was zwischen der letzten Fundstelle und der aktuellen liegt.
	                buff.append(text.substring(start, index));    
	            }
	            start = index + len;
	            // Ersetze die aktuelle Fundstelle durch den Ersatz.
	            buff.append(replace);
	        }
	        if (start < text.length()) {
	            // Kopiere, was auf die letzte Fundstelle noch folgt.
	            buff.append(text.substring(start, text.length()));
	        }
	        
	        System.out.println(text); // erst das Original
	        System.out.println(buff.toString()); // dann das veränderte
	        
	        Editor.appendText2(buff.toString());    
	        }

 

[outbreaker]

1.) JFrame search und nicht JFrame Search
2.) füge mal erst deine Buttons und co zum JFrame hinzu und mache dann das

Search.setSize( 350, 200 );
Search.setVisible( true );
Search.setLocation(350,250) ;

dann sind diese auch sichtbar

also Zeile 2-4 in Zeile 22 verschieben

 

[Oger]

Die Buttons und Co sind doch auf dem JPanel und das ist im JFrame, reicht das nicht oder steht ich grad aufm Schlauch ?

 

[outbreaker]

ja das reicht aber du zeigst das JFrame ja schon an bevor du deine Buttons usw hinzufügst somit werden die nicht angezeigt da ein repaint fehlt.

mache die Reihenfolge so:

JFrame Search = new JFrame( "Suchen und Ersetzen" );
		              
JPanel SuEr = new JPanel();
Search.add(SuEr);

JTextField SuchText = new JTextField(30);
SuEr.add (SuchText);
		              
JButton suchen = new JButton( "Suchen" );
SuEr.add( suchen );
JButton weiter = new JButton( "Weiter Suchen" );
SuEr.add( weiter );
		              
JTextField ErsetzText = new JTextField(30);
SuEr.add (ErsetzText);
	              
JButton ersetzen = new JButton( "Ersetzen" );
SuEr.add( ersetzen );

Search.setSize( 350, 200 );
Search.setVisible( true );
Search.setLocation(350,250) ;

 

[Oger]

Ok, danke, aber das löst mein Problem nicht. Die beiden Fenster hängen sich immer noch auf.

 

[outbreaker]

deine while-Schleife in Zeile 34 ist eine Endlosschleife
weil index immer 0 ist
sobald ud die Anwendung startest suchst du nach dem index
beim starten der Anwendung ist aber der Inhalt von search = "" somit bleibt er immer in der schleife

du solltest das suchen erst starten wenn der Anwender auf einen Button klickt und nicht schon vorher

 

[outbreaker]

Also wenn du in der while abfrage das = wegnimmst startet es würde ich sagen

while ( (index = text.indexOf(search, start)) >0 ) {

 

[Oger]

Öhm, das geht nicht. Was soll ich denn mit dem index dann tun ?

Außerdem kommt der Fehler schon bei der Darstellung des Fensters, da es sich gar nicht erst aufbaut. Wenn ich nur ein JTextField habe, dauert es ein paar Sekunden und dann geht es. Nur wenn ich das zweite mit einbeziehe, dann scheint er sich tot zu laden....

 

[Oger]

deine while-Schleife in Zeile 34 ist eine Endlosschleife
weil index immer 0 ist
sobald ud die Anwendung startest suchst du nach dem index
beim starten der Anwendung ist aber der Inhalt von search = "" somit bleibt er immer in der schleife

du solltest das suchen erst starten wenn der Anwender auf einen Button klickt und nicht schon vorher

Ohne gui läuft das Ding aber und funktioniert tadellos... Das mit dem Button draufklicken habe ich wohl vergessen..

 

[outbreaker]

du hast diese Zeile

int len = search.length();

und in deiner Schleife machst du das:

start = index + len;

wenn nun search.length() == 0 ist dann bleibt dein startwert immer 0

 

[Oger]

Hey, ich bin ein Noob und habe keine ahnung... aber ohne die Gui Kacke läuft dieser Teil wunderbar und ich kann ein vorgegbenes Wort suchen und auch ersetzen....


Das ist komisch:
Wenn ich

String search = "Fester String";

mach statt

String search = SuchText.getText();;

dann lädt er die Gui ohne Probleme

 

[outbreaker]

ja aber wenn du

String search = SuchText.getText();;

machst beim start dann ist der Inhalt von search leer
und dann suchst du nach dem vorkommen von keinem Zeichen in deiner while schleife

und wenn du schon von Anfang an einen String reinschreibst dann klappt es weill der ja nicht leer ist

mache deine while schleife erst nach drücken eines buttons und wenn SuchText.getText().length > 0 ist

um dem Button eine funktion zu geben braucht du einen ActionListener

den bekommst du z.b. so in deine Klasse:

public class Klasse implements ActionListener{
...
}

dann hast du eine MEthode actionPerformed() und in der sagst du dann was der Button machen soll
und das ganze muss dann noch an den Button gehängt werden mit

Button myB = new Button();
myB.addActionListener(this);

oder du machts es zum Testen erstmal so:

while ( (index = text.indexOf(search, start)) >=0 && SuchText.getText().length > 0){

 

[Oger]

Erstmal vielen Dank, es klappt nun....

Ich habe aber nu ein anderes Problem.
Wenn ich die Suchfunktion anklicke, dann wird ja das JFrame mit Inhalt geladen. Wenn ich aber während der Laufzeit ein weiteres Mal die Suchfunktion aktiviere, dann ist alles doppelt da. Kann man das JFrame am Ende irgendiwe zurück setzen oder löschen ?

 

[Tobias]

Zunächst einmal solltest du statt JFrame einen JDialog benutzen - der funktioniert ähnlich wie ein JFrame, ist aber extra dafür gedacht, als "Unterfenster" zu einem JFrame zu agieren. Und ansonsten: Der Suchdialog enthält bei nochmaligen Aufruf doppelt so viele Element wie gewollt, hab ich das richtig verstanden? Dann benutzt du vermutlich irgendwo das selbe Such-JFrame wieder und wieder, addest aber jedesmal die GUI-Elemente erneut - Suchdialog als eigene Klasse schreiben, Objekt nicht wiederverwenden.

mpG
Tobias

 

[Oger]

Ich habe eine Klasse mit dem Namen SuchGui die durch einen klick im Menü aufgerufen wird.... und in der Klasse wird das komplette Suchfenster aufgebaut.... Wie soll ich es statt dessen machen ?

 

[Tobias]

Das klingt vernünftig. Gib mal den Code von SuchGUI ...

 

[Oger]

Nicht wundern, 2 Buttons funktionieren im Moment noch nicht....

public class SuchGui extends JPanel implements ActionListener{
	
	private Editor test;
	
	JFrame SuchFenster = new JFrame( "Suchen und Ersetzen" );
	JPanel SuchPanel = new JPanel();	
	
	
	
	
	
	
	 
	
	public  SuchGui(Editor editor){
		 
		   test = editor;
		   
	   }
	
	public void actionPerformed(ActionEvent e){
		String cmd = e.getActionCommand() ;
		
		if (cmd.equals("suche") ) {
			
			SuchFenster.add(SuchPanel);
			final JTextField SuchText = new JTextField(30);
			SuchPanel.add(SuchText);
			JButton Suchen = new JButton( "Suchen" );
			SuchPanel.add( Suchen );
			JButton SuchenWeiter = new JButton( "Weiter Suchen" );
			SuchPanel.add( SuchenWeiter );
			
			final JTextField ErsetzText = new JTextField(30);
			SuchPanel.add(ErsetzText);
			JButton Ersetzen = new JButton( "Ersetzen" ); 
			SuchPanel.add(Ersetzen);
		   
			
			ActionListener SucheText = new ActionListener() { 
		    	public void actionPerformed( ActionEvent e ) { 
		    		System.exit( 0 ); 
		    	} 
		    };
		    Suchen.addActionListener( SucheText );

		    ActionListener SucheWeiterText = new ActionListener() { 
		    	public void actionPerformed( ActionEvent e ) { 
		    		System.exit( 0 ); 
		    	} 
		    };
		    SuchenWeiter.addActionListener( SucheWeiterText );
		    
		    
		    ActionListener ErsetzeText = new ActionListener() { 
		    	public void actionPerformed( ActionEvent e ) { 
		    		
		    		String text = Editor.getTextFromtArea();
			        int index;
			        int start = 0;
			        String search = SuchText.getText();
			        int len = search.length();
			        String replace = ErsetzText.getText();

			        StringBuffer buff = new StringBuffer(text.length()+30);
			            
			        while ( (index = text.indexOf(search, start)) >=0 ) {
			            if (index > start) {
			                // Kopiere, was zwischen der letzten Fundstelle und der aktuellen liegt.
			                buff.append(text.substring(start, index));    
			            }
			            start = index + len;
			            // Ersetze die aktuelle Fundstelle durch den Ersatz.
			            buff.append(replace);
			        }
			        if (start < text.length()) {
			            // Kopiere, was auf die letzte Fundstelle noch folgt.
			            buff.append(text.substring(start, text.length()));
			        }
			        
			        System.out.println(text); // erst das Original
			        System.out.println(buff.toString()); // dann das veränderte
			        
			        Editor.appendText2(buff.toString());    
			        }
		    };
		    Ersetzen.addActionListener( ErsetzeText ); 
		    
		    
		 
		    SuchFenster.setSize( 350, 200 );
		    SuchFenster.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
			SuchFenster.setVisible( true );
			SuchFenster.setLocation(350,250) ;
			
			SuchPanel.setVisible(true);
			
		
			
		}
	}
		
		
	
	
}

 

[Tobias]

Eieiei, da ist aber ganz schön was durcheinander gegangen ...

Mach folgendes:

public class SuchGUI extends JDialog implements ActionListener {

    private String suchString;

    public SuchGUI(SwingEditorInterface parent) {
         super(parent.getFrame(), true);  // getFrame() liefert das JFrame vom SwingEditor zurück
         // Gui zusammenbauen
         suchButton.addActionListener(this); // suchButton ist der Button, der die Suche startet
         setVisible(true);
    }

    public String getSuchString() {
        return suchString;
    }

    public void actionPerformed(ActionEvent e) {
         suchString = suchTextField.getText();
         setVisible(false);
         dispose();
    }

}

In der Klasse Globals eine Methode, die den Suchstring als Parameter annimmt und die eigentliche Suche ausführt. Im MenueListener dann sowas a la:

... if(e.getActionCommand().equals("Suchen")) {
    SuchGUI sGui = new SuchGUI(f); // f ist das SwingEditorInterface
    Glogabls.getGlobals().search(sGui.getSuchString());
}

SuchGUI macht also nichts anderes als den Aufbau der GUI, Globals macht die eigentliche Arbeit und MenueListener verkabelt das Ganze. Ich wünsche viel Erfolg. Wenn es nicht klappt: Morgen bin ich ja mit Leib und Seele wieder da .

mpG
Tobias

 

[Oger]

Naja gut, erstmal danke für die Hinweise... Das umzusetzen dauert wohl noch ne Weile