Få ny linje til at virke I JavA

Skal du ikke bare splitte strengen pr. nylinje-tegn: "\\n", i stedet for ved: "\\s"? Og hvorfor har du egentlig firkantede parenteser rundt om "\\s", når du alligevel 'escaper' indholdet af dem?
Sidst men ikke mindst, ville jeg nok lave en liste af de ord, som skal indgå i strengen i stedet for at kopiere "s.contains();" N gange. Dernæst skal man så løbe listen igennem og søge i den opdelte streng, som du lavede til at starte med.

Gætter på at det er Text og TextFlow fra JavaFX du arbejder med.

Som Jannik siger skal du ikke bruge "[" og "]" i dette tilfælde. De tegn bruges til at lave character classes, som er en mængde af tegn. Men \s er allerede en mængde af tegn, de tegn som betegner whitespace.

Når du splitter strengene forsvinder selve teksten du splitter med, så hvis du splitter på \s forsvinder alt whitespace inklusiv newline. Derfor bliver de ikke tegnet.

For at beholde newlines kan du splitte først udelukkende på dem og manuelt genindsætte newlines. For hver linje kan du så splitte på \s for at finde tokens.

En mindre detalje er at i stedet for at splitte på \s bør du splitte på \s+. Så kommer der færre tomme arrays hvis der f.eks. er mere end 1 whitespace mellem to. Visuelt gør det nok ingen forskel men du slipper for at lave en masse Text objekter med tomme strenge.

Jeg har ikke testet koden nedenunder. Men først splittes der på linje. Linje splittes for token med \s+ og der laves Text-objekter. Efter alle tokens på den linje er behandlet indsættes der en newline Text objekt.

 public HashSet getKeywords() {
         HashSet<String> keywords = new HashSet<>();
         keywords.add("if");
         keywords.add("else if");
         keywords.add("else");
         keywords.add("public");
         keywords.add("private");
         keywords.add("protected");
         keywords.add("try");
         keywords.add("catch");
         keywords.add("final");
         keywords.add("abstract");
         keywords.add("extends");
         keywords.add("implements");
         keywords.add("import");
         keywords.add("package");
         keywords.add("System");
         return keywords;
     }
     
     public void unknownMethod() {
         String text = "Hell World \n if public";
         final HashSet<String> keywords = getKeywords();
         for (String line : text.split("\n")) {
             for (String token : line.split("\\s+")) {
                 Text textToken = new Text(token + " ");
                 String id = "StandardText";
                 if (keywords.contains(token)) {
                     id = token;
                 }
                 textToken.setId(id);
                 getChildren().add(textToken);
             }
             //Add the newline
             getChildren().add(new Text("\n"));
         }
     }

Gætter på at det er Text og TextFlow fra JavaFX du arbejder med.

Som Jannik siger skal du ikke bruge "[" og "]" i dette tilfælde. De tegn bruges til at lave character classes, som er en mængde af tegn. Men \s er allerede en mængde af tegn, de tegn som betegner whitespace.

Når du splitter strengene forsvinder selve teksten du splitter med, så hvis du splitter på \s forsvinder alt whitespace inklusiv newline. Derfor bliver de ikke tegnet.

For at beholde newlines kan du splitte først udelukkende på dem og manuelt genindsætte newlines. For hver linje kan du så splitte på \s for at finde tokens.

En mindre detalje er at i stedet for at splitte på \s bør du splitte på \s+. Så kommer der færre tomme arrays hvis der f.eks. er mere end 1 whitespace mellem to. Visuelt gør det nok ingen forskel men du slipper for at lave en masse Text objekter med tomme strenge.

Jeg har ikke testet koden nedenunder. Men først splittes der på linje. Linje splittes for token med \s+ og der laves Text-objekter. Efter alle tokens på den linje er behandlet indsættes der en newline Text objekt.

Java kode 

 public HashSet getKeywords() {
         HashSet<String> keywords = new HashSet<>();
         keywords.add("if");
         keywords.add("else if");
         keywords.add("else");
         keywords.add("public");
         keywords.add("private");
         keywords.add("protected");
         keywords.add("try");
         keywords.add("catch");
         keywords.add("final");
         keywords.add("abstract");
         keywords.add("extends");
         keywords.add("implements");
         keywords.add("import");
         keywords.add("package");
         keywords.add("System");
         return keywords;
     }
     
     public void unknownMethod() {
         String text = "Hell World \n if public";
         final HashSet<String> keywords = getKeywords();
         for (String line : text.split("\n")) {
             for (String token : line.split("\\s+")) {
                 Text textToken = new Text(token + " ");
                 String id = "StandardText";
                 if (keywords.contains(token)) {
                     id = token;
                 }
                 textToken.setId(id);
                 getChildren().add(textToken);
             }
             //Add the newline
             getChildren().add(new Text("\n"));
         }
     }

Det virker næsten. Bortset fra den laver en ny linje per bogstav/tal så teksten vil istedet for ligne noget som:

i
f

g
o
o
g
l
e

Her er min fulde kode:

 /*
  * To change this license header, choose License Headers in Project Properties.
  * To change this template file, choose Tools | Templates
  * and open the template in the editor.
  */
 package fxarea;
 
 import java.util.HashSet;
 import javafx.scene.text.Text;
 import javafx.scene.text.TextFlow;
 
 /**
  *
  * @author Mathias - ZinXan
  */
 public class FXArea extends TextFlow {
 
     public FXArea() {
         autosize();
         ReadText();
     }
 
     public void ReadText() {
         String text = "Hell World \n if public";
         final HashSet<String> keywords = getKeywords();
         for (String line : text.split("\n")) {
             for (String token : line.split("\\s+")) {
                 Text textToken = new Text(token + " ");
                 String id = "StandardText";
                 if (keywords.contains(token)) {
                     id = token;
                 }
                 textToken.setId(id);
                 System.out.println(textToken.getId());
                 getChildren().add(textToken);
             }
             //Add the newline
             getChildren().add(new Text("\n"));
         }
     }
 
     public HashSet getKeywords() {
         HashSet<String> keywords = new HashSet<>();
         keywords.add("if");
         keywords.add("else if");
         keywords.add("else");
         keywords.add("public");
         keywords.add("private");
         keywords.add("protected");
         keywords.add("try");
         keywords.add("catch");
         keywords.add("final");
         keywords.add("abstract");
         keywords.add("extends");
         keywords.add("implements");
         keywords.add("import");
         keywords.add("package");
         keywords.add("System");
         return keywords;
     }
 }

Det lyder meget overraskende at de skulle give noget forskelligt. De eneste to forskelle, er at du har en ekstra println inde i løkken, og kalder autosize() først. Så prøvede lige dem selv og ved mig giver begge løsninger samme resultat.

Men glad for at høre du fik det til at virke.