Regex to replace logging

Question

I would like to add stacktrace to error logging, so anywhere we have

try {
    doSomething();
} catch (Exception ex) {
    something();
    logger.severe("oops");
    something();
}

I would like to have this:

try {
    doSomething();
} catch (Exception ex) {
    something();
    logger.log(Level.SEVERE,"oops",ex);
    something();
}

The exception variable can be anything ex, e etc.

I created a regex, but it is quite slow and does not work as I expect.

Regex:

((?s:.)*catch\s*\(\s*Exception\s*(.+)\s*\)\s*\{(?s:.)*logger\.)severe\(((?s:.)*?)\);((?s:.)*)

Replace:

$1log(Level.SEVERE,$3,$2); $4

It replaces

try {
  doSomething();
} catch (Exception ex) {
    something();
    logger.severe("oops");
    something();
} 
{
   logger.severe("oops2");
}

to

try {
  doSomething();
} catch (Exception ex) {
    something();
    logger.severe("oops");
    something();
} 
{
   logger.log(Level.SEVERE,"oops2",ex); 
}

So you're writing code to replace code? Does your development environment not include a find/replace all function? — Shar1er80
– Shar1er80, Commented Jun 19, 2015 at 15:14
No. I want to replace code with regex using it in the replace all command of my IDE. — HamoriZ
– HamoriZ, Commented Jun 19, 2015 at 15:33

dzidzitop · Accepted Answer · 2015-06-19 15:08:43Z

1

Use lazy matchers instead of greedy ones. Refer to the java.util.regex.Pattern documentation to get more details.

answered Jun 19, 2015 at 15:08

dzidzitop

3852 silver badges11 bronze badges

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

1 Comment

HamoriZ Over a year ago

yes, thanks I forgot to use lazy matcher by the "logger" word

Collectives™ on Stack Overflow

Regex to replace logging

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related