String split und replace geht nicht

[0plan]

Hallo, ich lese in einem Programm aus der CMD Computernamen im Netzwerk aus. Diese beginnen alle mit "\\". Da ich allerdings nur die Namen haben möchte, wollte ich den String splitten. Ich hab schon herrausgefunden das \\ in Java \\\\ gleich ist. Allerdings funktioniert es nicht.

Mein String sieht z.B. so aus \\PC1\\PC2\\PC3\\PC4\\ usw

So versuche ich es:

			String split[] = null;
			split = group.split("\\\\");
			
for(int i=0; i<split.length;i++){
    System.out.println(split[i]);
}

Wenn ich den String so splitte, bekom ich in der Console allerdings nach jedem PC Namen einen Zeilenumbruch. Weiss nicht woran das liegt.

Durch [group.replace("\\", "/"); funktionierts auch nicht, mit einem Slash vor den Rechner Namen könnte ich leben.

Die Ausgabe in der Konsole ist wie folgt:

PC1

PC2

PC3

anstatt

PC1
PC2
PC3

Bitte um Klärung, das macht mich wahnsinnig

 

[Naaram]

In Java werden in Strings Escape-Sequenzen mit \ eingeläutet. Also '\n' ist z.B. ein Zeilenumbruch und '\a' ein Piepser. Um ein Backslash hinzuschreiben schreibt man '\\'. Also probiere es doch mal mit "\\\\" als splitter.

 

[0plan]

Entschuldige, ich habe im splitter schon \\\\hatte mich vertippt. Mit \\\\ erhalte ich die ausgabe wie beschrieben.

 

[HoaX]

split verwendet ja eine Regex zum Splitten, und auch dort ist \ ein Escapezeichen, also brauchst du 8 mal den \

 

[0plan]

super vielen dank, mit 8x \ klappts. Bis 6 x bin ich noch mit gegangen, danach hab ich mir schon gedacht das klappt eh nicht

Danke für die Hilfe

 

[0plan]

ich seh grade, nun ist bevor dem ersten string aber immer nach ein zeilenumbruch. Alle anderen sind weg jedoch ist es jetzt wie folgt:

---------------------------------
UMBRUCH
pc1
pc2
pc3
pc4
-------------------------

der index 0 im split array ist also ein Zeilenumbruch.

 

[Gast2]

Der Zeilenumbruch kommt durch [c]System.out.println[/c]. Nimm [c]System.out.print[/c]. Dann hast du keinen Zeilenumbruch. In dem Array split sind KEINE Zeilenumbrüche drin. Der erste Eintrag ist halt ein Leerstring

 

[chalkbag]

du könntest auch einfach prüfen, ob du eine Leerzeile (spllit.length()==0) hast.
Diese willst dann natürlich nicht drucken.

 

[0plan]

Wie kommt es dann wenn ich den i als 1 initialisiere keinen habe oder "hallo in der konsole ausgebe"

 

[Gast2]

Wie gesagt - durch dein Split ist split[0] = ""

 

[0plan]

Okay danke, ist ja auch nicht weiterschlimm ich lass den Zähler bei 1 starten.

 

[mvitzATwork]

Das liegt nunmal daran, dass dein String mit \\ anfängt und split dort das erste mal Splittet.

Mögliche Lösung:

...
String split[] = group.split("\\\\\\\\");
for(int i=0; i < split.length; i++) {
  if (split[i] != null && !split[i].equals("")) {
    System.out.println(split[i]);
  }
}
...

 

[Gast2]

Okay danke, ist ja auch nicht weiterschlimm ich lass den Zähler bei 1 starten.

Oder du schneidest einfach den ersten \ weg:

[c]split = group.substring(1).split("\\\\");[/c]

 

[Naaram]

Man kann die Probleme mit Google Guava (Bibliothek) generell vermeiden. Hier ein Beispiel:

import com.google.common.base.Splitter;
import com.google.common.collect.Iterables;

public static void main(String[] args) throws URISyntaxException {
    String group = "\\\\PC1\\\\PC2\\\\PC3\\\\PC4\\\\";
    System.out.println(group);
    String split[] = Iterables.toArray(Splitter.on("\\\\").omitEmptyStrings().split(group), String.class);
    for (int i = 0; i < split.length; i++) {
        System.out.println(split[i]);
    }
}

 

[chalkbag]

Man braucht doch nicht für alles gleich ein extra Framework, vor allem wenn es die Java Api sowieso schon kann.

String s = "\\pc1\\pc2\\pc3\\\\";
StringTokenizer strt = new StringTokenizer(s,"\\");
while(strt.hasMoreTokens()){
	System.out.println(strt.nextToken());
}

 

[c_sidi90]

Man braucht doch nicht für alles gleich ein extra Framework, vor allem wenn es die Java Api sowieso schon kann.

Meine Meinung^^

 

[Naaram]

Man braucht doch nicht für alles gleich ein extra Framework, vor allem wenn es die Java Api sowieso schon kann.

Sorry aber das ist doch totaler Käse. Ich meine natürlich will man nicht für jeden kleinen Use Case eine Bibliothek importieren, so dass man den Überblick verliert, aber man sollte auf jeden Fall gut getestete 3rd Party Bibliotheken nutzen um sich Arbeit zu sparen.

Ich programmiere ja auch nicht das HTTP Protokoll nach (was ich aber dank der o. G. Lebenseinstellung schon 100 mal gesehen habe) sondern benutze da eine HTTP Client Bibliothek.

Genau so finde ich es Wichtig, dass man sich neben der Java Standard Bibliothek in ein Paar bekannte Frameworks (neben Guava ist auch apache-commons-* ein Kandidat) einarbeitet, anstatt Arbeit zu machen, die 100 andere auch schon gemacht haben. Dies habe ich früher auch nicht gerne gemacht, aber mittlerweile glaube ich dass dies ein wichtiger Teil des allgemeinen Lernprozesses ist (genau so, wie das Wissen wann man ein Bibliothek nicht benutzen sollte).

Ich habe das Beispiel gebracht, um zu zeigen, das man das Spezielle Problem (Leere Zeile) durch die verwendung einer Bibliothek mit wenigen sehr gut lesbaren Zeilen lösen kann. Außerdem kann man sich bei der Bibliothek sicher sein, dass der Code von hunderten Nutzern getestet ist.


Es ist für mich auch ein großer Unterschied, ob ich den Code produziere:

public List<String> getPcNames(String pcNames) {
    List<String> result = new ArrayList<String>();

    StringTokenizer tokenizer = new StringTokenizer(pcNames, "\\\\");
    while (tokenizer.hasMoreTokens()) {
        String token = tokenizer.nextToken();
        if(!token.isEmpty()) {
            result.add(token);
        }
    }
    return result;
}

Einen Code der Kommentare bedarf um zu verstehen was da passiert. Ein Code der einiger Tests bedarf, damit ich mich sicher fühle. Einen Code bei dem ich leicht einen Fehler einbaue.

Oder einen Code wie diesen:

public List<String> getPcNames(String pcNames) {
    List<String> result = new ArrayList<String>();
    Iterables.addAll(result, Splitter.on("\\\\").omitEmptyStrings().split(pcNames));
    return result;
}

In einem großen Projekt ist Lesbarkeit alles, sonst kostet jedes Feature abartige Summen, und ich finde, dass das untere Beispiel viel einfacher Lesbar ist, als das obere. Das ist meiner Meinung nach das wichtigste hier.

 

[Gast2]

Um für eine kleine Sache (4 Zeilen Code) gleich eine 3rd Party lib zu bemühen find ich ist dann doch ein wenig mit Kanonen auf Spatzen zu schießen. Wenn ich weitere Klassen und Methoden aus der lib in einem größeren Umfang nutze macht das Sinn. Wenn es bei der einen einzige Methode bleibt nicht. Da hol ich mir unnötige Abhängigkiten rein und schleppe jede Menge Ballast rum der einfach nur unnötig ist. Natürlich muss er seine Implementierung testen. Aber da kommst du bei einer lib auch nicht drumrum wenn du es ordentlich machen willst.

Der Vergleich von einem HTTP Client hakt hier gewaltig da dies nicht mal eben in einem 3 Zeiler zu erledigen ist.

Grade in einem großen Projekt muss man sich überlegen welche Abhängigkeiten man in Kauf nimmt. Von Lizenzproblematiken mal gar nicht zu reden.

Und für einen Anfänger (was der TO wohl eindeutig ist, sonst hätter er das Problem nicht) ist deine Nutzung der Bibliothek wohl kaum leichter zu verstehen als es mal eben selber zu machen und evtl dabei noch was zu lernen.

Just my 2 cents