java String manipulation, change multiple slashes to one slash

Question

Silly mistake ＋ is used instead of ＋

all, I am trying to convert all the "/+ "in input path to "/" to simplify unix - style path.

 path.replaceAll( "/+", "/"); 
 path.replaceAll( "\\/+", "/");

turns out not doing anything, what is the right way of doing this?

public class SimplifyPath {
public String simplifyPath(String path) {
    Stack<String> direc = new Stack<String> ();
    path = path.replaceAll("/＋", "/");
    System.out.println("now path becomes " + path);  // here path remains "///"

    int i = 0;
    while (i < path.length() - 1) {
        int slash = path.indexOf("/", i + 1);
        if (slash == -1) break;
        String tmp = path.substring(i, slash);
        if (tmp.equals("/.")){
            continue;
        } else if (tmp.equals("/..")) {
            if (! direc.empty()){
                direc.pop();
            }
            return "/";
        } else {
            direc.push(tmp);
        }
        i = slash;
    }
    if (direc.empty()) return "/";
    String ans = "";
    while (!direc.empty()) {
        ans = direc.pop() + ans;
    }
    return ans;
}

public static void main(String[] args){
    String input = "///";
    SimplifyPath test = new SimplifyPath();
    test.simplifyPath(input);
 }
}

Try path.replaceAll( "\\/+", "/");

Arnaud
– Arnaud

2015-12-09 09:11:31 +00:00
Commented Dec 9, 2015 at 9:11 — Arnaud
– Arnaud, Commented Dec 9, 2015 at 9:11
this not working either, is this a java version issue ?

sthbuilder
– sthbuilder

2015-12-09 09:14:13 +00:00
Commented Dec 9, 2015 at 9:14 — sthbuilder
– sthbuilder, Commented Dec 9, 2015 at 9:14

dejvuth · Accepted Answer · 2015-12-09 09:17:54Z

7

You're using ＋, not +. It's a different character.

Replace

path = path.replaceAll("/＋", "/");

with

path = path.replaceAll("/+", "/");

answered Dec 9, 2015 at 9:17

dejvuth

7,1743 gold badges37 silver badges37 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Jan · Accepted Answer · 2015-12-09 09:19:19Z

0

So you want //a//b//c to be converted to /a/b/c ?

public static void main(String[] args) {
    String x = "///a//b//c";
    System.out.println(x.replaceAll("/+", "/"));
}

Should to the trick.

If in fact you want /+ -> / conversion you'd need to escape the +, not the /

public static void main(String[] args) {
    String x = "/+/+/a//b/+/c";
    System.out.println(x.replaceAll("/\\+", "/"));
}

answered Dec 9, 2015 at 9:19

Jan

13.9k3 gold badges35 silver badges58 bronze badges

Comments

Haseeb Anser · Accepted Answer · 2015-12-09 09:18:43Z

-1

Have you tried using File.separator ... This is safer than \ or / because Linux and Windows use different file separators. Using File.separator will make your program run regardless of the platform it is being run on, after all, that is the point of the JVM. -- forward slash will work, however, File.separator will make you end users more confident that it will. For example for the path : "Test/World"

String fileP = "Test" + File.separator + "World";

answered Dec 9, 2015 at 9:18

Haseeb Anser

5041 gold badge5 silver badges19 bronze badges

1 Comment

BalusC Over a year ago

This user has plagiarized this answer: stackoverflow.com/a/24643179

Collectives™ on Stack Overflow

java String manipulation, change multiple slashes to one slash

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related