replace all with java matcher

Question

I'm trying to add hyperlink on stock information using JAVA matcher.

For example, this string will be changed

How do you think about Samsung and LG? I think Samsung is good.

to

How do you think about <a href="" kospi-code="005930">Samsung</a> and <a href="" kospi-code="003550">LG</a>? I think <a href="" kospi-code="005930">Samsung</a> is good.

But, result was not I expected. :( It was only added 005930.

Here is output.

How do you think about <a href="" kospi-code="005930">Samsung</a> and <a href="" kospi-code="005930">LG</a>? I think <a href="" kospi-code="005930">Samsung</a> is good.

Here is my code snippets. What did I wrong?

String multipleStocks = "How do you think about Samsung and LG? I think Samsung is good.";
Pattern p = Pattern.compile("Hansum|LG|Samsung");
Matcher m = p.matcher(multipleStocks);

HashMap<String, String> stocks = new HashMap<String, String>();

stocks.put("Hansum", "020000");
stocks.put("Samsung", "005930");
stocks.put("LG", "003550");
String ts = null;

while(m.find()){
    System.out.println(m.group());
    ts = m.replaceAll("<a "+stocks.get(m.group(0))+">$0</a>"); 
}
System.out.println(ts);

Rohit Gupta · Accepted Answer · 2016-07-18 11:21:11Z

2

try this in place of your loop.

for (String key : stocks.keySet()) {
    multipleStocks=multipleStocks.replaceAll(key, "<a "+stocks.get(key)+">$0</a>");
}

System.out.println(multipleStocks);

edited Jul 18, 2016 at 11:21

answered Jul 18, 2016 at 9:31

Rohit Gupta

4533 silver badges17 bronze badges

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

2 Comments

Minkyu Kim Over a year ago

Appricate your answer. I found another solution but your's works well. Thank you @rht.

Minkyu Kim Over a year ago

Oh, I found a problem that is existing similar keys in stocks. For instance, if LG Elotronics is exists in stocks, then link will be <a href="" kospi-code="005930"><a href="" kospi-code="066570">LG</a> Eloctronics</a>

Minkyu Kim · Accepted Answer · 2016-07-18 09:32:15Z

0

I asked this to other communitiy and someone gave me an answer.

I used Matcher.replaceAll but what had needed to me was Matcher.appendReplacement.

    while(m.find()){
        m.appendReplacement(sb, "<a "+stocks.get(m.group(0))+">$0</a>");
    }
    m.appendTail(sb);

answered Jul 18, 2016 at 9:32

Minkyu Kim

1,1543 gold badges21 silver badges44 bronze badges

1 Comment

Minkyu Kim Over a year ago

@Gab It needs 2 days for accept own answer. Thank you Gab.

ONLY_LTY · Accepted Answer · 2016-07-18 10:39:12Z

0

replaceAll() is

Replaces every subsequence of the input sequence that matches the pattern with the given replacement string.

the first loop m.find() is matcher Samsung , but replaceAll() is matcher Samsung and LG .

the second loop m.find() will return false,because m is changed.

you can annotation this code // ts = m.replaceAll("");

and execute , will print Samsung LG Samsung

answered Jul 18, 2016 at 10:39

ONLY_LTY

12 bronze badges

1 Comment

Minkyu Kim Over a year ago

Thank you but I want to attach html tag to such as Samsung and LG, not want to watch what has printed out.

Collectives™ on Stack Overflow

replace all with java matcher

3 Answers 3

2 Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related