Java : replaceAll and .split newline doesn't work

Question

I am wanting to split a line up (inputLine) which is

Country: United Kingdom
City: London

so I'm using this code:

public void ReadURL() {

    try {
        URL url = new URL("http://api.hostip.info/get_html.php?ip=");
        BufferedReader in = new BufferedReader(
        new InputStreamReader(url.openStream()));

        String inputLine = "";
        while ((inputLine = in.readLine()) != null) {
            String line = inputLine.replaceAll("\n", " ");
            System.out.println(line);
        }

        in.close();
    }   catch ( Exception e ) {
        System.err.println( e.getMessage() );
    }
}

when you run the method the the output is still

Country: United Kingdom
City: London

not like it's ment to be:

Country: United Kingdom City: London

now i've tried using

\n,\\n,\r,\r\n

and

System.getProperty("line.separator")

but none of them work and using replace, split and replaceAll but nothing works.

so how do I remove the newlines to make one line of a String?

more detail: I am wanting it so I have two separate strings

String Country = "Country: United Kingdom";

and

String City = "City: London";

that would be great

Umm, well you are using System.out.println NOT System.out.print — thatidiotguy
– thatidiotguy, Commented Mar 27, 2013 at 20:58

tmwanik · Accepted Answer · 2013-03-27 20:59:28Z

1

You should instead of using System.out.println(line); use System.out.print(line);.

The new line is caused by the println() method which terminates the current line by writing the line separator string.

answered Mar 27, 2013 at 20:59

tmwanik

1,66114 silver badges20 bronze badges

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

1 Comment

String.userName Over a year ago

Wow and I didn't even realize, I knew it would have been something obvious -.-

Daniel Williams · Accepted Answer · 2013-03-27 21:00:27Z

0

http://docs.oracle.com/javase/1.5.0/docs/api/java/io/BufferedReader.html#readLine()

Read that. readLine method will not return any carriage returns or new lines in the text and will break input by newline. So your loop does take in your entire blob of text but it reads it line by line.

You are also getting extra newlines from calling println. It will print your line as read in, add a new line, then print your blank line + newline and then your end line + newline giving you exactly the same output as your input (minus a few spaces).

You should use print instead of println.

answered Mar 27, 2013 at 21:00

Daniel Williams

8,9254 gold badges40 silver badges49 bronze badges

Comments

user180100 · Accepted Answer · 2013-03-27 21:10:17Z

0

I would advise taking a look at Guava Splitter.MapSplitter

In your case:

// input = "Country: United Kingdom\nCity: London"
final Map<String, String> split = Splitter.on('\n')
    .omitEmptyStrings().trimResults().withKeyValueSeparator(": ").split(input);
// ... (use split.get("Country") or split.get("City")

answered Mar 27, 2013 at 21:10

user180100

Collectives™ on Stack Overflow

Java : replaceAll and .split newline doesn't work

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related