Editor - Zeilenumbrüche checken

[billbill]

Hallo zusammen,

ich programmiere grade einen Editor und ich habe die Aufgabe, eine Fehlermeldung auspucken zu lassen falls eine Datei nicht korekt geladen werden kann (wegen den Zeilenumbrüchen, Linux="\n" und windows="\r").
Also wenn eine Datei in Windows gespeichert aber in Linux geöffnet wird und umgekehrt.
Ich habe folgendes gemacht.

public class ZeilenumbruchKonvention {
     /**
     * Das Textpane vom MainWindow.
     */
    private JTextPane textpane;
    String BS;
    String Zeilenumbruch;
    
    public String ermitleZeilenumbruch() {
		BS = System.getProperty("os.name");
                if (BS.equals("Linux")){
                    Zeilenumbruch="\n";
                }
                else if(BS.substring(0,6).equals("Windows")){
                    Zeilenumbruch="\r";
                }
    return Zeilenumbruch;
    }
    
    public int vergleicheZeilenUmbruch(String Zeilenumbruch,JTextPane text){
        String zeilen[];
        zeilen=text.getText().split(Zeilenumbruch);
        int ergebnis=1;
        if (zeilen[1]==null) {
            ergebnis=0;
        }
        else;
        System.out.println(ergebnis);
        return ergebnis;
    }
}

                    String Zeilenumbruch = this.ZK.ermitleZeilenumbruch();
                    int ZeilenUmbruchKorrekt=this.ZK.vergleicheZeilenUmbruch(Zeilenumbruch, Inhalt);
                    if (ZeilenUmbruchKorrekt==0){
                       JOptionPane.showMessageDialog(Inhalt, "Textformat nicht korrekt", actCom, JOptionPane.ERROR_MESSAGE);
                    }

Also der Text wird gesplitet sofern der Zeilenumbruch stimmt, sonst nicht.
Dementsprecht wird 1 oder 0 zurückgegeben falls zeilen[1] existiert und gehandelt.

Leider bekomm ich eine NullPointerException. Ich checke nicht wieso.

Ist mein Programm sonst ok? Was meint ihr?

Danke schonmal,

Grüsse
billbill

 

[Ghostfish]

Hm. Warum prüfst du denn zeilen[1] und nicht zeilen oder zeilen[0]?

Variablennamen schreibt man klein.

Den System-Zeilenumbruch kann man IMHO direkt abfragen.

Aus Usability-Gründen halte ich es für klüger, die Zeilenumbrüche beim Laden des Textes einfach umzuwandeln statt eine Fehlermeldung auszugeben.

 

[Ghostfish]

Hm. Warum prüfst du denn zeilen[1] und nicht zeilen oder zeilen[0]?

Ach so, verstanden. Wenn auch fehlerträchtig wenn der Text nur aus einer zeile besteht.

Bei Windows ist der Zeilenumbruch übrigens "\r\n".

 

[billbill]

Ach so, verstanden. Wenn auch fehlerträchtig wenn der Text nur aus einer zeile besteht.

Bei Windows ist der Zeilenumbruch übrigens "\r\n".

genau, wenn er aus einer zeile, also zeilen[0] besteht, bedeutet das, dass der zeilenumbruch falsch ist und die fehlermeldung erscheint

du hast recht "\r\n" ist für windows, danke

irgend eine ahnung warum die exception auftaucht?

 

[xehpuk]

Du hast uns nicht gesagt, wo der Fehler auftritt. Ich tippe mal auf diese Zeile:
[JAVA=22] zeilen=text.getText().split(Zeilenumbruch);[/code]

Zeilenumbruch

wird nämlich

null

sein. Etwas wie

BS.substring(0,6).equals("Windows")

wird nie

true

ergeben, denn "Windows" hat sieben, nicht sechs Zeichen.
Wenn Java 7 vorhanden, verwende [JAPI]System#lineSeparator()[/JAPI], ansonsten

System.getProperty("line.separator")

.

zeilen[1]==null

ergibt auch keinen Sinn. Greifst du auf einen nicht existierenden Index eines Arrays zu, wird nicht

null

zurückgegeben, sondern es fliegt eine [JAPI]ArrayIndexOutOfBoundsException[/JAPI].
Im Übrigen werden durch deine Methode Windows-Zeilenumbrüche auch unter Linux und Mac OS angenommen.

Des Weiteren sei auf den primitiven Datentyp

boolean

verwiesen. Die Methode sollte nämlich keinen

int

zurückgeben.

 

[Gelöschtes Mitglied 17508]

Hallo!

Ich weiss, dass das jetzt nicht soviel ausmacht, aber als kleine Anmerkung:

Wenn die Methode vergleicheZeilenUmbruch true oder false zurückliefern sollte, dann wäre es auch gleich angebracht, diese Methode anders zu benennen, zumindest vll. mit einem vorangestellten: isxxxx

    public int isvergleicheZeilenUmbruch(String Zeilenumbruch,JTextPane text){
        String zeilen[];
        zeilen=text.getText().split(Zeilenumbruch);
        boolean ergebnis=true;
        if (zeilen[1]==null) {
            return ergebnis;
        }
        else {
        System.out.println(ergebnis);
        ergebnis=false;
        return ergebnis;
    }

So in der Art zB.

mfG

 

[billbill]

Du hast uns nicht gesagt, wo der Fehler auftritt. Ich tippe mal auf diese Zeile:
[JAVA=22] zeilen=text.getText().split(Zeilenumbruch);[/code]

Zeilenumbruch

wird nämlich

null

sein. Etwas wie

BS.substring(0,6).equals("Windows")

wird nie

true

ergeben, denn "Windows" hat sieben, nicht sechs Zeichen.
Wenn Java 7 vorhanden, verwende [JAPI]System#lineSeparator()[/JAPI], ansonsten

System.getProperty("line.separator")

.

zeilen[1]==null

ergibt auch keinen Sinn. Greifst du auf einen nicht existierenden Index eines Arrays zu, wird nicht

null

zurückgegeben, sondern es fliegt eine [JAPI]ArrayIndexOutOfBoundsException[/JAPI].
Im Übrigen werden durch deine Methode Windows-Zeilenumbrüche auch unter Linux und Mac OS angenommen.

Des Weiteren sei auf den primitiven Datentyp

boolean

verwiesen. Die Methode sollte nämlich keinen

int

zurückgeben.

Ach so, ich wusste nicht das ich den line.seperator direkt entnehmen kann.
Die Fkt lautet jetzt

    public String ermitleZeilenumbruch() {
		zeilenumbruch = System.getProperty("line.separator");
    return zeilenumbruch;
    }

Die 2. Fkt hat den Rückgabewert boolean statt int.
Die Exception kommt genau an der stelle die du getippt hast, nach wie vor.

Ist das nicht so dass der ganze Text in Zeilen[0] gespeichert wird wenn das split-Argument nicht vorkommt?

Wie kann ich denn checken ob zeilen[1] existiert?

 

[xehpuk]

Ist das nicht so dass der ganze Text in Zeilen[0] gespeichert wird wenn das split-Argument nicht vorkommt?

Ja.

Wie kann ich denn checken ob zeilen[1] existiert?

zeilen.length > 1

Was mir noch auffällt: Eine Textkomponente sollte normalerweise nur '\n' als Zeilenumbruch haben (siehe [JAPI]DefaultEditorKit[/JAPI]). Wie fügst du Text ein? Was kriegst du bei

textPane.getDocument().getProperty(DefaultEditorKit.EndOfLineStringProperty)

zurück?

 

[billbill]

Es kommt eine neue NullPointerException wenn ich das ausführe.
Meine "alte" Exception ist jedenfalls aufgehoben.

Text wird in einem JTextPane geladen oder neu geschrieben, den entnehme ich einfach mit einem getText().

Also mir wurde gesagt dass es zu jedem Betriebssystem einen entsprechenden Zeilenumbruch gibt und das sollte halt behandelt werden.